Re: [PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-07 Thread Sakari Ailus
On Fri, Apr 07, 2017 at 01:36:34PM +0300, Sakari Ailus wrote: ... > > > + if (is_of_node(fwn)) { > > > + if (of_node_cmp(to_of_node(fwn)->name, "ports") == 0) > > > + fwn = fwnode_get_next_parent(fwn); > > > + } else { > > > + /* The "ports" node is always there in

Re: [PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-07 Thread Sakari Ailus
Hi Laurent, On Fri, Apr 07, 2017 at 01:54:58PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > On Friday 07 Apr 2017 13:36:34 Sakari Ailus wrote: > > On Fri, Apr 07, 2017 at 12:44:27PM +0300, Laurent Pinchart wrote: > > > On Thursday 06 Apr 2017 16:12:04 Sakari Ailus wrote: > > > > The

Re: [PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-07 Thread Laurent Pinchart
Hi Sakari, On Friday 07 Apr 2017 13:36:34 Sakari Ailus wrote: > On Fri, Apr 07, 2017 at 12:44:27PM +0300, Laurent Pinchart wrote: > > On Thursday 06 Apr 2017 16:12:04 Sakari Ailus wrote: > > > The fwnode_handle is a more generic way than OF device_node to describe > > > firmware nodes. Instead of

Re: [PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-07 Thread Sakari Ailus
Hi Laurent, On Fri, Apr 07, 2017 at 12:44:27PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Thursday 06 Apr 2017 16:12:04 Sakari Ailus wrote: > > The fwnode_handle is a more generic way than OF device_node to describe > > firmware nodes. Instead of the OF API,

Re: [PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-07 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Thursday 06 Apr 2017 16:12:04 Sakari Ailus wrote: > The fwnode_handle is a more generic way than OF device_node to describe > firmware nodes. Instead of the OF API, use more generic fwnode API to > obtain the same information. I would mention that this is

[PATCH v2 2/8] v4l: fwnode: Support generic fwnode for parsing standardised properties

2017-04-06 Thread Sakari Ailus
The fwnode_handle is a more generic way than OF device_node to describe firmware nodes. Instead of the OF API, use more generic fwnode API to obtain the same information. As the V4L2 fwnode support will be required by a small minority of e.g. ACPI based systems (the same might actually go for