From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: [pinmux scripts PATCH] board-to-kernel-dt: don't generate outer state node
Date: Thu, 14 May 2015 09:52:14 -0600 [thread overview]
Message-ID: <5554C4AE.1090704@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ0H+LwDSoMTCMNnKe+4LjwRF8kT7EE5O+GDkCr7eWDHTA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 05/14/2015 08:29 AM, Simon Glass wrote:
> On 14 May 2015 at 08:20, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> The important thing for board-to-kernel-dt to generate is the content of
>> the pinmux "state" node in DT. The name and label of the node that
>> contains this information is essentially irrelevant. Since this
>> irrelevant name and label varies between boards in existing DTs, update
>> board-to-kernel-dt so it doesn't generate it. This makes it obvious that
>> the name and label shouldn't be modified when importing new versions of
>> board-to-kernel-dt's output.
>>
>> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>> board-to-kernel-dt.py | 4 ----
>> 1 file changed, 4 deletions(-)
>
> Reviewed-by: Simon Glass <sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
>
> I don't suppose this stuff can go in a separate .dtsi include?
That might make sense. I guess send a patch and see if Thierry takes it:-)
next prev parent reply other threads:[~2015-05-14 15:52 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-14 14:20 [pinmux scripts PATCH] board-to-kernel-dt: don't generate outer state node Stephen Warren
[not found] ` <1431613216-19745-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2015-05-14 14:29 ` Simon Glass
[not found] ` <CAPnjgZ0H+LwDSoMTCMNnKe+4LjwRF8kT7EE5O+GDkCr7eWDHTA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-05-14 15:52 ` Stephen Warren [this message]
2015-05-14 15:57 ` Stephen Warren
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=5554C4AE.1090704@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=swarren-DDmLM1+adcrQT0dZR+AlfA@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).