* [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support
@ 2025-08-05 6:39 Ryan Chen
2025-08-06 1:15 ` Andrew Jeffery
0 siblings, 1 reply; 3+ messages in thread
From: Ryan Chen @ 2025-08-05 6:39 UTC (permalink / raw)
To: ryan_chen, Joel Stanley, Andrew Jeffery, linux-arm-kernel,
linux-aspeed, linux-kernel
Extend the ASPEED SoC info driver to support AST27XX silicon IDs.
Signed-off-by: Ryan Chen <ryan_chen@aspeedtech.com>
---
drivers/soc/aspeed/aspeed-socinfo.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/soc/aspeed/aspeed-socinfo.c b/drivers/soc/aspeed/aspeed-socinfo.c
index 3f759121dc00..67e9ac3d08ec 100644
--- a/drivers/soc/aspeed/aspeed-socinfo.c
+++ b/drivers/soc/aspeed/aspeed-socinfo.c
@@ -27,6 +27,10 @@ static struct {
{ "AST2620", 0x05010203 },
{ "AST2605", 0x05030103 },
{ "AST2625", 0x05030403 },
+ /* AST2700 */
+ { "AST2750", 0x06000003 },
+ { "AST2700", 0x06000103 },
+ { "AST2720", 0x06000203 },
};
static const char *siliconid_to_name(u32 siliconid)
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support
2025-08-05 6:39 [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support Ryan Chen
@ 2025-08-06 1:15 ` Andrew Jeffery
2025-08-06 2:54 ` Ryan Chen
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Jeffery @ 2025-08-06 1:15 UTC (permalink / raw)
To: Ryan Chen, Joel Stanley, linux-arm-kernel, linux-aspeed,
linux-kernel
Hi Ryan,
Regarding the patch subject, can you please follow the patterns
established by other commits under drivers/soc ?
Separately, can we concentrate efforts on trying to get the platform
definition bits upstream for the AST2700? Arnd's recent newsoc PR adds
several new SoCs (as the tag name suggests), which is a helpful
reference:
https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/log/?h=soc-newsoc-6.17
Cheers,
Andrew
On Tue, 2025-08-05 at 14:39 +0800, Ryan Chen wrote:
> Extend the ASPEED SoC info driver to support AST27XX silicon IDs.
>
> Signed-off-by: Ryan Chen <ryan_chen@aspeedtech.com>
> ---
> drivers/soc/aspeed/aspeed-socinfo.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/soc/aspeed/aspeed-socinfo.c b/drivers/soc/aspeed/aspeed-socinfo.c
> index 3f759121dc00..67e9ac3d08ec 100644
> --- a/drivers/soc/aspeed/aspeed-socinfo.c
> +++ b/drivers/soc/aspeed/aspeed-socinfo.c
> @@ -27,6 +27,10 @@ static struct {
> { "AST2620", 0x05010203 },
> { "AST2605", 0x05030103 },
> { "AST2625", 0x05030403 },
> + /* AST2700 */
> + { "AST2750", 0x06000003 },
> + { "AST2700", 0x06000103 },
> + { "AST2720", 0x06000203 },
> };
>
> static const char *siliconid_to_name(u32 siliconid)
^ permalink raw reply [flat|nested] 3+ messages in thread* RE: [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support
2025-08-06 1:15 ` Andrew Jeffery
@ 2025-08-06 2:54 ` Ryan Chen
0 siblings, 0 replies; 3+ messages in thread
From: Ryan Chen @ 2025-08-06 2:54 UTC (permalink / raw)
To: Andrew Jeffery, Joel Stanley,
linux-arm-kernel@lists.infradead.org,
linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support
>
> Hi Ryan,
>
> Regarding the patch subject, can you please follow the patterns established by
> other commits under drivers/soc ?
My mistake.
Will update subject : Soc: aspeed: socinfo: Add AST27xx silicon IDs
>
> Separately, can we concentrate efforts on trying to get the platform definition
> bits upstream for the AST2700? Arnd's recent newsoc PR adds several new
> SoCs (as the tag name suggests), which is a helpful
> reference:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git/log/?h=soc-newsoc-
> 6.17
Yes, will take look into this.
It will be my another patch reference for ast2700.
https://patchwork.ozlabs.org/project/linux-aspeed/list/?series=460647
>
> Cheers,
>
> Andrew
>
> On Tue, 2025-08-05 at 14:39 +0800, Ryan Chen wrote:
> > Extend the ASPEED SoC info driver to support AST27XX silicon IDs.
> >
> > Signed-off-by: Ryan Chen <ryan_chen@aspeedtech.com>
> > ---
> > drivers/soc/aspeed/aspeed-socinfo.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/soc/aspeed/aspeed-socinfo.c
> > b/drivers/soc/aspeed/aspeed-socinfo.c
> > index 3f759121dc00..67e9ac3d08ec 100644
> > --- a/drivers/soc/aspeed/aspeed-socinfo.c
> > +++ b/drivers/soc/aspeed/aspeed-socinfo.c
> > @@ -27,6 +27,10 @@ static struct {
> > { "AST2620", 0x05010203 },
> > { "AST2605", 0x05030103 },
> > { "AST2625", 0x05030403 },
> > + /* AST2700 */
> > + { "AST2750", 0x06000003 },
> > + { "AST2700", 0x06000103 },
> > + { "AST2720", 0x06000203 },
> > };
> >
> > static const char *siliconid_to_name(u32 siliconid)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-08-06 2:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-05 6:39 [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support Ryan Chen
2025-08-06 1:15 ` Andrew Jeffery
2025-08-06 2:54 ` Ryan Chen
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).