dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property
@ 2024-01-31  9:28 Dario Binacchi
  2024-01-31 15:53 ` Conor Dooley
  0 siblings, 1 reply; 5+ messages in thread
From: Dario Binacchi @ 2024-01-31  9:28 UTC (permalink / raw)
  To: linux-kernel
  Cc: Neil Armstrong, Conor Dooley, Krzysztof Kozlowski,
	Thomas Zimmermann, devicetree, Sam Ravnborg, Alexandre Torgue,
	Maxime Ripard, Rob Herring, dri-devel, Daniel Vetter,
	Jessica Zhang, Dario Binacchi

Allow 'port' property (coming from panel-common.yaml) to be used in DTS:

  st/stm32f769-disco-mb1166-reva09.dtb: panel@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>

---

Changes in v2:
- Rework the patch to drop errors found by command
  'make DT_CHECKER_FLAGS=-m dt_binding_check'.

 .../devicetree/bindings/display/panel/novatek,nt35510.yaml       | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
index a4afaff483b7..91921f4b0e5f 100644
--- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
+++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
@@ -31,6 +31,7 @@ properties:
   vddi-supply:
     description: regulator that supplies the vddi voltage
   backlight: true
+  port: true
 
 required:
   - compatible
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property
  2024-01-31  9:28 [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property Dario Binacchi
@ 2024-01-31 15:53 ` Conor Dooley
  2024-02-14  9:46   ` Alexandre TORGUE
  0 siblings, 1 reply; 5+ messages in thread
From: Conor Dooley @ 2024-01-31 15:53 UTC (permalink / raw)
  To: Dario Binacchi
  Cc: Neil Armstrong, Conor Dooley, Krzysztof Kozlowski,
	Thomas Zimmermann, devicetree, Sam Ravnborg, linux-kernel,
	Maxime Ripard, Alexandre Torgue, Rob Herring, dri-devel,
	Daniel Vetter, Jessica Zhang

[-- Attachment #1: Type: text/plain, Size: 1267 bytes --]

On Wed, Jan 31, 2024 at 10:28:44AM +0100, Dario Binacchi wrote:
> Allow 'port' property (coming from panel-common.yaml) to be used in DTS:
> 
>   st/stm32f769-disco-mb1166-reva09.dtb: panel@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>

Acked-by: Conor Dooley <conor.dooley@microchip.com>


> 
> ---
> 
> Changes in v2:
> - Rework the patch to drop errors found by command
>   'make DT_CHECKER_FLAGS=-m dt_binding_check'.
> 
>  .../devicetree/bindings/display/panel/novatek,nt35510.yaml       | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> index a4afaff483b7..91921f4b0e5f 100644
> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> @@ -31,6 +31,7 @@ properties:
>    vddi-supply:
>      description: regulator that supplies the vddi voltage
>    backlight: true
> +  port: true
>  
>  required:
>    - compatible
> -- 
> 2.43.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property
  2024-01-31 15:53 ` Conor Dooley
@ 2024-02-14  9:46   ` Alexandre TORGUE
  2024-02-25  9:01     ` Dario Binacchi
  0 siblings, 1 reply; 5+ messages in thread
From: Alexandre TORGUE @ 2024-02-14  9:46 UTC (permalink / raw)
  To: Conor Dooley, Dario Binacchi, Heiko Stuebner
  Cc: linux-kernel, Conor Dooley, Daniel Vetter, David Airlie,
	Jessica Zhang, Krzysztof Kozlowski, Linus Walleij,
	Maarten Lankhorst, Maxime Ripard, Neil Armstrong, Rob Herring,
	Sam Ravnborg, Thomas Zimmermann, devicetree, dri-devel

Hi Heiko

On 1/31/24 16:53, Conor Dooley wrote:
> On Wed, Jan 31, 2024 at 10:28:44AM +0100, Dario Binacchi wrote:
>> Allow 'port' property (coming from panel-common.yaml) to be used in DTS:
>>
>>    st/stm32f769-disco-mb1166-reva09.dtb: panel@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
>>
>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> 
>>
>> ---
>>
>> Changes in v2:
>> - Rework the patch to drop errors found by command
>>    'make DT_CHECKER_FLAGS=-m dt_binding_check'.
>>
>>   .../devicetree/bindings/display/panel/novatek,nt35510.yaml       | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>> index a4afaff483b7..91921f4b0e5f 100644
>> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>> @@ -31,6 +31,7 @@ properties:
>>     vddi-supply:
>>       description: regulator that supplies the vddi voltage
>>     backlight: true
>> +  port: true
>>   
>>   required:
>>     - compatible
>> -- 
>> 2.43.0
>>

Do you plan to take this patch in drm-misc next branch ? As I have a 
dependency with it to merge a DT patch I can take in my tree 
(stm32-next) if you prefer. Let me know.

Cheers
Alex

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property
  2024-02-14  9:46   ` Alexandre TORGUE
@ 2024-02-25  9:01     ` Dario Binacchi
  2024-02-29  8:54       ` Alexandre TORGUE
  0 siblings, 1 reply; 5+ messages in thread
From: Dario Binacchi @ 2024-02-25  9:01 UTC (permalink / raw)
  To: Alexandre TORGUE, heiko, Linus Walleij, Rob Herring
  Cc: Conor Dooley, linux-kernel, Conor Dooley, Daniel Vetter,
	David Airlie, Jessica Zhang, Krzysztof Kozlowski,
	Maarten Lankhorst, Maxime Ripard, Neil Armstrong, Sam Ravnborg,
	Thomas Zimmermann, devicetree, dri-devel

Hi,

On Wed, Feb 14, 2024 at 10:47 AM Alexandre TORGUE
<alexandre.torgue@foss.st.com> wrote:
>
> Hi Heiko
>
> On 1/31/24 16:53, Conor Dooley wrote:
> > On Wed, Jan 31, 2024 at 10:28:44AM +0100, Dario Binacchi wrote:
> >> Allow 'port' property (coming from panel-common.yaml) to be used in DTS:
> >>
> >>    st/stm32f769-disco-mb1166-reva09.dtb: panel@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
> >>
> >> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> >> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
> >
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> >
> >
> >>
> >> ---
> >>
> >> Changes in v2:
> >> - Rework the patch to drop errors found by command
> >>    'make DT_CHECKER_FLAGS=-m dt_binding_check'.
> >>
> >>   .../devicetree/bindings/display/panel/novatek,nt35510.yaml       | 1 +
> >>   1 file changed, 1 insertion(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> >> index a4afaff483b7..91921f4b0e5f 100644
> >> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> >> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
> >> @@ -31,6 +31,7 @@ properties:
> >>     vddi-supply:
> >>       description: regulator that supplies the vddi voltage
> >>     backlight: true
> >> +  port: true
> >>
> >>   required:
> >>     - compatible
> >> --
> >> 2.43.0
> >>
>
> Do you plan to take this patch in drm-misc next branch ? As I have a
> dependency with it to merge a DT patch I can take in my tree
> (stm32-next) if you prefer. Let me know.
>
> Cheers
> Alex

It's been some weeks, so a gentle ping seems in order :)

Thanks and regards,
Dario

-- 

Dario Binacchi

Senior Embedded Linux Developer

dario.binacchi@amarulasolutions.com

__________________________________


Amarula Solutions SRL

Via Le Canevare 30, 31100 Treviso, Veneto, IT

T. +39 042 243 5310
info@amarulasolutions.com

www.amarulasolutions.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property
  2024-02-25  9:01     ` Dario Binacchi
@ 2024-02-29  8:54       ` Alexandre TORGUE
  0 siblings, 0 replies; 5+ messages in thread
From: Alexandre TORGUE @ 2024-02-29  8:54 UTC (permalink / raw)
  To: Dario Binacchi, heiko, Linus Walleij, Rob Herring
  Cc: Conor Dooley, linux-kernel, Conor Dooley, Daniel Vetter,
	David Airlie, Jessica Zhang, Krzysztof Kozlowski,
	Maarten Lankhorst, Maxime Ripard, Neil Armstrong, Sam Ravnborg,
	Thomas Zimmermann, devicetree, dri-devel

Hi

On 2/25/24 10:01, Dario Binacchi wrote:
> Hi,
> 
> On Wed, Feb 14, 2024 at 10:47 AM Alexandre TORGUE
> <alexandre.torgue@foss.st.com> wrote:
>>
>> Hi Heiko
>>
>> On 1/31/24 16:53, Conor Dooley wrote:
>>> On Wed, Jan 31, 2024 at 10:28:44AM +0100, Dario Binacchi wrote:
>>>> Allow 'port' property (coming from panel-common.yaml) to be used in DTS:
>>>>
>>>>     st/stm32f769-disco-mb1166-reva09.dtb: panel@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
>>>>
>>>> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
>>>> Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>
>>>
>>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>>>
>>>
>>>>
>>>> ---
>>>>
>>>> Changes in v2:
>>>> - Rework the patch to drop errors found by command
>>>>     'make DT_CHECKER_FLAGS=-m dt_binding_check'.
>>>>
>>>>    .../devicetree/bindings/display/panel/novatek,nt35510.yaml       | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>>>> index a4afaff483b7..91921f4b0e5f 100644
>>>> --- a/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>>>> +++ b/Documentation/devicetree/bindings/display/panel/novatek,nt35510.yaml
>>>> @@ -31,6 +31,7 @@ properties:
>>>>      vddi-supply:
>>>>        description: regulator that supplies the vddi voltage
>>>>      backlight: true
>>>> +  port: true
>>>>
>>>>    required:
>>>>      - compatible
>>>> --
>>>> 2.43.0
>>>>
>>
>> Do you plan to take this patch in drm-misc next branch ? As I have a
>> dependency with it to merge a DT patch I can take in my tree
>> (stm32-next) if you prefer. Let me know.
>>
>> Cheers
>> Alex
> 
> It's been some weeks, so a gentle ping seems in order :)

Applied on stm32-next.

Thanks
Alex


> 
> Thanks and regards,
> Dario
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-02-29  8:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-31  9:28 [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property Dario Binacchi
2024-01-31 15:53 ` Conor Dooley
2024-02-14  9:46   ` Alexandre TORGUE
2024-02-25  9:01     ` Dario Binacchi
2024-02-29  8:54       ` Alexandre TORGUE

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).