Re: [PATCH v3 05/24] media: i2c: switch to use of_graph_get_next_device_endpoint()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Hi Rob

> This is assuming there's just 1 port and 1 endpoint, but let's be 
> specific as the bindings are (first endpoint on port 0):
> of_graph_get_endpoint_by_regs(client->dev.of_node, 0, -1);
> Note we could ask for endpoint 0 here, but the bindings generally allow 
> for more than 1.
> I imagine most of the other cases here are the same.

I will do it on new patch-set

> > -	for_each_endpoint_of_node(state->dev->of_node, ep_np) {
> > +	for_each_device_endpoint_of_node(state->dev->of_node, ep_np) {
> I would skip the rename.

It is needed to avoid confuse, because new function will add
another endpoint loop.


Thank you for your help !!

Best regards
Renesas Electronics
Ph.D. Kuninori Morimoto

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux