[PATCH v3 05/12] platform: generic: Add SiFive development platform

[Nick Hu]

On Mon, Aug 4, 2025 at 4:37 PM Anup Patel <anup at brainfault.org> wrote:
>
> On Tue, Jul 8, 2025 at 1:20 PM Nick Hu <nick.hu at sifive.com> wrote:
> >
> > From: Vincent Chen <vincent.chen at sifive.com>
> >
> > The sifive_dev_platform is a new platform which is compatible to the
> > generic platform to place some features which are only for the SiFive
> > development machine.
> >
> > Signed-off-by: Vincent Chen <vincent.chen at sifive.com>
> > Co-developed-by: Nick Hu <nick.hu at sifive.com>
> > Signed-off-by: Nick Hu <nick.hu at sifive.com>
> > ---
> >  platform/generic/Kconfig                      |  5 +++
> >  platform/generic/configs/defconfig            |  1 +
> >  platform/generic/sifive/objects.mk            |  3 ++
> >  platform/generic/sifive/sifive_dev_platform.c | 41 +++++++++++++++++++
> >  4 files changed, 50 insertions(+)
> >  create mode 100644 platform/generic/sifive/sifive_dev_platform.c
> >
> > diff --git a/platform/generic/Kconfig b/platform/generic/Kconfig
> > index a24d6ab2..20c25a83 100644
> > --- a/platform/generic/Kconfig
> > +++ b/platform/generic/Kconfig
> > @@ -43,6 +43,11 @@ config PLATFORM_RENESAS_RZFIVE
> >         select ANDES_PMU
> >         default n
> >
> > +config PLATFORM_SIFIVE_DEV
> > +       bool "SiFive development platform support"
> > +       depends on FDT_CACHE
> > +       default n
> > +
> >  config PLATFORM_SIFIVE_FU540
> >         bool "SiFive FU540 support"
> >         default n
> > diff --git a/platform/generic/configs/defconfig b/platform/generic/configs/defconfig
> > index 6b219cfb..42e7ef75 100644
> > --- a/platform/generic/configs/defconfig
> > +++ b/platform/generic/configs/defconfig
> > @@ -1,6 +1,7 @@
> >  CONFIG_PLATFORM_ALLWINNER_D1=y
> >  CONFIG_PLATFORM_ANDES_AE350=y
> >  CONFIG_PLATFORM_RENESAS_RZFIVE=y
> > +CONFIG_PLATFORM_SIFIVE_DEV=y
> >  CONFIG_PLATFORM_SIFIVE_FU540=y
> >  CONFIG_PLATFORM_SIFIVE_FU740=y
> >  CONFIG_PLATFORM_SOPHGO_SG2042=y
> > diff --git a/platform/generic/sifive/objects.mk b/platform/generic/sifive/objects.mk
> > index d75e444a..d32e1273 100644
> > --- a/platform/generic/sifive/objects.mk
> > +++ b/platform/generic/sifive/objects.mk
> > @@ -2,6 +2,9 @@
> >  # SPDX-License-Identifier: BSD-2-Clause
> >  #
> >
> > +carray-platform_override_modules-$(CONFIG_PLATFORM_SIFIVE_DEV) += sifive_dev_platform
> > +platform-objs-$(CONFIG_PLATFORM_SIFIVE_DEV) += sifive/sifive_dev_platform.o
> > +
> >  carray-platform_override_modules-$(CONFIG_PLATFORM_SIFIVE_FU540) += sifive_fu540
> >  platform-objs-$(CONFIG_PLATFORM_SIFIVE_FU540) += sifive/fu540.o
> >
> > diff --git a/platform/generic/sifive/sifive_dev_platform.c b/platform/generic/sifive/sifive_dev_platform.c
> > new file mode 100644
> > index 00000000..ac659868
> > --- /dev/null
> > +++ b/platform/generic/sifive/sifive_dev_platform.c
> > @@ -0,0 +1,41 @@
> > +/*
> > + * SPDX-License-Identifier: BSD-2-Clause
> > + *
> > + * Copyright (c) 2025 SiFive Inc.
> > + */
> > +
> > +#include <platform_override.h>
> > +#include <sbi_utils/cache/fdt_cmo_helper.h>
> > +
> > +static int sifive_early_init(bool cold_boot)
> > +{
> > +       int rc;
> > +
> > +       rc = generic_early_init(cold_boot);
> > +       if (rc)
> > +               return rc;
> > +
> > +       rc = fdt_cmo_init(cold_boot);
> > +       if (rc)
> > +               return rc;
>
> Why not call fdt_cmo_init() directly as part of generic_early_init() ?
>
Good point! I'll move it to the generic_early_init().

> > +
> > +       return 0;
> > +}
> > +
> > +static int sifive_platform_init(const void *fdt, int nodeoff,
> > +                               const struct fdt_match *match)
> > +{
> > +       generic_platform_ops.early_init = sifive_early_init;
> > +
> > +       return 0;
> > +}
> > +
> > +static const struct fdt_match sifive_dev_platform_match[] = {
> > +       { .compatible = "sifive-dev" },
> > +       { },
> > +};
> > +
> > +const struct fdt_driver sifive_dev_platform = {
> > +       .match_table = sifive_dev_platform_match,
> > +       .init = sifive_platform_init,
> > +};
> > --
> > 2.17.1
> >
>
> Regards,
> Anup