| Message ID | 20250617090651.55-3-raviteja.laggyshetty@oss.qualcomm.com |
|---|---|
| State | New |
| Headers | show |
| Series | Add EPSS L3 provider support on QCS8300 SoC | expand |
On 17/06/2025 11:06, Raviteja Laggyshetty wrote: > Add Epoch Subsystem (EPSS) L3 interconnect provider support on > QCS8300 SoC. > > Signed-off-by: Raviteja Laggyshetty <raviteja.laggyshetty@oss.qualcomm.com> > --- > drivers/interconnect/qcom/osm-l3.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/interconnect/qcom/osm-l3.c b/drivers/interconnect/qcom/osm-l3.c > index baecbf2533f7..d8f1e0a4617b 100644 > --- a/drivers/interconnect/qcom/osm-l3.c > +++ b/drivers/interconnect/qcom/osm-l3.c > @@ -270,6 +270,7 @@ static const struct of_device_id osm_l3_of_match[] = { > { .compatible = "qcom,sm8150-osm-l3", .data = &osm_l3 }, > { .compatible = "qcom,sc8180x-osm-l3", .data = &osm_l3 }, > { .compatible = "qcom,sm8250-epss-l3", .data = &epss_l3_perf_state }, > + { .compatible = "qcom,qcs8300-epss-l3", .data = &epss_l3_perf_state }, Heh, the same as some time ago. We discussed this. No, stop adding more redundant entries. For explanation look at previous discussions. Best regards, Krzysztof
On 17/06/2025 11:26, Krzysztof Kozlowski wrote: > On 17/06/2025 11:06, Raviteja Laggyshetty wrote: >> Add Epoch Subsystem (EPSS) L3 interconnect provider support on >> QCS8300 SoC. >> >> Signed-off-by: Raviteja Laggyshetty <raviteja.laggyshetty@oss.qualcomm.com> >> --- >> drivers/interconnect/qcom/osm-l3.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/interconnect/qcom/osm-l3.c b/drivers/interconnect/qcom/osm-l3.c >> index baecbf2533f7..d8f1e0a4617b 100644 >> --- a/drivers/interconnect/qcom/osm-l3.c >> +++ b/drivers/interconnect/qcom/osm-l3.c >> @@ -270,6 +270,7 @@ static const struct of_device_id osm_l3_of_match[] = { >> { .compatible = "qcom,sm8150-osm-l3", .data = &osm_l3 }, >> { .compatible = "qcom,sc8180x-osm-l3", .data = &osm_l3 }, >> { .compatible = "qcom,sm8250-epss-l3", .data = &epss_l3_perf_state }, >> + { .compatible = "qcom,qcs8300-epss-l3", .data = &epss_l3_perf_state }, > Heh, the same as some time ago. We discussed this. > > No, stop adding more redundant entries. For explanation look at previous > discussions. > You already received exactly the same comments. https://lore.kernel.org/all/51653aac-76e0-4da2-aea8-16d62b570155@kernel.org/ Best regards, Krzysztof
diff --git a/drivers/interconnect/qcom/osm-l3.c b/drivers/interconnect/qcom/osm-l3.c index baecbf2533f7..d8f1e0a4617b 100644 --- a/drivers/interconnect/qcom/osm-l3.c +++ b/drivers/interconnect/qcom/osm-l3.c @@ -270,6 +270,7 @@ static const struct of_device_id osm_l3_of_match[] = { { .compatible = "qcom,sm8150-osm-l3", .data = &osm_l3 }, { .compatible = "qcom,sc8180x-osm-l3", .data = &osm_l3 }, { .compatible = "qcom,sm8250-epss-l3", .data = &epss_l3_perf_state }, + { .compatible = "qcom,qcs8300-epss-l3", .data = &epss_l3_perf_state }, { } }; MODULE_DEVICE_TABLE(of, osm_l3_of_match);
Add Epoch Subsystem (EPSS) L3 interconnect provider support on QCS8300 SoC. Signed-off-by: Raviteja Laggyshetty <raviteja.laggyshetty@oss.qualcomm.com> --- drivers/interconnect/qcom/osm-l3.c | 1 + 1 file changed, 1 insertion(+)