| Message ID | 20221118224540.619276-289-uwe@kleine-koenig.org |
|---|---|
| State | New |
| Headers | show |
| Series | i2c: Complete conversion to i2c_probe_new | expand |
On Fri, 18 Nov 2022 23:40:22 +0100 Uwe Kleine-König <uwe@kleine-koenig.org> wrote: > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Reviewed-by: Marek Behún <kabel@kernel.org>
Hello Pavel, On Fri, Nov 18, 2022 at 11:40:22PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> I didn't get any feedback from your side about this patch set. In case the problem is "only" to get the led patches out of this series, I recommend: b4 am -l -s -P268-288 20221118224540.619276-1-uwe@kleine-koenig.org Best regards Uwe
Hi! > On Fri, Nov 18, 2022 at 11:40:22PM +0100, Uwe Kleine-König wrote: > > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > The probe function doesn't make use of the i2c_device_id * parameter so it > > can be trivially converted. > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > I didn't get any feedback from your side about this patch set. > > In case the problem is "only" to get the led patches out of this > series, I recommend: > > b4 am -l -s -P268-288 20221118224540.619276-1-uwe@kleine-koenig.org Yeah, well, I guess: # A part of the patches depend on commit 662233731d66 ("i2c: core: # Introduce i2c_client_get_device_id helper function"), there is a branch that # you can pull into your tree to get it: this is a bit of a problem because Linus will be unhappy if I pull into my for-next branch. So... I can get them in the next cycle, or I can ack them and get them via some special arangement with linux, or... Best regards, Pavel
Hello Pavel, On Fri, Dec 02, 2022 at 12:12:21PM +0100, Uwe Kleine-König wrote: > On Fri, Nov 18, 2022 at 11:40:22PM +0100, Uwe Kleine-König wrote: > > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > The probe function doesn't make use of the i2c_device_id * parameter so it > > can be trivially converted. > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > I didn't get any feedback from your side about this patch set. > > In case the problem is "only" to get the led patches out of this > series, I recommend: > > b4 am -l -s -P268-288 20221118224540.619276-1-uwe@kleine-koenig.org Still no feedback. A big part of the series was applied to the various trees, but the led patches are not. :-\ Would be great if you queued them for next after the merge window closed. (Or if you have concerns, speak up.) Best regards Uwe
On Thu, 22 Dec 2022, Uwe Kleine-König wrote: > Hello Pavel, > > On Fri, Dec 02, 2022 at 12:12:21PM +0100, Uwe Kleine-König wrote: > > On Fri, Nov 18, 2022 at 11:40:22PM +0100, Uwe Kleine-König wrote: > > > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > > > The probe function doesn't make use of the i2c_device_id * parameter so it > > > can be trivially converted. > > > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > I didn't get any feedback from your side about this patch set. > > > > In case the problem is "only" to get the led patches out of this > > series, I recommend: > > > > b4 am -l -s -P268-288 20221118224540.619276-1-uwe@kleine-koenig.org > > Still no feedback. A big part of the series was applied to the various > trees, but the led patches are not. :-\ Would be great if you queued > them for next after the merge window closed. (Or if you have concerns, > speak up.) If it's okay with Pavel, I can pick these up after the holidays.
On Fri, 23 Dec 2022, Lee Jones wrote: > On Thu, 22 Dec 2022, Uwe Kleine-König wrote: > > > Hello Pavel, > > > > On Fri, Dec 02, 2022 at 12:12:21PM +0100, Uwe Kleine-König wrote: > > > On Fri, Nov 18, 2022 at 11:40:22PM +0100, Uwe Kleine-König wrote: > > > > From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > > > > > The probe function doesn't make use of the i2c_device_id * parameter so it > > > > can be trivially converted. > > > > > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > > > > > I didn't get any feedback from your side about this patch set. > > > > > > In case the problem is "only" to get the led patches out of this > > > series, I recommend: > > > > > > b4 am -l -s -P268-288 20221118224540.619276-1-uwe@kleine-koenig.org > > > > Still no feedback. A big part of the series was applied to the various > > trees, but the led patches are not. :-\ Would be great if you queued > > them for next after the merge window closed. (Or if you have concerns, > > speak up.) > > If it's okay with Pavel, I can pick these up after the holidays. Applied now, thanks.
diff --git a/drivers/leds/leds-turris-omnia.c b/drivers/leds/leds-turris-omnia.c index c7c9851c894a..013f551b32b2 100644 --- a/drivers/leds/leds-turris-omnia.c +++ b/drivers/leds/leds-turris-omnia.c @@ -201,8 +201,7 @@ static struct attribute *omnia_led_controller_attrs[] = { }; ATTRIBUTE_GROUPS(omnia_led_controller); -static int omnia_leds_probe(struct i2c_client *client, - const struct i2c_device_id *id) +static int omnia_leds_probe(struct i2c_client *client) { struct device *dev = &client->dev; struct device_node *np = dev_of_node(dev), *child; @@ -272,7 +271,7 @@ static const struct i2c_device_id omnia_id[] = { MODULE_DEVICE_TABLE(i2c, omnia_id); static struct i2c_driver omnia_leds_driver = { - .probe = omnia_leds_probe, + .probe_new = omnia_leds_probe, .remove = omnia_leds_remove, .id_table = omnia_id, .driver = {