* [PATCH 513/606] power: supply: adp5061: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-21 9:46 ` Hennerich, Michael
2022-11-18 22:44 ` [PATCH 514/606] power: supply: bq2415x: " Uwe Kleine-König
` (30 subsequent siblings)
31 siblings, 1 reply; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Michael Hennerich, Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/adp5061.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/adp5061.c b/drivers/power/supply/adp5061.c
index fcf8ff0bc974..840db629a46c 100644
--- a/drivers/power/supply/adp5061.c
+++ b/drivers/power/supply/adp5061.c
@@ -694,8 +694,7 @@ static const struct power_supply_desc adp5061_desc = {
.num_properties = ARRAY_SIZE(adp5061_props),
};
-static int adp5061_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int adp5061_probe(struct i2c_client *client)
{
struct power_supply_config psy_cfg = {};
struct adp5061_state *st;
@@ -737,7 +736,7 @@ static struct i2c_driver adp5061_driver = {
.driver = {
.name = KBUILD_MODNAME,
},
- .probe = adp5061_probe,
+ .probe_new = adp5061_probe,
.id_table = adp5061_id,
};
module_i2c_driver(adp5061_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* RE: [PATCH 513/606] power: supply: adp5061: Convert to i2c's .probe_new()
2022-11-18 22:44 ` [PATCH 513/606] power: supply: adp5061: Convert to i2c's .probe_new() Uwe Kleine-König
@ 2022-11-21 9:46 ` Hennerich, Michael
0 siblings, 0 replies; 37+ messages in thread
From: Hennerich, Michael @ 2022-11-21 9:46 UTC (permalink / raw)
To: Uwe Kleine-König, Angel Iglesias, Lee Jones, Grant Likely,
Wolfram Sang, Sebastian Reichel
Cc: linux-i2c@vger.kernel.org, kernel@pengutronix.de,
Uwe Kleine-König, linux-pm@vger.kernel.org,
linux-kernel@vger.kernel.org
> -----Original Message-----
> From: Uwe Kleine-König <uwe@kleine-koenig.org>
> Sent: Freitag, 18. November 2022 23:44
> To: Angel Iglesias <ang.iglesiasg@gmail.com>; Lee Jones
> <lee.jones@linaro.org>; Grant Likely <grant.likely@linaro.org>; Wolfram
> Sang <wsa@kernel.org>; Hennerich, Michael
> <Michael.Hennerich@analog.com>; Sebastian Reichel <sre@kernel.org>
> Cc: linux-i2c@vger.kernel.org; kernel@pengutronix.de; Uwe Kleine-König
> <u.kleine-koenig@pengutronix.de>; linux-pm@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: [PATCH 513/606] power: supply: adp5061: Convert to i2c's
> .probe_new()
>
>
> 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>
Acked-by: Michael Hennerich <michael.hennerich@analog.com>
> ---
> drivers/power/supply/adp5061.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/power/supply/adp5061.c
> b/drivers/power/supply/adp5061.c index fcf8ff0bc974..840db629a46c
> 100644
> --- a/drivers/power/supply/adp5061.c
> +++ b/drivers/power/supply/adp5061.c
> @@ -694,8 +694,7 @@ static const struct power_supply_desc adp5061_desc
> = {
> .num_properties = ARRAY_SIZE(adp5061_props),
> };
>
> -static int adp5061_probe(struct i2c_client *client,
> - const struct i2c_device_id *id)
> +static int adp5061_probe(struct i2c_client *client)
> {
> struct power_supply_config psy_cfg = {};
> struct adp5061_state *st;
> @@ -737,7 +736,7 @@ static struct i2c_driver adp5061_driver = {
> .driver = {
> .name = KBUILD_MODNAME,
> },
> - .probe = adp5061_probe,
> + .probe_new = adp5061_probe,
> .id_table = adp5061_id,
> };
> module_i2c_driver(adp5061_driver);
> --
> 2.38.1
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 514/606] power: supply: bq2415x: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 513/606] power: supply: adp5061: Convert to i2c's .probe_new() Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 515/606] power: supply: bq24190: " Uwe Kleine-König
` (29 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, Pali Rohár,
linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq2415x_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq2415x_charger.c b/drivers/power/supply/bq2415x_charger.c
index 6b99e1c675b8..d2cb7431dced 100644
--- a/drivers/power/supply/bq2415x_charger.c
+++ b/drivers/power/supply/bq2415x_charger.c
@@ -1520,9 +1520,9 @@ static int bq2415x_power_supply_init(struct bq2415x_device *bq)
}
/* main bq2415x probe function */
-static int bq2415x_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq2415x_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
int ret;
int num;
char *name = NULL;
@@ -1780,7 +1780,7 @@ static struct i2c_driver bq2415x_driver = {
.of_match_table = of_match_ptr(bq2415x_of_match_table),
.acpi_match_table = ACPI_PTR(bq2415x_i2c_acpi_match),
},
- .probe = bq2415x_probe,
+ .probe_new = bq2415x_probe,
.remove = bq2415x_remove,
.id_table = bq2415x_i2c_id_table,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 515/606] power: supply: bq24190: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 513/606] power: supply: adp5061: Convert to i2c's .probe_new() Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 514/606] power: supply: bq2415x: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 516/606] power: supply: bq24257: " Uwe Kleine-König
` (28 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq24190_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq24190_charger.c b/drivers/power/supply/bq24190_charger.c
index 2274679c5ddd..2b2c3a4391c1 100644
--- a/drivers/power/supply/bq24190_charger.c
+++ b/drivers/power/supply/bq24190_charger.c
@@ -1767,9 +1767,9 @@ static int bq24190_get_config(struct bq24190_dev_info *bdi)
return 0;
}
-static int bq24190_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq24190_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct i2c_adapter *adapter = client->adapter;
struct device *dev = &client->dev;
struct power_supply_config charger_cfg = {}, battery_cfg = {};
@@ -2032,7 +2032,7 @@ static const struct of_device_id bq24190_of_match[] = {
MODULE_DEVICE_TABLE(of, bq24190_of_match);
static struct i2c_driver bq24190_driver = {
- .probe = bq24190_probe,
+ .probe_new = bq24190_probe,
.remove = bq24190_remove,
.shutdown = bq24190_shutdown,
.id_table = bq24190_i2c_ids,
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 516/606] power: supply: bq24257: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (2 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 515/606] power: supply: bq24190: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 517/606] power: supply: bq24735: " Uwe Kleine-König
` (27 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq24257_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq24257_charger.c b/drivers/power/supply/bq24257_charger.c
index a309bbedfe52..ab4c49788c58 100644
--- a/drivers/power/supply/bq24257_charger.c
+++ b/drivers/power/supply/bq24257_charger.c
@@ -947,9 +947,9 @@ static int bq24257_fw_probe(struct bq24257_device *bq)
return 0;
}
-static int bq24257_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq24257_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct i2c_adapter *adapter = client->adapter;
struct device *dev = &client->dev;
const struct acpi_device_id *acpi_id;
@@ -1167,7 +1167,7 @@ static struct i2c_driver bq24257_driver = {
.acpi_match_table = ACPI_PTR(bq24257_acpi_match),
.pm = &bq24257_pm,
},
- .probe = bq24257_probe,
+ .probe_new = bq24257_probe,
.remove = bq24257_remove,
.id_table = bq24257_i2c_ids,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 517/606] power: supply: bq24735: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (3 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 516/606] power: supply: bq24257: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 518/606] power: supply: bq2515x: " Uwe Kleine-König
` (26 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/bq24735-charger.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq24735-charger.c b/drivers/power/supply/bq24735-charger.c
index 3ce36d09c017..cfca3a82d5a8 100644
--- a/drivers/power/supply/bq24735-charger.c
+++ b/drivers/power/supply/bq24735-charger.c
@@ -352,8 +352,7 @@ static struct bq24735_platform *bq24735_parse_dt_data(struct i2c_client *client)
return pdata;
}
-static int bq24735_charger_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq24735_charger_probe(struct i2c_client *client)
{
int ret;
struct bq24735 *charger;
@@ -506,7 +505,7 @@ static struct i2c_driver bq24735_charger_driver = {
.name = "bq24735-charger",
.of_match_table = bq24735_match_ids,
},
- .probe = bq24735_charger_probe,
+ .probe_new = bq24735_charger_probe,
.id_table = bq24735_charger_id,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 518/606] power: supply: bq2515x: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (4 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 517/606] power: supply: bq24735: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 519/606] power: supply: bq256xx: " Uwe Kleine-König
` (25 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq2515x_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq2515x_charger.c b/drivers/power/supply/bq2515x_charger.c
index 4f76ad9c2f18..da224ae8dc61 100644
--- a/drivers/power/supply/bq2515x_charger.c
+++ b/drivers/power/supply/bq2515x_charger.c
@@ -1078,9 +1078,9 @@ static const struct regmap_config bq25155_regmap_config = {
.volatile_reg = bq2515x_volatile_register,
};
-static int bq2515x_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq2515x_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct device *dev = &client->dev;
struct bq2515x_device *bq2515x;
struct power_supply_config charger_cfg = {};
@@ -1158,7 +1158,7 @@ static struct i2c_driver bq2515x_driver = {
.name = "bq2515x-charger",
.of_match_table = bq2515x_of_match,
},
- .probe = bq2515x_probe,
+ .probe_new = bq2515x_probe,
.id_table = bq2515x_i2c_ids,
};
module_i2c_driver(bq2515x_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 519/606] power: supply: bq256xx: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (5 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 518/606] power: supply: bq2515x: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 520/606] power: supply: bq25890: " Uwe Kleine-König
` (24 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq256xx_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq256xx_charger.c b/drivers/power/supply/bq256xx_charger.c
index 01ad84fd147c..db13e288e439 100644
--- a/drivers/power/supply/bq256xx_charger.c
+++ b/drivers/power/supply/bq256xx_charger.c
@@ -1619,9 +1619,9 @@ static int bq256xx_parse_dt(struct bq256xx_device *bq,
return 0;
}
-static int bq256xx_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq256xx_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct device *dev = &client->dev;
struct bq256xx_device *bq;
struct power_supply_config psy_cfg = { };
@@ -1744,7 +1744,7 @@ static struct i2c_driver bq256xx_driver = {
.of_match_table = bq256xx_of_match,
.acpi_match_table = bq256xx_acpi_match,
},
- .probe = bq256xx_probe,
+ .probe_new = bq256xx_probe,
.id_table = bq256xx_i2c_ids,
};
module_i2c_driver(bq256xx_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 520/606] power: supply: bq25890: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (6 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 519/606] power: supply: bq256xx: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 521/606] power: supply: bq25980: " Uwe Kleine-König
` (23 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/bq25890_charger.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq25890_charger.c b/drivers/power/supply/bq25890_charger.c
index f0362dcb935e..bfdd2213ba69 100644
--- a/drivers/power/supply/bq25890_charger.c
+++ b/drivers/power/supply/bq25890_charger.c
@@ -1315,8 +1315,7 @@ static int bq25890_fw_probe(struct bq25890_device *bq)
return 0;
}
-static int bq25890_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq25890_probe(struct i2c_client *client)
{
struct device *dev = &client->dev;
struct bq25890_device *bq;
@@ -1515,7 +1514,7 @@ static struct i2c_driver bq25890_driver = {
.acpi_match_table = ACPI_PTR(bq25890_acpi_match),
.pm = &bq25890_pm,
},
- .probe = bq25890_probe,
+ .probe_new = bq25890_probe,
.remove = bq25890_remove,
.shutdown = bq25890_shutdown,
.id_table = bq25890_i2c_ids,
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 521/606] power: supply: bq25980: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (7 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 520/606] power: supply: bq25890: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 522/606] power: supply: bq27xxx: " Uwe Kleine-König
` (22 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq25980_charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq25980_charger.c b/drivers/power/supply/bq25980_charger.c
index 9339f5649282..a59d9762bc91 100644
--- a/drivers/power/supply/bq25980_charger.c
+++ b/drivers/power/supply/bq25980_charger.c
@@ -1207,9 +1207,9 @@ static int bq25980_parse_dt(struct bq25980_device *bq)
return 0;
}
-static int bq25980_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq25980_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct device *dev = &client->dev;
struct bq25980_device *bq;
int ret;
@@ -1287,7 +1287,7 @@ static struct i2c_driver bq25980_driver = {
.name = "bq25980-charger",
.of_match_table = bq25980_of_match,
},
- .probe = bq25980_probe,
+ .probe_new = bq25980_probe,
.id_table = bq25980_i2c_ids,
};
module_i2c_driver(bq25980_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 522/606] power: supply: bq27xxx: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (8 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 521/606] power: supply: bq25980: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 523/606] power: supply: ds2782: " Uwe Kleine-König
` (21 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, Pali Rohár,
linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/bq27xxx_battery_i2c.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/bq27xxx_battery_i2c.c b/drivers/power/supply/bq27xxx_battery_i2c.c
index 94b00bb89c17..f8768997333b 100644
--- a/drivers/power/supply/bq27xxx_battery_i2c.c
+++ b/drivers/power/supply/bq27xxx_battery_i2c.c
@@ -136,9 +136,9 @@ static int bq27xxx_battery_i2c_bulk_write(struct bq27xxx_device_info *di,
return 0;
}
-static int bq27xxx_battery_i2c_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int bq27xxx_battery_i2c_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct bq27xxx_device_info *di;
int ret;
char *name;
@@ -295,7 +295,7 @@ static struct i2c_driver bq27xxx_battery_i2c_driver = {
.name = "bq27xxx-battery",
.of_match_table = of_match_ptr(bq27xxx_battery_i2c_of_match_table),
},
- .probe = bq27xxx_battery_i2c_probe,
+ .probe_new = bq27xxx_battery_i2c_probe,
.remove = bq27xxx_battery_i2c_remove,
.id_table = bq27xxx_i2c_id_table,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 523/606] power: supply: ds2782: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (9 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 522/606] power: supply: bq27xxx: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 524/606] power: supply: lp8727: " Uwe Kleine-König
` (20 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/ds2782_battery.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/ds2782_battery.c b/drivers/power/supply/ds2782_battery.c
index d78cd05402f6..9b9619246902 100644
--- a/drivers/power/supply/ds2782_battery.c
+++ b/drivers/power/supply/ds2782_battery.c
@@ -368,9 +368,9 @@ static const struct ds278x_battery_ops ds278x_ops[] = {
}
};
-static int ds278x_battery_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int ds278x_battery_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct ds278x_platform_data *pdata = client->dev.platform_data;
struct power_supply_config psy_cfg = {};
struct ds278x_info *info;
@@ -458,7 +458,7 @@ static struct i2c_driver ds278x_battery_driver = {
.name = "ds2782-battery",
.pm = &ds278x_battery_pm_ops,
},
- .probe = ds278x_battery_probe,
+ .probe_new = ds278x_battery_probe,
.remove = ds278x_battery_remove,
.id_table = ds278x_id,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 524/606] power: supply: lp8727: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (10 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 523/606] power: supply: ds2782: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 525/606] power: supply: ltc2941: " Uwe Kleine-König
` (19 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/lp8727_charger.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/power/supply/lp8727_charger.c b/drivers/power/supply/lp8727_charger.c
index 384a374b52c1..e6c21377d53c 100644
--- a/drivers/power/supply/lp8727_charger.c
+++ b/drivers/power/supply/lp8727_charger.c
@@ -540,7 +540,7 @@ static struct lp8727_platform_data *lp8727_parse_dt(struct device *dev)
}
#endif
-static int lp8727_probe(struct i2c_client *cl, const struct i2c_device_id *id)
+static int lp8727_probe(struct i2c_client *cl)
{
struct lp8727_chg *pchg;
struct lp8727_platform_data *pdata;
@@ -615,7 +615,7 @@ static struct i2c_driver lp8727_driver = {
.name = "lp8727",
.of_match_table = of_match_ptr(lp8727_dt_ids),
},
- .probe = lp8727_probe,
+ .probe_new = lp8727_probe,
.remove = lp8727_remove,
.id_table = lp8727_ids,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 525/606] power: supply: ltc2941: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (11 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 524/606] power: supply: lp8727: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 526/606] power: supply: ltc4162-l: " Uwe Kleine-König
` (18 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/ltc2941-battery-gauge.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/ltc2941-battery-gauge.c b/drivers/power/supply/ltc2941-battery-gauge.c
index 657305214d68..d3fb42825983 100644
--- a/drivers/power/supply/ltc2941-battery-gauge.c
+++ b/drivers/power/supply/ltc2941-battery-gauge.c
@@ -439,8 +439,7 @@ static enum power_supply_property ltc294x_properties[] = {
POWER_SUPPLY_PROP_CURRENT_NOW,
};
-static int ltc294x_i2c_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int ltc294x_i2c_probe(struct i2c_client *client)
{
struct power_supply_config psy_cfg = {};
struct ltc294x_info *info;
@@ -636,7 +635,7 @@ static struct i2c_driver ltc294x_driver = {
.of_match_table = ltc294x_i2c_of_match,
.pm = LTC294X_PM_OPS,
},
- .probe = ltc294x_i2c_probe,
+ .probe_new = ltc294x_i2c_probe,
.shutdown = ltc294x_i2c_shutdown,
.id_table = ltc294x_i2c_id,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 526/606] power: supply: ltc4162-l: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (12 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 525/606] power: supply: ltc2941: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 527/606] power: supply: max14656: " Uwe Kleine-König
` (17 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/ltc4162-l-charger.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/ltc4162-l-charger.c b/drivers/power/supply/ltc4162-l-charger.c
index 1a5cb4405ee3..db2bb5233570 100644
--- a/drivers/power/supply/ltc4162-l-charger.c
+++ b/drivers/power/supply/ltc4162-l-charger.c
@@ -819,8 +819,7 @@ static void ltc4162l_clear_interrupts(struct ltc4162l_info *info)
regmap_write(info->regmap, LTC4162L_CHARGE_STATUS_ALERTS_REG, 0);
}
-static int ltc4162l_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int ltc4162l_probe(struct i2c_client *client)
{
struct i2c_adapter *adapter = client->adapter;
struct device *dev = &client->dev;
@@ -916,7 +915,7 @@ static const struct of_device_id ltc4162l_of_match[] = {
MODULE_DEVICE_TABLE(of, ltc4162l_of_match);
static struct i2c_driver ltc4162l_driver = {
- .probe = ltc4162l_probe,
+ .probe_new = ltc4162l_probe,
.alert = ltc4162l_alert,
.id_table = ltc4162l_i2c_id_table,
.driver = {
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 527/606] power: supply: max14656: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (13 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 526/606] power: supply: ltc4162-l: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 528/606] power: supply: max17040: " Uwe Kleine-König
` (16 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/max14656_charger_detector.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/max14656_charger_detector.c b/drivers/power/supply/max14656_charger_detector.c
index fc36828895bf..0d0180fcfa63 100644
--- a/drivers/power/supply/max14656_charger_detector.c
+++ b/drivers/power/supply/max14656_charger_detector.c
@@ -234,8 +234,7 @@ static enum power_supply_property max14656_battery_props[] = {
POWER_SUPPLY_PROP_MANUFACTURER,
};
-static int max14656_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int max14656_probe(struct i2c_client *client)
{
struct i2c_adapter *adapter = client->adapter;
struct device *dev = &client->dev;
@@ -317,7 +316,7 @@ static struct i2c_driver max14656_i2c_driver = {
.name = "max14656",
.of_match_table = max14656_match_table,
},
- .probe = max14656_probe,
+ .probe_new = max14656_probe,
.id_table = max14656_id,
};
module_i2c_driver(max14656_i2c_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 528/606] power: supply: max17040: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (14 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 527/606] power: supply: max14656: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 529/606] power: supply: max17042_battery: " Uwe Kleine-König
` (15 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, Iskren Chernev,
Krzysztof Kozlowski, Marek Szyprowski, Matheus Castello, linux-pm,
linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/max17040_battery.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/max17040_battery.c b/drivers/power/supply/max17040_battery.c
index a9aef1e8b186..d1075959dd46 100644
--- a/drivers/power/supply/max17040_battery.c
+++ b/drivers/power/supply/max17040_battery.c
@@ -430,9 +430,9 @@ static const struct power_supply_desc max17040_battery_desc = {
.num_properties = ARRAY_SIZE(max17040_battery_props),
};
-static int max17040_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int max17040_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct i2c_adapter *adapter = client->adapter;
struct power_supply_config psy_cfg = {};
struct max17040_chip *chip;
@@ -599,7 +599,7 @@ static struct i2c_driver max17040_i2c_driver = {
.of_match_table = max17040_of_match,
.pm = MAX17040_PM_OPS,
},
- .probe = max17040_probe,
+ .probe_new = max17040_probe,
.id_table = max17040_id,
};
module_i2c_driver(max17040_i2c_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 529/606] power: supply: max17042_battery: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (15 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 528/606] power: supply: max17040: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-19 12:58 ` Hans de Goede
2022-11-18 22:44 ` [PATCH 530/606] power: supply: rt5033_battery: " Uwe Kleine-König
` (14 subsequent siblings)
31 siblings, 1 reply; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, Hans de Goede,
Krzysztof Kozlowski, Marek Szyprowski, Sebastian Krzyszkowiak,
Purism Kernel Team, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/max17042_battery.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
index ab031bbfbe78..89cabe8ed3b0 100644
--- a/drivers/power/supply/max17042_battery.c
+++ b/drivers/power/supply/max17042_battery.c
@@ -1031,9 +1031,9 @@ static const struct power_supply_desc max17042_no_current_sense_psy_desc = {
.num_properties = ARRAY_SIZE(max17042_battery_props) - 2,
};
-static int max17042_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int max17042_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct i2c_adapter *adapter = client->adapter;
const struct power_supply_desc *max17042_desc = &max17042_psy_desc;
struct power_supply_config psy_cfg = {};
@@ -1220,7 +1220,7 @@ static struct i2c_driver max17042_i2c_driver = {
.of_match_table = of_match_ptr(max17042_dt_match),
.pm = &max17042_pm_ops,
},
- .probe = max17042_probe,
+ .probe_new = max17042_probe,
.id_table = max17042_id,
};
module_i2c_driver(max17042_i2c_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* Re: [PATCH 529/606] power: supply: max17042_battery: Convert to i2c's .probe_new()
2022-11-18 22:44 ` [PATCH 529/606] power: supply: max17042_battery: " Uwe Kleine-König
@ 2022-11-19 12:58 ` Hans de Goede
0 siblings, 0 replies; 37+ messages in thread
From: Hans de Goede @ 2022-11-19 12:58 UTC (permalink / raw)
To: Uwe Kleine-König, Angel Iglesias, Lee Jones, Grant Likely,
Wolfram Sang, Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, Krzysztof Kozlowski,
Marek Szyprowski, Sebastian Krzyszkowiak, Purism Kernel Team,
linux-pm, linux-kernel
Hi,
On 11/18/22 23:44, Uwe Kleine-König wrote:
> From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
>
> .probe_new() doesn't get the i2c_device_id * parameter, so determine
> that explicitly in the probe function.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Regards,
Hans
> ---
> drivers/power/supply/max17042_battery.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
> index ab031bbfbe78..89cabe8ed3b0 100644
> --- a/drivers/power/supply/max17042_battery.c
> +++ b/drivers/power/supply/max17042_battery.c
> @@ -1031,9 +1031,9 @@ static const struct power_supply_desc max17042_no_current_sense_psy_desc = {
> .num_properties = ARRAY_SIZE(max17042_battery_props) - 2,
> };
>
> -static int max17042_probe(struct i2c_client *client,
> - const struct i2c_device_id *id)
> +static int max17042_probe(struct i2c_client *client)
> {
> + const struct i2c_device_id *id = i2c_client_get_device_id(client);
> struct i2c_adapter *adapter = client->adapter;
> const struct power_supply_desc *max17042_desc = &max17042_psy_desc;
> struct power_supply_config psy_cfg = {};
> @@ -1220,7 +1220,7 @@ static struct i2c_driver max17042_i2c_driver = {
> .of_match_table = of_match_ptr(max17042_dt_match),
> .pm = &max17042_pm_ops,
> },
> - .probe = max17042_probe,
> + .probe_new = max17042_probe,
> .id_table = max17042_id,
> };
> module_i2c_driver(max17042_i2c_driver);
^ permalink raw reply [flat|nested] 37+ messages in thread
* [PATCH 530/606] power: supply: rt5033_battery: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (16 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 529/606] power: supply: max17042_battery: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 531/606] power: supply: rt9455: " Uwe Kleine-König
` (13 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/rt5033_battery.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/rt5033_battery.c b/drivers/power/supply/rt5033_battery.c
index 736dec608ff6..5c04cf305219 100644
--- a/drivers/power/supply/rt5033_battery.c
+++ b/drivers/power/supply/rt5033_battery.c
@@ -112,8 +112,7 @@ static const struct power_supply_desc rt5033_battery_desc = {
.num_properties = ARRAY_SIZE(rt5033_battery_props),
};
-static int rt5033_battery_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int rt5033_battery_probe(struct i2c_client *client)
{
struct i2c_adapter *adapter = client->adapter;
struct power_supply_config psy_cfg = {};
@@ -173,7 +172,7 @@ static struct i2c_driver rt5033_battery_driver = {
.name = "rt5033-battery",
.of_match_table = rt5033_battery_of_match,
},
- .probe = rt5033_battery_probe,
+ .probe_new = rt5033_battery_probe,
.remove = rt5033_battery_remove,
.id_table = rt5033_battery_id,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 531/606] power: supply: rt9455: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (17 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 530/606] power: supply: rt5033_battery: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 532/606] power: supply: sbs: " Uwe Kleine-König
` (12 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/rt9455_charger.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/rt9455_charger.c b/drivers/power/supply/rt9455_charger.c
index 72962286d704..31fb6526a1fd 100644
--- a/drivers/power/supply/rt9455_charger.c
+++ b/drivers/power/supply/rt9455_charger.c
@@ -1581,8 +1581,7 @@ static const struct regmap_config rt9455_regmap_config = {
.cache_type = REGCACHE_RBTREE,
};
-static int rt9455_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int rt9455_probe(struct i2c_client *client)
{
struct i2c_adapter *adapter = client->adapter;
struct device *dev = &client->dev;
@@ -1738,7 +1737,7 @@ MODULE_DEVICE_TABLE(acpi, rt9455_i2c_acpi_match);
#endif
static struct i2c_driver rt9455_driver = {
- .probe = rt9455_probe,
+ .probe_new = rt9455_probe,
.remove = rt9455_remove,
.id_table = rt9455_i2c_id_table,
.driver = {
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 532/606] power: supply: sbs: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (18 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 531/606] power: supply: rt9455: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 533/606] power: supply: sbs-manager: " Uwe Kleine-König
` (11 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/sbs-charger.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/sbs-charger.c b/drivers/power/supply/sbs-charger.c
index b08f7d0c4181..75ebcbf0a788 100644
--- a/drivers/power/supply/sbs-charger.c
+++ b/drivers/power/supply/sbs-charger.c
@@ -162,8 +162,7 @@ static const struct power_supply_desc sbs_desc = {
.get_property = sbs_get_property,
};
-static int sbs_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int sbs_probe(struct i2c_client *client)
{
struct power_supply_config psy_cfg = {};
struct sbs_info *chip;
@@ -241,7 +240,7 @@ static const struct i2c_device_id sbs_id[] = {
MODULE_DEVICE_TABLE(i2c, sbs_id);
static struct i2c_driver sbs_driver = {
- .probe = sbs_probe,
+ .probe_new = sbs_probe,
.id_table = sbs_id,
.driver = {
.name = "sbs-charger",
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 533/606] power: supply: sbs-manager: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (19 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 532/606] power: supply: sbs: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 534/606] power: supply: smb347: " Uwe Kleine-König
` (10 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/sbs-manager.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/sbs-manager.c b/drivers/power/supply/sbs-manager.c
index 71ec8f74f835..bde977391fd4 100644
--- a/drivers/power/supply/sbs-manager.c
+++ b/drivers/power/supply/sbs-manager.c
@@ -315,9 +315,9 @@ static void sbsm_del_mux_adapter(void *data)
i2c_mux_del_adapters(sbsm->muxc);
}
-static int sbsm_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int sbsm_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct i2c_adapter *adapter = client->adapter;
struct sbsm_data *data;
struct device *dev = &client->dev;
@@ -409,7 +409,7 @@ static struct i2c_driver sbsm_driver = {
.name = "sbsm",
.of_match_table = of_match_ptr(sbsm_dt_ids),
},
- .probe = sbsm_probe,
+ .probe_new = sbsm_probe,
.alert = sbsm_alert,
.id_table = sbsm_ids
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 534/606] power: supply: smb347: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (20 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 533/606] power: supply: sbs-manager: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 535/606] power: supply: ucs1002: " Uwe Kleine-König
` (9 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
.probe_new() doesn't get the i2c_device_id * parameter, so determine
that explicitly in the probe function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/power/supply/smb347-charger.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/smb347-charger.c b/drivers/power/supply/smb347-charger.c
index 996a82f8a2a1..b5f038310282 100644
--- a/drivers/power/supply/smb347-charger.c
+++ b/drivers/power/supply/smb347-charger.c
@@ -1528,9 +1528,9 @@ static const struct regulator_desc smb347_usb_vbus_regulator_desc = {
.n_voltages = 1,
};
-static int smb347_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int smb347_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_client_get_device_id(client);
struct power_supply_config mains_usb_cfg = {};
struct regulator_config usb_rdev_cfg = {};
struct device *dev = &client->dev;
@@ -1629,7 +1629,7 @@ static struct i2c_driver smb347_driver = {
.name = "smb347",
.of_match_table = smb3xx_of_match,
},
- .probe = smb347_probe,
+ .probe_new = smb347_probe,
.remove = smb347_remove,
.shutdown = smb347_shutdown,
.id_table = smb347_id,
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 535/606] power: supply: ucs1002: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (21 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 534/606] power: supply: smb347: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-18 22:44 ` [PATCH 536/606] power: supply: z2_battery: " Uwe Kleine-König
` (8 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/ucs1002_power.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/ucs1002_power.c b/drivers/power/supply/ucs1002_power.c
index ef673ec3db56..836d44c9fb74 100644
--- a/drivers/power/supply/ucs1002_power.c
+++ b/drivers/power/supply/ucs1002_power.c
@@ -532,8 +532,7 @@ static const struct regulator_desc ucs1002_regulator_descriptor = {
.n_voltages = 1,
};
-static int ucs1002_probe(struct i2c_client *client,
- const struct i2c_device_id *dev_id)
+static int ucs1002_probe(struct i2c_client *client)
{
struct device *dev = &client->dev;
struct power_supply_config charger_config = {};
@@ -681,7 +680,7 @@ static struct i2c_driver ucs1002_driver = {
.name = "ucs1002",
.of_match_table = ucs1002_of_match,
},
- .probe = ucs1002_probe,
+ .probe_new = ucs1002_probe,
};
module_i2c_driver(ucs1002_driver);
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* [PATCH 536/606] power: supply: z2_battery: Convert to i2c's .probe_new()
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (22 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 535/606] power: supply: ucs1002: " Uwe Kleine-König
@ 2022-11-18 22:44 ` Uwe Kleine-König
2022-11-19 6:50 ` [PATCH 000/606] i2c: Complete conversion to i2c_probe_new patchwork-bot+chrome-platform
` (7 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-18 22:44 UTC (permalink / raw)
To: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang,
Sebastian Reichel
Cc: linux-i2c, kernel, Uwe Kleine-König, linux-pm, linux-kernel
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>
---
drivers/power/supply/z2_battery.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/power/supply/z2_battery.c b/drivers/power/supply/z2_battery.c
index 1897c2984860..eb01b01bf593 100644
--- a/drivers/power/supply/z2_battery.c
+++ b/drivers/power/supply/z2_battery.c
@@ -176,8 +176,7 @@ static int z2_batt_ps_init(struct z2_charger *charger, int props)
return 0;
}
-static int z2_batt_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int z2_batt_probe(struct i2c_client *client)
{
int ret = 0;
int props = 1; /* POWER_SUPPLY_PROP_PRESENT */
@@ -306,7 +305,7 @@ static struct i2c_driver z2_batt_driver = {
.name = "z2-battery",
.pm = Z2_BATTERY_PM_OPS
},
- .probe = z2_batt_probe,
+ .probe_new = z2_batt_probe,
.remove = z2_batt_remove,
.id_table = z2_batt_id,
};
--
2.38.1
^ permalink raw reply related [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (23 preceding siblings ...)
2022-11-18 22:44 ` [PATCH 536/606] power: supply: z2_battery: " Uwe Kleine-König
@ 2022-11-19 6:50 ` patchwork-bot+chrome-platform
2022-11-19 6:50 ` patchwork-bot+chrome-platform
` (6 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: patchwork-bot+chrome-platform @ 2022-11-19 6:50 UTC (permalink / raw)
To: =?utf-8?q?Uwe_Kleine-K=C3=B6nig_=3Cuwe=40kleine-koenig=2Eorg=3E?=
Cc: ang.iglesiasg, lee.jones, grant.likely, wsa, linux-i2c, kernel,
linux-integrity, linux-kernel, linux-arm-kernel, linux-crypto,
linux-gpio, bcm-kernel-feedback-list, linux-rpi-kernel, linux-iio,
linux-input, platform-driver-x86, linux-stm32, linux-leds,
linuxppc-dev, linux-media, patches, linux-actions,
linux-renesas-soc, linux-amlogic, alsa-devel, linux-omap,
linux-samsung-soc, linux-mtd, netdev, devicetree, chrome-platform,
linux-pm, kernel, linux-pwm, linux-rtc, linux-spi, linux-staging,
linux-serial, linux-usb, linux-fbdev, linux-watchdog,
openipmi-developer
Hello:
This patch was applied to chrome-platform/linux.git (for-next)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 18 Nov 2022 23:35:34 +0100 you wrote:
> Hello,
>
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
>
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> [...]
Here is the summary with links:
- [512/606] platform/chrome: cros_ec: Convert to i2c's .probe_new()
https://git.kernel.org/chrome-platform/c/f9e510dc92df
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (24 preceding siblings ...)
2022-11-19 6:50 ` [PATCH 000/606] i2c: Complete conversion to i2c_probe_new patchwork-bot+chrome-platform
@ 2022-11-19 6:50 ` patchwork-bot+chrome-platform
2022-11-19 11:10 ` Wolfram Sang
` (5 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: patchwork-bot+chrome-platform @ 2022-11-19 6:50 UTC (permalink / raw)
To: =?utf-8?q?Uwe_Kleine-K=C3=B6nig_=3Cuwe=40kleine-koenig=2Eorg=3E?=
Cc: ang.iglesiasg, lee.jones, grant.likely, wsa, linux-i2c, kernel,
linux-integrity, linux-kernel, linux-arm-kernel, linux-crypto,
linux-gpio, bcm-kernel-feedback-list, linux-rpi-kernel, linux-iio,
linux-input, platform-driver-x86, linux-stm32, linux-leds,
linuxppc-dev, linux-media, patches, linux-actions,
linux-renesas-soc, linux-amlogic, alsa-devel, linux-omap,
linux-samsung-soc, linux-mtd, netdev, devicetree, chrome-platform,
linux-pm, kernel, linux-pwm, linux-rtc, linux-spi, linux-staging,
linux-serial, linux-usb, linux-fbdev, linux-watchdog,
openipmi-developer
Hello:
This patch was applied to chrome-platform/linux.git (for-kernelci)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 18 Nov 2022 23:35:34 +0100 you wrote:
> Hello,
>
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
>
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> [...]
Here is the summary with links:
- [512/606] platform/chrome: cros_ec: Convert to i2c's .probe_new()
https://git.kernel.org/chrome-platform/c/f9e510dc92df
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (25 preceding siblings ...)
2022-11-19 6:50 ` patchwork-bot+chrome-platform
@ 2022-11-19 11:10 ` Wolfram Sang
2022-11-20 19:43 ` Sebastian Reichel
` (4 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Wolfram Sang @ 2022-11-19 11:10 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Angel Iglesias, Lee Jones, Grant Likely, linux-i2c, kernel,
linux-integrity, linux-kernel, linux-arm-kernel, linux-crypto,
linux-gpio, Broadcom internal kernel review list,
linux-rpi-kernel, linux-iio, linux-input, platform-driver-x86,
linux-stm32, linux-leds, linuxppc-dev, linux-media, patches,
linux-actions, linux-renesas-soc, linux-amlogic, alsa-devel,
linux-omap, linux-samsung-soc, linux-mtd, netdev, devicetree,
chrome-platform, linux-pm, Purism Kernel Team, linux-pwm,
linux-rtc, linux-spi, linux-staging, linux-serial, linux-usb,
linux-fbdev, linux-watchdog, openipmi-developer
[-- Attachment #1: Type: text/plain, Size: 647 bytes --]
Hi Uwe,
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
Thanks for this work, really, but oh my poor inbox...
> I don't think it's feasable to apply this series in one go, so I ask the
> maintainers of the changed files to apply via their tree.
This seems reasonable. It would have made sense to send "patch series
per subsystem" then. So people only see the subset they are interested
in. I know filename-to-subsys mapping is hardly ever perfect. But in my
experience, even imperfect, it is more convenient than such a huge patch
series.
Happy hacking,
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (26 preceding siblings ...)
2022-11-19 11:10 ` Wolfram Sang
@ 2022-11-20 19:43 ` Sebastian Reichel
2022-11-21 9:53 ` Lee Jones
` (3 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Sebastian Reichel @ 2022-11-20 19:43 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang, linux-i2c,
kernel, linux-integrity, linux-kernel, linux-arm-kernel,
linux-crypto, linux-gpio, Broadcom internal kernel review list,
linux-rpi-kernel, linux-iio, linux-input, platform-driver-x86,
linux-stm32, linux-leds, linuxppc-dev, linux-media, patches,
linux-actions, linux-renesas-soc, linux-amlogic, alsa-devel,
linux-omap, linux-samsung-soc, linux-mtd, netdev, devicetree,
chrome-platform, linux-pm, Purism Kernel Team, linux-pwm,
linux-rtc, linux-spi, linux-staging, linux-serial, linux-usb,
linux-fbdev, linux-watchdog, openipmi-developer
[-- Attachment #1: Type: text/plain, Size: 2954 bytes --]
Hi,
On Fri, Nov 18, 2022 at 11:35:34PM +0100, Uwe Kleine-König wrote:
> Hello,
>
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
>
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/client_device_id_helper-immutable
>
> I don't think it's feasable to apply this series in one go, so I ask the
> maintainers of the changed files to apply via their tree. I guess it
> will take a few kernel release iterations until all patch are in, but I
> think a single tree creates too much conflicts.
>
> The last patch changes i2c_driver::probe, all non-converted drivers will
> fail to compile then. So I hope the build bots will tell me about any
> driver I missed to convert. This patch is obviously not for application
> now.
>
> I dropped most individuals from the recipents of this mail to not
> challenge the mail servers and mailing list filters too much. Sorry if
> you had extra efforts to find this mail.
>
> Best regards
> Uwe
...
> power: supply: adp5061: Convert to i2c's .probe_new()
> power: supply: bq2415x: Convert to i2c's .probe_new()
> power: supply: bq24190: Convert to i2c's .probe_new()
> power: supply: bq24257: Convert to i2c's .probe_new()
> power: supply: bq24735: Convert to i2c's .probe_new()
> power: supply: bq2515x: Convert to i2c's .probe_new()
> power: supply: bq256xx: Convert to i2c's .probe_new()
> power: supply: bq25890: Convert to i2c's .probe_new()
> power: supply: bq25980: Convert to i2c's .probe_new()
> power: supply: bq27xxx: Convert to i2c's .probe_new()
> power: supply: ds2782: Convert to i2c's .probe_new()
> power: supply: lp8727: Convert to i2c's .probe_new()
> power: supply: ltc2941: Convert to i2c's .probe_new()
> power: supply: ltc4162-l: Convert to i2c's .probe_new()
> power: supply: max14656: Convert to i2c's .probe_new()
> power: supply: max17040: Convert to i2c's .probe_new()
> power: supply: max17042_battery: Convert to i2c's .probe_new()
> power: supply: rt5033_battery: Convert to i2c's .probe_new()
> power: supply: rt9455: Convert to i2c's .probe_new()
> power: supply: sbs: Convert to i2c's .probe_new()
> power: supply: sbs-manager: Convert to i2c's .probe_new()
> power: supply: smb347: Convert to i2c's .probe_new()
> power: supply: ucs1002: Convert to i2c's .probe_new()
> power: supply: z2_battery: Convert to i2c's .probe_new()
> [...]
Thanks, I queued patches 513-536 to the power-supply subsystem.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (27 preceding siblings ...)
2022-11-20 19:43 ` Sebastian Reichel
@ 2022-11-21 9:53 ` Lee Jones
2022-11-22 18:58 ` Jonathan Cameron
` (2 subsequent siblings)
31 siblings, 0 replies; 37+ messages in thread
From: Lee Jones @ 2022-11-21 9:53 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang, linux-i2c,
kernel, linux-integrity, linux-kernel, linux-arm-kernel,
linux-crypto, linux-gpio, Broadcom internal kernel review list,
linux-rpi-kernel, linux-iio, linux-input, platform-driver-x86,
linux-stm32, linux-leds, linuxppc-dev, linux-media, patches,
linux-actions, linux-renesas-soc, linux-amlogic, alsa-devel,
linux-omap, linux-samsung-soc, linux-mtd, netdev, devicetree,
chrome-platform, linux-pm, Purism Kernel Team, linux-pwm,
linux-rtc, linux-spi, linux-staging, linux-serial, linux-usb,
linux-fbdev, linux-watchdog, openipmi-developer
On Fri, 18 Nov 2022, Uwe Kleine-König wrote:
> Hello,
>
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
Oh yeah, this! Thanks for picking this up Uwe, I guess I've been
distracted for the past 6 years or so. :)
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/client_device_id_helper-immutable
>
> I don't think it's feasable to apply this series in one go, so I ask the
> maintainers of the changed files to apply via their tree. I guess it
> will take a few kernel release iterations until all patch are in, but I
> think a single tree creates too much conflicts.
>
> The last patch changes i2c_driver::probe, all non-converted drivers will
> fail to compile then. So I hope the build bots will tell me about any
> driver I missed to convert. This patch is obviously not for application
> now.
>
> I dropped most individuals from the recipents of this mail to not
> challenge the mail servers and mailing list filters too much. Sorry if
> you had extra efforts to find this mail.
[...]
> drivers/mfd/88pm800.c | 5 ++---
> drivers/mfd/88pm805.c | 5 ++---
> drivers/mfd/aat2870-core.c | 5 ++---
> drivers/mfd/act8945a.c | 5 ++---
> drivers/mfd/adp5520.c | 6 +++---
> drivers/mfd/arizona-i2c.c | 6 +++---
> drivers/mfd/as3711.c | 5 ++---
> drivers/mfd/as3722.c | 5 ++---
> drivers/mfd/atc260x-i2c.c | 5 ++---
> drivers/mfd/axp20x-i2c.c | 5 ++---
> drivers/mfd/bcm590xx.c | 5 ++---
> drivers/mfd/bd9571mwv.c | 5 ++---
> drivers/mfd/da903x.c | 6 +++---
> drivers/mfd/da9052-i2c.c | 6 +++---
> drivers/mfd/da9055-i2c.c | 5 ++---
> drivers/mfd/da9062-core.c | 6 +++---
> drivers/mfd/da9063-i2c.c | 6 +++---
> drivers/mfd/da9150-core.c | 5 ++---
> drivers/mfd/khadas-mcu.c | 5 ++---
> drivers/mfd/lm3533-core.c | 5 ++---
> drivers/mfd/lp3943.c | 4 ++--
> drivers/mfd/lp873x.c | 5 ++---
> drivers/mfd/lp87565.c | 5 ++---
> drivers/mfd/lp8788.c | 4 ++--
> drivers/mfd/madera-i2c.c | 6 +++---
> drivers/mfd/max14577.c | 6 +++---
> drivers/mfd/max77620.c | 6 +++---
> drivers/mfd/max77693.c | 6 +++---
> drivers/mfd/max77843.c | 6 +++---
> drivers/mfd/max8907.c | 5 ++---
> drivers/mfd/max8925-i2c.c | 5 ++---
> drivers/mfd/max8997.c | 6 +++---
> drivers/mfd/max8998.c | 6 +++---
> drivers/mfd/mc13xxx-i2c.c | 6 +++---
> drivers/mfd/menelaus.c | 5 ++---
> drivers/mfd/menf21bmc.c | 4 ++--
> drivers/mfd/palmas.c | 5 ++---
> drivers/mfd/pcf50633-core.c | 5 ++---
> drivers/mfd/rc5t583.c | 5 ++---
> drivers/mfd/retu-mfd.c | 4 ++--
> drivers/mfd/rk808.c | 5 ++---
> drivers/mfd/rohm-bd718x7.c | 5 ++---
> drivers/mfd/rsmu_i2c.c | 6 +++---
> drivers/mfd/rt5033.c | 5 ++---
> drivers/mfd/sec-core.c | 5 ++---
> drivers/mfd/si476x-i2c.c | 6 +++---
> drivers/mfd/sky81452.c | 5 ++---
> drivers/mfd/stmfx.c | 5 ++---
> drivers/mfd/stmpe-i2c.c | 5 +++--
> drivers/mfd/stpmic1.c | 5 ++---
> drivers/mfd/stw481x.c | 5 ++---
> drivers/mfd/tc3589x.c | 6 +++---
> drivers/mfd/ti-lmu.c | 5 +++--
> drivers/mfd/tps6105x.c | 5 ++---
> drivers/mfd/tps65010.c | 6 +++---
> drivers/mfd/tps6507x.c | 5 ++---
> drivers/mfd/tps65086.c | 5 ++---
> drivers/mfd/tps65090.c | 5 ++---
> drivers/mfd/tps65218.c | 5 ++---
> drivers/mfd/tps6586x.c | 5 ++---
> drivers/mfd/tps65910.c | 6 +++---
> drivers/mfd/tps65912-i2c.c | 5 ++---
> drivers/mfd/twl-core.c | 5 +++--
> drivers/mfd/twl6040.c | 5 ++---
> drivers/mfd/wl1273-core.c | 5 ++---
> drivers/mfd/wm831x-i2c.c | 6 +++---
> drivers/mfd/wm8350-i2c.c | 5 ++---
> drivers/mfd/wm8400-core.c | 5 ++---
> drivers/mfd/wm8994-core.c | 6 +++---
For my own reference (apply this as-is to your sign-off block):
Acked-for-MFD-by: Lee Jones <lee@kernel.org>
> drivers/video/backlight/adp8860_bl.c | 6 +++---
> drivers/video/backlight/adp8870_bl.c | 6 +++---
> drivers/video/backlight/arcxcnn_bl.c | 4 ++--
> drivers/video/backlight/bd6107.c | 5 ++---
> drivers/video/backlight/lm3630a_bl.c | 5 ++---
> drivers/video/backlight/lm3639_bl.c | 5 ++---
> drivers/video/backlight/lp855x_bl.c | 5 +++--
> drivers/video/backlight/lv5207lp.c | 5 ++---
> drivers/video/backlight/tosa_bl.c | 5 ++---
> drivers/video/fbdev/matrox/matroxfb_maven.c | 5 ++---
For my own reference (apply this as-is to your sign-off block):
Acked-for-Backlight-by: Lee Jones <lee@kernel.org>
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (28 preceding siblings ...)
2022-11-21 9:53 ` Lee Jones
@ 2022-11-22 18:58 ` Jonathan Cameron
2022-11-22 20:16 ` Uwe Kleine-König
2022-12-09 12:00 ` (subset) " Robert Foss
2022-12-12 16:36 ` Robert Foss
31 siblings, 1 reply; 37+ messages in thread
From: Jonathan Cameron @ 2022-11-22 18:58 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Angel Iglesias, Lee Jones, Grant Likely, Wolfram Sang, linux-i2c,
kernel, linux-integrity, linux-kernel, linux-arm-kernel,
linux-crypto, linux-gpio, Broadcom internal kernel review list,
linux-rpi-kernel, linux-iio, linux-input, platform-driver-x86,
linux-stm32, linux-leds, linuxppc-dev, linux-media, patches,
linux-actions, linux-renesas-soc, linux-amlogic, alsa-devel,
linux-omap, linux-samsung-soc, linux-mtd, netdev, devicetree,
chrome-platform, linux-pm, Purism Kernel Team, linux-pwm,
linux-rtc, linux-spi, linux-staging, linux-serial, linux-usb,
linux-fbdev, linux-watchdog, openipmi-developer, gregkh
Queued all of the below:
with one tweaked as per your suggestion and the highlighted one dropped on basis
I was already carrying the equivalent - as you pointed out.
I was already carrying the required dependency.
Includes the IIO ones in staging.
Thanks,
Jonathan
p.s. I perhaps foolishly did this in a highly manual way so as to
also pick up Andy's RB. So might have dropped one...
Definitely would have been better as one patch per subsystem with
a cover letter suitable for replies like Andy's to be picked up
by b4.
> iio: accel: adxl372_i2c: Convert to i2c's .probe_new()
> iio: accel: bma180: Convert to i2c's .probe_new()
> iio: accel: bma400: Convert to i2c's .probe_new()
> iio: accel: bmc150: Convert to i2c's .probe_new()
> iio: accel: da280: Convert to i2c's .probe_new()
> iio: accel: kxcjk-1013: Convert to i2c's .probe_new()
> iio: accel: mma7455_i2c: Convert to i2c's .probe_new()
> iio: accel: mma8452: Convert to i2c's .probe_new()
> iio: accel: mma9551: Convert to i2c's .probe_new()
> iio: accel: mma9553: Convert to i2c's .probe_new()
> iio: adc: ad7091r5: Convert to i2c's .probe_new()
> iio: adc: ad7291: Convert to i2c's .probe_new()
> iio: adc: ad799x: Convert to i2c's .probe_new()
> iio: adc: ina2xx-adc: Convert to i2c's .probe_new()
> iio: adc: ltc2471: Convert to i2c's .probe_new()
> iio: adc: ltc2485: Convert to i2c's .probe_new()
> iio: adc: ltc2497: Convert to i2c's .probe_new()
> iio: adc: max1363: Convert to i2c's .probe_new()
> iio: adc: max9611: Convert to i2c's .probe_new()
> iio: adc: mcp3422: Convert to i2c's .probe_new()
> iio: adc: ti-adc081c: Convert to i2c's .probe_new()
> iio: adc: ti-ads1015: Convert to i2c's .probe_new()
> iio: cdc: ad7150: Convert to i2c's .probe_new()
> iio: cdc: ad7746: Convert to i2c's .probe_new()
> iio: chemical: ams-iaq-core: Convert to i2c's .probe_new()
> iio: chemical: atlas-ezo-sensor: Convert to i2c's .probe_new()
> iio: chemical: atlas-sensor: Convert to i2c's .probe_new()
> iio: chemical: bme680_i2c: Convert to i2c's .probe_new()
> iio: chemical: ccs811: Convert to i2c's .probe_new()
> iio: chemical: scd4x: Convert to i2c's .probe_new()
> iio: chemical: sgp30: Convert to i2c's .probe_new()
> iio: chemical: sgp40: Convert to i2c's .probe_new()
> iio: chemical: vz89x: Convert to i2c's .probe_new()
> iio: dac: ad5064: Convert to i2c's .probe_new()
> iio: dac: ad5380: Convert to i2c's .probe_new()
> iio: dac: ad5446: Convert to i2c's .probe_new()
> iio: dac: ad5593r: Convert to i2c's .probe_new()
> iio: dac: ad5696-i2c: Convert to i2c's .probe_new()
> iio: dac: ds4424: Convert to i2c's .probe_new()
> iio: dac: m62332: Convert to i2c's .probe_new()
> iio: dac: max517: Convert to i2c's .probe_new()
> iio: dac: max5821: Convert to i2c's .probe_new()
> iio: dac: mcp4725: Convert to i2c's .probe_new()
> iio: dac: ti-dac5571: Convert to i2c's .probe_new()
> iio: gyro: bmg160_i2c: Convert to i2c's .probe_new()
> iio: gyro: itg3200_core: Convert to i2c's .probe_new()
> iio: gyro: mpu3050-i2c: Convert to i2c's .probe_new()
> iio: gyro: st_gyro_i2c: Convert to i2c's .probe_new()
> iio: health: afe4404: Convert to i2c's .probe_new()
> iio: health: max30100: Convert to i2c's .probe_new()
> iio: health: max30102: Convert to i2c's .probe_new()
> iio: humidity: am2315: Convert to i2c's .probe_new()
> iio: humidity: hdc100x: Convert to i2c's .probe_new()
> iio: humidity: hdc2010: Convert to i2c's .probe_new()
> iio: humidity: hts221_i2c: Convert to i2c's .probe_new()
> iio: humidity: htu21: Convert to i2c's .probe_new()
> iio: humidity: si7005: Convert to i2c's .probe_new()
> iio: humidity: si7020: Convert to i2c's .probe_new()
> iio: imu: bmi160/bmi160_i2c: Convert to i2c's .probe_new()
> iio: imu: fxos8700_i2c: Convert to i2c's .probe_new()
> iio: imu: inv_mpu6050: Convert to i2c's .probe_new()
> iio: imu: kmx61: Convert to i2c's .probe_new()
> iio: imu: st_lsm6dsx: Convert to i2c's .probe_new()
> iio: light: adjd_s311: Convert to i2c's .probe_new()
> iio: light: adux1020: Convert to i2c's .probe_new()
> iio: light: al3010: Convert to i2c's .probe_new()
> iio: light: al3320a: Convert to i2c's .probe_new()
> iio: light: apds9300: Convert to i2c's .probe_new()
> iio: light: apds9960: Convert to i2c's .probe_new()
> iio: light: bh1750: Convert to i2c's .probe_new()
> iio: light: bh1780: Convert to i2c's .probe_new()
> iio: light: cm3232: Convert to i2c's .probe_new()
> iio: light: cm3323: Convert to i2c's .probe_new()
> iio: light: cm36651: Convert to i2c's .probe_new()
> iio: light: gp2ap002: Convert to i2c's .probe_new()
> iio: light: gp2ap020a00f: Convert to i2c's .probe_new()
> iio: light: isl29018: Convert to i2c's .probe_new()
> iio: light: isl29028: Convert to i2c's .probe_new()
> iio: light: isl29125: Convert to i2c's .probe_new()
> iio: light: jsa1212: Convert to i2c's .probe_new()
> iio: light: ltr501: Convert to i2c's .probe_new()
> iio: light: lv0104cs: Convert to i2c's .probe_new()
> iio: light: max44000: Convert to i2c's .probe_new()
> iio: light: max44009: Convert to i2c's .probe_new()
> iio: light: noa1305: Convert to i2c's .probe_new()
> iio: light: opt3001: Convert to i2c's .probe_new()
> iio: light: pa12203001: Convert to i2c's .probe_new()
> iio: light: rpr0521: Convert to i2c's .probe_new()
> iio: light: si1133: Convert to i2c's .probe_new()
> iio: light: si1145: Convert to i2c's .probe_new()
> iio: light: st_uvis25_i2c: Convert to i2c's .probe_new()
> iio: light: stk3310: Convert to i2c's .probe_new()
> iio: light: tcs3414: Convert to i2c's .probe_new()
> iio: light: tcs3472: Convert to i2c's .probe_new()
> iio: light: tsl2563: Convert to i2c's .probe_new()
> iio: light: tsl2583: Convert to i2c's .probe_new()
> iio: light: tsl2772: Convert to i2c's .probe_new()
> iio: light: tsl4531: Convert to i2c's .probe_new()
> iio: light: us5182d: Convert to i2c's .probe_new()
> iio: light: vcnl4000: Convert to i2c's .probe_new()
> iio: light: vcnl4035: Convert to i2c's .probe_new()
> iio: light: veml6030: Convert to i2c's .probe_new()
> iio: light: veml6070: Convert to i2c's .probe_new()
> iio: light: zopt2201: Convert to i2c's .probe_new()
> iio: magnetometer: ak8974: Convert to i2c's .probe_new()
> iio: magnetometer: ak8975: Convert to i2c's .probe_new()
> iio: magnetometer: bmc150_magn_i2c: Convert to i2c's .probe_new()
> iio: magnetometer: hmc5843: Convert to i2c's .probe_new()
> iio: magnetometer: mag3110: Convert to i2c's .probe_new()
> iio: magnetometer: mmc35240: Convert to i2c's .probe_new()
> iio: magnetometer: yamaha-yas530: Convert to i2c's .probe_new()
> iio: potentiometer: ad5272: Convert to i2c's .probe_new()
> iio: potentiometer: ds1803: Convert to i2c's .probe_new()
> iio: potentiometer: max5432: Convert to i2c's .probe_new()
> iio: potentiometer: tpl0102: Convert to i2c's .probe_new()
> iio: potentiostat: lmp91000: Convert to i2c's .probe_new()
> iio: pressure: abp060mg: Convert to i2c's .probe_new()
Not this one > iio: pressure: bmp280-i2c: Convert to i2c's .probe_new()
> iio: pressure: dlhl60d: Convert to i2c's .probe_new()
> iio: pressure: dps310: Convert to i2c's .probe_new()
> iio: pressure: hp03: Convert to i2c's .probe_new()
> iio: pressure: hp206c: Convert to i2c's .probe_new()
> iio: pressure: icp10100: Convert to i2c's .probe_new()
> iio: pressure: mpl115_i2c: Convert to i2c's .probe_new()
> iio: pressure: mpl3115: Convert to i2c's .probe_new()
> iio: pressure: ms5611_i2c: Convert to i2c's .probe_new()
> iio: pressure: ms5637: Convert to i2c's .probe_new()
> iio: pressure: st_pressure_i2c: Convert to i2c's .probe_new()
> iio: pressure: t5403: Convert to i2c's .probe_new()
> iio: pressure: zpa2326_i2c: Convert to i2c's .probe_new()
> iio: proximity: isl29501: Convert to i2c's .probe_new()
> iio: proximity: mb1232: Convert to i2c's .probe_new()
> iio: proximity: pulsedlight-lidar-lite-v2: Convert to i2c's
> .probe_new()
> iio: proximity: rfd77402: Convert to i2c's .probe_new()
> iio: proximity: srf08: Convert to i2c's .probe_new()
> iio: proximity: sx9500: Convert to i2c's .probe_new()
> iio: temperature: mlx90614: Convert to i2c's .probe_new()
> iio: temperature: mlx90632: Convert to i2c's .probe_new()
> iio: temperature: tmp006: Convert to i2c's .probe_new()
> iio: temperature: tmp007: Convert to i2c's .probe_new()
> iio: temperature: tsys01: Convert to i2c's .probe_new()
> iio: temperature: tsys02d: Convert to i2c's .probe_new()
...
> staging: iio: adt7316: Convert to i2c's .probe_new()
> staging: iio: ad5933: Convert to i2c's .probe_new()
> staging: iio: ade7854: Convert to i2c's .probe_new()
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-22 18:58 ` Jonathan Cameron
@ 2022-11-22 20:16 ` Uwe Kleine-König
2022-11-26 15:43 ` Andy Shevchenko
0 siblings, 1 reply; 37+ messages in thread
From: Uwe Kleine-König @ 2022-11-22 20:16 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Uwe Kleine-König, alsa-devel, linux-staging, linux-pwm,
linux-iio, linux-fbdev, platform-driver-x86, linux-mtd, linux-i2c,
Lee Jones, linux-stm32, linux-leds, linux-rtc, linux-renesas-soc,
linux-samsung-soc, linux-pm, Broadcom internal kernel review list,
linux-serial, linux-input, Grant Likely, linux-media, devicetree,
linux-watchdog, chrome-platform, linux-actions, linux-gpio,
Angel Iglesias, gregkh, linux-rpi-kernel, linux-amlogic,
openipmi-developer, linux-omap, linux-arm-kernel,
Purism Kernel Team, patches, linux-usb, linux-kernel, linux-spi,
Wolfram Sang, linux-crypto, kernel, netdev, linux-integrity,
linuxppc-dev
[-- Attachment #1: Type: text/plain, Size: 1306 bytes --]
On Tue, Nov 22, 2022 at 06:58:18PM +0000, Jonathan Cameron wrote:
>
> Queued all of the below:
> with one tweaked as per your suggestion and the highlighted one dropped on basis
> I was already carrying the equivalent - as you pointed out.
>
> I was already carrying the required dependency.
>
> Includes the IIO ones in staging.
>
> Thanks,
>
> Jonathan
>
> p.s. I perhaps foolishly did this in a highly manual way so as to
> also pick up Andy's RB. So might have dropped one...
You could have done:
H=$(git rev-parse @)
b4 am -P 49-190 20221118224540.619276-1-uwe@kleine-koenig.org
git am ...
git filter-branch -f --msg-filter "grep -v 'Signed-off-by: Jonathan'; echo 'Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>'; echo 'Signed-off-by: Jonathan Cameron <jic23@kernel.org>'" $H..
(untested, but you get the idea).
> Definitely would have been better as one patch per subsystem with
> a cover letter suitable for replies like Andy's to be picked up
> by b4.
Next time I will go for one series per subsystem which I like better
than one patch per subsystem.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-22 20:16 ` Uwe Kleine-König
@ 2022-11-26 15:43 ` Andy Shevchenko
0 siblings, 0 replies; 37+ messages in thread
From: Andy Shevchenko @ 2022-11-26 15:43 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Jonathan Cameron, Uwe Kleine-König, alsa-devel,
linux-staging, linux-pwm, linux-iio, linux-fbdev,
platform-driver-x86, linux-mtd, linux-i2c, Lee Jones, linux-stm32,
linux-leds, linux-rtc, linux-renesas-soc, linux-samsung-soc,
linux-pm, Broadcom internal kernel review list, linux-serial,
linux-input, Grant Likely, linux-media, devicetree,
linux-watchdog, chrome-platform, linux-actions, linux-gpio,
Angel Iglesias, gregkh, linux-rpi-kernel, linux-amlogic,
openipmi-developer, linux-omap, linux-arm-kernel,
Purism Kernel Team, patches, linux-usb, linux-kernel, linux-spi,
Wolfram Sang, linux-crypto, kernel, netdev, linux-integrity,
linuxppc-dev
On Tue, Nov 22, 2022 at 09:16:54PM +0100, Uwe Kleine-König wrote:
> On Tue, Nov 22, 2022 at 06:58:18PM +0000, Jonathan Cameron wrote:
> > Queued all of the below:
> > with one tweaked as per your suggestion and the highlighted one dropped on basis
> > I was already carrying the equivalent - as you pointed out.
> >
> > I was already carrying the required dependency.
> >
> > Includes the IIO ones in staging.
> >
> > p.s. I perhaps foolishly did this in a highly manual way so as to
> > also pick up Andy's RB. So might have dropped one...
>
> You could have done:
>
> H=$(git rev-parse @)
> b4 am -P 49-190 20221118224540.619276-1-uwe@kleine-koenig.org
> git am ...
> git filter-branch -f --msg-filter "grep -v 'Signed-off-by: Jonathan'; echo 'Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>'; echo 'Signed-off-by: Jonathan Cameron <jic23@kernel.org>'" $H..
>
> (untested, but you get the idea).
That's, for example (just last from the history as is), how I usually do it
(tested):
git filter-branch --msg-filter 'sed -e "/Signed-off-by: Andy Shevchenko/ a Tested-by: Daniel Scally <dan.scally@ideasonboard.com>"' -f HEAD~4..HEAD
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 37+ messages in thread
* Re: (subset) [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (29 preceding siblings ...)
2022-11-22 18:58 ` Jonathan Cameron
@ 2022-12-09 12:00 ` Robert Foss
2022-12-12 16:36 ` Robert Foss
31 siblings, 0 replies; 37+ messages in thread
From: Robert Foss @ 2022-12-09 12:00 UTC (permalink / raw)
To: Uwe Kleine-König, Wolfram Sang, Angel Iglesias, Grant Likely,
Lee Jones
Cc: Robert Foss, Broadcom internal kernel review list, linux-gpio,
linux-rtc, linux-arm-kernel, linux-input, linux-integrity,
linux-media, openipmi-developer, linux-serial, linux-usb,
linux-spi, kernel, Purism Kernel Team, linux-rpi-kernel,
linux-leds, linux-actions, netdev, linux-iio, linux-pwm,
linux-staging, chrome-platform, linux-crypto, linux-watchdog,
linux-amlogic, linux-fbdev, linux-renesas-soc, linux-i2c,
linux-kernel, alsa-devel, linux-stm32, linuxppc-dev, patches,
linux-omap, linux-mtd, linux-samsung-soc, linux-pm,
platform-driver-x86, devicetree
On Fri, 18 Nov 2022 23:35:34 +0100, Uwe Kleine-König wrote:
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
>
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> [...]
Applied all patches that build.
Patches excluded:
- ps8622
- ti-sn65dsi83
- adv7511
Repo: https://cgit.freedesktop.org/drm/drm-misc/
[014/606] drm/bridge: adv7511: Convert to i2c's .probe_new()
(no commit info)
[015/606] drm/bridge/analogix/anx6345: Convert to i2c's .probe_new()
(no commit info)
[016/606] drm/bridge/analogix/anx78xx: Convert to i2c's .probe_new()
(no commit info)
[017/606] drm/bridge: anx7625: Convert to i2c's .probe_new()
(no commit info)
[018/606] drm/bridge: icn6211: Convert to i2c's .probe_new()
(no commit info)
[019/606] drm/bridge: chrontel-ch7033: Convert to i2c's .probe_new()
commit: 8dc6de280f01c0f7b8d40435736f3c975368ad70
[020/606] drm/bridge: it6505: Convert to i2c's .probe_new()
(no commit info)
[021/606] drm/bridge: it66121: Convert to i2c's .probe_new()
(no commit info)
[022/606] drm/bridge: lt8912b: Convert to i2c's .probe_new()
(no commit info)
[023/606] drm/bridge: lt9211: Convert to i2c's .probe_new()
(no commit info)
[024/606] drm/bridge: lt9611: Convert to i2c's .probe_new()
(no commit info)
[025/606] drm/bridge: lt9611uxc: Convert to i2c's .probe_new()
(no commit info)
[026/606] drm/bridge: megachips: Convert to i2c's .probe_new()
(no commit info)
[027/606] drm/bridge: nxp-ptn3460: Convert to i2c's .probe_new()
(no commit info)
[028/606] drm/bridge: parade-ps8622: Convert to i2c's .probe_new()
(no commit info)
[029/606] drm/bridge: sii902x: Convert to i2c's .probe_new()
(no commit info)
[030/606] drm/bridge: sii9234: Convert to i2c's .probe_new()
(no commit info)
[031/606] drm/bridge: sii8620: Convert to i2c's .probe_new()
(no commit info)
[032/606] drm/bridge: tc358767: Convert to i2c's .probe_new()
(no commit info)
[033/606] drm/bridge: tc358768: Convert to i2c's .probe_new()
(no commit info)
[034/606] drm/bridge/tc358775: Convert to i2c's .probe_new()
(no commit info)
[035/606] drm/bridge: ti-sn65dsi83: Convert to i2c's .probe_new()
(no commit info)
[037/606] drm/bridge: tfp410: Convert to i2c's .probe_new()
(no commit info)
rob
^ permalink raw reply [flat|nested] 37+ messages in thread* Re: (subset) [PATCH 000/606] i2c: Complete conversion to i2c_probe_new
2022-11-18 22:35 [PATCH 000/606] i2c: Complete conversion to i2c_probe_new Uwe Kleine-König
` (30 preceding siblings ...)
2022-12-09 12:00 ` (subset) " Robert Foss
@ 2022-12-12 16:36 ` Robert Foss
31 siblings, 0 replies; 37+ messages in thread
From: Robert Foss @ 2022-12-12 16:36 UTC (permalink / raw)
To: Lee Jones, Uwe Kleine-König, Angel Iglesias, Wolfram Sang,
Grant Likely
Cc: Robert Foss, linuxppc-dev, linux-actions, linux-spi, devicetree,
linux-media, linux-stm32, linux-crypto, chrome-platform,
Broadcom internal kernel review list, linux-input,
linux-rpi-kernel, linux-integrity, linux-omap, linux-serial,
Purism Kernel Team, linux-staging, alsa-devel, linux-watchdog,
linux-gpio, linux-rtc, linux-leds, linux-iio, linux-kernel,
linux-usb, linux-pm, platform-driver-x86, patches, linux-mtd,
linux-renesas-soc, linux-amlogic, linux-pwm, linux-samsung-soc,
linux-i2c, kernel, netdev, linux-arm-kernel, openipmi-developer,
linux-fbdev
On Fri, 18 Nov 2022 23:35:34 +0100, Uwe Kleine-König wrote:
> since commit b8a1a4cd5a98 ("i2c: Provide a temporary .probe_new()
> call-back type") from 2016 there is a "temporary" alternative probe
> callback for i2c drivers.
>
> This series completes all drivers to this new callback (unless I missed
> something). It's based on current next/master.
> 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:
>
> [...]
Applied, thanks!
Repo: https://cgit.freedesktop.org/drm/drm-misc/
[014/606] drm/bridge: adv7511: Convert to i2c's .probe_new()
commit: 1c546894ff82f8b7c070998c03f9b15a3499f326
[028/606] drm/bridge: parade-ps8622: Convert to i2c's .probe_new()
commit: d6b522e9bbb0cca1aeae4ef6188800534794836f
[035/606] drm/bridge: ti-sn65dsi83: Convert to i2c's .probe_new()
commit: 0f6548807fa77e87bbc37964c6b1ed9ba6e1155d
rob
^ permalink raw reply [flat|nested] 37+ messages in thread