2018-07-23 12:39:45

by Yue Haibing

Subject: [PATCH] cpufreq: qcom-kryo: add NULL entry to the end of_device_id array

Make sure of_device_id tables are NULL terminated
Found by coccinelle spatch "misc/of_table.cocci"

Signed-off-by: YueHaibing <[email protected]>
---
drivers/cpufreq/qcom-cpufreq-kryo.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/cpufreq/qcom-cpufreq-kryo.c b/drivers/cpufreq/qcom-cpufreq-kryo.c
index 29389ac..efc9a7a 100644
--- a/drivers/cpufreq/qcom-cpufreq-kryo.c
+++ b/drivers/cpufreq/qcom-cpufreq-kryo.c
@@ -183,6 +183,7 @@ static struct platform_driver qcom_cpufreq_kryo_driver = {
static const struct of_device_id qcom_cpufreq_kryo_match_list[] __initconst = {
{ .compatible = "qcom,apq8096", },
{ .compatible = "qcom,msm8996", },
+ {}
};

/*
--
2.7.0

2018-07-23 12:46:30

by Ilia Lin

Subject: Re: [PATCH] cpufreq: qcom-kryo: add NULL entry to the end of_device_id array

On Mon, Jul 23, 2018 at 3:38 PM YueHaibing <[email protected]> wrote:
>
> Make sure of_device_id tables are NULL terminated
> Found by coccinelle spatch "misc/of_table.cocci"
>
> Signed-off-by: YueHaibing <[email protected]>
> ---
> drivers/cpufreq/qcom-cpufreq-kryo.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/cpufreq/qcom-cpufreq-kryo.c b/drivers/cpufreq/qcom-cpufreq-kryo.c
> index 29389ac..efc9a7a 100644
> --- a/drivers/cpufreq/qcom-cpufreq-kryo.c
> +++ b/drivers/cpufreq/qcom-cpufreq-kryo.c
> @@ -183,6 +183,7 @@ static struct platform_driver qcom_cpufreq_kryo_driver = {
> static const struct of_device_id qcom_cpufreq_kryo_match_list[] __initconst = {
> { .compatible = "qcom,apq8096", },
> { .compatible = "qcom,msm8996", },
> + {}
> };
>
> /*
> --
> 2.7.0
>
>

Acked-by: Ilia Lin <[email protected]>

2018-07-24 03:46:19

by Viresh Kumar

Subject: Re: [PATCH] cpufreq: qcom-kryo: add NULL entry to the end of_device_id array

On 23-07-18, 20:34, YueHaibing wrote:
> Make sure of_device_id tables are NULL terminated
> Found by coccinelle spatch "misc/of_table.cocci"
>
> Signed-off-by: YueHaibing <[email protected]>
> ---
> drivers/cpufreq/qcom-cpufreq-kryo.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/cpufreq/qcom-cpufreq-kryo.c b/drivers/cpufreq/qcom-cpufreq-kryo.c
> index 29389ac..efc9a7a 100644
> --- a/drivers/cpufreq/qcom-cpufreq-kryo.c
> +++ b/drivers/cpufreq/qcom-cpufreq-kryo.c
> @@ -183,6 +183,7 @@ static struct platform_driver qcom_cpufreq_kryo_driver = {
> static const struct of_device_id qcom_cpufreq_kryo_match_list[] __initconst = {
> { .compatible = "qcom,apq8096", },
> { .compatible = "qcom,msm8996", },
> + {}
> };
>
> /*

Acked-by: Viresh Kumar <[email protected]>

--
viresh

2018-07-26 09:07:59

by Rafael J. Wysocki

Subject: Re: [PATCH] cpufreq: qcom-kryo: add NULL entry to the end of_device_id array

On Tuesday, July 24, 2018 5:43:35 AM CEST Viresh Kumar wrote:
> On 23-07-18, 20:34, YueHaibing wrote:
> > Make sure of_device_id tables are NULL terminated
> > Found by coccinelle spatch "misc/of_table.cocci"
> >
> > Signed-off-by: YueHaibing <[email protected]>
> > ---
> > drivers/cpufreq/qcom-cpufreq-kryo.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/cpufreq/qcom-cpufreq-kryo.c b/drivers/cpufreq/qcom-cpufreq-kryo.c
> > index 29389ac..efc9a7a 100644
> > --- a/drivers/cpufreq/qcom-cpufreq-kryo.c
> > +++ b/drivers/cpufreq/qcom-cpufreq-kryo.c
> > @@ -183,6 +183,7 @@ static struct platform_driver qcom_cpufreq_kryo_driver = {
> > static const struct of_device_id qcom_cpufreq_kryo_match_list[] __initconst = {
> > { .compatible = "qcom,apq8096", },
> > { .compatible = "qcom,msm8996", },
> > + {}
> > };
> >
> > /*
>
> Acked-by: Viresh Kumar <[email protected]>
>
>

Patch applied, thanks!