Re: [PATCH v1 07/35] drm/modes: Only consider bpp and refresh before options

2022-08-16 Thread Maxime Ripard
Hi Geert, Thanks for your review On Fri, Aug 12, 2022 at 03:25:39PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > > Some video= options might have a value that contains a dash. However, the > > command line parsing mode considers all

Re: [PATCH v1 07/35] drm/modes: Only consider bpp and refresh before options

2022-08-12 Thread Geert Uytterhoeven
Hi Maxime, On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > Some video= options might have a value that contains a dash. However, the > command line parsing mode considers all dashes as the separator between the > mode and the bpp count. > > Let's rework the parsing code a bit to only

[PATCH v1 07/35] drm/modes: Only consider bpp and refresh before options

2022-07-29 Thread Maxime Ripard
Some video= options might have a value that contains a dash. However, the command line parsing mode considers all dashes as the separator between the mode and the bpp count. Let's rework the parsing code a bit to only consider a dash as the bpp separator if it before a comma, the options