2021-09-05 17:13:44

by Robert Marko

Subject: [PATCH] soc: qcom: socinfo: Add IPQ8074 family ID-s

IPQ8074 family SoC ID-s are missing, so lets add them based on
the downstream driver.

Signed-off-by: Robert Marko <[email protected]>
---
drivers/soc/qcom/socinfo.c | 12 ++++++++++++
1 file changed, 12 insertions(+)

diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
index 9faf48302f4b..288897868435 100644
--- a/drivers/soc/qcom/socinfo.c
+++ b/drivers/soc/qcom/socinfo.c
@@ -281,19 +281,31 @@ static const struct soc_id soc_id[] = {
{ 319, "APQ8098" },
{ 321, "SDM845" },
{ 322, "MDM9206" },
+ { 323, "IPQ8074" },
{ 324, "SDA660" },
{ 325, "SDM658" },
{ 326, "SDA658" },
{ 327, "SDA630" },
{ 338, "SDM450" },
{ 341, "SDA845" },
+ { 342, "IPQ8072" },
+ { 343, "IPQ8076" },
+ { 344, "IPQ8078" },
{ 345, "SDM636" },
{ 346, "SDA636" },
{ 349, "SDM632" },
{ 350, "SDA632" },
{ 351, "SDA450" },
{ 356, "SM8250" },
+ { 375, "IPQ8070" },
+ { 376, "IPQ8071" },
+ { 389, "IPQ8072A" },
+ { 390, "IPQ8074A" },
+ { 391, "IPQ8076A" },
+ { 392, "IPQ8078A" },
{ 394, "SM6125" },
+ { 395, "IPQ8070A" },
+ { 396, "IPQ8071A" },
{ 402, "IPQ6018" },
{ 403, "IPQ6028" },
{ 421, "IPQ6000" },
--
2.31.1

2021-09-06 15:53:58

by Kathiravan T

Subject: Re: [PATCH] soc: qcom: socinfo: Add IPQ8074 family ID-s

On 2021-09-05 22:41, Robert Marko wrote:
> IPQ8074 family SoC ID-s are missing, so lets add them based on
> the downstream driver.
>

Did you intentionally left out the IDs for below variants?

IPQ8172 - 397
IPQ8173 - 398
IPQ8174 - 399

Reviewed-by: Kathiravan T <[email protected]>

> Signed-off-by: Robert Marko <[email protected]>
> ---
> drivers/soc/qcom/socinfo.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
> index 9faf48302f4b..288897868435 100644
> --- a/drivers/soc/qcom/socinfo.c
> +++ b/drivers/soc/qcom/socinfo.c
> @@ -281,19 +281,31 @@ static const struct soc_id soc_id[] = {
> { 319, "APQ8098" },
> { 321, "SDM845" },
> { 322, "MDM9206" },
> + { 323, "IPQ8074" },
> { 324, "SDA660" },
> { 325, "SDM658" },
> { 326, "SDA658" },
> { 327, "SDA630" },
> { 338, "SDM450" },
> { 341, "SDA845" },
> + { 342, "IPQ8072" },
> + { 343, "IPQ8076" },
> + { 344, "IPQ8078" },
> { 345, "SDM636" },
> { 346, "SDA636" },
> { 349, "SDM632" },
> { 350, "SDA632" },
> { 351, "SDA450" },
> { 356, "SM8250" },
> + { 375, "IPQ8070" },
> + { 376, "IPQ8071" },
> + { 389, "IPQ8072A" },
> + { 390, "IPQ8074A" },
> + { 391, "IPQ8076A" },
> + { 392, "IPQ8078A" },
> { 394, "SM6125" },
> + { 395, "IPQ8070A" },
> + { 396, "IPQ8071A" },
> { 402, "IPQ6018" },
> { 403, "IPQ6028" },
> { 421, "IPQ6000" },

--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member of Code Aurora Forum, hosted by The Linux Foundation

2021-09-06 16:50:03

by Robert Marko

Subject: Re: [PATCH] soc: qcom: socinfo: Add IPQ8074 family ID-s

On Mon, 6 Sept 2021 at 17:50, Kathiravan T <[email protected]> wrote:
>
> On 2021-09-05 22:41, Robert Marko wrote:
> > IPQ8074 family SoC ID-s are missing, so lets add them based on
> > the downstream driver.
> >
>
> Did you intentionally left out the IDs for below variants?
>
> IPQ8172 - 397
> IPQ8173 - 398
> IPQ8174 - 399

Yes, I wasn't really sure what to do with those.

Regards,
Robert
>
> Reviewed-by: Kathiravan T <[email protected]>
>
> > Signed-off-by: Robert Marko <[email protected]>
> > ---
> > drivers/soc/qcom/socinfo.c | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c
> > index 9faf48302f4b..288897868435 100644
> > --- a/drivers/soc/qcom/socinfo.c
> > +++ b/drivers/soc/qcom/socinfo.c
> > @@ -281,19 +281,31 @@ static const struct soc_id soc_id[] = {
> > { 319, "APQ8098" },
> > { 321, "SDM845" },
> > { 322, "MDM9206" },
> > + { 323, "IPQ8074" },
> > { 324, "SDA660" },
> > { 325, "SDM658" },
> > { 326, "SDA658" },
> > { 327, "SDA630" },
> > { 338, "SDM450" },
> > { 341, "SDA845" },
> > + { 342, "IPQ8072" },
> > + { 343, "IPQ8076" },
> > + { 344, "IPQ8078" },
> > { 345, "SDM636" },
> > { 346, "SDA636" },
> > { 349, "SDM632" },
> > { 350, "SDA632" },
> > { 351, "SDA450" },
> > { 356, "SM8250" },
> > + { 375, "IPQ8070" },
> > + { 376, "IPQ8071" },
> > + { 389, "IPQ8072A" },
> > + { 390, "IPQ8074A" },
> > + { 391, "IPQ8076A" },
> > + { 392, "IPQ8078A" },
> > { 394, "SM6125" },
> > + { 395, "IPQ8070A" },
> > + { 396, "IPQ8071A" },
> > { 402, "IPQ6018" },
> > { 403, "IPQ6028" },
> > { 421, "IPQ6000" },
>
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
> member of Code Aurora Forum, hosted by The Linux Foundation