mbox series

[0/3] ACPI / utils: Add acpi_evaluate_reg() helper

Message ID 20200505132128.19476-1-hdegoede@redhat.com (mailing list archive)
Headers show
Series ACPI / utils: Add acpi_evaluate_reg() helper | expand

Message

Hans de Goede May 5, 2020, 1:21 p.m. UTC
Hi All,

Here is a small series adding an acpi_evaluate_reg() helper, note
the third patch sits on top of a fix for the pinctrl-cherryview
driver which I recently submitted and which is still finding its
way upstream.

Since this is not urgent (just a small code cleanup) I suggest
that the ACPI people can pick up patches 1-2 and then the last patch
can be merged post 5.8-rc1, at which point all the dependencies for
it should have landed already.

Regards,

Hans

Comments

Andy Shevchenko May 5, 2020, 3:44 p.m. UTC | #1
On Tue, May 05, 2020 at 03:21:25PM +0200, Hans de Goede wrote:
> Hi All,
> 
> Here is a small series adding an acpi_evaluate_reg() helper, note
> the third patch sits on top of a fix for the pinctrl-cherryview
> driver which I recently submitted and which is still finding its
> way upstream.
> 
> Since this is not urgent (just a small code cleanup) I suggest
> that the ACPI people can pick up patches 1-2 and then the last patch
> can be merged post 5.8-rc1, at which point all the dependencies for
> it should have landed already.

Thank you!

Some minor comments to be addressed.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Hans de Goede May 7, 2020, 10:22 a.m. UTC | #2
Hi,

On 5/5/20 5:44 PM, Andy Shevchenko wrote:
> On Tue, May 05, 2020 at 03:21:25PM +0200, Hans de Goede wrote:
>> Hi All,
>>
>> Here is a small series adding an acpi_evaluate_reg() helper, note
>> the third patch sits on top of a fix for the pinctrl-cherryview
>> driver which I recently submitted and which is still finding its
>> way upstream.
>>
>> Since this is not urgent (just a small code cleanup) I suggest
>> that the ACPI people can pick up patches 1-2 and then the last patch
>> can be merged post 5.8-rc1, at which point all the dependencies for
>> it should have landed already.
> 
> Thank you!
> 
> Some minor comments to be addressed.
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Thank you, I will fix the remarks you have and send out a v2
with just patches 1 and 2 for now, then we can move forward
with those.

How to deal with the _REG not being called issue by the ACPICA
core issue on Cherryview devices is still being discussed, so
lets wait for the final fix for that and then I can send out
a new version of the 3th patch on top of that once the other
bits are merged.

Regards,

Hans