Re: [PATCH] gpio: pca953x: add a comment explaining the need for a lockdep subclass
From: Bartosz Golaszewski
Date: Thu Sep 29 2016 - 03:57:09 EST
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!
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.