[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190627081831.GF54126@ediswmail.ad.cirrus.com>
Date: Thu, 27 Jun 2019 09:18:31 +0100
From: Charles Keepax <ckeepax@...nsource.cirrus.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
CC: <wsa@...-dreams.de>, <mika.westerberg@...ux.intel.com>,
<jarkko.nikula@...ux.intel.com>, <linux-i2c@...r.kernel.org>,
<linux-acpi@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<benjamin.tissoires@...hat.com>, <jbroadus@...il.com>,
<patches@...nsource.cirrus.com>
Subject: Re: [PATCH v7 2/6] i2c: acpi: Use available IRQ helper functions
On Wed, Jun 26, 2019 at 10:09:06PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 26, 2019 at 04:02:58PM +0100, Charles Keepax wrote:
> > Use the available IRQ helper functions, most of the functions have
> > additional helpful side affects like configuring the trigger type of the
> > IRQ.
>
> > - acpi_dev_free_resource_list(&resource_list);
>
> I have thought that we can drop completely the resource_list, though ACPI
> doesn't allow this. So, I would rather withdraw this change to be on the safe
> (symmetrical) side.
>
Yeah would be my preference as well, will do another spin.
Thanks,
Charles
Powered by blists - more mailing lists