Thread (68 messages) 68 messages, 9 authors, 2016-02-11

[PATCH 5/5] arm: boot: store ATAGs structure into DT "/chosen/linux, atags" entry

From: Frank Rowand <hidden>
Date: 2015-11-26 04:19:38
Also in: linux-devicetree, linux-omap, lkml

On 11/25/2015 1:03 PM, Tony Lindgren wrote:
* Arnd Bergmann [off-list ref] [151125 11:50]:
quoted
On Wednesday 25 November 2015 10:16:44 Tony Lindgren wrote:
quoted
* Pali Roh?r [off-list ref] [151123 06:46]:
quoted
On Sunday 22 November 2015 07:51:46 Pavel Machek wrote:
quoted
On Wed 2015-11-11 17:10:46, Frank Rowand wrote:
quoted
Adding devicetree list.

Thread starts at
http://lists.infradead.org/pipermail/linux-arm-kernel/2015-July/354459.html

On 11/5/2015 8:17 AM, Tony Lindgren wrote:
quoted
* Pali Roh?r [off-list ref] [151105 03:41]:
quoted
On Tuesday 13 October 2015 16:37:46 Pali Roh?r wrote:
quoted
On Monday 12 October 2015 13:45:09 Tony Lindgren wrote:
quoted
* Pali Roh?r [off-list ref] [151012 13:29]:
quoted
On Monday 12 October 2015 22:16:40 Tony Lindgren wrote:
quoted
Pali, any news on posting an updated series with the comments
addressed in this thread? It seems that we all pretty much agree
what needs to be done.
I'm not real happy with the concept of patches 4 and 5 in this series.
My concern is that those two patches are using the FDT as a transport
mechanism for a binary blob (the atags object).
Umm. Ok. Do you have alternative proposal that works for everyone?

I mean. This discussion was going for quite a long time, and it would
be nice to have some solution... patch proposal... something.
                                                                    Pavel
Yes, discussion is going for a long time! So should I spend time for
adding documentation to my solution (this is last one thing which is
missing)? Or my solution is wrong and somebody else will propose new?
I do not want to spend time on something which will be rejected and
discarded.
At least I don't have better solutions in mind.
I would be happier if we could restrict this as much as possible to the
boards that need it, as an opt-in. That way it doesn't become an ABI
The feature (in whatever form it takes) should be definitely be highly
restricted and marked as deprecated.
quoted
for people that don't already rely in this information. How about
adding a check the code adds the linux,atags property to do it
only for a whitelist of board numbers?
Or populate /proc/atags only for the ones that need it from machine
specific init_early?
This is circling back to the first comment from Russell King where
he suggested a legacy file for the N900 which calls save_atags():

    Are the ATAGs at a fixed address on the N900?  Can that be handled in
    some kind of legacy file for the N900 which calls save_atags() on it, so
    we don't end up introducing yet more stuff that we have to maintain into
    the distant future?  If not, what about copying a known working atag
    structure into a legacy file for the N900?

It seems to me that patches 1, 2, 4, and 5 could be replaced by this
approach.

Regards,

Frank
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help