[v2] dt-bindings: nt35510: document 'port' property

[drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property

Posted by Dario Binacchi 1 year, 10 months ago

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

Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property

Posted by Conor Dooley 1 year, 10 months ago

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
>

Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property

Posted by Alexandre TORGUE 1 year, 10 months ago

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

Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property

Posted by Dario Binacchi 1 year, 10 months ago

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

Re: [drm-drm-misc:drm-misc-next v2] dt-bindings: nt35510: document 'port' property

Posted by Alexandre TORGUE 1 year, 9 months ago

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
>