[GIT PULL] SPEAr platform fixes over 3.5-rc7
Olof Johansson
olof at lixom.net
Tue Jul 17 23:43:34 EDT 2012
Hi Shiraz,
I see that every commit in that branch is done by you, but not a
single one has a Signed-off-by by you. Please fix that, it's important
to track the history of how code is introduced to the kernel.
Also, we are _very_ late in the 3.5 release cycle now. Only truly
critical fixes can go in (3.5 is likely to come out by the weekend). I
have some comments about the patches below.
In general, to make our life easier, please make sure the commit
message for the patch describes why the fix is needed when it's not
obvious.
I'll wait with pulling until you have a chance to fix up your branch
based on the below comments. Please make sure you do it with extreme
expediency though, or chances are it'll miss 3.5.
Thanks,
-Olof
On Tue, Jul 17, 2012 at 6:01 AM, Shiraz Hashim <shiraz.hashim at st.com> wrote:
> Vipul Kumar Samar (10):
> clk:spear1340:Fix: Rename clk ids within predefined limit
> clk:spear1310:Fix: Rename clk ids within predefined limit
> Clk:spear3xx:Fix: Rename clk ids within predefined limit
> Clk:spear6xx:Fix: Rename clk ids within predefined limit
Ok, those 4 are quite obvious why they're needed.
> ARM: SPEAr13xx: Fix Interrupt bindings
> clk: SPEAr1340: Fix clk enable register for uart1 and i2c1.
Those two as well.
> clk: SPEAr13xx: Add localtimer (twd) clock support
This doesn't look like a critical regression fix, does it?
> Clk: SPEAr1340: Update sys clock parent array
Please help us out here by describing what is broken by the way it was
before the patch.
> ARM: dts: SPEAr320: Fix compatible string
This one seems straightforward.
> ARM: dts: SPEAr320: Boot the board in EXTENDED_MODE
Same here, please elaborate on why / how it is broken today.
Thanks,
-Olof
More information about the linux-arm-kernel
mailing list