lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 22 Mar 2021 02:05:10 +0000 From: Ran Wang <ran.wang_1@....com> To: Andy Shevchenko <andy.shevchenko@...il.com> CC: Linus Walleij <linus.walleij@...aro.org>, Bartosz Golaszewski <bgolaszewski@...libre.com>, Michael Walle <michael@...le.cc>, "open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org> Subject: RE: [PATCH v3] gpio: mpc8xxx: Add ACPI support 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@....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
Powered by blists - more mailing lists