Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

On 25.01.2018 16:55, Philippe Cornu wrote: > The "adjusted_mode" clock value (ie the real pixel clock) is more > accurate than "mode" clock value (ie the panel/bridge requested > clock value). It offers a better preciseness for timing > computations and allows to reduce the extra dsi bandwidth in >

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Philippe, I've tried your patch on both 4.17-rc1 & drm-misc-next and it works fine. So, Reviewed-by: Yannick Fertré Tested-by: Yannick Fertré BR Yannick Fertré On 04/23/2018 12:10 PM, Laurent Pinchart wrote: > Hi Philippe, > > On Thursday, 8 February 2018 15:15:40 EEST Laurent Pinchart wr

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Philippe, On Thursday, 8 February 2018 15:15:40 EEST Laurent Pinchart wrote: > On Saturday, 3 February 2018 00:41:26 EET Philippe CORNU wrote: > > On 01/29/2018 11:40 AM, Laurent Pinchart wrote: > > > On Monday, 29 January 2018 12:17:37 EET Philippe CORNU wrote: > > >> On 01/29/2018 10:46 AM, L

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Philippe, On Saturday, 3 February 2018 00:41:26 EET Philippe CORNU wrote: > On 01/29/2018 11:40 AM, Laurent Pinchart wrote: > > On Monday, 29 January 2018 12:17:37 EET Philippe CORNU wrote: > >> On 01/29/2018 10:46 AM, Laurent Pinchart wrote: > >>> On Thursday, 25 January 2018 17:55:04 EET Phil

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Laurent & Daniel :-) On 01/29/2018 11:40 AM, Laurent Pinchart wrote: > Hi Philippe, > > On Monday, 29 January 2018 12:17:37 EET Philippe CORNU wrote: >> On 01/29/2018 10:46 AM, Laurent Pinchart wrote: >>> On Thursday, 25 January 2018 17:55:04 EET Philippe Cornu wrote: >>> The "adjusted_mo

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Philippe, On Monday, 29 January 2018 12:17:37 EET Philippe CORNU wrote: > On 01/29/2018 10:46 AM, Laurent Pinchart wrote: > > On Thursday, 25 January 2018 17:55:04 EET Philippe Cornu wrote: > > > >> The "adjusted_mode" clock value (ie the real pixel clock) is more > >> accurate than "mode" clo

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Laurent, And many thanks for your comments :) On 01/29/2018 10:46 AM, Laurent Pinchart wrote: > Hi Philippe, > > (CC'ing Daniel Vetter) > > Thank you for the patch. > > On Thursday, 25 January 2018 17:55:04 EET Philippe Cornu wrote: >> The "adjusted_mode" clock value (ie the real pixel clo

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Philippe, (CC'ing Daniel Vetter) Thank you for the patch. On Thursday, 25 January 2018 17:55:04 EET Philippe Cornu wrote: > The "adjusted_mode" clock value (ie the real pixel clock) is more > accurate than "mode" clock value (ie the panel/bridge requested > clock value). It offers a better pr

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

On 01/26/2018 03:24 PM, Philippe CORNU wrote: Hi Brian, And a big thanks for your Tested-by On 01/25/2018 11:47 PM, Brian Norris wrote: On Thu, Jan 25, 2018 at 7:55 AM, Philippe Cornu wrote: The "adjusted_mode" clock value (ie the real pixel clock) is more accurate than "mode" clock value (

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

Hi Brian, And a big thanks for your Tested-by On 01/25/2018 11:47 PM, Brian Norris wrote: > On Thu, Jan 25, 2018 at 7:55 AM, Philippe Cornu wrote: >> The "adjusted_mode" clock value (ie the real pixel clock) is more >> accurate than "mode" clock value (ie the panel/bridge requested >> clock value

Re: [PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

On Thu, Jan 25, 2018 at 7:55 AM, Philippe Cornu wrote: > The "adjusted_mode" clock value (ie the real pixel clock) is more > accurate than "mode" clock value (ie the panel/bridge requested > clock value). It offers a better preciseness for timing > computations and allows to reduce the extra dsi b

[PATCH] drm/bridge/synopsys: dsi: use adjusted_mode in mode_set

The "adjusted_mode" clock value (ie the real pixel clock) is more accurate than "mode" clock value (ie the panel/bridge requested clock value). It offers a better preciseness for timing computations and allows to reduce the extra dsi bandwidth in burst mode (from ~20% to ~10-12%, hw platform depend