mbox series

[RESEND,v3,0/2] Check for endpoints in fwnode->secondary more sensibly

Message ID 20210806220906.326758-1-djrscally@gmail.com
Headers show
Series Check for endpoints in fwnode->secondary more sensibly | expand

Message

Daniel Scally Aug. 6, 2021, 10:09 p.m. UTC
Hello all

A while ago I patched fwnode_graph_get_endpoint_by_id() to check for endpoints
against fwnode->secondary if none was found against the primary. It's actually
better to do this in fwnode_graph_get_next_endpoint() instead, since that
function is called by fwnode_graph_get_endpoint_by_id() and also directly called
in a bunch of other places (primarily sensor drivers checking that they have
endpoints connected during probe). This small series just adds the equivalent
functionality to fwnode_graph_get_next_endpoint() and reverts the earlier
commit.

Resending with +CC linux-acpi

Thanks
Dan


Daniel Scally (2):
  device property: Check fwnode->secondary in
    fwnode_graph_get_next_endpoint()
  Revert "media: device property: Call fwnode_graph_get_endpoint_by_id()
    for fwnode->secondary"

 drivers/base/property.c | 30 +++++++++++++++++++++---------
 1 file changed, 21 insertions(+), 9 deletions(-)

Comments

Rafael J. Wysocki Aug. 16, 2021, 4:36 p.m. UTC | #1
On Sat, Aug 7, 2021 at 12:09 AM Daniel Scally <djrscally@gmail.com> wrote:
>

> Hello all

>

> A while ago I patched fwnode_graph_get_endpoint_by_id() to check for endpoints

> against fwnode->secondary if none was found against the primary. It's actually

> better to do this in fwnode_graph_get_next_endpoint() instead, since that

> function is called by fwnode_graph_get_endpoint_by_id() and also directly called

> in a bunch of other places (primarily sensor drivers checking that they have

> endpoints connected during probe). This small series just adds the equivalent

> functionality to fwnode_graph_get_next_endpoint() and reverts the earlier

> commit.

>

> Resending with +CC linux-acpi


Both patches applied as 5.15 material, thanks!