On Thu, Sep 29, 2016 at 09:56:58AM +0200, Bartosz Golaszewski wrote: > 2016-09-29 8:06 GMT+02:00 Wolfram Sang <wsa@xxxxxxxxxxxxx>: > > On Mon, Sep 26, 2016 at 12:00:30PM +0200, Wolfram Sang wrote: > >> On Mon, Sep 26, 2016 at 11:54:15AM +0200, Bartosz Golaszewski wrote: > >> > This is a follow-up to commit 559b46990e76 ("gpio: pca953x: fix an > >> > incorrect lockdep warning"). The reason for calling > >> > lockdep_set_subclass() in pca953x_probe() is not explained in > >> > the code. > >> > > >> > Add a comment describing the problem, partial solution and required > >> > future extensions. > >> > > >> > Signed-off-by: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> > >> > >> Acked-by: Wolfram Sang <wsa@xxxxxxxxxxxxx> > >> > >> Linus, because of dependencies, I should probably pick it up? > > > > Linus, ping! > > > > Hi Wolfram, > > this patch will not apply to the gpio tree. If Linus is too busy to > comment, maybe you could pick it up anyway - it doesn't change > anything in terms of functionality and it's better to have the code > commented when it's not obvious what it does. I want to pick it up and am likely to do so anyway, but to be formally perfect, I need an ack from Linus.
Attachment:
signature.asc
Description: PGP signature