| Message ID | 20240802153609.296197-2-detlev.casanova@collabora.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add dw_mmc support for rk3576 | expand |
> [PATCH 1/3] dt-bindings: pinctrl: Add rk3576 dw-mshc bindings s/pinctrl/mmc On Friday, 2 August 2024 11:31:27 EDT Detlev Casanova wrote: > Add the compatible string for rockchip,rk3576-dw-mshc and add support > for the rockchip,use-v2-tuning flag, a new feature of this core. > > Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> > --- > .../devicetree/bindings/mmc/rockchip-dw-mshc.yaml | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml index > 211cd0b0bc5f3..dd8d1e773bb38 100644 > --- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > +++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > @@ -39,6 +39,7 @@ properties: > - rockchip,rk3368-dw-mshc > - rockchip,rk3399-dw-mshc > - rockchip,rk3568-dw-mshc > + - rockchip,rk3576-dw-mshc > - rockchip,rk3588-dw-mshc > - rockchip,rv1108-dw-mshc > - rockchip,rv1126-dw-mshc > @@ -95,6 +96,16 @@ properties: > If not specified, the host will do tuning for 360 times, > namely tuning for each degree. > > + rockchip,use-v2-tuning: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + If present, use tuning version 2. > + v2 tuning will inherit pre-stage loader's phase settings for the > first + time, and do re-tune if necessary. > + Re-tune will still try the rough degrees, for instance, 90, 180, 270, > + 360 but continue to do the fine tuning if sample window isn't good + > enough. > + > required: > - compatible > - reg
On 02/08/2024 17:31, Detlev Casanova wrote: > Add the compatible string for rockchip,rk3576-dw-mshc and add support > for the rockchip,use-v2-tuning flag, a new feature of this core. > > Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> A nit, subject: drop second/last, redundant "bindings". The "dt-bindings" prefix is already stating that these are bindings. See also: https://elixir.bootlin.com/linux/v6.7-rc8/source/Documentation/devicetree/bindings/submitting-patches.rst#L18 > --- > .../devicetree/bindings/mmc/rockchip-dw-mshc.yaml | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > index 211cd0b0bc5f3..dd8d1e773bb38 100644 > --- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > +++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml > @@ -39,6 +39,7 @@ properties: > - rockchip,rk3368-dw-mshc > - rockchip,rk3399-dw-mshc > - rockchip,rk3568-dw-mshc > + - rockchip,rk3576-dw-mshc > - rockchip,rk3588-dw-mshc > - rockchip,rv1108-dw-mshc > - rockchip,rv1126-dw-mshc > @@ -95,6 +96,16 @@ properties: > If not specified, the host will do tuning for 360 times, > namely tuning for each degree. > > + rockchip,use-v2-tuning: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + If present, use tuning version 2. > + v2 tuning will inherit pre-stage loader's phase settings for the first > + time, and do re-tune if necessary. > + Re-tune will still try the rough degrees, for instance, 90, 180, 270, > + 360 but continue to do the fine tuning if sample window isn't good > + enough. You described the desired Linux feature or behavior, not the actual hardware. The bindings are about the latter, so instead you need to rephrase the property and its description to match actual hardware capabilities/features/configuration etc. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml index 211cd0b0bc5f3..dd8d1e773bb38 100644 --- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml +++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml @@ -39,6 +39,7 @@ properties: - rockchip,rk3368-dw-mshc - rockchip,rk3399-dw-mshc - rockchip,rk3568-dw-mshc + - rockchip,rk3576-dw-mshc - rockchip,rk3588-dw-mshc - rockchip,rv1108-dw-mshc - rockchip,rv1126-dw-mshc @@ -95,6 +96,16 @@ properties: If not specified, the host will do tuning for 360 times, namely tuning for each degree. + rockchip,use-v2-tuning: + $ref: /schemas/types.yaml#/definitions/flag + description: + If present, use tuning version 2. + v2 tuning will inherit pre-stage loader's phase settings for the first + time, and do re-tune if necessary. + Re-tune will still try the rough degrees, for instance, 90, 180, 270, + 360 but continue to do the fine tuning if sample window isn't good + enough. + required: - compatible - reg
Add the compatible string for rockchip,rk3576-dw-mshc and add support for the rockchip,use-v2-tuning flag, a new feature of this core. Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com> --- .../devicetree/bindings/mmc/rockchip-dw-mshc.yaml | 11 +++++++++++ 1 file changed, 11 insertions(+)