On Tue, 2017-03-07 at 12:13 +0200, Andy Shevchenko wrote: > Seems this driver has never been even compile tested. > > Fix obvious typo in macro to make it defined.
Ah, now I got it. It's defined in removed platform data file :-(. I will fix this in v2. > > Signed-off-by: Andy Shevchenko <[email protected]> > --- > drivers/nfc/st21nfca/i2c.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/nfc/st21nfca/i2c.c b/drivers/nfc/st21nfca/i2c.c > index 737384d287aa..7f68a5fe15d8 100644 > --- a/drivers/nfc/st21nfca/i2c.c > +++ b/drivers/nfc/st21nfca/i2c.c > @@ -286,7 +286,7 @@ static int check_crc(u8 *buf, int buflen) > crc = ~crc; > > if (buf[buflen - 2] != (crc & 0xff) || buf[buflen - 1] != > (crc >> 8)) { > - pr_err(ST21NFCA_HCI_DRIVER_NAME > + pr_err(ST21NFCA_HCI_I2C_DRIVER_NAME > ": CRC error 0x%x != 0x%x 0x%x\n", crc, > buf[buflen - 1], > buf[buflen - 2]); > > @@ -672,7 +672,7 @@ static int st21nfca_hci_i2c_probe(struct > i2c_client *client, > r = devm_request_threaded_irq(&client->dev, client->irq, > NULL, > st21nfca_hci_irq_thread_fn, > phy->irq_polarity | IRQF_ONESHOT, > - ST21NFCA_HCI_DRIVER_NAME, phy); > + ST21NFCA_HCI_I2C_DRIVER_NAME, phy); > if (r < 0) { > nfc_err(&client->dev, "Unable to register IRQ > handler\n"); > return r; > @@ -701,7 +701,7 @@ static int st21nfca_hci_i2c_remove(struct > i2c_client *client) > } > > static struct i2c_device_id st21nfca_hci_i2c_id_table[] = { > - {ST21NFCA_HCI_DRIVER_NAME, 0}, > + {ST21NFCA_HCI_I2C_DRIVER_NAME, 0}, > {} > }; > MODULE_DEVICE_TABLE(i2c, st21nfca_hci_i2c_id_table); -- Andy Shevchenko <[email protected]> Intel Finland Oy
