From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH v3 4/4] i2c: Add multi-instantiate pseudo driver Date: Wed, 8 Aug 2018 10:05:56 +0200 Message-ID: References: <20180807080539.17811-1-hdegoede@redhat.com> <20180807080539.17811-5-hdegoede@redhat.com> <20180807112701.5iukqa6m5atf22fw@ninjato> <9e17dfca-901b-79c9-b562-4f893d7ae44b@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Andy Shevchenko , Wolfram Sang Cc: "Rafael J . Wysocki" , Len Brown , Mika Westerberg , Darren Hart , Srinivas Pandruvada , linux-acpi@vger.kernel.org, platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, Heikki Krogerus , John Garry , linux-i2c@vger.kernel.org List-Id: linux-i2c@vger.kernel.org Hi, On 07-08-18 13:45, Andy Shevchenko wrote: > On Tue, 2018-08-07 at 13:33 +0200, Hans de Goede wrote: >> Hi, >> >> On 07-08-18 13:27, Wolfram Sang wrote: >>> Hi Hans, >>> >>>> +I2C MULTI INSTANTIATE DRIVER > > In the case below, I think it would be slightly better to mention ACPI > here, something like > > ACPI I2C ... > > Otherwise it sounds a bit confusing why it's in PDx86. Ok, fixed for the upcoming v4 of this patchset. Regards, Hans > >>>> +M: Hans de Goede >>>> +L: linux-i2c@vger.kernel.org >>> >>> Since this driver is about FW bugs, I wonder if >>> platform-driver-x86@vger.kernel.org isn't the better list here? >> >> Ok, fixed for v4. >> >>>> +S: Maintained >>>> +F: drivers/drivers/platform/x86/i2c-multi-instantiate.c >>> >>> I2C parts of the rest looks good from a glimpse... >> >> Thanks. >> >> Regards, >> >> Hans >