Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-05-16 Thread Maxime Ripard
Hi, On Wed, May 15, 2024 at 09:42:34AM +0200, Sean Nyekjaer wrote: > On Wed, May 15, 2024 at 08:39:49AM UTC, Yannick FERTRE wrote: > > Hi Sean, > > > > thanks for your patch. > > > > Tested-by: Yannick Fertre > > > > I think that a helper could be useful in simplifying this part. > > This

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-05-15 Thread Yannick FERTRE
Hi Sean, thanks for your patch. Tested-by: Yannick Fertre I think that a helper could be useful in simplifying this part. This might be reworked when a new helper will be implemented. Best regards On 4/22/24 16:05, Sean Nyekjaer wrote: On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-05-15 Thread Sean Nyekjaer
On Wed, May 15, 2024 at 08:39:49AM UTC, Yannick FERTRE wrote: > Hi Sean, > > thanks for your patch. > > Tested-by: Yannick Fertre > > I think that a helper could be useful in simplifying this part. > This might be reworked when a new helper will be implemented. > > Best regards Hi Yannick,

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-04-30 Thread Maxime Ripard
Hi, On Mon, Apr 29, 2024 at 10:17:45AM +0200, Sean Nyekjaer wrote: > On Wed, Apr 24, 2024 at 09:21:17AM UTC, Maxime Ripard wrote: > > Hi, > > > > Sorry, my previous review didn't go through. > > > > On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean Nyekjaer wrote: > > > When using the DSI

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-04-29 Thread Sean Nyekjaer
Hi, On Wed, Apr 24, 2024 at 09:21:17AM UTC, Maxime Ripard wrote: > Hi, > > Sorry, my previous review didn't go through. > > On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean Nyekjaer wrote: > > When using the DSI interface via DSI2LVDS bridge, it seems a bit harsh > > to reguire the requested and

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-04-24 Thread Maxime Ripard
Hi, Sorry, my previous review didn't go through. On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean Nyekjaer wrote: > When using the DSI interface via DSI2LVDS bridge, it seems a bit harsh > to reguire the requested and the actual px clock to be within > 50Hz. A typical LVDS display requires the px

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-04-23 Thread Robert Foss
On Mon, Apr 22, 2024 at 4:06 PM Sean Nyekjaer wrote: > > On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean Nyekjaer wrote: > > When using the DSI interface via DSI2LVDS bridge, it seems a bit harsh > > to reguire the requested and the actual px clock to be within > > 50Hz. A typical LVDS display

Re: [PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-04-22 Thread Sean Nyekjaer
On Fri, Mar 22, 2024 at 11:47:31AM +0100, Sean Nyekjaer wrote: > When using the DSI interface via DSI2LVDS bridge, it seems a bit harsh > to reguire the requested and the actual px clock to be within > 50Hz. A typical LVDS display requires the px clock to be within +-10%. > > In case for HDMI .5%

[PATCH] drm/stm: dsi: relax mode_valid clock tolerance

2024-03-22 Thread Sean Nyekjaer
When using the DSI interface via DSI2LVDS bridge, it seems a bit harsh to reguire the requested and the actual px clock to be within 50Hz. A typical LVDS display requires the px clock to be within +-10%. In case for HDMI .5% tolerance is required. Fixes: e01356d18273 ("drm/stm: dsi: provide the