Re: [PATCH RFC leds-next] leds: initial support for Turris Omnia LEDs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 28 Mar 2020 13:08:02 +0100
Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx> wrote:

> Hi Pavel.
> 
> On 3/21/20 4:33 PM, Pavel Machek wrote:
> > Hi!
> >   
> >> This adds basic support for LEDs on the front side of CZ.NIC's Turris
> >> Omnia router.  
> > 
> > Looks good, I had to apply it by hand, by I took it. I realize that
> > dts interface may not be final, but I do not want to solve Makefile
> > rejects in future.  
> 
> I don't think it is a good idea to merge it without bindings,
> and with DT parser that will certainly undergo essential changes.
> 
> Besides, you appear as the commit author in your tree and moreover
> the patch has a broken title. Also the commit message contains
> DT bindings that will have different shape eventually.
> 
> NACK for merging this patch in this shape.
> 

This was only RFC, please do not merge.



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux