[PATCH v2 2/5] soc: amlogic: meson-gx-socinfo: add new SoCs id

Xianwei Zhao via B4 Relay posted 5 patches 1 week, 6 days ago
There is a newer version of this series
[PATCH v2 2/5] soc: amlogic: meson-gx-socinfo: add new SoCs id
Posted by Xianwei Zhao via B4 Relay 1 week, 6 days ago
From: Xianwei Zhao <xianwei.zhao@amlogic.com>

Add new definition for Amlogi SoCs, include S6, S7, S7D.

Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
---
 drivers/soc/amlogic/meson-gx-socinfo.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/soc/amlogic/meson-gx-socinfo.c b/drivers/soc/amlogic/meson-gx-socinfo.c
index 7549f1644e5e..2a54ca43cd13 100644
--- a/drivers/soc/amlogic/meson-gx-socinfo.c
+++ b/drivers/soc/amlogic/meson-gx-socinfo.c
@@ -46,6 +46,9 @@ static const struct meson_gx_soc_id {
 	{ "A5", 0x3c },
 	{ "C3", 0x3d },
 	{ "A4", 0x40 },
+	{ "S7", 0x46 },
+	{ "S7D", 0x47 },
+	{ "S6", 0x48 },
 };
 
 static const struct meson_gx_package_id {
@@ -86,6 +89,9 @@ static const struct meson_gx_package_id {
 	{ "A311D2", 0x36, 0x1, 0xf },
 	{ "A113X2", 0x3c, 0x1, 0xf },
 	{ "A113L2", 0x40, 0x1, 0xf },
+	{ "S805X3", 0x46, 0x3, 0xf },
+	{ "S905X5M", 0x47, 0x1, 0xf },
+	{ "S905X5", 0x48, 0x1, 0xf },
 };
 
 static inline unsigned int socinfo_to_major(u32 socinfo)

-- 
2.37.1
Re: [PATCH v2 2/5] soc: amlogic: meson-gx-socinfo: add new SoCs id
Posted by Martin Blumenstingl 1 week, 6 days ago
On Tue, Nov 18, 2025 at 6:59 AM Xianwei Zhao via B4 Relay
<devnull+xianwei.zhao.amlogic.com@kernel.org> wrote:
>
> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
>
> Add new definition for Amlogi SoCs, include S6, S7, S7D.
s/Amlogi/Amlogic/

Neil, is this something you can fixup while applying the patch or do
we need a v3?

with that:
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Re: [PATCH v2 2/5] soc: amlogic: meson-gx-socinfo: add new SoCs id
Posted by Xianwei Zhao 1 week, 6 days ago
Hi Martin,
    Thanks for your reply.

On 2025/11/19 05:43, Martin Blumenstingl wrote:
> [ EXTERNAL EMAIL ]
> 
> On Tue, Nov 18, 2025 at 6:59 AM Xianwei Zhao via B4 Relay
> <devnull+xianwei.zhao.amlogic.com@kernel.org> wrote:
>>
>> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
>>
>> Add new definition for Amlogi SoCs, include S6, S7, S7D.
> s/Amlogi/Amlogic/
> 
> Neil, is this something you can fixup while applying the patch or do
> we need a v3?

Thanks for points out, this patch is not applied, I will send v3 to fix it.

> 
> with that:
>> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Re: [PATCH v2 2/5] soc: amlogic: meson-gx-socinfo: add new SoCs id
Posted by Neil Armstrong 1 week, 6 days ago
On 11/18/25 06:59, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
> 
> Add new definition for Amlogi SoCs, include S6, S7, S7D.
> 
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
> ---
>   drivers/soc/amlogic/meson-gx-socinfo.c | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/soc/amlogic/meson-gx-socinfo.c b/drivers/soc/amlogic/meson-gx-socinfo.c
> index 7549f1644e5e..2a54ca43cd13 100644
> --- a/drivers/soc/amlogic/meson-gx-socinfo.c
> +++ b/drivers/soc/amlogic/meson-gx-socinfo.c
> @@ -46,6 +46,9 @@ static const struct meson_gx_soc_id {
>   	{ "A5", 0x3c },
>   	{ "C3", 0x3d },
>   	{ "A4", 0x40 },
> +	{ "S7", 0x46 },
> +	{ "S7D", 0x47 },
> +	{ "S6", 0x48 },
>   };
>   
>   static const struct meson_gx_package_id {
> @@ -86,6 +89,9 @@ static const struct meson_gx_package_id {
>   	{ "A311D2", 0x36, 0x1, 0xf },
>   	{ "A113X2", 0x3c, 0x1, 0xf },
>   	{ "A113L2", 0x40, 0x1, 0xf },
> +	{ "S805X3", 0x46, 0x3, 0xf },
> +	{ "S905X5M", 0x47, 0x1, 0xf },
> +	{ "S905X5", 0x48, 0x1, 0xf },
>   };
>   
>   static inline unsigned int socinfo_to_major(u32 socinfo)
> 

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>