devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
To: Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
Cc: Phil Elwell
	<philip.j.elwell-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Frank Rowand
	<frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Nishanth Menon <nm-l0cyMroinI0@public.gmane.org>,
	Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Devicetree Compiler
	<devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Pantelis Antoniou
	<pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>,
	Tero Kristo <t-kristo-l0cyMroinI0@public.gmane.org>,
	Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Subject: Re: [PATCH 1/2] fdt: Allow stacked overlays phandle references
Date: Thu, 27 Jul 2017 17:25:34 +1000	[thread overview]
Message-ID: <20170727072534.GC7970@umbus.fritz.box> (raw)
In-Reply-To: <20170726150148.GD26163@bill-the-cat>

[-- Attachment #1: Type: text/plain, Size: 1521 bytes --]

On Wed, Jul 26, 2017 at 11:01:48AM -0400, Tom Rini wrote:
> On Wed, Jul 26, 2017 at 02:23:15PM +1000, David Gibson wrote:
> > On Thu, Jul 13, 2017 at 08:38:10PM +0100, Phil Elwell wrote:
> > > Can we also consider a mechanism for overlay-local symbols, i.e. symbols
> > > that are used purely to create links within an overlay - perhaps using a
> > > particular naming convention? This would make it easier to instantiate an
> > > overlay multiple times without having to uniquify all symbols, and it would
> > > avoid polluting the global namespace without reason.
> > 
> > I'd really prefer not to add yet more features and complexity to the
> > existing shoddily designed overlay mechanism.  I'd prefer for people
> > to focus on a better replacement - which includes considering these
> > sorts of use cases and how to handle them.
> 
> Can we go for incremental progress over time instead of replacing what
> was finally accepted?  Maybe one or two concrete examples that need
> improvement upon, and go from there?  Thanks!

As a general rule, I'd agree with that approach, but not this time.
Incremental improvements are great if you have a solid base from which
to work, at the moment we don't.  What we have is a half-arsed hack
that become widely deployed enough that we have to live with it.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2017-07-27  7:25 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-14 14:52 [PATCH 0/2] stacked overlay support Pantelis Antoniou
     [not found] ` <1497451946-15443-1-git-send-email-pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2017-06-14 14:52   ` [PATCH 1/2] fdt: Allow stacked overlays phandle references Pantelis Antoniou
     [not found]     ` <1497451946-15443-2-git-send-email-pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>
2017-07-03  9:06       ` David Gibson
     [not found]         ` <20170703090648.GV13989-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-03 12:41           ` Pantelis Antoniou
2017-07-07  7:09             ` David Gibson
     [not found]               ` <20170707070915.GD24325-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-07 14:01                 ` Tom Rini
2017-07-13 19:51                   ` Frank Rowand
2017-07-13 19:40                 ` Frank Rowand
     [not found]                   ` <5967CCA8.6030406-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-26  4:18                     ` David Gibson
2017-07-13 19:35             ` Frank Rowand
2017-07-13 19:31           ` Frank Rowand
2017-07-13 19:38             ` Phil Elwell
     [not found]               ` <CAPhXvM4NzU61dENLeJ2Xt=arKqYFjXaPBvzrjxAJ7h3Y-gT4Nw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-07-13 20:07                 ` Frank Rowand
     [not found]                   ` <5967D2F7.60303-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-13 20:08                     ` Frank Rowand
2017-07-13 21:22                     ` Phil Elwell
     [not found]                       ` <f06fe24c-7f32-4e7d-c28b-2e5b31c5dbf0-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-13 21:40                         ` Frank Rowand
     [not found]                           ` <5967E8BC.4090307-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-14  7:21                             ` Pantelis Antoniou
2017-07-24 18:06                               ` Frank Rowand
     [not found]                                 ` <59763739.4070708-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-24 20:51                                   ` Phil Elwell
     [not found]                                     ` <7b6a51ad-70a4-efaf-0a11-c576a95fd222-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-24 22:44                                       ` Frank Rowand
2017-07-26  4:55                                   ` David Gibson
     [not found]                                     ` <20170726045533.GD8978-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-26 14:03                                       ` Frank Rowand
     [not found]                                         ` <5978A11F.1010008-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-27  7:24                                           ` David Gibson
2017-07-26  4:32                               ` David Gibson
     [not found]                                 ` <20170726043227.GC8978-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-26 13:59                                   ` Frank Rowand
     [not found]                                     ` <5978A047.6060406-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-27  7:23                                       ` David Gibson
     [not found]                                         ` <20170727072351.GA7970-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-27 17:24                                           ` Frank Rowand
     [not found]                                             ` <597A21C1.6030104-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-31  4:06                                               ` David Gibson
2017-07-26  4:28                             ` David Gibson
2017-07-26  4:23                 ` David Gibson
     [not found]                   ` <20170726042315.GA8978-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-26 15:01                     ` Tom Rini
2017-07-27  7:25                       ` David Gibson [this message]
     [not found]                         ` <20170727072534.GC7970-K0bRW+63XPQe6aEkudXLsA@public.gmane.org>
2017-07-27 11:24                           ` Tom Rini
     [not found]             ` <5967CAA6.6010801-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-07-26  4:21               ` David Gibson
2017-06-14 14:52   ` [PATCH 2/2] tests: Add stacked overlay tests on fdtoverlay Pantelis Antoniou

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170727072534.GC7970@umbus.fritz.box \
    --to=david-xt8fgy+axnrb3ne2bgzf6laj5h9x9tb+@public.gmane.org \
    --cc=devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=frowand.list-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=nm-l0cyMroinI0@public.gmane.org \
    --cc=pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org \
    --cc=philip.j.elwell-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=t-kristo-l0cyMroinI0@public.gmane.org \
    --cc=trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).