* [PATCH v2] dt-bindings: ata: Add UniPhier controller binding @ 2023-02-13 4:54 Kunihiko Hayashi 2023-02-13 9:10 ` Krzysztof Kozlowski 0 siblings, 1 reply; 7+ messages in thread From: Kunihiko Hayashi @ 2023-02-13 4:54 UTC (permalink / raw) To: Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel, Kunihiko Hayashi Add UniPhier SATA controller compatible string to the platform binding. This controller needs maximum three reset controls. Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> --- .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) Changes since v1: - Restrict resets property changes with compatible strings - Fix maxItems from two to three diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml b/Documentation/devicetree/bindings/ata/ahci-platform.yaml index 7dc2a2e8f598..25dd5ffaa517 100644 --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml @@ -45,6 +45,9 @@ properties: - marvell,armada-8k-ahci - marvell,berlin2-ahci - marvell,berlin2q-ahci + - socionext,uniphier-pro4-ahci + - socionext,uniphier-pxs2-ahci + - socionext,uniphier-pxs3-ahci - const: generic-ahci - enum: - cavium,octeon-7130-ahci @@ -76,6 +79,20 @@ properties: resets: maxItems: 1 +if: + properties: + compatible: + contains: + enum: + - socionext,uniphier-pro4-ahci + - socionext,uniphier-pxs2-ahci + - socionext,uniphier-pxs3-ahci +then: + properties: + resets: + minItems: 1 + maxItems: 3 + patternProperties: "^sata-port@[0-9a-f]+$": $ref: /schemas/ata/ahci-common.yaml#/$defs/ahci-port -- 2.25.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-13 4:54 [PATCH v2] dt-bindings: ata: Add UniPhier controller binding Kunihiko Hayashi @ 2023-02-13 9:10 ` Krzysztof Kozlowski 2023-02-14 9:33 ` Kunihiko Hayashi 0 siblings, 1 reply; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-02-13 9:10 UTC (permalink / raw) To: Kunihiko Hayashi, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel On 13/02/2023 05:54, Kunihiko Hayashi wrote: > Add UniPhier SATA controller compatible string to the platform binding. > This controller needs maximum three reset controls. > > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> > --- > .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > Changes since v1: > - Restrict resets property changes with compatible strings > - Fix maxItems from two to three > > diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml b/Documentation/devicetree/bindings/ata/ahci-platform.yaml > index 7dc2a2e8f598..25dd5ffaa517 100644 > --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml > +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml > @@ -45,6 +45,9 @@ properties: > - marvell,armada-8k-ahci > - marvell,berlin2-ahci > - marvell,berlin2q-ahci > + - socionext,uniphier-pro4-ahci > + - socionext,uniphier-pxs2-ahci > + - socionext,uniphier-pxs3-ahci > - const: generic-ahci > - enum: Top level is saying reset=1, so did you test your bindings? > - cavium,octeon-7130-ahci > @@ -76,6 +79,20 @@ properties: > resets: > maxItems: 1 > > +if: Keep it in allOf like in example-schema. Will save you one re-indent pretty soon... > + properties: > + compatible: > + contains: > + enum: > + - socionext,uniphier-pro4-ahci > + - socionext,uniphier-pxs2-ahci > + - socionext,uniphier-pxs3-ahci > +then: > + properties: > + resets: > + minItems: 1 > + maxItems: 3 And entire allOf goes to the place like in example-schema. > + > patternProperties: > "^sata-port@[0-9a-f]+$": > $ref: /schemas/ata/ahci-common.yaml#/$defs/ahci-port Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-13 9:10 ` Krzysztof Kozlowski @ 2023-02-14 9:33 ` Kunihiko Hayashi 2023-02-14 9:42 ` Krzysztof Kozlowski 0 siblings, 1 reply; 7+ messages in thread From: Kunihiko Hayashi @ 2023-02-14 9:33 UTC (permalink / raw) To: Krzysztof Kozlowski, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel Hi Krzysztof, On 2023/02/13 18:10, Krzysztof Kozlowski wrote: > On 13/02/2023 05:54, Kunihiko Hayashi wrote: >> Add UniPhier SATA controller compatible string to the platform binding. >> This controller needs maximum three reset controls. >> >> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> >> --- >> .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> Changes since v1: >> - Restrict resets property changes with compatible strings >> - Fix maxItems from two to three >> >> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >> b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >> index 7dc2a2e8f598..25dd5ffaa517 100644 >> --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >> @@ -45,6 +45,9 @@ properties: >> - marvell,armada-8k-ahci >> - marvell,berlin2-ahci >> - marvell,berlin2q-ahci >> + - socionext,uniphier-pro4-ahci >> + - socionext,uniphier-pxs2-ahci >> + - socionext,uniphier-pxs3-ahci >> - const: generic-ahci >> - enum: > > Top level is saying reset=1, so did you test your bindings? Umm, I didn't see any errors on dt_binding_check, anyway I'll add initial minItems:1 and maxItems:3 on top level first. > >> - cavium,octeon-7130-ahci >> @@ -76,6 +79,20 @@ properties: >> resets: >> maxItems: 1 >> >> +if: > Keep it in allOf like in example-schema. Will save you one re-indent > pretty soon... I see. I'll move if clause into allOf, and also move the original allOf. >> + properties: >> + compatible: >> + contains: >> + enum: >> + - socionext,uniphier-pro4-ahci >> + - socionext,uniphier-pxs2-ahci >> + - socionext,uniphier-pxs3-ahci >> +then: >> + properties: >> + resets: >> + minItems: 1 >> + maxItems: 3 > > And entire allOf goes to the place like in example-schema. I've got it. and I'll fix it (minItems:2). Thank you, --- Best Regards Kunihiko Hayashi ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-14 9:33 ` Kunihiko Hayashi @ 2023-02-14 9:42 ` Krzysztof Kozlowski 2023-02-16 17:23 ` Kunihiko Hayashi 0 siblings, 1 reply; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-02-14 9:42 UTC (permalink / raw) To: Kunihiko Hayashi, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel On 14/02/2023 10:33, Kunihiko Hayashi wrote: > Hi Krzysztof, > > On 2023/02/13 18:10, Krzysztof Kozlowski wrote: >> On 13/02/2023 05:54, Kunihiko Hayashi wrote: >>> Add UniPhier SATA controller compatible string to the platform binding. >>> This controller needs maximum three reset controls. >>> >>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> >>> --- >>> .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ >>> 1 file changed, 17 insertions(+) >>> >>> Changes since v1: >>> - Restrict resets property changes with compatible strings >>> - Fix maxItems from two to three >>> >>> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>> b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>> index 7dc2a2e8f598..25dd5ffaa517 100644 >>> --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>> @@ -45,6 +45,9 @@ properties: >>> - marvell,armada-8k-ahci >>> - marvell,berlin2-ahci >>> - marvell,berlin2q-ahci >>> + - socionext,uniphier-pro4-ahci >>> + - socionext,uniphier-pxs2-ahci >>> + - socionext,uniphier-pxs3-ahci >>> - const: generic-ahci >>> - enum: >> >> Top level is saying reset=1, so did you test your bindings? > > Umm, I didn't see any errors on dt_binding_check, anyway I'll add > initial minItems:1 and maxItems:3 on top level first. You need to test also all DTS using these bindings. Yours and others. If you tested the DTS (with proper binding, not one which is basically noop): uniphier-pro4-ace.dtb: sata@65600000: resets: [[27, 12], [27, 28], [37, 3]] is too long BTW, the patch has other errors - just look at the beginning of the file. I cannot see it here in the diff, but when you open the file you should notice it. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-14 9:42 ` Krzysztof Kozlowski @ 2023-02-16 17:23 ` Kunihiko Hayashi 2023-02-17 8:42 ` Krzysztof Kozlowski 0 siblings, 1 reply; 7+ messages in thread From: Kunihiko Hayashi @ 2023-02-16 17:23 UTC (permalink / raw) To: Krzysztof Kozlowski, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel On 2023/02/14 18:42, Krzysztof Kozlowski wrote: > On 14/02/2023 10:33, Kunihiko Hayashi wrote: >> Hi Krzysztof, >> >> On 2023/02/13 18:10, Krzysztof Kozlowski wrote: >>> On 13/02/2023 05:54, Kunihiko Hayashi wrote: >>>> Add UniPhier SATA controller compatible string to the platform binding. >>>> This controller needs maximum three reset controls. >>>> >>>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> >>>> --- >>>> .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ >>>> 1 file changed, 17 insertions(+) >>>> >>>> Changes since v1: >>>> - Restrict resets property changes with compatible strings >>>> - Fix maxItems from two to three >>>> >>>> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>> b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>> index 7dc2a2e8f598..25dd5ffaa517 100644 >>>> --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>> @@ -45,6 +45,9 @@ properties: >>>> - marvell,armada-8k-ahci >>>> - marvell,berlin2-ahci >>>> - marvell,berlin2q-ahci >>>> + - socionext,uniphier-pro4-ahci >>>> + - socionext,uniphier-pxs2-ahci >>>> + - socionext,uniphier-pxs3-ahci >>>> - const: generic-ahci >>>> - enum: >>> >>> Top level is saying reset=1, so did you test your bindings? >> >> Umm, I didn't see any errors on dt_binding_check, anyway I'll add >> initial minItems:1 and maxItems:3 on top level first. > > You need to test also all DTS using these bindings. Yours and others. > If you tested the DTS (with proper binding, not one which is basically > noop): > > uniphier-pro4-ace.dtb: sata@65600000: resets: [[27, 12], [27, 28], [37, > 3]] is too long I've tried updating tools and doing dtbs_check, but I couldn't find this error. It seems that this error can't be detected unless there is the specified compatible in "select:". > BTW, the patch has other errors - just look at the beginning of the > file. I cannot see it here in the diff, but when you open the file you > should notice it. Sorry, but I cannot see anything wrong. I'll check the header or something... Thank you, --- Best Regards Kunihiko Hayashi ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-16 17:23 ` Kunihiko Hayashi @ 2023-02-17 8:42 ` Krzysztof Kozlowski 2023-02-17 11:14 ` Kunihiko Hayashi 0 siblings, 1 reply; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-02-17 8:42 UTC (permalink / raw) To: Kunihiko Hayashi, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel On 16/02/2023 18:23, Kunihiko Hayashi wrote: > On 2023/02/14 18:42, Krzysztof Kozlowski wrote: >> On 14/02/2023 10:33, Kunihiko Hayashi wrote: >>> Hi Krzysztof, >>> >>> On 2023/02/13 18:10, Krzysztof Kozlowski wrote: >>>> On 13/02/2023 05:54, Kunihiko Hayashi wrote: >>>>> Add UniPhier SATA controller compatible string to the platform binding. >>>>> This controller needs maximum three reset controls. >>>>> >>>>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> >>>>> --- >>>>> .../devicetree/bindings/ata/ahci-platform.yaml | 17 +++++++++++++++++ >>>>> 1 file changed, 17 insertions(+) >>>>> >>>>> Changes since v1: >>>>> - Restrict resets property changes with compatible strings >>>>> - Fix maxItems from two to three >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>> b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>> index 7dc2a2e8f598..25dd5ffaa517 100644 >>>>> --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>> @@ -45,6 +45,9 @@ properties: >>>>> - marvell,armada-8k-ahci >>>>> - marvell,berlin2-ahci >>>>> - marvell,berlin2q-ahci >>>>> + - socionext,uniphier-pro4-ahci >>>>> + - socionext,uniphier-pxs2-ahci >>>>> + - socionext,uniphier-pxs3-ahci >>>>> - const: generic-ahci >>>>> - enum: >>>> >>>> Top level is saying reset=1, so did you test your bindings? >>> >>> Umm, I didn't see any errors on dt_binding_check, anyway I'll add >>> initial minItems:1 and maxItems:3 on top level first. >> >> You need to test also all DTS using these bindings. Yours and others. >> If you tested the DTS (with proper binding, not one which is basically >> noop): >> >> uniphier-pro4-ace.dtb: sata@65600000: resets: [[27, 12], [27, 28], [37, >> 3]] is too long > > I've tried updating tools and doing dtbs_check, but I couldn't find this > error. It seems that this error can't be detected unless there is the > specified compatible in "select:". > >> BTW, the patch has other errors - just look at the beginning of the >> file. I cannot see it here in the diff, but when you open the file you >> should notice it. > > Sorry, but I cannot see anything wrong. > I'll check the header or something... If you open the file, you will notice the same compatibles in two places. In select and in properties. You added your compatibles only to one place, so not symmetrically. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] dt-bindings: ata: Add UniPhier controller binding 2023-02-17 8:42 ` Krzysztof Kozlowski @ 2023-02-17 11:14 ` Kunihiko Hayashi 0 siblings, 0 replies; 7+ messages in thread From: Kunihiko Hayashi @ 2023-02-17 11:14 UTC (permalink / raw) To: Krzysztof Kozlowski, Damien Le Moal, Rob Herring, Krzysztof Kozlowski, Hans de Goede, Jens Axboe Cc: linux-ide, devicetree, linux-kernel On 2023/02/17 17:42, Krzysztof Kozlowski wrote: > On 16/02/2023 18:23, Kunihiko Hayashi wrote: >> On 2023/02/14 18:42, Krzysztof Kozlowski wrote: >>> On 14/02/2023 10:33, Kunihiko Hayashi wrote: >>>> Hi Krzysztof, >>>> >>>> On 2023/02/13 18:10, Krzysztof Kozlowski wrote: >>>>> On 13/02/2023 05:54, Kunihiko Hayashi wrote: >>>>>> Add UniPhier SATA controller compatible string to the platform >>>>>> binding. >>>>>> This controller needs maximum three reset controls. >>>>>> >>>>>> Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com> >>>>>> --- >>>>>> .../devicetree/bindings/ata/ahci-platform.yaml | 17 >>>>>> +++++++++++++++++ >>>>>> 1 file changed, 17 insertions(+) >>>>>> >>>>>> Changes since v1: >>>>>> - Restrict resets property changes with compatible strings >>>>>> - Fix maxItems from two to three >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>>> b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>>> index 7dc2a2e8f598..25dd5ffaa517 100644 >>>>>> --- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>>> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml >>>>>> @@ -45,6 +45,9 @@ properties: >>>>>> - marvell,armada-8k-ahci >>>>>> - marvell,berlin2-ahci >>>>>> - marvell,berlin2q-ahci >>>>>> + - socionext,uniphier-pro4-ahci >>>>>> + - socionext,uniphier-pxs2-ahci >>>>>> + - socionext,uniphier-pxs3-ahci >>>>>> - const: generic-ahci >>>>>> - enum: >>>>> >>>>> Top level is saying reset=1, so did you test your bindings? >>>> >>>> Umm, I didn't see any errors on dt_binding_check, anyway I'll add >>>> initial minItems:1 and maxItems:3 on top level first. >>> >>> You need to test also all DTS using these bindings. Yours and others. >>> If you tested the DTS (with proper binding, not one which is basically >>> noop): >>> >>> uniphier-pro4-ace.dtb: sata@65600000: resets: [[27, 12], [27, 28], [37, >>> 3]] is too long >> >> I've tried updating tools and doing dtbs_check, but I couldn't find this >> error. It seems that this error can't be detected unless there is the >> specified compatible in "select:". >> >>> BTW, the patch has other errors - just look at the beginning of the >>> file. I cannot see it here in the diff, but when you open the file you >>> should notice it. >> >> Sorry, but I cannot see anything wrong. >> I'll check the header or something... > > If you open the file, you will notice the same compatibles in two > places. In select and in properties. You added your compatibles only to > one place, so not symmetrically. OK, I understand. I'll add them in both. Thank you, --- Best Regards Kunihiko Hayashi ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-02-17 11:15 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-02-13 4:54 [PATCH v2] dt-bindings: ata: Add UniPhier controller binding Kunihiko Hayashi 2023-02-13 9:10 ` Krzysztof Kozlowski 2023-02-14 9:33 ` Kunihiko Hayashi 2023-02-14 9:42 ` Krzysztof Kozlowski 2023-02-16 17:23 ` Kunihiko Hayashi 2023-02-17 8:42 ` Krzysztof Kozlowski 2023-02-17 11:14 ` Kunihiko Hayashi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox