* [PATCH 0/2] led: is31fl32xx: minor improvements
@ 2016-07-18 16:39 H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 1/2] led: is31fl32xx: fix typo in id and match table names H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 2/2] led: is31fl32xx: define complete i2c_device_id table H. Nikolaus Schaller
0 siblings, 2 replies; 3+ messages in thread
From: H. Nikolaus Schaller @ 2016-07-18 16:39 UTC (permalink / raw)
To: Richard Purdie, Jacek Anaszewski, Rob Herring, Pawel Moll,
Mark Rutland, Ian Campbell, Kumar Gala, drivshin.allworx
Cc: linux-leds, linux-kernel, kernel, marek, letux-kernel,
Andrey Utkin, H. Nikolaus Schaller
while developing the is31fl319x driver we found some minor issues relevant in the is31fl32xx driver.
H. Nikolaus Schaller (2):
led: is31fl32xx: fix typo in id and match table names
led: is31fl32xx: define complete i2c_device_id table
drivers/leds/leds-is31fl32xx.c | 24 +++++++++++++++---------
1 file changed, 15 insertions(+), 9 deletions(-)
--
2.7.3
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 1/2] led: is31fl32xx: fix typo in id and match table names
2016-07-18 16:39 [PATCH 0/2] led: is31fl32xx: minor improvements H. Nikolaus Schaller
@ 2016-07-18 16:39 ` H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 2/2] led: is31fl32xx: define complete i2c_device_id table H. Nikolaus Schaller
1 sibling, 0 replies; 3+ messages in thread
From: H. Nikolaus Schaller @ 2016-07-18 16:39 UTC (permalink / raw)
To: Richard Purdie, Jacek Anaszewski, Rob Herring, Pawel Moll,
Mark Rutland, Ian Campbell, Kumar Gala, drivshin.allworx
Cc: linux-leds, linux-kernel, kernel, marek, letux-kernel,
Andrey Utkin, H. Nikolaus Schaller
Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
---
drivers/leds/leds-is31fl32xx.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/leds/leds-is31fl32xx.c b/drivers/leds/leds-is31fl32xx.c
index c901d13..a53b71e 100644
--- a/drivers/leds/leds-is31fl32xx.c
+++ b/drivers/leds/leds-is31fl32xx.c
@@ -422,7 +422,7 @@ err:
return ret;
}
-static const struct of_device_id of_is31fl31xx_match[] = {
+static const struct of_device_id of_is31fl32xx_match[] = {
{ .compatible = "issi,is31fl3236", .data = &is31fl3236_cdef, },
{ .compatible = "issi,is31fl3235", .data = &is31fl3235_cdef, },
{ .compatible = "issi,is31fl3218", .data = &is31fl3218_cdef, },
@@ -432,7 +432,7 @@ static const struct of_device_id of_is31fl31xx_match[] = {
{},
};
-MODULE_DEVICE_TABLE(of, of_is31fl31xx_match);
+MODULE_DEVICE_TABLE(of, of_is31fl32xx_match);
static int is31fl32xx_probe(struct i2c_client *client,
const struct i2c_device_id *id)
@@ -444,7 +444,7 @@ static int is31fl32xx_probe(struct i2c_client *client,
int count;
int ret = 0;
- of_dev_id = of_match_device(of_is31fl31xx_match, dev);
+ of_dev_id = of_match_device(of_is31fl32xx_match, dev);
if (!of_dev_id)
return -EINVAL;
@@ -485,20 +485,20 @@ static int is31fl32xx_remove(struct i2c_client *client)
* i2c-core requires that id_table be non-NULL, even though
* it is not used for DeviceTree based instantiation.
*/
-static const struct i2c_device_id is31fl31xx_id[] = {
+static const struct i2c_device_id is31fl32xx_id[] = {
{},
};
-MODULE_DEVICE_TABLE(i2c, is31fl31xx_id);
+MODULE_DEVICE_TABLE(i2c, is31fl32xx_id);
static struct i2c_driver is31fl32xx_driver = {
.driver = {
.name = "is31fl32xx",
- .of_match_table = of_is31fl31xx_match,
+ .of_match_table = of_is31fl32xx_match,
},
.probe = is31fl32xx_probe,
.remove = is31fl32xx_remove,
- .id_table = is31fl31xx_id,
+ .id_table = is31fl32xx_id,
};
module_i2c_driver(is31fl32xx_driver);
--
2.7.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] led: is31fl32xx: define complete i2c_device_id table
2016-07-18 16:39 [PATCH 0/2] led: is31fl32xx: minor improvements H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 1/2] led: is31fl32xx: fix typo in id and match table names H. Nikolaus Schaller
@ 2016-07-18 16:39 ` H. Nikolaus Schaller
1 sibling, 0 replies; 3+ messages in thread
From: H. Nikolaus Schaller @ 2016-07-18 16:39 UTC (permalink / raw)
To: Richard Purdie, Jacek Anaszewski, Rob Herring, Pawel Moll,
Mark Rutland, Ian Campbell, Kumar Gala, drivshin.allworx
Cc: linux-leds, linux-kernel, kernel, marek, letux-kernel,
Andrey Utkin, H. Nikolaus Schaller
Otherwise the driver module will not be automatically probed.
Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
---
drivers/leds/leds-is31fl32xx.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/leds/leds-is31fl32xx.c b/drivers/leds/leds-is31fl32xx.c
index a53b71e..478844c 100644
--- a/drivers/leds/leds-is31fl32xx.c
+++ b/drivers/leds/leds-is31fl32xx.c
@@ -482,10 +482,16 @@ static int is31fl32xx_remove(struct i2c_client *client)
}
/*
- * i2c-core requires that id_table be non-NULL, even though
- * it is not used for DeviceTree based instantiation.
+ * i2c-core (and modalias) requires that id_table be properly filled,
+ * even though it is not used for DeviceTree based instantiation.
*/
static const struct i2c_device_id is31fl32xx_id[] = {
+ { "is31fl3236" },
+ { "is31fl3235" },
+ { "is31fl3218" },
+ { "sn3218" },
+ { "is31fl3216" },
+ { "sn3216" },
{},
};
--
2.7.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-18 16:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-18 16:39 [PATCH 0/2] led: is31fl32xx: minor improvements H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 1/2] led: is31fl32xx: fix typo in id and match table names H. Nikolaus Schaller
2016-07-18 16:39 ` [PATCH 2/2] led: is31fl32xx: define complete i2c_device_id table H. Nikolaus Schaller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).