| Message ID | 20230915021509.25773-10-quic_tengfan@quicinc.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | soc: qcom: Add uart console support for SM4450 | expand |
On 15/09/2023 04:15, Tengfei Fan wrote: > Add the SM4450 interconnect and pinctrl drivers as built-in for > support the Qualcomm SM4450 platform to boot to uart shell. > > Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> > --- > arch/arm64/configs/defconfig | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index ec59174b14db..e91993de865e 100644 > --- a/arch/arm64/configs/defconfig > +++ b/arch/arm64/configs/defconfig > @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y > CONFIG_PINCTRL_SDM660=y > CONFIG_PINCTRL_SDM670=y > CONFIG_PINCTRL_SDM845=y > +CONFIG_PINCTRL_SM4450=y > CONFIG_PINCTRL_SM6115=y > CONFIG_PINCTRL_SM6125=y > CONFIG_PINCTRL_SM6350=y > @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y > CONFIG_INTERCONNECT_QCOM_SC8180X=y > CONFIG_INTERCONNECT_QCOM_SC8280XP=y > CONFIG_INTERCONNECT_QCOM_SDM845=y > +CONFIG_INTERCONNECT_QCOM_SM4450=y Why it cannot be =m? Best regards, Krzysztof
在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: > On 15/09/2023 04:15, Tengfei Fan wrote: >> Add the SM4450 interconnect and pinctrl drivers as built-in for >> support the Qualcomm SM4450 platform to boot to uart shell. >> >> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >> --- >> arch/arm64/configs/defconfig | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >> index ec59174b14db..e91993de865e 100644 >> --- a/arch/arm64/configs/defconfig >> +++ b/arch/arm64/configs/defconfig >> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >> CONFIG_PINCTRL_SDM660=y >> CONFIG_PINCTRL_SDM670=y >> CONFIG_PINCTRL_SDM845=y >> +CONFIG_PINCTRL_SM4450=y >> CONFIG_PINCTRL_SM6115=y >> CONFIG_PINCTRL_SM6125=y >> CONFIG_PINCTRL_SM6350=y >> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >> CONFIG_INTERCONNECT_QCOM_SC8180X=y >> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >> CONFIG_INTERCONNECT_QCOM_SDM845=y >> +CONFIG_INTERCONNECT_QCOM_SM4450=y > > Why it cannot be =m? > > Best regards, > Krzysztof > Hi Krzysztof, Because system haven't capacity of loading ko files at this time on SM4450 platform, so setting to "Y".
On 15/09/2023 11:12, Tengfei Fan wrote: > > > 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >> On 15/09/2023 04:15, Tengfei Fan wrote: >>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>> support the Qualcomm SM4450 platform to boot to uart shell. >>> >>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>> --- >>> arch/arm64/configs/defconfig | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>> index ec59174b14db..e91993de865e 100644 >>> --- a/arch/arm64/configs/defconfig >>> +++ b/arch/arm64/configs/defconfig >>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>> CONFIG_PINCTRL_SDM660=y >>> CONFIG_PINCTRL_SDM670=y >>> CONFIG_PINCTRL_SDM845=y >>> +CONFIG_PINCTRL_SM4450=y >>> CONFIG_PINCTRL_SM6115=y >>> CONFIG_PINCTRL_SM6125=y >>> CONFIG_PINCTRL_SM6350=y >>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >> >> Why it cannot be =m? >> >> Best regards, >> Krzysztof >> > > Hi Krzysztof, > Because system haven't capacity of loading ko files at this time on > SM4450 platform, so setting to "Y". Hm? System has this capability. All systems have. What is so different on SM4450 comparing to everything else we have here? No, this should be =m and you need to fix your system. Best regards, Krzysztof
在 9/15/2023 5:16 PM, Krzysztof Kozlowski 写道: > On 15/09/2023 11:12, Tengfei Fan wrote: >> >> >> 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >>> On 15/09/2023 04:15, Tengfei Fan wrote: >>>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>>> support the Qualcomm SM4450 platform to boot to uart shell. >>>> >>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>>> --- >>>> arch/arm64/configs/defconfig | 2 ++ >>>> 1 file changed, 2 insertions(+) >>>> >>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>>> index ec59174b14db..e91993de865e 100644 >>>> --- a/arch/arm64/configs/defconfig >>>> +++ b/arch/arm64/configs/defconfig >>>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>>> CONFIG_PINCTRL_SDM660=y >>>> CONFIG_PINCTRL_SDM670=y >>>> CONFIG_PINCTRL_SDM845=y >>>> +CONFIG_PINCTRL_SM4450=y >>>> CONFIG_PINCTRL_SM6115=y >>>> CONFIG_PINCTRL_SM6125=y >>>> CONFIG_PINCTRL_SM6350=y >>>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >>> >>> Why it cannot be =m? >>> >>> Best regards, >>> Krzysztof >>> >> >> Hi Krzysztof, >> Because system haven't capacity of loading ko files at this time on >> SM4450 platform, so setting to "Y". > > Hm? System has this capability. All systems have. What is so different > on SM4450 comparing to everything else we have here? > > No, this should be =m and you need to fix your system. > > Best regards, > Krzysztof > Hi Krzysztof, Find new way which can load ko files on SM4450 platform, still need use "Y", because of some other modules have dependence to these two config, like scm, smmu module drivers, uart shell console cannot be got if set to "m". Also do test for setting these two config to "m" on SM8450 platform, get uart shell consle failed if so setting.
On 15/09/2023 13:18, Tengfei Fan wrote: > > > 在 9/15/2023 5:16 PM, Krzysztof Kozlowski 写道: >> On 15/09/2023 11:12, Tengfei Fan wrote: >>> >>> >>> 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >>>> On 15/09/2023 04:15, Tengfei Fan wrote: >>>>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>>>> support the Qualcomm SM4450 platform to boot to uart shell. >>>>> >>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>>>> --- >>>>> arch/arm64/configs/defconfig | 2 ++ >>>>> 1 file changed, 2 insertions(+) >>>>> >>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>>>> index ec59174b14db..e91993de865e 100644 >>>>> --- a/arch/arm64/configs/defconfig >>>>> +++ b/arch/arm64/configs/defconfig >>>>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>>>> CONFIG_PINCTRL_SDM660=y >>>>> CONFIG_PINCTRL_SDM670=y >>>>> CONFIG_PINCTRL_SDM845=y >>>>> +CONFIG_PINCTRL_SM4450=y >>>>> CONFIG_PINCTRL_SM6115=y >>>>> CONFIG_PINCTRL_SM6125=y >>>>> CONFIG_PINCTRL_SM6350=y >>>>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >>>> >>>> Why it cannot be =m? >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> >>> Hi Krzysztof, >>> Because system haven't capacity of loading ko files at this time on >>> SM4450 platform, so setting to "Y". >> >> Hm? System has this capability. All systems have. What is so different >> on SM4450 comparing to everything else we have here? >> >> No, this should be =m and you need to fix your system. >> >> Best regards, >> Krzysztof >> > Hi Krzysztof, > Find new way which can load ko files on SM4450 platform, still need use > "Y", because of some other modules have dependence to these two config, > like scm, smmu module drivers, uart shell console cannot be got if set > to "m". That's what I am asking, which device exactly needs it. > > Also do test for setting these two config to "m" on SM8450 platform, get > uart shell consle failed if so setting. Yeah, this we know, I did this. I am asking about SM4450. Best regards, Krzysztof
On 9/15/2023 4:29 AM, Krzysztof Kozlowski wrote: > On 15/09/2023 13:18, Tengfei Fan wrote: >> >> >> 在 9/15/2023 5:16 PM, Krzysztof Kozlowski 写道: >>> On 15/09/2023 11:12, Tengfei Fan wrote: >>>> >>>> >>>> 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >>>>> On 15/09/2023 04:15, Tengfei Fan wrote: >>>>>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>>>>> support the Qualcomm SM4450 platform to boot to uart shell. >>>>>> >>>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>>>>> --- >>>>>> arch/arm64/configs/defconfig | 2 ++ >>>>>> 1 file changed, 2 insertions(+) >>>>>> >>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>>>>> index ec59174b14db..e91993de865e 100644 >>>>>> --- a/arch/arm64/configs/defconfig >>>>>> +++ b/arch/arm64/configs/defconfig >>>>>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>>>>> CONFIG_PINCTRL_SDM660=y >>>>>> CONFIG_PINCTRL_SDM670=y >>>>>> CONFIG_PINCTRL_SDM845=y >>>>>> +CONFIG_PINCTRL_SM4450=y >>>>>> CONFIG_PINCTRL_SM6115=y >>>>>> CONFIG_PINCTRL_SM6125=y >>>>>> CONFIG_PINCTRL_SM6350=y >>>>>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>>>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>>>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >>>>> >>>>> Why it cannot be =m? >>>>> >>>>> Best regards, >>>>> Krzysztof >>>>> >>>> >>>> Hi Krzysztof, >>>> Because system haven't capacity of loading ko files at this time on >>>> SM4450 platform, so setting to "Y". >>> >>> Hm? System has this capability. All systems have. What is so different >>> on SM4450 comparing to everything else we have here? >>> >>> No, this should be =m and you need to fix your system. >>> >>> Best regards, >>> Krzysztof >>> >> Hi Krzysztof, >> Find new way which can load ko files on SM4450 platform, still need use >> "Y", because of some other modules have dependence to these two config, >> like scm, smmu module drivers, uart shell console cannot be got if set >> to "m". > > That's what I am asking, which device exactly needs it. > >> >> Also do test for setting these two config to "m" on SM8450 platform, get >> uart shell consle failed if so setting. > > Yeah, this we know, I did this. I am asking about SM4450. Why we have =m requirement when other drivers above are =Y?. Can we confirm w/ Georgi? I am not aware that Interconnect drivers needs to be =m only. CONFIG_INTERCONNECT_QCOM_SC8180X=y CONFIG_INTERCONNECT_QCOM_SC8280XP=y CONFIG_INTERCONNECT_QCOM_SDM845=y
On 9/15/2023 7:21 PM, Trilok Soni wrote: > On 9/15/2023 4:29 AM, Krzysztof Kozlowski wrote: >> On 15/09/2023 13:18, Tengfei Fan wrote: >>> >>> >>> 在 9/15/2023 5:16 PM, Krzysztof Kozlowski 写道: >>>> On 15/09/2023 11:12, Tengfei Fan wrote: >>>>> >>>>> >>>>> 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >>>>>> On 15/09/2023 04:15, Tengfei Fan wrote: >>>>>>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>>>>>> support the Qualcomm SM4450 platform to boot to uart shell. >>>>>>> >>>>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>>>>>> --- >>>>>>> arch/arm64/configs/defconfig | 2 ++ >>>>>>> 1 file changed, 2 insertions(+) >>>>>>> >>>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>>>>>> index ec59174b14db..e91993de865e 100644 >>>>>>> --- a/arch/arm64/configs/defconfig >>>>>>> +++ b/arch/arm64/configs/defconfig >>>>>>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>>>>>> CONFIG_PINCTRL_SDM660=y >>>>>>> CONFIG_PINCTRL_SDM670=y >>>>>>> CONFIG_PINCTRL_SDM845=y >>>>>>> +CONFIG_PINCTRL_SM4450=y >>>>>>> CONFIG_PINCTRL_SM6115=y >>>>>>> CONFIG_PINCTRL_SM6125=y >>>>>>> CONFIG_PINCTRL_SM6350=y >>>>>>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>>>>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>>>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>>>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>>>>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >>>>>> >>>>>> Why it cannot be =m? >>>>>> >>>>>> Best regards, >>>>>> Krzysztof >>>>>> >>>>> >>>>> Hi Krzysztof, >>>>> Because system haven't capacity of loading ko files at this time on >>>>> SM4450 platform, so setting to "Y". >>>> >>>> Hm? System has this capability. All systems have. What is so different >>>> on SM4450 comparing to everything else we have here? >>>> >>>> No, this should be =m and you need to fix your system. >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> Hi Krzysztof, >>> Find new way which can load ko files on SM4450 platform, still need use >>> "Y", because of some other modules have dependence to these two config, >>> like scm, smmu module drivers, uart shell console cannot be got if set >>> to "m". >> >> That's what I am asking, which device exactly needs it. >> >>> >>> Also do test for setting these two config to "m" on SM8450 platform, get >>> uart shell consle failed if so setting. >> >> Yeah, this we know, I did this. I am asking about SM4450. > > Why we have =m requirement when other drivers above are =Y?. Can we confirm > w/ Georgi? I am not aware that Interconnect drivers needs to be =m only. > > CONFIG_INTERCONNECT_QCOM_SC8180X=y > CONFIG_INTERCONNECT_QCOM_SC8280XP=y > CONFIG_INTERCONNECT_QCOM_SDM845=y Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document the reasons somewhere on why they are added as =y? CONFIG_INTERCONNECT_QCOM=y CONFIG_INTERCONNECT_QCOM_MSM8916=m CONFIG_INTERCONNECT_QCOM_MSM8996=m CONFIG_INTERCONNECT_QCOM_OSM_L3=m CONFIG_INTERCONNECT_QCOM_QCM2290=m CONFIG_INTERCONNECT_QCOM_QCS404=m CONFIG_INTERCONNECT_QCOM_SA8775P=y CONFIG_INTERCONNECT_QCOM_SC7180=y CONFIG_INTERCONNECT_QCOM_SC7280=y CONFIG_INTERCONNECT_QCOM_SC8180X=y CONFIG_INTERCONNECT_QCOM_SC8280XP=y CONFIG_INTERCONNECT_QCOM_SDM845=y CONFIG_INTERCONNECT_QCOM_SM8150=m CONFIG_INTERCONNECT_QCOM_SM8250=m CONFIG_INTERCONNECT_QCOM_SM8350=m CONFIG_INTERCONNECT_QCOM_SM8450=y CONFIG_INTERCONNECT_QCOM_SM8550=y
Hi Trilok, On 16.09.23 5:25, Trilok Soni wrote: > On 9/15/2023 7:21 PM, Trilok Soni wrote: >> On 9/15/2023 4:29 AM, Krzysztof Kozlowski wrote: >>> On 15/09/2023 13:18, Tengfei Fan wrote: >>>> >>>> >>>> 在 9/15/2023 5:16 PM, Krzysztof Kozlowski 写道: >>>>> On 15/09/2023 11:12, Tengfei Fan wrote: >>>>>> >>>>>> >>>>>> 在 9/15/2023 3:21 PM, Krzysztof Kozlowski 写道: >>>>>>> On 15/09/2023 04:15, Tengfei Fan wrote: >>>>>>>> Add the SM4450 interconnect and pinctrl drivers as built-in for >>>>>>>> support the Qualcomm SM4450 platform to boot to uart shell. >>>>>>>> >>>>>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> >>>>>>>> --- >>>>>>>> arch/arm64/configs/defconfig | 2 ++ >>>>>>>> 1 file changed, 2 insertions(+) >>>>>>>> >>>>>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>>>>>>> index ec59174b14db..e91993de865e 100644 >>>>>>>> --- a/arch/arm64/configs/defconfig >>>>>>>> +++ b/arch/arm64/configs/defconfig >>>>>>>> @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y >>>>>>>> CONFIG_PINCTRL_SDM660=y >>>>>>>> CONFIG_PINCTRL_SDM670=y >>>>>>>> CONFIG_PINCTRL_SDM845=y >>>>>>>> +CONFIG_PINCTRL_SM4450=y >>>>>>>> CONFIG_PINCTRL_SM6115=y >>>>>>>> CONFIG_PINCTRL_SM6125=y >>>>>>>> CONFIG_PINCTRL_SM6350=y >>>>>>>> @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y >>>>>>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>>>>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>>>>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>>>>>> +CONFIG_INTERCONNECT_QCOM_SM4450=y >>>>>>> >>>>>>> Why it cannot be =m? >>>>>>> >>>>>>> Best regards, >>>>>>> Krzysztof >>>>>>> >>>>>> >>>>>> Hi Krzysztof, >>>>>> Because system haven't capacity of loading ko files at this time on >>>>>> SM4450 platform, so setting to "Y". >>>>> >>>>> Hm? System has this capability. All systems have. What is so different >>>>> on SM4450 comparing to everything else we have here? >>>>> >>>>> No, this should be =m and you need to fix your system. >>>>> >>>>> Best regards, >>>>> Krzysztof >>>>> >>>> Hi Krzysztof, >>>> Find new way which can load ko files on SM4450 platform, still need use >>>> "Y", because of some other modules have dependence to these two config, >>>> like scm, smmu module drivers, uart shell console cannot be got if set >>>> to "m". >>> >>> That's what I am asking, which device exactly needs it. >>> >>>> >>>> Also do test for setting these two config to "m" on SM8450 platform, get >>>> uart shell consle failed if so setting. >>> >>> Yeah, this we know, I did this. I am asking about SM4450. >> >> Why we have =m requirement when other drivers above are =Y?. Can we confirm >> w/ Georgi? I am not aware that Interconnect drivers needs to be =m only. >> >> CONFIG_INTERCONNECT_QCOM_SC8180X=y >> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >> CONFIG_INTERCONNECT_QCOM_SDM845=y > > Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document > the reasons somewhere on why they are added as =y? > > CONFIG_INTERCONNECT_QCOM=y > CONFIG_INTERCONNECT_QCOM_MSM8916=m > CONFIG_INTERCONNECT_QCOM_MSM8996=m > CONFIG_INTERCONNECT_QCOM_OSM_L3=m > CONFIG_INTERCONNECT_QCOM_QCM2290=m > CONFIG_INTERCONNECT_QCOM_QCS404=m > CONFIG_INTERCONNECT_QCOM_SA8775P=y > CONFIG_INTERCONNECT_QCOM_SC7180=y > CONFIG_INTERCONNECT_QCOM_SC7280=y > CONFIG_INTERCONNECT_QCOM_SC8180X=y > CONFIG_INTERCONNECT_QCOM_SC8280XP=y > CONFIG_INTERCONNECT_QCOM_SDM845=y > CONFIG_INTERCONNECT_QCOM_SM8150=m > CONFIG_INTERCONNECT_QCOM_SM8250=m > CONFIG_INTERCONNECT_QCOM_SM8350=m > CONFIG_INTERCONNECT_QCOM_SM8450=y > CONFIG_INTERCONNECT_QCOM_SM8550=y If the device can boot (to console/initramfs) with =m, we go with that. But if something critical like the UART depends on the interconnect provider, then we make it built-in. On SM8550 for example, we have enabled bandwidth scaling support for QUP and that's why it needs to be =y. It looks like on SM4450 this should be =y too. Thanks, Georgi
On 16/09/2023 04:21, Trilok Soni wrote: >>>> No, this should be =m and you need to fix your system. >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> Hi Krzysztof, >>> Find new way which can load ko files on SM4450 platform, still need use >>> "Y", because of some other modules have dependence to these two config, >>> like scm, smmu module drivers, uart shell console cannot be got if set >>> to "m". >> >> That's what I am asking, which device exactly needs it. >> >>> >>> Also do test for setting these two config to "m" on SM8450 platform, get >>> uart shell consle failed if so setting. >> >> Yeah, this we know, I did this. I am asking about SM4450. > > Why we have =m requirement when other drivers above are =Y?. Can we confirm > w/ Georgi? I am not aware that Interconnect drivers needs to be =m only. There is a requirement of all drivers which can be a module, to be a module. Best regards, Krzysztof
On 16/09/2023 17:13, Georgi Djakov wrote: > >> >> Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document >> the reasons somewhere on why they are added as =y? >> >> CONFIG_INTERCONNECT_QCOM=y >> CONFIG_INTERCONNECT_QCOM_MSM8916=m >> CONFIG_INTERCONNECT_QCOM_MSM8996=m >> CONFIG_INTERCONNECT_QCOM_OSM_L3=m >> CONFIG_INTERCONNECT_QCOM_QCM2290=m >> CONFIG_INTERCONNECT_QCOM_QCS404=m >> CONFIG_INTERCONNECT_QCOM_SA8775P=y >> CONFIG_INTERCONNECT_QCOM_SC7180=y >> CONFIG_INTERCONNECT_QCOM_SC7280=y >> CONFIG_INTERCONNECT_QCOM_SC8180X=y >> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >> CONFIG_INTERCONNECT_QCOM_SDM845=y >> CONFIG_INTERCONNECT_QCOM_SM8150=m >> CONFIG_INTERCONNECT_QCOM_SM8250=m >> CONFIG_INTERCONNECT_QCOM_SM8350=m >> CONFIG_INTERCONNECT_QCOM_SM8450=y >> CONFIG_INTERCONNECT_QCOM_SM8550=y > > If the device can boot (to console/initramfs) with =m, we go with that. > But if something critical like the UART depends on the interconnect > provider, then we make it built-in. > > On SM8550 for example, we have enabled bandwidth scaling support for QUP > and that's why it needs to be =y. > > It looks like on SM4450 this should be =y too. I asked why SM4450 has to be =y and there was no answer. The argument that SM8450 is a module, is not applicable. Best regards, Krzysztof
Hi Krzysztof, On 16.09.23 23:32, Krzysztof Kozlowski wrote: > On 16/09/2023 17:13, Georgi Djakov wrote: >> >>> >>> Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document >>> the reasons somewhere on why they are added as =y? >>> >>> CONFIG_INTERCONNECT_QCOM=y >>> CONFIG_INTERCONNECT_QCOM_MSM8916=m >>> CONFIG_INTERCONNECT_QCOM_MSM8996=m >>> CONFIG_INTERCONNECT_QCOM_OSM_L3=m >>> CONFIG_INTERCONNECT_QCOM_QCM2290=m >>> CONFIG_INTERCONNECT_QCOM_QCS404=m >>> CONFIG_INTERCONNECT_QCOM_SA8775P=y >>> CONFIG_INTERCONNECT_QCOM_SC7180=y >>> CONFIG_INTERCONNECT_QCOM_SC7280=y >>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>> CONFIG_INTERCONNECT_QCOM_SM8150=m >>> CONFIG_INTERCONNECT_QCOM_SM8250=m >>> CONFIG_INTERCONNECT_QCOM_SM8350=m >>> CONFIG_INTERCONNECT_QCOM_SM8450=y >>> CONFIG_INTERCONNECT_QCOM_SM8550=y >> >> If the device can boot (to console/initramfs) with =m, we go with that. >> But if something critical like the UART depends on the interconnect >> provider, then we make it built-in. >> >> On SM8550 for example, we have enabled bandwidth scaling support for QUP >> and that's why it needs to be =y. >> >> It looks like on SM4450 this should be =y too. > > I asked why SM4450 has to be =y and there was no answer. The argument > that SM8450 is a module, is not applicable. From the hardware description i see in patch 7, the serial engine depends on some interconnect provider. But as the serial console driver is only available as built-in, the interconnect provider also needs be built-in for the UART device to probe and register the console. So the answer to your question should be that this is needed by the UART device (at least). Such details of course deserve to be mentioned in the commit message of this patch. BR, Georgi
On 17/09/2023 00:55, Georgi Djakov wrote: > Hi Krzysztof, > > On 16.09.23 23:32, Krzysztof Kozlowski wrote: >> On 16/09/2023 17:13, Georgi Djakov wrote: >>> >>>> >>>> Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document >>>> the reasons somewhere on why they are added as =y? >>>> >>>> CONFIG_INTERCONNECT_QCOM=y >>>> CONFIG_INTERCONNECT_QCOM_MSM8916=m >>>> CONFIG_INTERCONNECT_QCOM_MSM8996=m >>>> CONFIG_INTERCONNECT_QCOM_OSM_L3=m >>>> CONFIG_INTERCONNECT_QCOM_QCM2290=m >>>> CONFIG_INTERCONNECT_QCOM_QCS404=m >>>> CONFIG_INTERCONNECT_QCOM_SA8775P=y >>>> CONFIG_INTERCONNECT_QCOM_SC7180=y >>>> CONFIG_INTERCONNECT_QCOM_SC7280=y >>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>> CONFIG_INTERCONNECT_QCOM_SM8150=m >>>> CONFIG_INTERCONNECT_QCOM_SM8250=m >>>> CONFIG_INTERCONNECT_QCOM_SM8350=m >>>> CONFIG_INTERCONNECT_QCOM_SM8450=y >>>> CONFIG_INTERCONNECT_QCOM_SM8550=y >>> >>> If the device can boot (to console/initramfs) with =m, we go with that. >>> But if something critical like the UART depends on the interconnect >>> provider, then we make it built-in. >>> >>> On SM8550 for example, we have enabled bandwidth scaling support for QUP >>> and that's why it needs to be =y. >>> >>> It looks like on SM4450 this should be =y too. >> >> I asked why SM4450 has to be =y and there was no answer. The argument >> that SM8450 is a module, is not applicable. > > From the hardware description i see in patch 7, the serial engine depends > on some interconnect provider. But as the serial console driver is only > available as built-in, the interconnect provider also needs be built-in > for the UART device to probe and register the console. > > So the answer to your question should be that this is needed by the UART > device (at least). > > Such details of course deserve to be mentioned in the commit message of > this patch. If you mean here the debug UART with console, then it is the same valid reason as in my change for others. This should be mentioned in commit msg. Best regards, Krzysztof
在 9/17/2023 3:35 PM, Krzysztof Kozlowski 写道: > On 17/09/2023 00:55, Georgi Djakov wrote: >> Hi Krzysztof, >> >> On 16.09.23 23:32, Krzysztof Kozlowski wrote: >>> On 16/09/2023 17:13, Georgi Djakov wrote: >>>> >>>>> >>>>> Complete list here, and it is inconsistent. Latest 8550 is also =y. Do we document >>>>> the reasons somewhere on why they are added as =y? >>>>> >>>>> CONFIG_INTERCONNECT_QCOM=y >>>>> CONFIG_INTERCONNECT_QCOM_MSM8916=m >>>>> CONFIG_INTERCONNECT_QCOM_MSM8996=m >>>>> CONFIG_INTERCONNECT_QCOM_OSM_L3=m >>>>> CONFIG_INTERCONNECT_QCOM_QCM2290=m >>>>> CONFIG_INTERCONNECT_QCOM_QCS404=m >>>>> CONFIG_INTERCONNECT_QCOM_SA8775P=y >>>>> CONFIG_INTERCONNECT_QCOM_SC7180=y >>>>> CONFIG_INTERCONNECT_QCOM_SC7280=y >>>>> CONFIG_INTERCONNECT_QCOM_SC8180X=y >>>>> CONFIG_INTERCONNECT_QCOM_SC8280XP=y >>>>> CONFIG_INTERCONNECT_QCOM_SDM845=y >>>>> CONFIG_INTERCONNECT_QCOM_SM8150=m >>>>> CONFIG_INTERCONNECT_QCOM_SM8250=m >>>>> CONFIG_INTERCONNECT_QCOM_SM8350=m >>>>> CONFIG_INTERCONNECT_QCOM_SM8450=y >>>>> CONFIG_INTERCONNECT_QCOM_SM8550=y >>>> >>>> If the device can boot (to console/initramfs) with =m, we go with that. >>>> But if something critical like the UART depends on the interconnect >>>> provider, then we make it built-in. >>>> >>>> On SM8550 for example, we have enabled bandwidth scaling support for QUP >>>> and that's why it needs to be =y. >>>> >>>> It looks like on SM4450 this should be =y too. >>> >>> I asked why SM4450 has to be =y and there was no answer. The argument >>> that SM8450 is a module, is not applicable. >> >> From the hardware description i see in patch 7, the serial engine depends >> on some interconnect provider. But as the serial console driver is only >> available as built-in, the interconnect provider also needs be built-in >> for the UART device to probe and register the console. >> >> So the answer to your question should be that this is needed by the UART >> device (at least). >> >> Such details of course deserve to be mentioned in the commit message of >> this patch. > > If you mean here the debug UART with console, then it is the same valid > reason as in my change for others. This should be mentioned in commit msg. > > Best regards, > Krzysztof > yes, will mention this valid reason in next patch commit message detailly.
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index ec59174b14db..e91993de865e 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -598,6 +598,7 @@ CONFIG_PINCTRL_SC8280XP=y CONFIG_PINCTRL_SDM660=y CONFIG_PINCTRL_SDM670=y CONFIG_PINCTRL_SDM845=y +CONFIG_PINCTRL_SM4450=y CONFIG_PINCTRL_SM6115=y CONFIG_PINCTRL_SM6125=y CONFIG_PINCTRL_SM6350=y @@ -1500,6 +1501,7 @@ CONFIG_INTERCONNECT_QCOM_SC7280=y CONFIG_INTERCONNECT_QCOM_SC8180X=y CONFIG_INTERCONNECT_QCOM_SC8280XP=y CONFIG_INTERCONNECT_QCOM_SDM845=y +CONFIG_INTERCONNECT_QCOM_SM4450=y CONFIG_INTERCONNECT_QCOM_SM8150=m CONFIG_INTERCONNECT_QCOM_SM8250=m CONFIG_INTERCONNECT_QCOM_SM8350=m
Add the SM4450 interconnect and pinctrl drivers as built-in for support the Qualcomm SM4450 platform to boot to uart shell. Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com> --- arch/arm64/configs/defconfig | 2 ++ 1 file changed, 2 insertions(+)