From: Yu-Chien Peter Lin <peterlin@andestech.com>
To: opensbi@lists.infradead.org
Subject: [PATCH v4 00/14] Add Andes AE350 fdt driver support
Date: Mon, 24 Oct 2022 15:10:48 +0000 [thread overview]
Message-ID: <Y1aq+LQ/GyvhCHDB@APC323> (raw)
In-Reply-To: <CAAhSdy2XvAA_1Qi9-b_d3H5st69ZbJbMdoho2X2fGHnzCY6A7g@mail.gmail.com>
On Sun, Oct 23, 2022 at 10:42:52AM +0530, Anup Patel wrote:
> On Fri, Oct 14, 2022 at 6:03 AM Yu Chien Peter Lin
> <peterlin@andestech.com> wrote:
> >
> > We move the devices used by AE350 to fdt driver framework, reuse
> > existing driver and add timer, ipi and reset drivers.
> > The OpenSBI domain support is also enabled. As of now, we can use
> > Kconfig to manage these configurations that can support variants of
> > Andes platform and SoC like Kendryte K510.
> >
> > Changes v1 -> v2
> > - Rename Andes-specific devices (PLMT, PLICSW) compatible string
> > - Update ISA string in CPU nodes
> > Changes v2 -> v3
> > - Add PATCH1 and PATCH2 to fix typos
> > - Add PATCH3 which factors out a function for adding mtimer and ipi
> > device regions to root domain
> > - Remove unused variable "num_src" and "riscv,ndev" property of ipi
> > device node
> > Changes v3 -> v4
> > - Add PATCH3 to fix grammar
> > - Move plmt_{cold,warm}_timer_init() to andes_plmt.c and define
> > fdt_plmt_cold_timer_init() which adds PLMT region and sets
> > timer device
> > - Move plicsw_{cold,warm}_ipi_init() to andes_plicsw.c and define
> > fdt_plicsw_cold_ipi_init() which adds PLICSW region and sets
> > ipi device
> >
> >
> > Yu Chien Peter Lin (14):
> > include: sbi: Fix typo in comment
> > lib: sbi: Fix typo in comment
> > include: sbi: Fix grammar in comment
> > lib: sbi: Add sbi_domain_root_add_memrange() API
> > platform: andes/ae350: Remove enabling cache from an350_final_init
> > platform: andes/ae350: Use kconfig to set platform version and default
> > name
> > platform: andes/ae350: Use fdt serial driver
> > lib: utils/timer: Add Andes fdt timer support
> > lib: utils/reset: Add Andes fdt reset driver support
> > platform: andes/ae350: Use fdt irqchip driver
> > platform: andes/ae350: Add fw_platform_init for platform
> > initialization
> > lib: utils/ipi: Add Andes fdt ipi driver support
> > platform: andes/ae350: Add AE350 domain support
> > docs: andes-ae350.md: Update ae350 documentation for fdt driver
> > support
>
> Applied this series to the riscv/opensbi repo.
>
> Any thoughts on using the generic platform ?
>
> Thanks,
> Anup
Hi Anup,
The FW_TEXT_START on AE350 is different from generic platform.
Will it become a configuration?
I think we would keep the platform specific files in andes/ae350
until we have to refactor the duplicate code.
Thanks,
Peter Lin
>
> >
> > docs/platform/andes-ae350.md | 184 +++++++++++++++++++++++-
> > include/sbi/sbi_domain.h | 16 ++-
> > include/sbi/sbi_scratch.h | 2 +-
> > include/sbi_utils/fdt/fdt_helper.h | 6 +
> > include/sbi_utils/ipi/andes_plicsw.h | 46 ++++++
> > include/sbi_utils/timer/aclint_mtimer.h | 2 +
> > include/sbi_utils/timer/andes_plmt.h | 29 ++++
> > lib/sbi/sbi_domain.c | 27 ++++
> > lib/sbi/sbi_expected_trap.S | 2 +-
> > lib/utils/fdt/fdt_helper.c | 108 ++++++++++++++
> > lib/utils/ipi/Kconfig | 9 ++
> > lib/utils/ipi/andes_plicsw.c | 137 ++++++++++++++++++
> > lib/utils/ipi/fdt_ipi_plicsw.c | 47 ++++++
> > lib/utils/ipi/objects.mk | 4 +
> > lib/utils/reset/Kconfig | 4 +
> > lib/utils/reset/fdt_reset_atcwdt200.c | 122 ++++++++++++++++
> > lib/utils/reset/objects.mk | 3 +
> > lib/utils/timer/Kconfig | 9 ++
> > lib/utils/timer/aclint_mtimer.c | 50 ++-----
> > lib/utils/timer/andes_plmt.c | 104 ++++++++++++++
> > lib/utils/timer/fdt_timer_plmt.c | 51 +++++++
> > lib/utils/timer/objects.mk | 4 +
> > platform/andes/ae350/Kconfig | 30 +++-
> > platform/andes/ae350/objects.mk | 2 +-
> > platform/andes/ae350/platform.c | 165 +++++++++------------
> > platform/andes/ae350/platform.h | 17 ---
> > platform/andes/ae350/plicsw.c | 139 ------------------
> > platform/andes/ae350/plicsw.h | 44 ------
> > platform/andes/ae350/plmt.c | 107 --------------
> > platform/andes/ae350/plmt.h | 17 ---
> > 30 files changed, 1021 insertions(+), 466 deletions(-)
> > create mode 100644 include/sbi_utils/ipi/andes_plicsw.h
> > create mode 100644 include/sbi_utils/timer/andes_plmt.h
> > create mode 100644 lib/utils/ipi/andes_plicsw.c
> > create mode 100644 lib/utils/ipi/fdt_ipi_plicsw.c
> > create mode 100644 lib/utils/reset/fdt_reset_atcwdt200.c
> > create mode 100644 lib/utils/timer/andes_plmt.c
> > create mode 100644 lib/utils/timer/fdt_timer_plmt.c
> > delete mode 100644 platform/andes/ae350/plicsw.c
> > delete mode 100644 platform/andes/ae350/plicsw.h
> > delete mode 100644 platform/andes/ae350/plmt.c
> > delete mode 100644 platform/andes/ae350/plmt.h
> >
> > --
> > 2.34.1
> >
> >
> > --
> > opensbi mailing list
> > opensbi at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/opensbi
next prev parent reply other threads:[~2022-10-24 15:10 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-14 0:32 [PATCH v4 00/14] Add Andes AE350 fdt driver support Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 01/14] include: sbi: Fix typo in comment Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 02/14] lib: " Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 03/14] include: sbi: Fix grammar " Yu Chien Peter Lin
2022-10-23 4:39 ` Anup Patel
2022-10-14 0:32 ` [PATCH v4 04/14] lib: sbi: Add sbi_domain_root_add_memrange() API Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 05/14] platform: andes/ae350: Remove enabling cache from an350_final_init Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 06/14] platform: andes/ae350: Use kconfig to set platform version and default name Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 07/14] platform: andes/ae350: Use fdt serial driver Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 08/14] lib: utils/timer: Add Andes fdt timer support Yu Chien Peter Lin
2022-10-16 19:15 ` Samuel Holland
2022-10-19 21:23 ` Yu-Chien Peter Lin
2022-10-21 5:11 ` Anup Patel
2022-10-21 6:01 ` Atish Patra
2022-10-29 15:51 ` Samuel Holland
2022-10-14 0:32 ` [PATCH v4 09/14] lib: utils/reset: Add Andes fdt reset driver support Yu Chien Peter Lin
2022-10-16 19:19 ` Samuel Holland
2022-10-14 0:32 ` [PATCH v4 10/14] platform: andes/ae350: Use fdt irqchip driver Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 11/14] platform: andes/ae350: Add fw_platform_init for platform initialization Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 12/14] lib: utils/ipi: Add Andes fdt ipi driver support Yu Chien Peter Lin
2022-10-23 4:43 ` Anup Patel
2022-10-14 0:32 ` [PATCH v4 13/14] platform: andes/ae350: Add AE350 domain support Yu Chien Peter Lin
2022-10-14 0:32 ` [PATCH v4 14/14] docs: andes-ae350.md: Update ae350 documentation for fdt driver support Yu Chien Peter Lin
2022-10-23 5:12 ` [PATCH v4 00/14] Add Andes AE350 " Anup Patel
2022-10-24 15:10 ` Yu-Chien Peter Lin [this message]
2022-10-24 8:10 ` Anup Patel
2022-10-27 20:21 ` Yu-Chien Peter Lin
2022-10-27 12:43 ` Anup Patel
2022-10-28 8:43 ` Yu-Chien Peter Lin
2022-10-28 16:38 ` Anup Patel
2022-10-29 10:01 ` Yu-Chien Peter Lin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y1aq+LQ/GyvhCHDB@APC323 \
--to=peterlin@andestech.com \
--cc=opensbi@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.