Re: [PATCH] ACPI / GPIO: Driver GPIO mappings for ACPI GPIOs

From: Alexandre Courbot
Date: Sun Nov 16 2014 - 23:29:22 EST


On Fri, Nov 14, 2014 at 5:58 PM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
> On Mon, Nov 3, 2014 at 11:56 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
>> On Monday, November 03, 2014 02:22:10 PM Linus Walleij wrote:
>
>>> With that change:
>>> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>>
>> OK, made the changes and added your Reviewed-by.
>>
>> One semi-related question though. Alexandre ACKed the patch before, so
>> what did that mean from the GPIO subsystem's perspective? Was the patch
>> approved, not approved, semi-approved?
>
> It just means we are not always entirely aligned, it's not like
> Alexandre and I have meetings on the side and decide what to
> say, it's just these mails. I do trust him, I wouldn't get mad if it
> had been merged just wanted some polishing when I could get
> it...

In case of conflicting feedback between me and Linus, it goes without
saying that Linus' opinion should prevail. I am trying to help a bit
and take some of the burden off his shoulders, but have neither his
skill nor his experience (which shows when you compare our patch
reviews). So for important stuff it is a good idea to wait until both
of us give a ack - the most likely conflict scenario being that I give
a pass overlooking something that Linus will catch.
--
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/