2024-01-31 09:29:37

by Dario Binacchi

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

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 <[email protected]>
Cc: Alexandre Torgue <[email protected]>

---

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



2024-01-31 15:55:11

by Conor Dooley

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

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 <[email protected]>
> Cc: Alexandre Torgue <[email protected]>

Acked-by: Conor Dooley <[email protected]>


>
> ---
>
> 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
>


Attachments:
(No filename) (1.24 kB)
signature.asc (235.00 B)
Download all attachments

2024-02-14 09:49:37

by Alexandre TORGUE

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

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 <[email protected]>
>> Cc: Alexandre Torgue <[email protected]>
>
> Acked-by: Conor Dooley <[email protected]>
>
>
>>
>> ---
>>
>> 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

2024-02-25 09:02:08

by Dario Binacchi

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

Hi,

On Wed, Feb 14, 2024 at 10:47 AM Alexandre TORGUE
<[email protected]> 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 <[email protected]>
> >> Cc: Alexandre Torgue <[email protected]>
> >
> > Acked-by: Conor Dooley <[email protected]>
> >
> >
> >>
> >> ---
> >>
> >> 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

[email protected]

__________________________________


Amarula Solutions SRL

Via Le Canevare 30, 31100 Treviso, Veneto, IT

T. +39 042 243 5310
[email protected]

http://www.amarulasolutions.com

2024-02-29 09:07:28

by Alexandre TORGUE

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

Hi

On 2/25/24 10:01, Dario Binacchi wrote:
> Hi,
>
> On Wed, Feb 14, 2024 at 10:47 AM Alexandre TORGUE
> <[email protected]> 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 <[email protected]>
>>>> Cc: Alexandre Torgue <[email protected]>
>>>
>>> Acked-by: Conor Dooley <[email protected]>
>>>
>>>
>>>>
>>>> ---
>>>>
>>>> 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
>