| Message ID | 20230830-fp5-initial-v1-9-5a954519bbad@fairphone.com |
|---|---|
| State | Accepted |
| Commit | aa01dd7bc100af0bd230fc498d4a5750d0be73b2 |
| Headers | show |
| Series | Initial support for the Fairphone 5 smartphone | expand |
On 30/08/2023 11:58, Luca Weiss wrote: > The Qualcomm QCM6490 platform uses the qcom-cpufreq-hw driver, so add it > to the cpufreq-dt-platdev driver's blocklist. > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> This patchset is collecting unnecessarily way too many subsystems. It creates some feeling of dependencies, which is not true and not correct. Putting DTS in the middle makes it more confusing - if driver depends on DTS, then it is NAK. Please split your patchset - independent changes touching separate subsystems should go to their own set. Best regards, Krzysztof
On 30-08-23, 11:58, Luca Weiss wrote: > The Qualcomm QCM6490 platform uses the qcom-cpufreq-hw driver, so add it > to the cpufreq-dt-platdev driver's blocklist. > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> > --- > drivers/cpufreq/cpufreq-dt-platdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > index fb2875ce1fdd..02ec58a8603b 100644 > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > @@ -145,6 +145,7 @@ static const struct of_device_id blocklist[] __initconst = { > { .compatible = "qcom,msm8996", }, > { .compatible = "qcom,msm8998", }, > { .compatible = "qcom,qcm2290", }, > + { .compatible = "qcom,qcm6490", }, > { .compatible = "qcom,qcs404", }, > { .compatible = "qcom,qdu1000", }, > { .compatible = "qcom,sa8155p" }, Applied. Thanks.
diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c index fb2875ce1fdd..02ec58a8603b 100644 --- a/drivers/cpufreq/cpufreq-dt-platdev.c +++ b/drivers/cpufreq/cpufreq-dt-platdev.c @@ -145,6 +145,7 @@ static const struct of_device_id blocklist[] __initconst = { { .compatible = "qcom,msm8996", }, { .compatible = "qcom,msm8998", }, { .compatible = "qcom,qcm2290", }, + { .compatible = "qcom,qcm6490", }, { .compatible = "qcom,qcs404", }, { .compatible = "qcom,qdu1000", }, { .compatible = "qcom,sa8155p" },
The Qualcomm QCM6490 platform uses the qcom-cpufreq-hw driver, so add it to the cpufreq-dt-platdev driver's blocklist. Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> --- drivers/cpufreq/cpufreq-dt-platdev.c | 1 + 1 file changed, 1 insertion(+)