On Tue, Oct 02, 2012 at 11:50:02AM -0600, Jason Gunthorpe wrote:

On Mon, Oct 01, 2012 at 07:31:58AM -0700, Olof Johansson wrote:

quoted

Ah. diff took out the #else in the middle and I didn't look at the
file. Obviously correct as-is.

In some other places we choose to put the ifdef outside the function
and provide a static inline stub in the header file instead. Either
way works fine though.

I don't actually expect anyone will run CONFIG_ARCH_KIRKWOOD_DT
without CACHE_FEROCEON_L2 - I had thought about just forcing kconfig
to include CACHE_FEROCEON_L2, but figured there was some reason they
were separate.. 

Is there anything further I should do to get this patch upstream?

nope, it's in the queue.

thx,

Jason.