[PATCH 1/2] cpufreq: ti-cpufreq: Add support for AM3517

[PATCH 1/2] cpufreq: ti-cpufreq: Add support for AM3517

[Adam Ford]

The AM3517 only lists 600MHz @ 1.2V, but the revister values for
0x4830A204 = 1b86 802f, it seems like am3517 might be a derivative
of the omap36 which has OPPs would be OPP50 (300 MHz) and OPP100
(600 MHz).

This patch simply enable adds the am3517 to the compatible table
using the omap3630 structure instead of the 3430.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c
index f2f58d689320..6b69fb1d6bdf 100644
--- a/drivers/cpufreq/ti-cpufreq.c
+++ b/drivers/cpufreq/ti-cpufreq.c
@@ -270,6 +270,7 @@ static int ti_cpufreq_setup_syscon_register(struct ti_cpufreq_data *opp_data)
 
 static const struct of_device_id ti_cpufreq_of_match[] = {
 	{ .compatible = "ti,am33xx", .data = &am3x_soc_data, },
+	{ .compatible = "ti,am3517", .data = &omap36xx_soc_data, },
 	{ .compatible = "ti,am43", .data = &am4x_soc_data, },
 	{ .compatible = "ti,dra7", .data = &dra7_soc_data },
 	{ .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, },
-- 
2.17.1

[PATCH 2/2] ARM: dts: Add OPP-V2 table for AM3517

[Adam Ford]

The AM3517 only lists 600MHz @ 1.2V, but the revister values for
0x4830A204 = 1b86 802f, it seems like am3517 might be a derivative
of the omap36 which has OPPs would be OPP50 (300 MHz) and OPP100
(600 MHz).

This patch sets up the OPP50 and OPP100 tables at 300MHz and 600MHz
for the AM3517 with each having an operating voltage at 1.2V.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi
index 23ea381d363f..a0f23361d4bc 100644
--- a/arch/arm/boot/dts/am3517.dtsi
+++ b/arch/arm/boot/dts/am3517.dtsi
@@ -16,6 +16,37 @@
 		can = &hecc;
 	};
 
+	cpus {
+		cpu: cpu@0 {
+			/* Based on OMAP3630 variants OPP50 and OPP100 */
+			operating-points-v2 = <&cpu0_opp_table>;
+
+			clock-latency = <300000>; /* From legacy driver */
+		};
+	};
+
+	cpu0_opp_table: opp-table {
+		compatible = "operating-points-v2-ti-cpu";
+		syscon = <&scm_conf>;
+		/*
+		 * AM3517 TRM only lists 600MHz @ 1.2V, but omap36xx
+		 * appear to operate at 300MHz as well. Since AM3517 only
+		 * lists one operating voltage, it will remain fixed at 1.2V
+		 */
+		opp50-300000000 {
+			opp-hz = /bits/ 64 <300000000>;
+			opp-microvolt = <1200000>;
+			opp-supported-hw = <0xffffffff 0xffffffff>;
+			opp-suspend;
+		};
+
+		opp100-600000000 {
+			opp-hz = /bits/ 64 <600000000>;
+			opp-microvolt = <1200000>;
+			opp-supported-hw = <0xffffffff 0xffffffff>;
+		};
+	};
+
 	ocp@68000000 {
 		am35x_otg_hs: am35x_otg_hs@5c040000 {
 			compatible = "ti,omap3-musb";
-- 
2.17.1

Re: [PATCH 1/2] cpufreq: ti-cpufreq: Add support for AM3517

[H. Nikolaus Schaller]

Hi Adam,

> Am 09.09.2019 um 17:45 schrieb Adam Ford <aford173@gmail.com>:
> 
> The AM3517 only lists 600MHz @ 1.2V, but the revister values for

a small typo...

s/revister/register/

> 0x4830A204 = 1b86 802f, it seems like am3517 might be a derivative
> of the omap36 which has OPPs would be OPP50 (300 MHz) and OPP100
> (600 MHz).
> 
> This patch simply enable adds the am3517 to the compatible table
> using the omap3630 structure instead of the 3430.
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>
> 
> diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c
> index f2f58d689320..6b69fb1d6bdf 100644
> --- a/drivers/cpufreq/ti-cpufreq.c
> +++ b/drivers/cpufreq/ti-cpufreq.c
> @@ -270,6 +270,7 @@ static int ti_cpufreq_setup_syscon_register(struct ti_cpufreq_data *opp_data)
> 
> static const struct of_device_id ti_cpufreq_of_match[] = {
> 	{ .compatible = "ti,am33xx", .data = &am3x_soc_data, },
> +	{ .compatible = "ti,am3517", .data = &omap36xx_soc_data, },
> 	{ .compatible = "ti,am43", .data = &am4x_soc_data, },
> 	{ .compatible = "ti,dra7", .data = &dra7_soc_data },
> 	{ .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, },
> -- 
> 2.17.1
> 

Looks good to me.

Should I include your two patches to my patch set (and resend)?
Or should we keep them separated?

BR and thanks,
Nikolaus

Re: [PATCH 1/2] cpufreq: ti-cpufreq: Add support for AM3517

[Adam Ford]

On Mon, Sep 9, 2019 at 1:13 PM H. Nikolaus Schaller <hns@goldelico.com> wrote:
>
> Hi Adam,
>
> > Am 09.09.2019 um 17:45 schrieb Adam Ford <aford173@gmail.com>:
> >
> > The AM3517 only lists 600MHz @ 1.2V, but the revister values for
>
> a small typo...
>
> s/revister/register/
>
> > 0x4830A204 = 1b86 802f, it seems like am3517 might be a derivative
> > of the omap36 which has OPPs would be OPP50 (300 MHz) and OPP100
> > (600 MHz).
> >
> > This patch simply enable adds the am3517 to the compatible table
> > using the omap3630 structure instead of the 3430.
> >
> > Signed-off-by: Adam Ford <aford173@gmail.com>
> >
> > diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c
> > index f2f58d689320..6b69fb1d6bdf 100644
> > --- a/drivers/cpufreq/ti-cpufreq.c
> > +++ b/drivers/cpufreq/ti-cpufreq.c
> > @@ -270,6 +270,7 @@ static int ti_cpufreq_setup_syscon_register(struct ti_cpufreq_data *opp_data)
> >
> > static const struct of_device_id ti_cpufreq_of_match[] = {
> >       { .compatible = "ti,am33xx", .data = &am3x_soc_data, },
> > +     { .compatible = "ti,am3517", .data = &omap36xx_soc_data, },
> >       { .compatible = "ti,am43", .data = &am4x_soc_data, },
> >       { .compatible = "ti,dra7", .data = &dra7_soc_data },
> >       { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, },
> > --
> > 2.17.1
> >
>
> Looks good to me.
>
> Should I include your two patches to my patch set (and resend)?
> Or should we keep them separated?

Go ahead and take credit for them.  I just did what you told me to do
and tested them.
Go ahead and mark it as Tested-by with my name.

adam
>
> BR and thanks,
> Nikolaus
>

Re: [PATCH 1/2] cpufreq: ti-cpufreq: Add support for AM3517

[H. Nikolaus Schaller]

Hi,

> Am 09.09.2019 um 20:17 schrieb Adam Ford <aford173@gmail.com>:
> 
> On Mon, Sep 9, 2019 at 1:13 PM H. Nikolaus Schaller <hns@goldelico.com> wrote:
>> 
>> Hi Adam,
>> 
>>> Am 09.09.2019 um 17:45 schrieb Adam Ford <aford173@gmail.com>:
>>> 
>>> The AM3517 only lists 600MHz @ 1.2V, but the revister values for
>> 
>> a small typo...
>> 
>> s/revister/register/
>> 
>>> 0x4830A204 = 1b86 802f, it seems like am3517 might be a derivative
>>> of the omap36 which has OPPs would be OPP50 (300 MHz) and OPP100
>>> (600 MHz).
>>> 
>>> This patch simply enable adds the am3517 to the compatible table
>>> using the omap3630 structure instead of the 3430.
>>> 
>>> Signed-off-by: Adam Ford <aford173@gmail.com>
>>> 
>>> diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c
>>> index f2f58d689320..6b69fb1d6bdf 100644
>>> --- a/drivers/cpufreq/ti-cpufreq.c
>>> +++ b/drivers/cpufreq/ti-cpufreq.c
>>> @@ -270,6 +270,7 @@ static int ti_cpufreq_setup_syscon_register(struct ti_cpufreq_data *opp_data)
>>> 
>>> static const struct of_device_id ti_cpufreq_of_match[] = {
>>>      { .compatible = "ti,am33xx", .data = &am3x_soc_data, },
>>> +     { .compatible = "ti,am3517", .data = &omap36xx_soc_data, },
>>>      { .compatible = "ti,am43", .data = &am4x_soc_data, },
>>>      { .compatible = "ti,dra7", .data = &dra7_soc_data },
>>>      { .compatible = "ti,omap34xx", .data = &omap34xx_soc_data, },
>>> --
>>> 2.17.1
>>> 
>> 
>> Looks good to me.
>> 
>> Should I include your two patches to my patch set (and resend)?
>> Or should we keep them separated?
> 
> Go ahead and take credit for them.  I just did what you told me to do
> and tested them.

Well, you are still the Author, the one who wrote down the ideas :)
I think git will even keep the author name untouched.

> Go ahead and mark it as Tested-by with my name.

Best is to keep your Signed-Off + mine and add a tested-by as well :)

BR and thanks,
Nikolaus