* [PATCH 0/2] Drop the IPQ5019 SoC ID @ 2023-07-10 10:54 Kathiravan T 2023-07-10 10:54 ` [PATCH 1/2] dt-bindings: arm: qcom,ids: drop " Kathiravan T 2023-07-10 10:54 ` [PATCH 2/2] soc: qcom: socinfo: " Kathiravan T 0 siblings, 2 replies; 8+ messages in thread From: Kathiravan T @ 2023-07-10 10:54 UTC (permalink / raw) To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma, Kathiravan T Kathiravan T (2): dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID soc: qcom: socinfo: drop the IPQ5019 SoC ID drivers/soc/qcom/socinfo.c | 1 - include/dt-bindings/arm/qcom,ids.h | 1 - 2 files changed, 2 deletions(-) -- 2.34.1 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/2] dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID 2023-07-10 10:54 [PATCH 0/2] Drop the IPQ5019 SoC ID Kathiravan T @ 2023-07-10 10:54 ` Kathiravan T 2023-07-10 11:37 ` Krzysztof Kozlowski 2023-07-10 10:54 ` [PATCH 2/2] soc: qcom: socinfo: " Kathiravan T 1 sibling, 1 reply; 8+ messages in thread From: Kathiravan T @ 2023-07-10 10:54 UTC (permalink / raw) To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma, Kathiravan T IPQ5019 SoC is not available in production. Lets drop it. Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> --- include/dt-bindings/arm/qcom,ids.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h index bcbe9ee2cdaf..179dd56b2d95 100644 --- a/include/dt-bindings/arm/qcom,ids.h +++ b/include/dt-bindings/arm/qcom,ids.h @@ -250,7 +250,6 @@ #define QCOM_ID_QRU1000 539 #define QCOM_ID_QDU1000 545 #define QCOM_ID_QDU1010 587 -#define QCOM_ID_IPQ5019 569 #define QCOM_ID_QRU1032 588 #define QCOM_ID_QRU1052 589 #define QCOM_ID_QRU1062 590 -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID 2023-07-10 10:54 ` [PATCH 1/2] dt-bindings: arm: qcom,ids: drop " Kathiravan T @ 2023-07-10 11:37 ` Krzysztof Kozlowski 2023-07-10 12:10 ` Konrad Dybcio 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-07-10 11:37 UTC (permalink / raw) To: Kathiravan T, Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma On 10/07/2023 12:54, Kathiravan T wrote: > IPQ5019 SoC is not available in production. Lets drop it. > > Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> > --- > include/dt-bindings/arm/qcom,ids.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h > index bcbe9ee2cdaf..179dd56b2d95 100644 > --- a/include/dt-bindings/arm/qcom,ids.h > +++ b/include/dt-bindings/arm/qcom,ids.h > @@ -250,7 +250,6 @@ > #define QCOM_ID_QRU1000 539 > #define QCOM_ID_QDU1000 545 > #define QCOM_ID_QDU1010 587 > -#define QCOM_ID_IPQ5019 569 What about users of this binding? What's the impact on them? When did the SoC become obsolete and unsupported by Qualcomm? Remember that ceasing a production does not mean that magically all users of a product disappear... Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID 2023-07-10 11:37 ` Krzysztof Kozlowski @ 2023-07-10 12:10 ` Konrad Dybcio 2023-07-11 11:02 ` Kathiravan T 0 siblings, 1 reply; 8+ messages in thread From: Konrad Dybcio @ 2023-07-10 12:10 UTC (permalink / raw) To: Krzysztof Kozlowski, Kathiravan T, Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma On 10.07.2023 13:37, Krzysztof Kozlowski wrote: > On 10/07/2023 12:54, Kathiravan T wrote: >> IPQ5019 SoC is not available in production. Lets drop it. >> >> Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> >> --- >> include/dt-bindings/arm/qcom,ids.h | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h >> index bcbe9ee2cdaf..179dd56b2d95 100644 >> --- a/include/dt-bindings/arm/qcom,ids.h >> +++ b/include/dt-bindings/arm/qcom,ids.h >> @@ -250,7 +250,6 @@ >> #define QCOM_ID_QRU1000 539 >> #define QCOM_ID_QDU1000 545 >> #define QCOM_ID_QDU1010 587 >> -#define QCOM_ID_IPQ5019 569 > > What about users of this binding? What's the impact on them? When did > the SoC become obsolete and unsupported by Qualcomm? Remember that > ceasing a production does not mean that magically all users of a product > disappear... This + from my experience, SOCID entries are set in stone and freed indices are never reclaimed Konrad > > > Best regards, > Krzysztof > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID 2023-07-10 12:10 ` Konrad Dybcio @ 2023-07-11 11:02 ` Kathiravan T 2023-07-11 11:11 ` Krzysztof Kozlowski 0 siblings, 1 reply; 8+ messages in thread From: Kathiravan T @ 2023-07-11 11:02 UTC (permalink / raw) To: Konrad Dybcio, Krzysztof Kozlowski, Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma On 7/10/2023 5:40 PM, Konrad Dybcio wrote: > On 10.07.2023 13:37, Krzysztof Kozlowski wrote: >> On 10/07/2023 12:54, Kathiravan T wrote: >>> IPQ5019 SoC is not available in production. Lets drop it. >>> >>> Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> >>> --- >>> include/dt-bindings/arm/qcom,ids.h | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h >>> index bcbe9ee2cdaf..179dd56b2d95 100644 >>> --- a/include/dt-bindings/arm/qcom,ids.h >>> +++ b/include/dt-bindings/arm/qcom,ids.h >>> @@ -250,7 +250,6 @@ >>> #define QCOM_ID_QRU1000 539 >>> #define QCOM_ID_QDU1000 545 >>> #define QCOM_ID_QDU1010 587 >>> -#define QCOM_ID_IPQ5019 569 >> What about users of this binding? What's the impact on them? When did >> the SoC become obsolete and unsupported by Qualcomm? Remember that >> ceasing a production does not mean that magically all users of a product >> disappear... > This + from my experience, SOCID entries are set in stone and freed > indices are never reclaimed This SKU is planned but never productized. That's why I removed it. May be I should be more precise in the commit title. Should I leave it as it is / remove it? Thanks, Kathiravan T. > > Konrad >> >> Best regards, >> Krzysztof >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: arm: qcom,ids: drop the IPQ5019 SoC ID 2023-07-11 11:02 ` Kathiravan T @ 2023-07-11 11:11 ` Krzysztof Kozlowski 0 siblings, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-07-11 11:11 UTC (permalink / raw) To: Kathiravan T, Konrad Dybcio, Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma On 11/07/2023 13:02, Kathiravan T wrote: > > On 7/10/2023 5:40 PM, Konrad Dybcio wrote: >> On 10.07.2023 13:37, Krzysztof Kozlowski wrote: >>> On 10/07/2023 12:54, Kathiravan T wrote: >>>> IPQ5019 SoC is not available in production. Lets drop it. >>>> >>>> Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> >>>> --- >>>> include/dt-bindings/arm/qcom,ids.h | 1 - >>>> 1 file changed, 1 deletion(-) >>>> >>>> diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h >>>> index bcbe9ee2cdaf..179dd56b2d95 100644 >>>> --- a/include/dt-bindings/arm/qcom,ids.h >>>> +++ b/include/dt-bindings/arm/qcom,ids.h >>>> @@ -250,7 +250,6 @@ >>>> #define QCOM_ID_QRU1000 539 >>>> #define QCOM_ID_QDU1000 545 >>>> #define QCOM_ID_QDU1010 587 >>>> -#define QCOM_ID_IPQ5019 569 >>> What about users of this binding? What's the impact on them? When did >>> the SoC become obsolete and unsupported by Qualcomm? Remember that >>> ceasing a production does not mean that magically all users of a product >>> disappear... >> This + from my experience, SOCID entries are set in stone and freed >> indices are never reclaimed > > > This SKU is planned but never productized. That's why I removed it. If you mean this was never produced, then yes, it can be removed and your commit msg should be a bit more precise about it. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] soc: qcom: socinfo: drop the IPQ5019 SoC ID 2023-07-10 10:54 [PATCH 0/2] Drop the IPQ5019 SoC ID Kathiravan T 2023-07-10 10:54 ` [PATCH 1/2] dt-bindings: arm: qcom,ids: drop " Kathiravan T @ 2023-07-10 10:54 ` Kathiravan T 2023-07-10 11:38 ` Krzysztof Kozlowski 1 sibling, 1 reply; 8+ messages in thread From: Kathiravan T @ 2023-07-10 10:54 UTC (permalink / raw) To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma, Kathiravan T IPQ5019 SoC is not available in production. Lets drop it. Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> --- drivers/soc/qcom/socinfo.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c index 4d49945b3a35..78293ab13c87 100644 --- a/drivers/soc/qcom/socinfo.c +++ b/drivers/soc/qcom/socinfo.c @@ -406,7 +406,6 @@ static const struct soc_id soc_id[] = { { qcom_board_id(QRU1000) }, { qcom_board_id(QDU1000) }, { qcom_board_id(QDU1010) }, - { qcom_board_id(IPQ5019) }, { qcom_board_id(QRU1032) }, { qcom_board_id(QRU1052) }, { qcom_board_id(QRU1062) }, -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] soc: qcom: socinfo: drop the IPQ5019 SoC ID 2023-07-10 10:54 ` [PATCH 2/2] soc: qcom: socinfo: " Kathiravan T @ 2023-07-10 11:38 ` Krzysztof Kozlowski 0 siblings, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2023-07-10 11:38 UTC (permalink / raw) To: Kathiravan T, Andy Gross, Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-kernel, devicetree Cc: quic_srichara, quic_sjaganat, quic_anusha, quic_saahtoma On 10/07/2023 12:54, Kathiravan T wrote: > IPQ5019 SoC is not available in production. Lets drop it. > > Signed-off-by: Kathiravan T <quic_kathirav@quicinc.com> > --- > drivers/soc/qcom/socinfo.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c > index 4d49945b3a35..78293ab13c87 100644 > --- a/drivers/soc/qcom/socinfo.c > +++ b/drivers/soc/qcom/socinfo.c > @@ -406,7 +406,6 @@ static const struct soc_id soc_id[] = { > { qcom_board_id(QRU1000) }, > { qcom_board_id(QDU1000) }, > { qcom_board_id(QDU1010) }, > - { qcom_board_id(IPQ5019) }, Similar concerns as for patch #1, although a bit smaller because this is not an ABI. Anyway, commit msg said only that it is not available anymore, but does not explain anything about existing users... Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-07-11 11:11 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-10 10:54 [PATCH 0/2] Drop the IPQ5019 SoC ID Kathiravan T 2023-07-10 10:54 ` [PATCH 1/2] dt-bindings: arm: qcom,ids: drop " Kathiravan T 2023-07-10 11:37 ` Krzysztof Kozlowski 2023-07-10 12:10 ` Konrad Dybcio 2023-07-11 11:02 ` Kathiravan T 2023-07-11 11:11 ` Krzysztof Kozlowski 2023-07-10 10:54 ` [PATCH 2/2] soc: qcom: socinfo: " Kathiravan T 2023-07-10 11:38 ` Krzysztof Kozlowski
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).