devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Proper ranges for memory controller (exynos5410-smdk5410)
@ 2020-08-30 14:52 Krzysztof Kozlowski
  0 siblings, 0 replies; only message in thread
From: Krzysztof Kozlowski @ 2020-08-30 14:52 UTC (permalink / raw)
  To: Rob Herring, linux-samsung-soc, linux-arm-kernel, devicetree

Hi Rob,

Maybe you could help me here as I am trying to understand where is the
error (cross compiling on ARM):

    $ make exynos_defconfig
    $ make dtbs_check

    arch/arm/boot/dts/exynos5410-smdk5410.dt.yaml: soc: memory-controller@12250000:ranges: [[0, 0, 67108864, 131072, 1, 0, 83886080, 131072, 2, 0, 100663296, 131072, 3, 0, 117440512, 131072]] is not valid under any of the given schemas (Possible causes of the failure):
    arch/arm/boot/dts/exynos5410-smdk5410.dt.yaml: soc: memory-controller@12250000:ranges: [[0, 0, 67108864, 131072, 1, 0, 83886080, 131072, 2, 0, 100663296, 131072, 3, 0, 117440512, 131072]] is not of type 'boolean'
    arch/arm/boot/dts/exynos5410-smdk5410.dt.yaml: soc: memory-controller@12250000:ranges:0: [0, 0, 67108864, 131072, 1, 0, 83886080, 131072, 2, 0, 100663296, 131072, 3, 0, 117440512, 131072] is too long
    From schema: local/lib/python3.6/site-packages/dtschema/schemas/simple-bus.yaml

This is a Exynos SROM memory controller with four banks like this:

arch/arm/boot/dts/exynos5410.dtsi
    376 &sromc {
    377         #address-cells = <2>;
    378         #size-cells = <1>;
    379         ranges = <0 0 0x04000000 0x20000
    380                   1 0 0x05000000 0x20000
    381                   2 0 0x06000000 0x20000
    382                   3 0 0x07000000 0x20000>;
    383 };

Converting it to:
    376 &sromc {
    377         #address-cells = <2>;
    378         #size-cells = <1>;
    379         ranges = <0 0 0x04000000 0x20000>,
    380                  <1 0 0x05000000 0x20000>,
    381                  <2 0 0x06000000 0x20000>,
    382                  <3 0 0x07000000 0x20000>;
    383 };
silences the error although I cannot check whether this even works
(I don't have this HW).  Is it because of simple-bus schema which
expects an array of arrays of 3 to 7 size?

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-08-30 14:52 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-08-30 14:52 Proper ranges for memory controller (exynos5410-smdk5410) Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).