On Mon, 9 Jun 2014 11:36:59 +0200 Hans de Goede <hdego...@redhat.com> wrote:
> From: Stefan Roese <s...@denx.de> > > There have been 3 versions of the sunxi_emac support patch during its > development. Somehow version 2 ended up in upstream u-boot where as > the u-boot-sunxi git repo got version 3. > > This bumps the version in upstream u-boot to version 3 of the patch: > - Initialize MII clock earlier so mii access to allow independent use > - Name change from WEMAC to EMAC to match mainline kernel & chip manual > - Cosmetic code cleanup > > Signed-off-by: Stefan Roese <s...@denx.de> > Signed-off-by: Henrik Nordstrom <hen...@henriknordstrom.net> > Signed-off-by: Oliver Schinagl <oli...@schinagl.nl> > Signed-off-by: Hans de Goede <hdego...@redhat.com> > Acked-by: Ian Campbell <i...@hellion.org.uk> Shouldn't the cosmetic and functional changes be normally split into separate patches? Indeed, it looks like the move of + /* Set MII clock */ + clrsetbits_le32(®s->mac_mcfg, 0xf << 2, 0xd << 2); is the only functional change in this rather large patch. And could you please elaborate on > - Initialize MII clock earlier so mii access to allow independent use was this causing any user visible problems? -- Best regards, Siarhei Siamashka _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot