Re: [PATCH - v2] spi: davinci: add OF support for the spi controller

From: Grant Likely
Date: Wed Dec 19 2012 - 08:18:50 EST


On Sat, 15 Dec 2012 00:27:58 +0000, Grant Likely <grant.likely@xxxxxxxxxxxx> wrote:
> On Tue, 11 Dec 2012 16:20:39 -0500, Murali Karicheri <m-karicheri2@xxxxxx> wrote:
> > This adds OF support to DaVinci SPI controller to configure platform
> > data through device bindings. Also replaces clk_enable() with
> > of clk_prepare_enable() as well as clk_disable() with
> > clk_disable_unprepare().
> >
> > Signed-off-by: Murali Karicheri <m-karicheri2@xxxxxx>
> > Reviewed-by : Grant Likely <grant.likely@xxxxxxxxxxxx>
>
> Applied, thanks.
>
> I did remove the OF_ALIAS_N property though. I know the COMPATIBLE one
> uses it, but it is actually kind of redundant since it can also be
> determined by counting the number of OF_ALIAS_* entries, and having the
> _N one in there means extra work needs to be done to filter it out.
>
> Also, I had to add a #ifndef _LINUX_OF_PRIVATE_H wrapper around the
> whole header file. This is needed for all header files to protect
> against multiple includes.

Ummm.... oops. I replied to the wrong email when applying another patch.
Sorry for the noise.

But I have applied your patch now. It will appear in linux-next after
the merge window closes.

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/