The example node in the binding uses the AST2500 compatible string for the SD controller with a 64kiB ranges property, but the SD controller is allocated 128kiB of MMIO space according to the AST2500 datasheet. Fix the example to correctly reflect the hardware in the AST2500, however it should be noted that the MMIO region is reduced to 64kiB in the AST2600 where a second SD controller block has been introduced into the address space. Also add the IBM copyright header that I left out of the initial patch. Suggested-by: Joel Stanley <joel@xxxxxxxxx> Signed-off-by: Andrew Jeffery <andrew@xxxxxxxx> --- Hi Ulf, this is the follow-up fix as discussed here: http://patchwork.ozlabs.org/patch/1143090/ Documentation/devicetree/bindings/mmc/aspeed,sdhci.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mmc/aspeed,sdhci.yaml b/Documentation/devicetree/bindings/mmc/aspeed,sdhci.yaml index 570f8c72662b..200de9396036 100644 --- a/Documentation/devicetree/bindings/mmc/aspeed,sdhci.yaml +++ b/Documentation/devicetree/bindings/mmc/aspeed,sdhci.yaml @@ -1,4 +1,5 @@ # SPDX-License-Identifier: GPL-2.0-or-later +# Copyright 2019 IBM Corp. %YAML 1.2 --- $id: http://devicetree.org/schemas/mmc/aspeed,sdhci.yaml# @@ -84,7 +85,7 @@ examples: reg = <0x1e740000 0x100>; #address-cells = <1>; #size-cells = <1>; - ranges = <0 0x1e740000 0x10000>; + ranges = <0 0x1e740000 0x20000>; clocks = <&syscon ASPEED_CLK_GATE_SDCLK>; sdhci0: sdhci@100 { -- 2.20.1