Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-29 Thread Maxime Ripard
On Thu, Aug 25, 2022 at 05:13:29PM +0200, Noralf Trønnes wrote: > > > Den 25.08.2022 15.44, skrev Maxime Ripard: > > Hi, > > > > On Sat, Aug 20, 2022 at 10:12:46PM +0200, Noralf Trønnes wrote: > >>> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > >>> index 1e9996b33cc8..

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-26 Thread Maxime Ripard
On Sun, Aug 21, 2022 at 01:43:40PM +0200, Noralf Trønnes wrote: > > > Den 18.08.2022 17.31, skrev Maxime Ripard: > > On Thu, Aug 18, 2022 at 05:01:38PM +0200, Noralf Trønnes wrote: > >> > >> > >> Den 18.08.2022 01.23, skrev Noralf Trønnes: > >>> > >>> > >>> Den 17.08.2022 15.11, skrev Noralf Trøn

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-25 Thread Noralf Trønnes
Den 25.08.2022 15.44, skrev Maxime Ripard: > Hi, > > On Sat, Aug 20, 2022 at 10:12:46PM +0200, Noralf Trønnes wrote: >>> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h >>> index 1e9996b33cc8..78275e68ff66 100644 >>> --- a/include/drm/drm_connector.h >>> +++ b/include/drm

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-25 Thread Maxime Ripard
Hi, On Sat, Aug 20, 2022 at 10:12:46PM +0200, Noralf Trønnes wrote: > > diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > > index 1e9996b33cc8..78275e68ff66 100644 > > --- a/include/drm/drm_connector.h > > +++ b/include/drm/drm_connector.h > > @@ -143,6 +143,32 @@ enum subpi

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-25 Thread Maxime Ripard
Hi, On Fri, Aug 19, 2022 at 11:35:42AM +0200, Geert Uytterhoeven wrote: > On Thu, Aug 18, 2022 at 5:34 PM Maxime Ripard wrote: > > On Thu, Aug 18, 2022 at 05:20:42PM +0200, Geert Uytterhoeven wrote: > > > On Thu, Aug 18, 2022 at 4:54 PM Maxime Ripard wrote: > > > > On Wed, Aug 17, 2022 at 04:04:

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-24 Thread Maxime Ripard
On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrote: > Den 16.08.2022 11.49, skrev Maxime Ripard: > > On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: > >> Den 16.08.2022 10.26, skrev Maxime Ripard: > >>> Hi, > >>> > >>> On Mon, Aug 08, 2022 at 02:44:56PM +0200, Noralf Trø

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-24 Thread Noralf Trønnes
Den 16.08.2022 10.26, skrev Maxime Ripard: > Hi, > > On Mon, Aug 08, 2022 at 02:44:56PM +0200, Noralf Trønnes wrote: >> Den 29.07.2022 18.34, skrev Maxime Ripard: >>> The TV mode property has been around for a while now to select and get the >>> current TV mode output on an analog TV connector.

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-21 Thread Noralf Trønnes
Den 18.08.2022 17.31, skrev Maxime Ripard: > On Thu, Aug 18, 2022 at 05:01:38PM +0200, Noralf Trønnes wrote: >> >> >> Den 18.08.2022 01.23, skrev Noralf Trønnes: >>> >>> >>> Den 17.08.2022 15.11, skrev Noralf Trønnes: Den 17.08.2022 13.46, skrev Maxime Ripard: > On Tue, Aug 16

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-20 Thread Noralf Trønnes
Den 29.07.2022 18.34, skrev Maxime Ripard: > The TV mode property has been around for a while now to select and get the > current TV mode output on an analog TV connector. > > Despite that property name being generic, its content isn't and has been > driver-specific which makes it hard to build

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-19 Thread Geert Uytterhoeven
Hi Maxime, On Thu, Aug 18, 2022 at 5:34 PM Maxime Ripard wrote: > On Thu, Aug 18, 2022 at 05:20:42PM +0200, Geert Uytterhoeven wrote: > > On Thu, Aug 18, 2022 at 4:54 PM Maxime Ripard wrote: > > > On Wed, Aug 17, 2022 at 04:04:24PM +0200, Geert Uytterhoeven wrote: > > > > On Wed, Aug 17, 2022 at

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Maxime Ripard
On Thu, Aug 18, 2022 at 05:20:42PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Thu, Aug 18, 2022 at 4:54 PM Maxime Ripard wrote: > > On Wed, Aug 17, 2022 at 04:04:24PM +0200, Geert Uytterhoeven wrote: > > > On Wed, Aug 17, 2022 at 3:19 PM Maxime Ripard wrote: > > > > On Wed, Aug 17, 202

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Maxime Ripard
On Thu, Aug 18, 2022 at 05:01:38PM +0200, Noralf Trønnes wrote: > > > Den 18.08.2022 01.23, skrev Noralf Trønnes: > > > > > > Den 17.08.2022 15.11, skrev Noralf Trønnes: > >> > >> > >> Den 17.08.2022 13.46, skrev Maxime Ripard: > >>> On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrot

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Geert Uytterhoeven
Hi Maxime, On Thu, Aug 18, 2022 at 4:54 PM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 04:04:24PM +0200, Geert Uytterhoeven wrote: > > On Wed, Aug 17, 2022 at 3:19 PM Maxime Ripard wrote: > > > On Wed, Aug 17, 2022 at 03:05:52PM +0200, Geert Uytterhoeven wrote: > > > > On Wed, Aug 17, 2022 at

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 03:11:56PM +0200, Noralf Trønnes wrote: > Den 17.08.2022 13.46, skrev Maxime Ripard: > > On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrote: > >> Den 16.08.2022 11.49, skrev Maxime Ripard: > >>> On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: > >>

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Noralf Trønnes
Den 18.08.2022 01.23, skrev Noralf Trønnes: > > > Den 17.08.2022 15.11, skrev Noralf Trønnes: >> >> >> Den 17.08.2022 13.46, skrev Maxime Ripard: >>> On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrote: Den 16.08.2022 11.49, skrev Maxime Ripard: > On Tue, Aug 16, 2022 at 11

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-18 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 04:04:24PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Wed, Aug 17, 2022 at 3:19 PM Maxime Ripard wrote: > > On Wed, Aug 17, 2022 at 03:05:52PM +0200, Geert Uytterhoeven wrote: > > > On Wed, Aug 17, 2022 at 1:15 PM Maxime Ripard wrote: > > > > On Wed, Aug 17, 202

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Noralf Trønnes
Den 17.08.2022 15.11, skrev Noralf Trønnes: > > > Den 17.08.2022 13.46, skrev Maxime Ripard: >> On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrote: >>> Den 16.08.2022 11.49, skrev Maxime Ripard: On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: > Den 16.08.20

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Geert Uytterhoeven
Hi Maxime, On Wed, Aug 17, 2022 at 3:19 PM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 03:05:52PM +0200, Geert Uytterhoeven wrote: > > On Wed, Aug 17, 2022 at 1:15 PM Maxime Ripard wrote: > > > On Wed, Aug 17, 2022 at 10:35:07AM +0200, Geert Uytterhoeven wrote: > > > > On Wed, Aug 17, 2022 at

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 03:05:52PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Wed, Aug 17, 2022 at 1:15 PM Maxime Ripard wrote: > > On Wed, Aug 17, 2022 at 10:35:07AM +0200, Geert Uytterhoeven wrote: > > > On Wed, Aug 17, 2022 at 9:47 AM Maxime Ripard wrote: > > > > On Wed, Aug 17, 202

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Noralf Trønnes
Den 17.08.2022 13.46, skrev Maxime Ripard: > On Tue, Aug 16, 2022 at 09:35:24PM +0200, Noralf Trønnes wrote: >> Den 16.08.2022 11.49, skrev Maxime Ripard: >>> On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: Den 16.08.2022 10.26, skrev Maxime Ripard: > Hi, > > On

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Geert Uytterhoeven
Hi Maxime, On Wed, Aug 17, 2022 at 1:15 PM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 10:35:07AM +0200, Geert Uytterhoeven wrote: > > On Wed, Aug 17, 2022 at 9:47 AM Maxime Ripard wrote: > > > On Wed, Aug 17, 2022 at 09:31:18AM +0200, Geert Uytterhoeven wrote: > > > > On Tue, Aug 16, 2022 at

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 10:35:07AM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Wed, Aug 17, 2022 at 9:47 AM Maxime Ripard wrote: > > On Wed, Aug 17, 2022 at 09:31:18AM +0200, Geert Uytterhoeven wrote: > > > On Tue, Aug 16, 2022 at 5:50 PM Maxime Ripard wrote: > > > > On Tue, Aug 16, 202

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Geert Uytterhoeven
Hi Maxime, On Wed, Aug 17, 2022 at 9:47 AM Maxime Ripard wrote: > On Wed, Aug 17, 2022 at 09:31:18AM +0200, Geert Uytterhoeven wrote: > > On Tue, Aug 16, 2022 at 5:50 PM Maxime Ripard wrote: > > > On Tue, Aug 16, 2022 at 04:43:44PM +0200, Geert Uytterhoeven wrote: > > > > > > > > Either you have

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Maxime Ripard
On Wed, Aug 17, 2022 at 09:31:18AM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Tue, Aug 16, 2022 at 5:50 PM Maxime Ripard wrote: > > On Tue, Aug 16, 2022 at 04:43:44PM +0200, Geert Uytterhoeven wrote: > > > > > > > Either you have to add them here (e.g. "hd720p50" and > > > > > > > "hd7

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Geert Uytterhoeven
Hi Maxime, On Tue, Aug 16, 2022 at 5:50 PM Maxime Ripard wrote: > On Tue, Aug 16, 2022 at 04:43:44PM +0200, Geert Uytterhoeven wrote: > > > > > > Either you have to add them here (e.g. "hd720p50" and "hd720p60"), > > > > > > or > > > > > > handle them through "@". The latter would impact "[PATC

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-17 Thread Geert Uytterhoeven
On Wed, Aug 17, 2022 at 9:31 AM Geert Uytterhoeven wrote: > On Tue, Aug 16, 2022 at 5:50 PM Maxime Ripard wrote: > > On Tue, Aug 16, 2022 at 04:43:44PM +0200, Geert Uytterhoeven wrote: > > > > > > > Either you have to add them here (e.g. "hd720p50" and > > > > > > > "hd720p60"), or > > > > > > >

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Noralf Trønnes
Den 16.08.2022 11.49, skrev Maxime Ripard: > On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: >> >> >> Den 16.08.2022 10.26, skrev Maxime Ripard: >>> Hi, >>> >>> On Mon, Aug 08, 2022 at 02:44:56PM +0200, Noralf Trønnes wrote: Den 29.07.2022 18.34, skrev Maxime Ripard: > T

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Maxime Ripard
On Tue, Aug 16, 2022 at 04:43:44PM +0200, Geert Uytterhoeven wrote: > > > > > Either you have to add them here (e.g. "hd720p50" and "hd720p60"), or > > > > > handle them through "@". The latter would impact "[PATCH v1 > > > > > 09/35] drm/modes: Move named modes parsing to a separate function", as

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Geert Uytterhoeven
Hi Maxime, On Tue, Aug 16, 2022 at 4:11 PM Maxime Ripard wrote: > On Tue, Aug 16, 2022 at 03:29:07PM +0200, Geert Uytterhoeven wrote: > > On Tue, Aug 16, 2022 at 3:20 PM Maxime Ripard wrote: > > > On Fri, Aug 12, 2022 at 03:25:18PM +0200, Geert Uytterhoeven wrote: > > > > > --- a/drivers/gpu/drm

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Maxime Ripard
On Tue, Aug 16, 2022 at 03:29:07PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Tue, Aug 16, 2022 at 3:20 PM Maxime Ripard wrote: > > On Fri, Aug 12, 2022 at 03:25:18PM +0200, Geert Uytterhoeven wrote: > > > > --- a/drivers/gpu/drm/drm_connector.c > > > > +++ b/drivers/gpu/drm/drm_connect

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Geert Uytterhoeven
Hi Maxime, On Tue, Aug 16, 2022 at 3:20 PM Maxime Ripard wrote: > On Fri, Aug 12, 2022 at 03:25:18PM +0200, Geert Uytterhoeven wrote: > > > --- a/drivers/gpu/drm/drm_connector.c > > > +++ b/drivers/gpu/drm/drm_connector.c > > > @@ -1649,11 +1650,40 @@ > > > EXPORT_SYMBOL(drm_mode_create_tv_margi

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Maxime Ripard
Hi Geert, On Fri, Aug 12, 2022 at 03:25:18PM +0200, Geert Uytterhoeven wrote: > > --- a/drivers/gpu/drm/drm_connector.c > > +++ b/drivers/gpu/drm/drm_connector.c > > @@ -1649,11 +1650,40 @@ EXPORT_SYMBOL(drm_mode_create_tv_margin_properties); > > * 0 on success or a negative error code on failur

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Maxime Ripard
On Tue, Aug 16, 2022 at 11:42:20AM +0200, Noralf Trønnes wrote: > > > Den 16.08.2022 10.26, skrev Maxime Ripard: > > Hi, > > > > On Mon, Aug 08, 2022 at 02:44:56PM +0200, Noralf Trønnes wrote: > >> Den 29.07.2022 18.34, skrev Maxime Ripard: > >>> The TV mode property has been around for a while

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-16 Thread Maxime Ripard
Hi, On Mon, Aug 08, 2022 at 02:44:56PM +0200, Noralf Trønnes wrote: > Den 29.07.2022 18.34, skrev Maxime Ripard: > > The TV mode property has been around for a while now to select and get the > > current TV mode output on an analog TV connector. > > > > Despite that property name being generic, i

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-12 Thread Geert Uytterhoeven
Hi Maxime, On Fri, Jul 29, 2022 at 6:35 PM Maxime Ripard wrote: > The TV mode property has been around for a while now to select and get the > current TV mode output on an analog TV connector. > > Despite that property name being generic, its content isn't and has been > driver-specific which mak

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-08 Thread Noralf Trønnes
Den 29.07.2022 18.34, skrev Maxime Ripard: > The TV mode property has been around for a while now to select and get the > current TV mode output on an analog TV connector. > > Despite that property name being generic, its content isn't and has been > driver-specific which makes it hard to build

[PATCH v1 05/35] drm/connector: Add TV standard property

2022-07-29 Thread Maxime Ripard
The TV mode property has been around for a while now to select and get the current TV mode output on an analog TV connector. Despite that property name being generic, its content isn't and has been driver-specific which makes it hard to build any generic behaviour on top of it, both in kernel and