Re: sh: x3 - fix setup_bootmem_node() compile error with shx3_defconfig

[Magnus D. <mag...@gm...>]

On 8/12/07, Paul Mundt <le...@li...> wrote:
> On Fri, Aug 10, 2007 at 08:31:55PM +0900, Magnus Damm wrote:
> > sh: x3 - fix setup_bootmem_node() compile error with shx3_defconfig
>
> On Fri, Aug 10, 2007 at 08:41:43PM +0900, Magnus Damm wrote:
> > sh: intc - add a clear register to struct intc_prio_reg
>
> On Fri, Aug 10, 2007 at 08:53:17PM +0900, Magnus Damm wrote:
> > sh: intc - rework core code
>
> On Fri, Aug 10, 2007 at 09:05:24PM +0900, Magnus Damm wrote:
> > sh: intc - convert board specific r2d code V2
>
> On Fri, Aug 10, 2007 at 09:10:27PM +0900, Magnus Damm wrote:
> > sh: replace sh specific CONFIG_VOYAGERGX with CONFIG_MFD_SM501
>
> On Fri, Aug 10, 2007 at 09:17:18PM +0900, Magnus Damm wrote:
> > sh: x3 - add ipi vectors
>
> These have been applied and pushed out, thanks.

Great, thank you!

> A few minor nits.. there was quite a bit of whitespace damage and other
> bits that checkpatch complained rather loudly about. Please remember to
> run the patches through that first so I don't have to clean them up by
> hand. While checkpatch does have a tendency to be stupid about some
> things, it does catch some fairly obvious stuff. In general, if
> checkpatch is happy, I'm happy.

Oh, right... I checked all my previous patches with checkpatch, but
with this batch I was in a hurry and forgot that step. Sorry about
that, I'll try to remember that in the future. I don't want to give
you boring whitespace work... =)

> Now that the R2D stuff is split up, you should provide updated defconfigs
> for both versions, rts7751r2d_defconfig should match the default choice
> for the board version (though it might be less ambiguous if we were to
> rename that particular defconfig to something closer to the config names).

Yep, good idea. I'll cook up a patch for that.

/ magnus