Hi,
On 24/05/17 10:14, Chen-Yu Tsai wrote:
On Wed, May 24, 2017 at 4:30 PM, Andre Przywara andre.przywara@arm.com wrote:
Hi,
On 23/05/17 23:26, Andreas Färber wrote:
Hi Andre,
Am 23.05.2017 um 12:09 schrieb Andre Przywara:
an updated version: the first patch is unchanged (still the exact same DT as in Linux), but the second patch reformats the Ethernet nodes as a .dtsi and uses the new U-Boot specific DT addon feature to make it clear that these nodes are for a (preliminary) U-Boot specific binding. Once the Ethernet driver made it into Linux, we will either adapt the U-Boot driver to cope with the new binding or get a new (Designware based) driver anyway. In any case this -u-boot.dtsi can then just be removed.
Could we squash the two? Or add the .dtsi first? That would keep the series bisectable.
Yeah, now with that separate file for the Ethernet node squashing the two patches into one would be the easiest.
Chen-Yu, Maxime, are you OK with this? I can send a new version then.
You're asking the wrong person. I'm not a maintainer for U-boot. :p
I know, but you commented on my initial post a few weeks ago (suggesting to keep the DT update in one patch) and are a valuable and knowledgeable resource anyway ;-)
Cheers, Andre.
Andreas' point is valid. Keeping it in one patch means the user retains ethernet support across the changes.
ChenYu