On Tue, May 14, 2019 at 7:01 PM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>
> On Tue, May 14, 2019 at 04:53:10PM +0200, Sedat Dilek wrote:
> > On Tue, May 14, 2019 at 3:57 PM Oleg Zhurakivskyy
> > <oleg.zhurakivskyy@xxxxxxxxx> wrote:
>
> > What about this one?
> >
> > diff --git a/drivers/nfc/nxp-nci/Kconfig b/drivers/nfc/nxp-nci/Kconfig
> > index a28c4265354d..b9e6486aa8fe 100644
> > --- a/drivers/nfc/nxp-nci/Kconfig
> > +++ b/drivers/nfc/nxp-nci/Kconfig
> > @@ -2,8 +2,8 @@ config NFC_NXP_NCI
> > tristate "NXP-NCI NFC driver"
> > depends on NFC_NCI
> > ---help---
> > - Generic core driver for NXP NCI chips such as the NPC100
> > - or PN7150 families.
> > + Generic core driver for NXP NCI chips such as the NPC100 (PN547),
> > + NPC300 (PN548) or PN7150 families.
> > This is a driver based on the NCI NFC kernel layers and
> > will thus not work with NXP libnfc library.
> >
> > @@ -19,6 +19,11 @@ config NFC_NXP_NCI_I2C
> > chips.
> > Select this if your platform is using the I2C bus.
> >
> > + Furthermore, the pin control and GPIO driver of the actual SoC or
> > + PCH is needed.
> > + For example set CONFIG_PINCTRL_SUNRISEPOINT=y to activate the
> > + Intel Sunrisepoint (PCH of Intel Skylake) pinctrl and GPIO driver.
> > +
>
> Besides some indentation problems (the help lines should be prefixed with
> 'TAB + 2 spaces'), this is not needed — it's obvious and usually distros
> provide all of pin control drivers anyway.
>
> For debugging one may check deferred devices via DebugFS, or use
> 'initcall_debug', or other facilities.
>
> > To compile this driver as a module, choose m here. The module will
> > be called nxp_nci_i2c.
> > - Say Y if unsure.
> > + Say N if unsure.
>
>
> --
> With Best Regards,
> Andy Shevchenko
>
>
Excellent eyes (TAB + 2 spaces)!
So, this is enough?
$ git diff
diff --git a/drivers/nfc/nxp-nci/Kconfig b/drivers/nfc/nxp-nci/Kconfig
index a28c4265354d..d85a4761e271 100644
--- a/drivers/nfc/nxp-nci/Kconfig
+++ b/drivers/nfc/nxp-nci/Kconfig
@@ -2,8 +2,8 @@ config NFC_NXP_NCI
tristate "NXP-NCI NFC driver"
depends on NFC_NCI
---help---
- Generic core driver for NXP NCI chips such as the NPC100
- or PN7150 families.
+ Generic core driver for NXP NCI chips such as the PN547 (NPC100),
+ PN548 (NPC300) or PN7150 families.
This is a driver based on the NCI NFC kernel layers and
will thus not work with NXP libnfc library.
@@ -21,4 +21,4 @@ config NFC_NXP_NCI_I2C
To compile this driver as a module, choose m here. The module will
be called nxp_nci_i2c.
- Say Y if unsure.
+ Say N if unsure.
Shall I sent a patch for this, or do you want to that yourself?
Thanks.
- Sedat -
