Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Maxime Ripard
On Mon, Aug 29, 2022 at 04:14:54PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Mon, Aug 29, 2022 at 3:30 PM Maxime Ripard wrote: > > On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > > > - Speaking of closed captioning... a lot of different stuff were put in > > >

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Geert Uytterhoeven
Hi Maxime, On Mon, Aug 29, 2022 at 3:30 PM Maxime Ripard wrote: > On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > > - Speaking of closed captioning... a lot of different stuff were put in the > > blanking interval over the years. Like teletext in Europe. There are > >

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Maxime Ripard
Hi Mateusz On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > Hi Maxime, > > W dniu 18.08.2022 o 17:56, Geert Uytterhoeven pisze: > > Hi Maxime, > > > > On Thu, Aug 18, 2022 at 5:46 PM Maxime Ripard wrote: > >> On Thu, Aug 18, 2022 at 05:34:30PM +0200, Geert Uytterhoeven

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-24 Thread Geert Uytterhoeven
Hi Maxime, On Wed, Aug 17, 2022 at 9:54 AM Maxime Ripard wrote: > On Tue, Aug 16, 2022 at 05:00:38PM +0200, Geert Uytterhoeven wrote: > > On Tue, Aug 16, 2022 at 3:26 PM Maxime Ripard wrote: > > > On Fri, Aug 12, 2022 at 03:18:58PM +0200, Geert Uytterhoeven wrote: > > > > On Fri, Jul 29, 2022

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-24 Thread Mateusz Kwiatkowski
Hi Maxime, W dniu 18.08.2022 o 17:56, Geert Uytterhoeven pisze: > Hi Maxime, > > On Thu, Aug 18, 2022 at 5:46 PM Maxime Ripard wrote: >> On Thu, Aug 18, 2022 at 05:34:30PM +0200, Geert Uytterhoeven wrote: >>> On Thu, Aug 18, 2022 at 3:42 PM Maxime Ripard wrote: I started adding more sanity

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Geert Uytterhoeven
Hi Maxime, On Thu, Aug 18, 2022 at 5:46 PM Maxime Ripard wrote: > On Thu, Aug 18, 2022 at 05:34:30PM +0200, Geert Uytterhoeven wrote: > > On Thu, Aug 18, 2022 at 3:42 PM Maxime Ripard wrote: > > > I started adding more sanity checks to my code, and I just realised I > > > don't seem to be able

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Maxime Ripard
On Thu, Aug 18, 2022 at 05:34:30PM +0200, Geert Uytterhoeven wrote: > On Thu, Aug 18, 2022 at 3:42 PM Maxime Ripard wrote: > > On Thu, Aug 18, 2022 at 02:57:55PM +0200, Geert Uytterhoeven wrote: > > > On Thu, Aug 18, 2022 at 2:39 PM Maxime Ripard wrote: > > > > On Wed, Aug 17, 2022 at 04:01:48PM

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Geert Uytterhoeven
Hi Maxime, On Thu, Aug 18, 2022 at 3:42 PM Maxime Ripard wrote: > On Thu, Aug 18, 2022 at 02:57:55PM +0200, Geert Uytterhoeven wrote: > > On Thu, Aug 18, 2022 at 2:39 PM Maxime Ripard wrote: > > > On Wed, Aug 17, 2022 at 04:01:48PM +0200, Geert Uytterhoeven wrote: > > > > > I've looked around

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Maxime Ripard
On Thu, Aug 18, 2022 at 02:57:55PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Thu, Aug 18, 2022 at 2:39 PM Maxime Ripard wrote: > > On Wed, Aug 17, 2022 at 04:01:48PM +0200, Geert Uytterhoeven wrote: > > > > I've looked around and it looks like the entire blanking area is > > > >

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Geert Uytterhoeven
Hi Maxime, On Thu, Aug 18, 2022 at 2:39 PM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 04:01:48PM +0200, Geert Uytterhoeven wrote: > > > I've looked around and it looks like the entire blanking area is > > > supposed to be 40 pixels in interlaced, but I couldn't find anywhere how > > > > 625

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-18 Thread Maxime Ripard
Hi! On Wed, Aug 17, 2022 at 04:01:48PM +0200, Geert Uytterhoeven wrote: > > > > > Vertically, it's simpler, as the number of lines is discrete. > > > > > You do have to take into account interlace and doublescan, and > > > > > progressive modes with 262/312 lines. > > > > > > > > So we only have

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-17 Thread Geert Uytterhoeven
Hi Maxime, On Wed, Aug 17, 2022 at 3:15 PM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 10:51:55AM +0200, Geert Uytterhoeven wrote: > > On Wed, Aug 17, 2022 at 9:54 AM Maxime Ripard wrote: > > > On Tue, Aug 16, 2022 at 05:00:38PM +0200, Geert Uytterhoeven wrote: > > > > On Tue, Aug 16, 2022

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-17 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 10:51:55AM +0200, Geert Uytterhoeven wrote: > On Wed, Aug 17, 2022 at 9:54 AM Maxime Ripard wrote: > > On Tue, Aug 16, 2022 at 05:00:38PM +0200, Geert Uytterhoeven wrote: > > > On Tue, Aug 16, 2022 at 3:26 PM Maxime Ripard wrote: > > > > On Fri, Aug 12, 2022 at 03:18:58PM

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-17 Thread Maxime Ripard
On Tue, Aug 16, 2022 at 05:00:38PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Tue, Aug 16, 2022 at 3:26 PM Maxime Ripard wrote: > > On Fri, Aug 12, 2022 at 03:18:58PM +0200, Geert Uytterhoeven wrote: > > > On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > > > > Multiple drivers

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-16 Thread Geert Uytterhoeven
Hi Maxime, On Tue, Aug 16, 2022 at 3:26 PM Maxime Ripard wrote: > On Fri, Aug 12, 2022 at 03:18:58PM +0200, Geert Uytterhoeven wrote: > > On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > > > Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines > > > modes in the

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-16 Thread Maxime Ripard
Hi Geert, On Fri, Aug 12, 2022 at 03:18:58PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > Thanks for your patch! > > On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > > Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines > > modes in the drivers. > > Nit:

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-16 Thread Maxime Ripard
On Tue, Aug 02, 2022 at 04:16:29PM +0200, Thomas Zimmermann wrote: > Hi > > Am 02.08.22 um 15:58 schrieb Jani Nikula: > > On Fri, 29 Jul 2022, Maxime Ripard wrote: > > > Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines > > > modes in the drivers. > > > > > > Since

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-12 Thread Geert Uytterhoeven
Hi Maxime, Thanks for your patch! On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines > modes in the drivers. Nit: strictly speaking these are not analog modes, but the digital variants (ITU-R BT.656 and DVD-Video

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-02 Thread Thomas Zimmermann
Hi Am 02.08.22 um 15:58 schrieb Jani Nikula: On Fri, 29 Jul 2022, Maxime Ripard wrote: Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines modes in the drivers. Since those modes are fairly standards, and that we'll need to use them in more places in the future, let's

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-02 Thread Jani Nikula
On Fri, 29 Jul 2022, Maxime Ripard wrote: > Multiple drivers (meson, vc4) define the analog TV 525-lines and 625-lines > modes in the drivers. > > Since those modes are fairly standards, and that we'll need to use them in > more places in the future, let's move the meson definition into the >