| Message ID | 20130308121317.4ee83c0a@armhf |
|---|---|
| State | New |
| Headers | show |
Hello. On 08-03-2013 15:13, Jean-Francois Moine wrote: > The commit 48be9ac930086f Please also specify the summary line for that commit in parens (or however you like). > removed the RTC initialization. > This patch re-enables the RTC via the DT. > Signed-off-by: Jean-François Moine <moinejf@free.fr> WBR, Sergei
On Fri, Mar 08, 2013 at 12:13:17PM +0100, Jean-Francois Moine wrote: > The commit 48be9ac930086f removed the RTC initialization. > This patch re-enables the RTC via the DT. > > Signed-off-by: Jean-François Moine <moinejf@free.fr> > --- > arch/arm/boot/dts/dove.dtsi | 5 +++++ > 1 file changed, 5 insertions(+) Applied to mvebu/fixes with the following commit entry: ARM: Dove: add RTC device node The commit: 48be9ac ARM: Dove: split legacy and DT setup removed the RTC initialization. This patch re-enables the RTC via the DT. thx, Jason.
On Friday 08 March 2013, Jason Cooper wrote: > Applied to mvebu/fixes with the following commit entry: > > ARM: Dove: add RTC device node > > The commit: > > 48be9ac ARM: Dove: split legacy and DT setup > > removed the RTC initialization. This patch re-enables the RTC > via the DT. Looks good. FWIW, the style I tend to use is |ARM: Dove: add RTC device node | |The commit 48be9ac "ARM: Dove: split legacy and DT setup" removed the RTC |initialization. This patch re-enables the RTC via the DT. I don't think that there is a wide agreement on how to refer to other patches yet, but I do a lot of regression fixes like that, so maybe I can start a trend here. ARnd
On Fri, Mar 08, 2013 at 11:12:02PM +0000, Arnd Bergmann wrote: > On Friday 08 March 2013, Jason Cooper wrote: > > Applied to mvebu/fixes with the following commit entry: > > > > ARM: Dove: add RTC device node > > > > The commit: > > > > 48be9ac ARM: Dove: split legacy and DT setup > > > > removed the RTC initialization. This patch re-enables the RTC > > via the DT. > > Looks good. FWIW, the style I tend to use is > > > |ARM: Dove: add RTC device node > | > |The commit 48be9ac "ARM: Dove: split legacy and DT setup" removed the RTC > |initialization. This patch re-enables the RTC via the DT. > > I don't think that there is a wide agreement on how to refer to other > patches yet, but I do a lot of regression fixes like that, so maybe I > can start a trend here. Looks good to me. flows a bit better than what I had. I'll use that from now on for anything I come across. thx, Jason.
diff --git a/arch/arm/boot/dts/dove.dtsi b/arch/arm/boot/dts/dove.dtsi index 46afade..28ff4dc 100644 --- a/arch/arm/boot/dts/dove.dtsi +++ b/arch/arm/boot/dts/dove.dtsi @@ -197,6 +197,11 @@ status = "disabled"; }; + rtc@d8500 { + compatible = "marvell,orion-rtc"; + reg = <0xd8500 0x20>; + }; + crypto: crypto@30000 { compatible = "marvell,orion-crypto"; reg = <0x30000 0x10000>,
The commit 48be9ac930086f removed the RTC initialization. This patch re-enables the RTC via the DT. Signed-off-by: Jean-François Moine <moinejf@free.fr> --- arch/arm/boot/dts/dove.dtsi | 5 +++++ 1 file changed, 5 insertions(+)