[PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support

[PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support

[Ryan Chen]

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

Re: [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support

[Andrew Jeffery]

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)

RE: [PATCH] drivers/soc/aspeed: Add AST27XX SoC ID support

[Ryan Chen]

> 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)