linux-kernel - Re: [PATCH v6 0/8] Move device tree graph parsing helpers to drivers/of

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20140307184138.7FF68C40CC5@trevor.secretlab.ca>
Date:	Fri, 07 Mar 2014 18:41:38 +0000
From:	Grant Likely <grant.likely@...aro.org>
To:	Mauro Carvalho Chehab <m.chehab@...sung.com>,
	Russell King - ARM Linux <linux@....linux.org.uk>
Cc:	Philipp Zabel <p.zabel@...gutronix.de>,
	Tomi Valkeinen <tomi.valkeinen@...com>,
	Guennadi Liakhovetski <g.liakhovetski@....de>,
	Rob Herring <robh+dt@...nel.org>,
	Sylwester Nawrocki <s.nawrocki@...sung.com>,
	Laurent Pinchart <laurent.pinchart@...asonboard.com>,
	Kyungmin Park <kyungmin.park@...sung.com>,
	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org,
	devicetree@...r.kernel.org
Subject: Re: [PATCH v6 0/8] Move device tree graph parsing helpers to drivers/of

On Thu, 06 Mar 2014 12:17:21 -0300, Mauro Carvalho Chehab <m.chehab@...sung.com> wrote:
> Em Thu, 06 Mar 2014 14:16:57 +0000
> Russell King - ARM Linux <linux@....linux.org.uk> escreveu:
> 
> > On Wed, Mar 05, 2014 at 03:42:34PM +0100, Philipp Zabel wrote:
> > > Am Mittwoch, den 05.03.2014, 13:35 +0200 schrieb Tomi Valkeinen:
> > > > Hi,
> > > > 
> > > > On 05/03/14 11:20, Philipp Zabel wrote:
> > > > > Hi,
> > > > > 
> > > > > this version of the OF graph helper move series further addresses a few of
> > > > > Tomi's and Sylwester's comments.
> > > > > 
> > > > > Changes since v5:
> > > > >  - Fixed spelling errors and a wrong device node name in the link section
> > > > >  - Added parentless previous endpoint's full name to warning
> > > > >  - Fixed documentation comment for of_graph_parse_endpoint
> > > > >  - Unrolled for-loop in of_graph_get_remote_port_parent
> > > > > 
> > > > > Philipp Zabel (8):
> > > > >   [media] of: move graph helpers from drivers/media/v4l2-core to
> > > > >     drivers/of
> > > > >   Documentation: of: Document graph bindings
> > > > >   of: Warn if of_graph_get_next_endpoint is called with the root node
> > > > >   of: Reduce indentation in of_graph_get_next_endpoint
> > > > >   [media] of: move common endpoint parsing to drivers/of
> > > > >   of: Implement simplified graph binding for single port devices
> > > > >   of: Document simplified graph binding for single port devices
> > > > >   of: Warn if of_graph_parse_endpoint is called with the root node
> > > > 
> > > > So, as I've pointed out, I don't agree with the API, as it's too limited
> > > > and I can't use it, but as this series is (mostly) about moving the
> > > > current API to a common place, it's fine for me.
> > > > 
> > > > Acked-by: Tomi Valkeinen <tomi.valkeinen@...com>
> > > 
> > > Thanks. I'll be happy to help expanding the API to parse ports
> > > individually, once this gets accepted.
> > > 
> > > Mauro, Guennadi, are you fine with how this turned out? I'd like to get
> > > your acks again, for the changed location.
> 
> From my side, there's nothing on such code that is V4L2 specific.
> Moving it to drivers/of makes sense on my eyes.
> 
> Acked-by: Mauro Carvalho Chehab <m.chehab@...sung.com>
> > 
> > I'll need those acks before I can even think about queuing up the
> > imx-drm bits.
> > 
> > Another way to deal with this is if this gets pulled into the V4L tree
> > from Philipp's git tree, I can also pull that in myself.  What mustn't
> > happen is for these to be committed independently as patches.
> 
> If everyone agrees, I actually prefer have this patch applied on my tree,
> in order to avoid some potential merge conflicts at the merge window,
> as we might have other drivers and changes there touching on those API
> calls (I'm aware of a series of patches from Sylwester with some DT
> stuff on it. Not sure if it would be affected by such changes or not).

No. I disagree. Aside from the api changes this is primarily a drivers/of
series. I should go via me or Rob Herring. If you're concerned about
resolving conflicts then I can put it into a separate branch that you
can merge into your tree also.

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/