[PATCH 1/6] ARM: mx27: Replace clk_register_clkdev with clock DT lookup
Fabio Estevam
festevam at gmail.com
Fri Jun 21 01:41:29 EST 2013
On Thu, Jun 20, 2013 at 12:35 PM, Fabio Estevam <festevam at gmail.com> wrote:
> On Thu, Jun 20, 2013 at 12:28 PM, Alexander Shiyan <shc_work at mail.ru> wrote:
>>> From: Fabio Estevam <fabio.estevam at freescale.com>
>>>
>>> Similarly as it was done for mx6q, use a DT lookup in order to make maintainance
>>> task for the clock devices easier.
>>>
>>> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
>>> [Markus Pargmann: Add gpio clocks]
>>> Signed-off-by: Markus Pargmann <mpa at pengutronix.de>
>>> ---
>>> arch/arm/boot/dts/imx27.dtsi | 12 ++++++++++++
>>> 1 file changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
>>> index 0695264..48c263c 100644
>>> --- a/arch/arm/boot/dts/imx27.dtsi
>>> +++ b/arch/arm/boot/dts/imx27.dtsi
>>> @@ -208,6 +208,8 @@
>>> #gpio-cells = <2>;
>>> interrupt-controller;
>>> #interrupt-cells = <2>;
>>> + clocks = <&clks 47>;
>>> + clock-names = "ipg";
>>
>> Clock is not used in the gpio driver, so I doubt the usefulness of this patch.
>
> I think you meant "this hunk", right?
>
> My original patch did not have this part. It was added by Markus as
> mentioned in the commit log, and I agree we should go with the
> original version that did not add the gpio clock.
Just checked and my original patch has already been applied:
commit c20736f1aba75bd4251e96ef3f487beebbaf41cc
Author: Fabio Estevam <fabio.estevam at freescale.com>
Date: Wed Nov 28 15:55:30 2012 -0200
ARM: mx27: Replace clk_register_clkdev with clock DT lookup
Similarly as it was done for mx6q, use a DT lookup in order to make maintain
task for the clock devices easier.
Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
Tested-by: Markus Pargmann <mpa at pengutronix.de>
Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
, and it does not add the gpio clocks.
More information about the devicetree-discuss
mailing list