Aw: Re: Re: Re: [PATCH v5 1/5] dt-bindings: ata: ahci-platform: Convert DT bindings to yaml

Gesendet: Sonntag, 06. März 2022 um 12:15 Uhr
Von: "Krzysztof Kozlowski" [off-list ref]
An: "Frank Wunderlich" [off-list ref]
Cc: "Frank Wunderlich" <redacted>, devicetree@vger.kernel.org, "Damien Le Moal" <redacted>, "Rob Herring" <robh+dt@kernel.org>, "Andrew Lunn" <andrew@lunn.ch>, "Gregory Clement" <gregory.clement@bootlin.com>, "Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com>, "Russell King" <linux@armlinux.org.uk>, "Heiko Stuebner" <heiko@sntech.de>, "Peter Geis" <redacted>, "Michael Riesch" <redacted>, "Hans de Goede" <redacted>, "Jens Axboe" <axboe@kernel.dk>, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org
Betreff: Re: Aw: Re: Re: [PATCH v5 1/5] dt-bindings: ata: ahci-platform: Convert DT bindings to yaml

On 06/03/2022 11:41, Frank Wunderlich wrote:

quoted

quoted

Gesendet: Sonntag, 06. März 2022 um 11:27 Uhr
Von: "Krzysztof Kozlowski" [off-list ref]

quoted

    add compatibles used together with generic-ahci
      - marvell,berlin2-ahci

This is fine, just mention it in commit msg.

quoted

      - qcom,apq8064-ahci
      - qcom,ipq806x-ahci

These you need to consult with qcom-sata.txt. This could be a following
commit which will integrate qcom-sata.txt and remove it.

this depends on Robs opinion

Then maybe precise the question for Rob...

do i need to fix the errors for qcom-compatibles/reg-count/clock-count (reported by your bot) *now*?

or is binding well enough with adding berlin-compatible and fixing the indentation error in example?

for the marvell anyof issue and the spear13xx i have a patch in my tree which i include in v6

quoted

quoted

Either you have
binding document for all devices or you create a common part, like for UFS:
https://lore.kernel.org/linux-devicetree/20220222145854.358646-1-krzysztof.kozlowski@canonical.com/ (local)
https://github.com/krzk/linux/commits/n/dt-bindings-ufs-v2

The choice depends more or less on complexity of bindings, IOW, how big
and complicated bindings would be if you combine everything to one YAML.

In the case of UFS, the devices differ - by clocks, resets, phys and
sometimes supplies. Therefore it easier to have one common shared part
and several device bindings.

AHCI looks more consistent - except that Qualcomm - so maybe better to
have one document.

quoted

    increase reg-count to 2 (used in omap5-l4.dtsi)
    increase clock-count to 5 (used in qcom-apq8064.dtsi)

This would need allOf+if.

if i get ok from rob i add only the berlin-compatible and skip the qcom+reg/clock-change in the first applied version. Adding the allOf/if (and making it right) will only delay the sata-binding/dts-change.

I don't get what is the problem with delaying this patch for the time
needed to make the bindings correct? Especially that alternative is to
add bindings document which soon will need to be modified, e.g. split
into common part. Is there a particular hurry with these bindings
conversion?

i see it as requirement for last part

"arm64: dts: rockchip: Add sata nodes to rk356x"

if this can applied without the bindings conversion there is nothing to hurry :)

regards Frank

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel