From: Tom Rini <trini@ti.com>
To: Robert Nelson <robertcnelson@gmail.com>,
Tony Lindgren <tony@atomide.com>
Cc: Matt Ranostay <mranostay@gmail.com>,
robh+dt@kernel.org, mark.rutland@arm.com,
Russell King <linux@arm.linux.org.uk>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
devicetree <devicetree@vger.kernel.org>,
linux kernel <linux-kernel@vger.kernel.org>,
Pantelis Antoniou <pantelis.antoniou@gmail.com>,
Matt Porter <matt.porter@linaro.org>
Subject: Re: [PATCH v2] ARM: dts: am335x-bone-common: Add i2c2 definition
Date: Tue, 13 May 2014 08:53:39 -0400 [thread overview]
Message-ID: <537215D3.70306@ti.com> (raw)
In-Reply-To: <CAOCHtYgg7jwxswH+mqF3RA7-0RP-nGuft+Ze-GRwu1fbA6+0Bw@mail.gmail.com>
On 05/12/2014 04:57 PM, Robert Nelson wrote:
>>> Either case if fine with me. As who knows when the dtc "overlay" will
>>> every truly make it mainline, as the capemgr was the only real kernel
>>> user of the i2c/at24 eeprom information.
>>
>> Sounds like we should keep it disabled though so u-boot can be used
>> to toggle it while waiting for the capemgr. That's because the board
>> has a header for pins, so it's not exactly limited to just the capes.
>>
>> Anybody working on enabling/disabling cape dtb configurations in u-boot?
>
> Well,
>
> Would Tom even approve of that in mainline u-boot? He didn't want my
> "invert" the gpio to enable the usb hub on the older beagle xm A/B..
>
> http://lists.denx.de/pipermail/u-boot/2014-January/172154.html
>
> http://lists.denx.de/pipermail/u-boot/2014-January/172274.html
I would think that using the 'fdt' command in U-Boot to add all
properties of every cape found on a running system would drive someone
to madness quite quickly. Moving all of Pantelis' work for dynamic
device trees from the kernel to N bootloaders (U-Boot, barebox, UEFI,
etc) sounds like a step in the wrong direction.
--
Tom
WARNING: multiple messages have this Message-ID (diff)
From: Tom Rini <trini@ti.com>
To: Robert Nelson <robertcnelson@gmail.com>,
Tony Lindgren <tony@atomide.com>
Cc: Matt Ranostay <mranostay@gmail.com>, <robh+dt@kernel.org>,
<mark.rutland@arm.com>, Russell King <linux@arm.linux.org.uk>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
devicetree <devicetree@vger.kernel.org>,
linux kernel <linux-kernel@vger.kernel.org>,
Pantelis Antoniou <pantelis.antoniou@gmail.com>,
Matt Porter <matt.porter@linaro.org>
Subject: Re: [PATCH v2] ARM: dts: am335x-bone-common: Add i2c2 definition
Date: Tue, 13 May 2014 08:53:39 -0400 [thread overview]
Message-ID: <537215D3.70306@ti.com> (raw)
In-Reply-To: <CAOCHtYgg7jwxswH+mqF3RA7-0RP-nGuft+Ze-GRwu1fbA6+0Bw@mail.gmail.com>
On 05/12/2014 04:57 PM, Robert Nelson wrote:
>>> Either case if fine with me. As who knows when the dtc "overlay" will
>>> every truly make it mainline, as the capemgr was the only real kernel
>>> user of the i2c/at24 eeprom information.
>>
>> Sounds like we should keep it disabled though so u-boot can be used
>> to toggle it while waiting for the capemgr. That's because the board
>> has a header for pins, so it's not exactly limited to just the capes.
>>
>> Anybody working on enabling/disabling cape dtb configurations in u-boot?
>
> Well,
>
> Would Tom even approve of that in mainline u-boot? He didn't want my
> "invert" the gpio to enable the usb hub on the older beagle xm A/B..
>
> http://lists.denx.de/pipermail/u-boot/2014-January/172154.html
>
> http://lists.denx.de/pipermail/u-boot/2014-January/172274.html
I would think that using the 'fdt' command in U-Boot to add all
properties of every cape found on a running system would drive someone
to madness quite quickly. Moving all of Pantelis' work for dynamic
device trees from the kernel to N bootloaders (U-Boot, barebox, UEFI,
etc) sounds like a step in the wrong direction.
--
Tom
next prev parent reply other threads:[~2014-05-13 12:53 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-10 1:43 [PATCH v2] ARM: dts: am335x-bone-common: Add i2c2 definition Matt Ranostay
2014-05-10 1:43 ` Matt Ranostay
2014-05-12 19:50 ` Tony Lindgren
2014-05-12 19:59 ` Robert Nelson
[not found] ` <CAOCHtYiYwx_bEsyqkbUTTAbhBYESzvfBOOYeVuFXPS1dDy0ZKA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-12 20:15 ` Tony Lindgren
2014-05-12 20:15 ` Tony Lindgren
[not found] ` <20140512201517.GC5668-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2014-05-12 20:27 ` Robert Nelson
2014-05-12 20:27 ` Robert Nelson
[not found] ` <CAOCHtYg6XpF0Gbo4iFqsFvP=QSivQQvCzov8UfxeLXMzrRYEWw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-12 20:42 ` Tony Lindgren
2014-05-12 20:42 ` Tony Lindgren
2014-05-12 20:57 ` Robert Nelson
2014-05-12 21:07 ` Tony Lindgren
2014-05-13 12:53 ` Tom Rini [this message]
2014-05-13 12:53 ` Tom Rini
2014-05-13 14:06 ` Javier Martinez Canillas
2014-05-13 14:13 ` Tom Rini
2014-05-13 14:13 ` Tom Rini
[not found] ` <CABxcv=nq9XvyN0aPOBFDm0UyoaW-Htyy0tyP3YhYaDnDUo3a5w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-13 14:22 ` Matt Porter
2014-05-13 14:22 ` Matt Porter
2014-05-13 14:39 ` Javier Martinez Canillas
2014-05-13 17:07 ` Pantelis Antoniou
2014-05-13 17:51 ` Javier Martinez Canillas
2014-05-13 20:24 ` John Syn
2014-05-13 20:24 ` John Syn
2014-05-14 3:39 ` Pantelis Antoniou
2014-05-14 3:39 ` Pantelis Antoniou
[not found] ` <2A5F30DB-EF92-4CFC-BA3C-4ECAE36FD4FF-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-05-14 5:44 ` John Syn
2014-05-14 5:44 ` John Syn
[not found] ` <CF9849B0.24764%john3909-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-05-14 7:12 ` Javier Martinez Canillas
2014-05-14 7:12 ` Javier Martinez Canillas
[not found] ` <CABxcv=mWqENSzODJYAJRgDYNy46X6WkQ+0e4NSDt8qE-ob1KuQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-14 3:24 ` Pantelis Antoniou
2014-05-14 3:24 ` 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=537215D3.70306@ti.com \
--to=trini@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mark.rutland@arm.com \
--cc=matt.porter@linaro.org \
--cc=mranostay@gmail.com \
--cc=pantelis.antoniou@gmail.com \
--cc=robertcnelson@gmail.com \
--cc=robh+dt@kernel.org \
--cc=tony@atomide.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.