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

[Xianwei Zhao]

Hi Neil,
   Thanks for your reply

On 2025/11/17 18:19, Neil Armstrong wrote:
> [ EXTERNAL EMAIL ]
>
> Hi,
>
> On 11/17/25 11:17, Xianwei Zhao via B4 Relay wrote:
> > From: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> >
> > Add new definition for Amlogi SoCs, include S6, S7, S7D.
> >
> > Signed-off-by: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> > ---
> >   drivers/soc/amlogic/meson-gx-socinfo.c | 7 ++++++-
> >   1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/soc/amlogic/meson-gx-socinfo.c 
> > b/drivers/soc/amlogic/meson-gx-socinfo.c
> > index 7549f1644e5e..6fbb70c8922d 100644
> > --- a/drivers/soc/amlogic/meson-gx-socinfo.c
> > +++ b/drivers/soc/amlogic/meson-gx-socinfo.c
> > @@ -45,7 +45,9 @@ static const struct meson_gx_soc_id {
> >       { "S4", 0x37 },
> >       { "A5", 0x3c },
> >       { "C3", 0x3d },
> > -     { "A4", 0x40 },
>
> Why did you drop A4 ?
This was my mistake, thanks for pointing out, I will restore it.
> > +     { "S7", 0x46 },
> > +     { "S7D", 0x47 },
> > +     { "S6", 0x48 },
> >   };
> >
> >   static const struct meson_gx_package_id {
> > @@ -86,6 +88,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)
>
> Thanks,
> Neil