lkml.org 
[lkml]   [2019]   [Jan]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 5/7] riscv: dts: add initial support for the SiFive FU540-C000 SoC
    On Fri, Dec 14, 2018 at 09:21:52PM -0800, Paul Walmsley wrote:
    > Add initial support for the SiFive FU540-C000 SoC. This is a 28nm SoC
    > based around the SiFive U54-MC core complex and a TileLink
    > interconnect.
    >
    > This file is expected to grow considerably as more device drivers are
    > added to the kernel.
    >
    > Cc: Rob Herring <robh+dt@kernel.org>
    > Cc: Mark Rutland <mark.rutland@arm.com>
    > Cc: Palmer Dabbelt <palmer@sifive.com>
    > Cc: Albert Ou <aou@eecs.berkeley.edu>
    > Cc: devicetree@vger.kernel.org
    > Cc: linux-riscv@lists.infradead.org
    > Cc: linux-kernel@vger.kernel.org
    > Signed-off-by: Paul Walmsley <paul.walmsley@sifive.com>
    > Signed-off-by: Paul Walmsley <paul@pwsan.com>
    > ---
    > arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 182 +++++++++++++++++++++
    > 1 file changed, 182 insertions(+)
    > create mode 100644 arch/riscv/boot/dts/sifive/fu540-c000.dtsi
    >
    > diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi
    > new file mode 100644
    > index 000000000000..0ef314cf17b6
    > --- /dev/null
    > +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi

    > + cpus {
    > + #address-cells = <1>;
    > + #size-cells = <0>;
    > + timebase-frequency = <1000000>;
    > + cpu0: cpu@0 {
    > + clock-frequency = <0>;
    > + compatible = "sifive,e51", "sifive,rocket0";

    Looks like you forgot the currently required "riscv" compatible here and
    below.

    But perhaps it's the binding and arch code that should be revised
    instead (e.g. as per your discussion with Rob elsewhere in this thread).

    Johan

    \
     
     \ /
      Last update: 2019-01-21 15:11    [W:2.753 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site