On Mon, Dec 05, 2016 at 10:13:38AM -0300, Javier Martinez Canillas wrote: > Hello Javi, > > On 12/05/2016 07:09 AM, Javi Merino wrote: > > In asds configured with V4L2_ASYNC_MATCH_OF, the v4l2 subdev can be > > part of a devicetree overlay, for example: > > > > &media_bridge { > > ... > > my_port: port@0 { > > #address-cells = <1>; > > #size-cells = <0>; > > reg = <0>; > > ep: endpoint@0 { > > remote-endpoint = <&camera0>; > > }; > > }; > > }; > > > > / { > > fragment@0 { > > target = <&i2c0>; > > __overlay__ { > > my_cam { > > compatible = "foo,bar"; > > port { > > camera0: endpoint { > > remote-endpoint = <&my_port>; > > ... > > }; > > }; > > }; > > }; > > }; > > }; > > > > Each time the overlay is applied, its of_node pointer will be > > different. We are not interested in matching the pointer, what we > > want to match is that the path is the one we are expecting. Change to > > use of_node_cmp() so that we continue matching after the overlay has > > been removed and reapplied. > > > > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > > Cc: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> > > Cc: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > Signed-off-by: Javi Merino <javi.merino@xxxxxxxxxx> > > --- > > I already reviewed v1 but you didn't carry the tag. So again: I forgot to add it :( > Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> Thanks! Javi -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html