Skip to content

Commit c188df3

Browse files
morimotoHans Verkuil
authored andcommitted
media: i2c: replace of_graph_get_next_endpoint()
From DT point of view, in general, drivers should be asking for a specific port number because their function is fixed in the binding. of_graph_get_next_endpoint() doesn't match to this concept. Simply replace - of_graph_get_next_endpoint(xxx, NULL); + of_graph_get_endpoint_by_regs(xxx, 0, -1); Link: https://lore.kernel.org/r/[email protected] Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Kuninori Morimoto <[email protected]> Signed-off-by: Sakari Ailus <[email protected]> Signed-off-by: Hans Verkuil <[email protected]>
1 parent 7b61650 commit c188df3

File tree

13 files changed

+14
-14
lines changed

13 files changed

+14
-14
lines changed

drivers/media/i2c/adv7343.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ adv7343_get_pdata(struct i2c_client *client)
403403
if (!IS_ENABLED(CONFIG_OF) || !client->dev.of_node)
404404
return client->dev.platform_data;
405405

406-
np = of_graph_get_next_endpoint(client->dev.of_node, NULL);
406+
np = of_graph_get_endpoint_by_regs(client->dev.of_node, 0, -1);
407407
if (!np)
408408
return NULL;
409409

drivers/media/i2c/adv7604.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3204,8 +3204,8 @@ static int adv76xx_parse_dt(struct adv76xx_state *state)
32043204

32053205
np = state->i2c_clients[ADV76XX_PAGE_IO]->dev.of_node;
32063206

3207-
/* Parse the endpoint. */
3208-
endpoint = of_graph_get_next_endpoint(np, NULL);
3207+
/* FIXME: Parse the endpoint. */
3208+
endpoint = of_graph_get_endpoint_by_regs(np, -1, -1);
32093209
if (!endpoint)
32103210
return -EINVAL;
32113211

drivers/media/i2c/mt9p031.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1078,7 +1078,7 @@ mt9p031_get_pdata(struct i2c_client *client)
10781078
if (!IS_ENABLED(CONFIG_OF) || !client->dev.of_node)
10791079
return client->dev.platform_data;
10801080

1081-
np = of_graph_get_next_endpoint(client->dev.of_node, NULL);
1081+
np = of_graph_get_endpoint_by_regs(client->dev.of_node, 0, -1);
10821082
if (!np)
10831083
return NULL;
10841084

drivers/media/i2c/mt9v032.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1006,7 +1006,7 @@ mt9v032_get_pdata(struct i2c_client *client)
10061006
if (!IS_ENABLED(CONFIG_OF) || !client->dev.of_node)
10071007
return client->dev.platform_data;
10081008

1009-
np = of_graph_get_next_endpoint(client->dev.of_node, NULL);
1009+
np = of_graph_get_endpoint_by_regs(client->dev.of_node, 0, -1);
10101010
if (!np)
10111011
return NULL;
10121012

drivers/media/i2c/ov2659.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1388,7 +1388,7 @@ ov2659_get_pdata(struct i2c_client *client)
13881388
if (!IS_ENABLED(CONFIG_OF) || !client->dev.of_node)
13891389
return client->dev.platform_data;
13901390

1391-
endpoint = of_graph_get_next_endpoint(client->dev.of_node, NULL);
1391+
endpoint = of_graph_get_endpoint_by_regs(client->dev.of_node, 0, -1);
13921392
if (!endpoint)
13931393
return NULL;
13941394

drivers/media/i2c/ov5645.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1059,7 +1059,7 @@ static int ov5645_probe(struct i2c_client *client)
10591059
ov5645->i2c_client = client;
10601060
ov5645->dev = dev;
10611061

1062-
endpoint = of_graph_get_next_endpoint(dev->of_node, NULL);
1062+
endpoint = of_graph_get_endpoint_by_regs(dev->of_node, 0, -1);
10631063
if (!endpoint) {
10641064
dev_err(dev, "endpoint node not found\n");
10651065
return -EINVAL;

drivers/media/i2c/ov5647.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1363,7 +1363,7 @@ static int ov5647_parse_dt(struct ov5647 *sensor, struct device_node *np)
13631363
struct device_node *ep;
13641364
int ret;
13651365

1366-
ep = of_graph_get_next_endpoint(np, NULL);
1366+
ep = of_graph_get_endpoint_by_regs(np, 0, -1);
13671367
if (!ep)
13681368
return -EINVAL;
13691369

drivers/media/i2c/s5c73m3/s5c73m3-core.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1568,7 +1568,7 @@ static int s5c73m3_get_dt_data(struct s5c73m3 *state)
15681568
"failed to request gpio S5C73M3_RST\n");
15691569
gpiod_set_consumer_name(state->reset, "S5C73M3_RST");
15701570

1571-
node_ep = of_graph_get_next_endpoint(node, NULL);
1571+
node_ep = of_graph_get_endpoint_by_regs(node, 0, -1);
15721572
if (!node_ep) {
15731573
dev_warn(dev, "no endpoint defined for node: %pOF\n", node);
15741574
return 0;

drivers/media/i2c/s5k5baf.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1849,7 +1849,7 @@ static int s5k5baf_parse_device_node(struct s5k5baf *state, struct device *dev)
18491849
state->mclk_frequency);
18501850
}
18511851

1852-
node_ep = of_graph_get_next_endpoint(node, NULL);
1852+
node_ep = of_graph_get_endpoint_by_regs(node, 0, -1);
18531853
if (!node_ep) {
18541854
dev_err(dev, "no endpoint defined at node %pOF\n", node);
18551855
return -EINVAL;

drivers/media/i2c/tc358743.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1895,7 +1895,7 @@ static int tc358743_probe_of(struct tc358743_state *state)
18951895
return dev_err_probe(dev, PTR_ERR(refclk),
18961896
"failed to get refclk\n");
18971897

1898-
ep = of_graph_get_next_endpoint(dev->of_node, NULL);
1898+
ep = of_graph_get_endpoint_by_regs(dev->of_node, 0, -1);
18991899
if (!ep) {
19001900
dev_err(dev, "missing endpoint node\n");
19011901
return -EINVAL;

0 commit comments

Comments
 (0)