lkml.org 
[lkml]   [2021]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v3] gpio: mpc8xxx: Add ACPI support
Date
Hi Andy,

On Friday, March 19, 2021 6:32 PM, Andy Shevchenko wrote:
>
> On Fri, Mar 19, 2021 at 10:10 AM Ran Wang <ran.wang_1@nxp.com> wrote:
> >
> > Current implementation only supports DT, now add ACPI support.
>
> Thanks for an update, my comments below.
>
> ...
>
> > +#include <linux/acpi.h>
>
> Missed
> property.h
> mod_devicetable.h.

Got it.

> ...
>
> > - mpc8xxx_gc->regs = of_iomap(np, 0);
> > + mpc8xxx_gc->regs = devm_platform_ioremap_resource(pdev, 0);
>
> > if (!mpc8xxx_gc->regs)
>
> This is wrong now.

Yes, will correct this in next version

> > return -ENOMEM;
>
> This too.

Got it.

> ...
>
> > + fwnode = dev_fwnode(&pdev->dev);
> > if (of_device_is_compatible(np, "fsl,qoriq-gpio") ||
> > of_device_is_compatible(np, "fsl,ls1028a-gpio") ||
> > - of_device_is_compatible(np, "fsl,ls1088a-gpio"))
> > + of_device_is_compatible(np, "fsl,ls1088a-gpio") ||
>
> > + !(IS_ERR_OR_NULL(fwnode) || is_of_node(fwnode)))
>
> Since you left acpi.h inclusion, you may switch this to
>
> is_acpi_node(fwnode)
>
> or drop fwnode and use
>
> has_acpi_companion(&pdev->dev)

OK, will use is_acpi_node(fwnode) instead (since fwnode is referred in following irq_domain_create_linear()

Thanks for your patient educate :)

Regards,
Ran

\
 
 \ /
  Last update: 2021-03-22 03:06    [W:0.038 / U:4.216 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site