* [PATCH v2 00/10] clk: switch to simple i2c probe function
@ 2022-04-07 15:18 Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
` (9 more replies)
0 siblings, 10 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
This series of patches updates i2c probes in clk to use the simple
probe variant (simple as in "single argument"), probe_new, following
one of two scenarios.
In the first scenario, the existing probe function
int (*probe)(struct i2c_client *client, const struct i2c_device_id *id);
(see struct i2c_driver in include/linux/i2c.h) doesn't use the id
argument, so it can be trivially converted to
int (*probe_new)(struct i2c_client *client);
and the i2c_driver declaration updated to use .probe_new instead of
.probe.
In the second scenario, the probe function does use the id argument.
In this case, the relevant part of the calling code,
/*
* When there are no more users of probe(),
* rename probe_new to probe.
*/
if (driver->probe_new)
status = driver->probe_new(client);
else if (driver->probe)
status = driver->probe(client,
i2c_match_id(driver->id_table, client));
else
status = -EINVAL;
(from drivers/i2c/i2c-core-base.c) is moved down into the probe
function, typically along the lines of
const struct i2c_device_id *id = i2c_match_id(ids, id);
where ids is the i2c_device_id table. The latter needs to be declared
before the probe function; as suggested by Wolfram Sang, the full
definition is moved above the probe function.
Changes since v1:
- the pair of patches have been split up into a single patch series
with one patch per modified file, and the commit titles adjusted
accordingly
- renesas-pcie has been added
Stephen Kitt (10):
clk: cdce706: use simple i2c probe function
clk: cdce925: use i2c_match_id and simple i2c probe
clk: cs2000-cp: use simple i2c probe function
clk: max9485: use simple i2c probe function
clk: si514: use simple i2c probe function
clk: si5341: use simple i2c probe function
clk: si5351: use i2c_match_id and simple i2c probe
clk: si544: use i2c_match_id and simple i2c probe
clk: si570: use i2c_match_id and simple i2c probe
clk: renesas-pcie: use simple i2c probe function
drivers/clk/clk-cdce706.c | 5 ++---
drivers/clk/clk-cdce925.c | 24 ++++++++++++------------
drivers/clk/clk-cs2000-cp.c | 5 ++---
drivers/clk/clk-max9485.c | 5 ++---
drivers/clk/clk-renesas-pcie.c | 4 ++--
drivers/clk/clk-si514.c | 5 ++---
drivers/clk/clk-si5341.c | 5 ++---
drivers/clk/clk-si5351.c | 24 ++++++++++++------------
drivers/clk/clk-si544.c | 22 +++++++++++-----------
drivers/clk/clk-si570.c | 24 ++++++++++++------------
10 files changed, 59 insertions(+), 64 deletions(-)
--
2.27.0
^ permalink raw reply [flat|nested] 32+ messages in thread
* [PATCH v2 01/10] clk: cdce706: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:09 ` Wolfram Sang
` (2 more replies)
2022-04-07 15:18 ` [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe Stephen Kitt
` (8 subsequent siblings)
9 siblings, 3 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-cdce706.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk-cdce706.c b/drivers/clk/clk-cdce706.c
index c91e9096b070..5467d941ddfd 100644
--- a/drivers/clk/clk-cdce706.c
+++ b/drivers/clk/clk-cdce706.c
@@ -627,8 +627,7 @@ of_clk_cdce_get(struct of_phandle_args *clkspec, void *data)
return &cdce->clkout[idx].hw;
}
-static int cdce706_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int cdce706_probe(struct i2c_client *client)
{
struct i2c_adapter *adapter = client->adapter;
struct cdce706_dev_data *cdce;
@@ -692,7 +691,7 @@ static struct i2c_driver cdce706_i2c_driver = {
.name = "cdce706",
.of_match_table = of_match_ptr(cdce706_dt_match),
},
- .probe = cdce706_probe,
+ .probe_new = cdce706_probe,
.remove = cdce706_remove,
.id_table = cdce706_id,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:13 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function Stephen Kitt
` (7 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
As part of the ongoing i2c transition to the simple probe
("probe_new"), this patch uses i2c_match_id to retrieve the
driver_data for the probed device. The id parameter is thus no longer
necessary and the simple probe can be used instead.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-cdce925.c | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/clk/clk-cdce925.c b/drivers/clk/clk-cdce925.c
index 308b353815e1..ef9a2d44e40c 100644
--- a/drivers/clk/clk-cdce925.c
+++ b/drivers/clk/clk-cdce925.c
@@ -634,11 +634,20 @@ static struct regmap_bus regmap_cdce925_bus = {
.read = cdce925_regmap_i2c_read,
};
-static int cdce925_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static const struct i2c_device_id cdce925_id[] = {
+ { "cdce913", CDCE913 },
+ { "cdce925", CDCE925 },
+ { "cdce937", CDCE937 },
+ { "cdce949", CDCE949 },
+ { }
+};
+MODULE_DEVICE_TABLE(i2c, cdce925_id);
+
+static int cdce925_probe(struct i2c_client *client)
{
struct clk_cdce925_chip *data;
struct device_node *node = client->dev.of_node;
+ const struct i2c_device_id *id = i2c_match_id(cdce925_id, client);
const char *parent_name;
const char *pll_clk_name[MAX_NUMBER_OF_PLLS] = {NULL,};
struct clk_init_data init;
@@ -814,15 +823,6 @@ static int cdce925_probe(struct i2c_client *client,
return err;
}
-static const struct i2c_device_id cdce925_id[] = {
- { "cdce913", CDCE913 },
- { "cdce925", CDCE925 },
- { "cdce937", CDCE937 },
- { "cdce949", CDCE949 },
- { }
-};
-MODULE_DEVICE_TABLE(i2c, cdce925_id);
-
static const struct of_device_id clk_cdce925_of_match[] = {
{ .compatible = "ti,cdce913" },
{ .compatible = "ti,cdce925" },
@@ -837,7 +837,7 @@ static struct i2c_driver cdce925_driver = {
.name = "cdce925",
.of_match_table = of_match_ptr(clk_cdce925_of_match),
},
- .probe = cdce925_probe,
+ .probe_new = cdce925_probe,
.id_table = cdce925_id,
};
module_i2c_driver(cdce925_driver);
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 04/10] clk: max9485: " Stephen Kitt
` (6 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-cs2000-cp.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk-cs2000-cp.c b/drivers/clk/clk-cs2000-cp.c
index dc5040a84dcc..aa5c72bab83e 100644
--- a/drivers/clk/clk-cs2000-cp.c
+++ b/drivers/clk/clk-cs2000-cp.c
@@ -570,8 +570,7 @@ static int cs2000_remove(struct i2c_client *client)
return 0;
}
-static int cs2000_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int cs2000_probe(struct i2c_client *client)
{
struct cs2000_priv *priv;
struct device *dev = &client->dev;
@@ -625,7 +624,7 @@ static struct i2c_driver cs2000_driver = {
.pm = &cs2000_pm_ops,
.of_match_table = cs2000_of_match,
},
- .probe = cs2000_probe,
+ .probe_new = cs2000_probe,
.remove = cs2000_remove,
.id_table = cs2000_id,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 04/10] clk: max9485: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (2 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 05/10] clk: si514: " Stephen Kitt
` (5 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-max9485.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk-max9485.c b/drivers/clk/clk-max9485.c
index 5e80f3d090f3..5f85b0a32872 100644
--- a/drivers/clk/clk-max9485.c
+++ b/drivers/clk/clk-max9485.c
@@ -254,8 +254,7 @@ max9485_of_clk_get(struct of_phandle_args *clkspec, void *data)
return &drvdata->hw[idx].hw;
}
-static int max9485_i2c_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int max9485_i2c_probe(struct i2c_client *client)
{
struct max9485_driver_data *drvdata;
struct device *dev = &client->dev;
@@ -377,7 +376,7 @@ static struct i2c_driver max9485_driver = {
.pm = &max9485_pm_ops,
.of_match_table = max9485_dt_ids,
},
- .probe = max9485_i2c_probe,
+ .probe_new = max9485_i2c_probe,
.id_table = max9485_i2c_ids,
};
module_i2c_driver(max9485_driver);
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 05/10] clk: si514: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (3 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 04/10] clk: max9485: " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 06/10] clk: si5341: " Stephen Kitt
` (4 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-si514.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk-si514.c b/drivers/clk/clk-si514.c
index 364b62b9928d..4481c4303534 100644
--- a/drivers/clk/clk-si514.c
+++ b/drivers/clk/clk-si514.c
@@ -327,8 +327,7 @@ static const struct regmap_config si514_regmap_config = {
.volatile_reg = si514_regmap_is_volatile,
};
-static int si514_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int si514_probe(struct i2c_client *client)
{
struct clk_si514 *data;
struct clk_init_data init;
@@ -394,7 +393,7 @@ static struct i2c_driver si514_driver = {
.name = "si514",
.of_match_table = clk_si514_of_match,
},
- .probe = si514_probe,
+ .probe_new = si514_probe,
.remove = si514_remove,
.id_table = si514_id,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 06/10] clk: si5341: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (4 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 05/10] clk: si514: " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe Stephen Kitt
` (3 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-si5341.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk-si5341.c b/drivers/clk/clk-si5341.c
index 41851f41b682..4bca73212662 100644
--- a/drivers/clk/clk-si5341.c
+++ b/drivers/clk/clk-si5341.c
@@ -1547,8 +1547,7 @@ static const struct attribute *si5341_attributes[] = {
NULL
};
-static int si5341_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static int si5341_probe(struct i2c_client *client)
{
struct clk_si5341 *data;
struct clk_init_data init;
@@ -1837,7 +1836,7 @@ static struct i2c_driver si5341_driver = {
.name = "si5341",
.of_match_table = clk_si5341_of_match,
},
- .probe = si5341_probe,
+ .probe_new = si5341_probe,
.remove = si5341_remove,
.id_table = si5341_id,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (5 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 06/10] clk: si5341: " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:18 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 08/10] clk: si544: " Stephen Kitt
` (2 subsequent siblings)
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
As part of the ongoing i2c transition to the simple probe
("probe_new"), this patch uses i2c_match_id to retrieve the
driver_data for the probed device. The id parameter is thus no longer
necessary and the simple probe can be used instead.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-si5351.c | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/clk/clk-si5351.c b/drivers/clk/clk-si5351.c
index 93fa8c9e11be..b9f088c4ba2f 100644
--- a/drivers/clk/clk-si5351.c
+++ b/drivers/clk/clk-si5351.c
@@ -1367,9 +1367,18 @@ si53351_of_clk_get(struct of_phandle_args *clkspec, void *data)
}
#endif /* CONFIG_OF */
-static int si5351_i2c_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static const struct i2c_device_id si5351_i2c_ids[] = {
+ { "si5351a", SI5351_VARIANT_A },
+ { "si5351a-msop", SI5351_VARIANT_A3 },
+ { "si5351b", SI5351_VARIANT_B },
+ { "si5351c", SI5351_VARIANT_C },
+ { }
+};
+MODULE_DEVICE_TABLE(i2c, si5351_i2c_ids);
+
+static int si5351_i2c_probe(struct i2c_client *client)
{
+ const struct i2c_device_id *id = i2c_match_id(si5351_i2c_ids, client);
enum si5351_variant variant = (enum si5351_variant)id->driver_data;
struct si5351_platform_data *pdata;
struct si5351_driver_data *drvdata;
@@ -1649,21 +1658,12 @@ static int si5351_i2c_remove(struct i2c_client *client)
return 0;
}
-static const struct i2c_device_id si5351_i2c_ids[] = {
- { "si5351a", SI5351_VARIANT_A },
- { "si5351a-msop", SI5351_VARIANT_A3 },
- { "si5351b", SI5351_VARIANT_B },
- { "si5351c", SI5351_VARIANT_C },
- { }
-};
-MODULE_DEVICE_TABLE(i2c, si5351_i2c_ids);
-
static struct i2c_driver si5351_driver = {
.driver = {
.name = "si5351",
.of_match_table = of_match_ptr(si5351_dt_ids),
},
- .probe = si5351_i2c_probe,
+ .probe_new = si5351_i2c_probe,
.remove = si5351_i2c_remove,
.id_table = si5351_i2c_ids,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 08/10] clk: si544: use i2c_match_id and simple i2c probe
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (6 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 09/10] clk: si570: " Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function Stephen Kitt
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
As part of the ongoing i2c transition to the simple probe
("probe_new"), this patch uses i2c_match_id to retrieve the
driver_data for the probed device. The id parameter is thus no longer
necessary and the simple probe can be used instead.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-si544.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/clk/clk-si544.c b/drivers/clk/clk-si544.c
index d9ec9086184d..089786907641 100644
--- a/drivers/clk/clk-si544.c
+++ b/drivers/clk/clk-si544.c
@@ -451,11 +451,19 @@ static const struct regmap_config si544_regmap_config = {
.volatile_reg = si544_regmap_is_volatile,
};
-static int si544_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static const struct i2c_device_id si544_id[] = {
+ { "si544a", si544a },
+ { "si544b", si544b },
+ { "si544c", si544c },
+ { }
+};
+MODULE_DEVICE_TABLE(i2c, si544_id);
+
+static int si544_probe(struct i2c_client *client)
{
struct clk_si544 *data;
struct clk_init_data init;
+ const struct i2c_device_id *id = i2c_match_id(si544_id, client);
int err;
data = devm_kzalloc(&client->dev, sizeof(*data), GFP_KERNEL);
@@ -499,14 +507,6 @@ static int si544_probe(struct i2c_client *client,
return 0;
}
-static const struct i2c_device_id si544_id[] = {
- { "si544a", si544a },
- { "si544b", si544b },
- { "si544c", si544c },
- { }
-};
-MODULE_DEVICE_TABLE(i2c, si544_id);
-
static const struct of_device_id clk_si544_of_match[] = {
{ .compatible = "silabs,si544a" },
{ .compatible = "silabs,si544b" },
@@ -520,7 +520,7 @@ static struct i2c_driver si544_driver = {
.name = "si544",
.of_match_table = clk_si544_of_match,
},
- .probe = si544_probe,
+ .probe_new = si544_probe,
.id_table = si544_id,
};
module_i2c_driver(si544_driver);
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 09/10] clk: si570: use i2c_match_id and simple i2c probe
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (7 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 08/10] clk: si544: " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function Stephen Kitt
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
As part of the ongoing i2c transition to the simple probe
("probe_new"), this patch uses i2c_match_id to retrieve the
driver_data for the probed device. The id parameter is thus no longer
necessary and the simple probe can be used instead.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-si570.c | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/clk/clk-si570.c b/drivers/clk/clk-si570.c
index eea50121718a..1ff8f32f734d 100644
--- a/drivers/clk/clk-si570.c
+++ b/drivers/clk/clk-si570.c
@@ -398,11 +398,20 @@ static const struct regmap_config si570_regmap_config = {
.volatile_reg = si570_regmap_is_volatile,
};
-static int si570_probe(struct i2c_client *client,
- const struct i2c_device_id *id)
+static const struct i2c_device_id si570_id[] = {
+ { "si570", si57x },
+ { "si571", si57x },
+ { "si598", si59x },
+ { "si599", si59x },
+ { }
+};
+MODULE_DEVICE_TABLE(i2c, si570_id);
+
+static int si570_probe(struct i2c_client *client)
{
struct clk_si570 *data;
struct clk_init_data init;
+ const struct i2c_device_id *id = i2c_match_id(si570_id, client);
u32 initial_fout, factory_fout, stability;
bool skip_recall;
int err;
@@ -495,15 +504,6 @@ static int si570_remove(struct i2c_client *client)
return 0;
}
-static const struct i2c_device_id si570_id[] = {
- { "si570", si57x },
- { "si571", si57x },
- { "si598", si59x },
- { "si599", si59x },
- { }
-};
-MODULE_DEVICE_TABLE(i2c, si570_id);
-
static const struct of_device_id clk_si570_of_match[] = {
{ .compatible = "silabs,si570" },
{ .compatible = "silabs,si571" },
@@ -518,7 +518,7 @@ static struct i2c_driver si570_driver = {
.name = "si570",
.of_match_table = clk_si570_of_match,
},
- .probe = si570_probe,
+ .probe_new = si570_probe,
.remove = si570_remove,
.id_table = si570_id,
};
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
` (8 preceding siblings ...)
2022-04-07 15:18 ` [PATCH v2 09/10] clk: si570: " Stephen Kitt
@ 2022-04-07 15:18 ` Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
9 siblings, 2 replies; 32+ messages in thread
From: Stephen Kitt @ 2022-04-07 15:18 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
The i2c probe function here doesn't use the id information provided in
its second argument, so the single-parameter i2c probe function
("probe_new") can be used instead.
This avoids scanning the identifier tables during probes.
Signed-off-by: Stephen Kitt <steve@sk2.org>
---
drivers/clk/clk-renesas-pcie.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/clk-renesas-pcie.c b/drivers/clk/clk-renesas-pcie.c
index 59d9cf0053eb..4f5df1fc74b4 100644
--- a/drivers/clk/clk-renesas-pcie.c
+++ b/drivers/clk/clk-renesas-pcie.c
@@ -213,7 +213,7 @@ rs9_of_clk_get(struct of_phandle_args *clkspec, void *data)
return rs9->clk_dif[idx];
}
-static int rs9_probe(struct i2c_client *client, const struct i2c_device_id *id)
+static int rs9_probe(struct i2c_client *client)
{
unsigned char name[5] = "DIF0";
struct rs9_driver_data *rs9;
@@ -312,7 +312,7 @@ static struct i2c_driver rs9_driver = {
.pm = &rs9_pm_ops,
.of_match_table = clk_rs9_of_match,
},
- .probe = rs9_probe,
+ .probe_new = rs9_probe,
.id_table = rs9_id,
};
module_i2c_driver(rs9_driver);
--
2.27.0
^ permalink raw reply related [flat|nested] 32+ messages in thread
* Re: [PATCH v2 01/10] clk: cdce706: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
@ 2022-04-07 20:09 ` Wolfram Sang
2022-04-07 20:15 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2 siblings, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:09 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:22PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe Stephen Kitt
@ 2022-04-07 20:13 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:13 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 432 bytes --]
On Thu, Apr 07, 2022 at 05:18:23PM +0200, Stephen Kitt wrote:
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function Stephen Kitt
@ 2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:14 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:24PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 04/10] clk: max9485: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 04/10] clk: max9485: " Stephen Kitt
@ 2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:14 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:25PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 05/10] clk: si514: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 05/10] clk: si514: " Stephen Kitt
@ 2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:14 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:26PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 06/10] clk: si5341: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 06/10] clk: si5341: " Stephen Kitt
@ 2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:14 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:27PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 01/10] clk: cdce706: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
2022-04-07 20:09 ` Wolfram Sang
@ 2022-04-07 20:15 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2 siblings, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:15 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:22PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe Stephen Kitt
@ 2022-04-07 20:18 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:18 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 432 bytes --]
On Thu, Apr 07, 2022 at 05:18:28PM +0200, Stephen Kitt wrote:
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 08/10] clk: si544: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 08/10] clk: si544: " Stephen Kitt
@ 2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:19 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 432 bytes --]
On Thu, Apr 07, 2022 at 05:18:29PM +0200, Stephen Kitt wrote:
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 09/10] clk: si570: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 09/10] clk: si570: " Stephen Kitt
@ 2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:19 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 432 bytes --]
On Thu, Apr 07, 2022 at 05:18:30PM +0200, Stephen Kitt wrote:
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function Stephen Kitt
@ 2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Wolfram Sang @ 2022-04-07 20:19 UTC (permalink / raw)
To: Stephen Kitt; +Cc: Michael Turquette, Stephen Boyd, linux-clk, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Thu, Apr 07, 2022 at 05:18:31PM +0200, Stephen Kitt wrote:
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 01/10] clk: cdce706: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
2022-04-07 20:09 ` Wolfram Sang
2022-04-07 20:15 ` Wolfram Sang
@ 2022-04-23 2:32 ` Stephen Boyd
2 siblings, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:32 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:22)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe Stephen Kitt
2022-04-07 20:13 ` Wolfram Sang
@ 2022-04-23 2:32 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:32 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:23)
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
@ 2022-04-23 2:32 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:32 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:24)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 04/10] clk: max9485: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 04/10] clk: max9485: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
@ 2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:33 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:25)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 05/10] clk: si514: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 05/10] clk: si514: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
@ 2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:33 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:26)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 06/10] clk: si5341: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 06/10] clk: si5341: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
@ 2022-04-23 2:33 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:33 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:27)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe Stephen Kitt
2022-04-07 20:18 ` Wolfram Sang
@ 2022-04-23 2:34 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:34 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:28)
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 08/10] clk: si544: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 08/10] clk: si544: " Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
@ 2022-04-23 2:34 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:34 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:29)
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 09/10] clk: si570: use i2c_match_id and simple i2c probe
2022-04-07 15:18 ` [PATCH v2 09/10] clk: si570: " Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
@ 2022-04-23 2:35 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:35 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:30)
> As part of the ongoing i2c transition to the simple probe
> ("probe_new"), this patch uses i2c_match_id to retrieve the
> driver_data for the probed device. The id parameter is thus no longer
> necessary and the simple probe can be used instead.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
* Re: [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function
2022-04-07 15:18 ` [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
@ 2022-04-23 2:35 ` Stephen Boyd
1 sibling, 0 replies; 32+ messages in thread
From: Stephen Boyd @ 2022-04-23 2:35 UTC (permalink / raw)
To: Michael Turquette, Stephen Kitt
Cc: linux-clk, linux-kernel, Wolfram Sang, Stephen Kitt
Quoting Stephen Kitt (2022-04-07 08:18:31)
> The i2c probe function here doesn't use the id information provided in
> its second argument, so the single-parameter i2c probe function
> ("probe_new") can be used instead.
>
> This avoids scanning the identifier tables during probes.
>
> Signed-off-by: Stephen Kitt <steve@sk2.org>
> ---
Applied to clk-next
^ permalink raw reply [flat|nested] 32+ messages in thread
end of thread, other threads:[~2022-04-23 2:35 UTC | newest]
Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-07 15:18 [PATCH v2 00/10] clk: switch to simple i2c probe function Stephen Kitt
2022-04-07 15:18 ` [PATCH v2 01/10] clk: cdce706: use " Stephen Kitt
2022-04-07 20:09 ` Wolfram Sang
2022-04-07 20:15 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 02/10] clk: cdce925: use i2c_match_id and simple i2c probe Stephen Kitt
2022-04-07 20:13 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 03/10] clk: cs2000-cp: use simple i2c probe function Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:32 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 04/10] clk: max9485: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 05/10] clk: si514: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 06/10] clk: si5341: " Stephen Kitt
2022-04-07 20:14 ` Wolfram Sang
2022-04-23 2:33 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 07/10] clk: si5351: use i2c_match_id and simple i2c probe Stephen Kitt
2022-04-07 20:18 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 08/10] clk: si544: " Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:34 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 09/10] clk: si570: " Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
2022-04-07 15:18 ` [PATCH v2 10/10] clk: renesas-pcie: use simple i2c probe function Stephen Kitt
2022-04-07 20:19 ` Wolfram Sang
2022-04-23 2:35 ` Stephen Boyd
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).