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 >

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

2022-07-29 Thread Maxime Ripard
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 framework. The meson one was chosen because vc4's isn't