2014-06-22 19:50:08

by Andreas Färber

Subject: [PATCH] ARM: dts: exynos5410: Fill in CPU clock-frequency

It's 1.6 GHz for the Cortex-A15.

Avoids warnings like "/cpus/cpu@0 missing clock-frequency property".

Signed-off-by: Andreas Färber <[email protected]>
---
arch/arm/boot/dts/exynos5410.dtsi | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5410.dtsi b/arch/arm/boot/dts/exynos5410.dtsi
index 3839c26..9d0b8cc 100644
--- a/arch/arm/boot/dts/exynos5410.dtsi
+++ b/arch/arm/boot/dts/exynos5410.dtsi
@@ -28,24 +28,28 @@
device_type = "cpu";
compatible = "arm,cortex-a15";
reg = <0x0>;
+ clock-frequency = <1600000000>;
};

CPU1: cpu@1 {
device_type = "cpu";
compatible = "arm,cortex-a15";
reg = <0x1>;
+ clock-frequency = <1600000000>;
};

CPU2: cpu@2 {
device_type = "cpu";
compatible = "arm,cortex-a15";
reg = <0x2>;
+ clock-frequency = <1600000000>;
};

CPU3: cpu@3 {
device_type = "cpu";
compatible = "arm,cortex-a15";
reg = <0x3>;
+ clock-frequency = <1600000000>;
};
};

--
1.9.3

2014-06-23 07:53:41

by Tarek Dakhran

Subject: Re: [PATCH] ARM: dts: exynos5410: Fill in CPU clock-frequency

On 06/22/2014 11:49 PM, Andreas Färber wrote:
> It's 1.6 GHz for the Cortex-A15.
>
> Avoids warnings like "/cpus/cpu@0 missing clock-frequency property".
>
> Signed-off-by: Andreas Färber <[email protected]>
> ---
> arch/arm/boot/dts/exynos5410.dtsi | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5410.dtsi b/arch/arm/boot/dts/exynos5410.dtsi
> index 3839c26..9d0b8cc 100644
> --- a/arch/arm/boot/dts/exynos5410.dtsi
> +++ b/arch/arm/boot/dts/exynos5410.dtsi
> @@ -28,24 +28,28 @@
> device_type = "cpu";
> compatible = "arm,cortex-a15";
> reg = <0x0>;
> + clock-frequency = <1600000000>;
>

Reviewed-by: Tarek Dakhran<[email protected]>

--
Best regards,
Tarek Dakhran

2014-07-04 11:32:15

by Andreas Färber

Subject: Re: [PATCH] ARM: dts: exynos5410: Fill in CPU clock-frequency

Am 23.06.2014 09:53, schrieb Tarek Dakhran:
> On 06/22/2014 11:49 PM, Andreas Färber wrote:
>> It's 1.6 GHz for the Cortex-A15.
>>
>> Avoids warnings like "/cpus/cpu@0 missing clock-frequency property".
>>
>> Signed-off-by: Andreas Färber <[email protected]>
>> ---
>> arch/arm/boot/dts/exynos5410.dtsi | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/exynos5410.dtsi
>> b/arch/arm/boot/dts/exynos5410.dtsi
>> index 3839c26..9d0b8cc 100644
>> --- a/arch/arm/boot/dts/exynos5410.dtsi
>> +++ b/arch/arm/boot/dts/exynos5410.dtsi
>> @@ -28,24 +28,28 @@
>> device_type = "cpu";
>> compatible = "arm,cortex-a15";
>> reg = <0x0>;
>> + clock-frequency = <1600000000>;
>>
>
> Reviewed-by: Tarek Dakhran<[email protected]>

Ping? Seems really trivial...

Andreas

--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

2014-07-04 21:06:46

by Kukjin Kim

Subject: Re: [PATCH] ARM: dts: exynos5410: Fill in CPU clock-frequency

On 07/04/14 20:32, Andreas Färber wrote:
> Am 23.06.2014 09:53, schrieb Tarek Dakhran:
>> On 06/22/2014 11:49 PM, Andreas Färber wrote:
>>> It's 1.6 GHz for the Cortex-A15.
>>>
>>> Avoids warnings like "/cpus/cpu@0 missing clock-frequency property".
>>>
>>> Signed-off-by: Andreas [email protected]>
>>> ---
>>> arch/arm/boot/dts/exynos5410.dtsi | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/exynos5410.dtsi
>>> b/arch/arm/boot/dts/exynos5410.dtsi
>>> index 3839c26..9d0b8cc 100644
>>> --- a/arch/arm/boot/dts/exynos5410.dtsi
>>> +++ b/arch/arm/boot/dts/exynos5410.dtsi
>>> @@ -28,24 +28,28 @@
>>> device_type = "cpu";
>>> compatible = "arm,cortex-a15";
>>> reg =<0x0>;
>>> + clock-frequency =<1600000000>;
>>>
>>
>> Reviewed-by: Tarek Dakhran<[email protected]>
>
> Ping? Seems really trivial...
>
Will apply.

BTW Andreas, I can't use 'Färber' in my git tree so which one is right
or more preferred? Farber, Faerber, Ferber? Sorry.

Thanks,
Kukjin

2014-07-15 12:57:21

by Andreas Färber

Subject: Re: [PATCH] ARM: dts: exynos5410: Fill in CPU clock-frequency

Hi Kukjin,

Am 04.07.2014 23:06, schrieb Kukjin Kim:
> BTW Andreas, I can't use 'Färber' in my git tree so which one is right
> or more preferred? Farber, Faerber, Ferber? Sorry.

Hm, git-am should in theory just work fine for this patch with UTF-8 on
Linux. As for Reviewed-by / Tested-by, some people had success with
copy&paste, otherwise Faerber as in the email please.

Thanks,
Andreas

--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg