Re: [PATCH 1/3] dt-bindings: pinctrl: Add rk3576 dw-mshc bindings
From: Krzysztof Kozlowski <krzk@kernel.org>
Date: 2024-08-04 09:55:42
Also in:
linux-devicetree, linux-mmc, linux-rockchip, lkml
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
quoted hunk ↗ jump to hunk
--- .../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