On Wed, May 24, 2017 at 4:30 PM, Andre Przywara <andre.przyw...@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 Andreas' point is valid. Keeping it in one patch means the user retains ethernet support across the changes. ChenYu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot