Re: [PATCH v2] ADP1653 board code for Nokia RX-51

[Pali RohÃr]

On Wednesday 18 September 2013 19:42:12 Tony Lindgren wrote:
> * Pali RohÃr <pali.rohar@xxxxxxxxx> [130918 09:08]:
> > On Wednesday 18 September 2013 15:16:44 Pavel Machek wrote:
> > > On Sun 2013-09-08 02:02:52, Aaro Koskinen wrote:
> > > > Hi,
> > > > 
> > > > On Fri, Sep 06, 2013 at 10:34:05PM +0200, Pali RohÃr 
wrote:
> > > > > > --- /dev/null
> > > > > > +++ b/arch/arm/mach-omap2/board-rx51-camera.c
> > > > 
> > > > [...]
> > > > 
> > > > > Ping, can you review this patch v2?
> > > > 
> > > > I don't think Tony will accept any new board stuff for
> > > > RX-51/N900. See for example:
> > > > http://marc.info/?l=linux-kernel&m=137629626213187&w=2
> > > > 
> > > > There should be initial Nokia N900 DTS file in 3.12-rc1,
> > > > and we should continue converting this board fully to
> > > > DT.
> > > 
> > > That's not reasonable. N900 is pretty far away from
> > > working with device tree. Yes, I was able to get some
> > > bits merged, but that should not be excuse to halt other
> > > N900 development.
> > > 
> > > 									Pavel
> > 
> > Right. So Tony, will you accept future patches for board
> > files?
> 
> Only fixes to board-*.c files please unless there's a _really_
> good reason to make things more complex with the platform
> data. Let's not make the DT conversion any more complex than
> it already is.
> 
> Getting things working with DT for the devices is pretty
> trivial in most cases. And we can also still initialize the
> legacy devices using platform data the same way as earlier.
> 
> Regards,
> 
> Tony

So, what to this with this patch v2 (it was sent 06.03.2013)?
It adding board support for ADP1653 flash torch which is part of 
rx51 camera code. Driver itself is already in kernel tree.
Can you comment/review this patch?

-- 
Pali RohÃr
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.