Dear Michal Simek, In message <1314362843-18214-4-git-send-email-mon...@monstr.eu> you wrote: > Use dev->iobase instead of baseaddress. > > Signed-off-by: Michal Simek <mon...@monstr.eu> ...
> - out_be32 (emaclite.baseaddress + XEL_TSR_OFFSET, 0); > + out_be32 (dev->iobase + XEL_TSR_OFFSET, 0); > /* Copy MAC address */ > xemaclite_alignedwrite (dev->enetaddr, > - emaclite.baseaddress, ENET_ADDR_LENGTH); > + dev->iobase, ENET_ADDR_LENGTH); > /* Set the length */ > - out_be32 (emaclite.baseaddress + XEL_TPLR_OFFSET, ENET_ADDR_LENGTH); > + out_be32 (dev->iobase + XEL_TPLR_OFFSET, ENET_ADDR_LENGTH); > /* Update the MAC address in the EMAC Lite */ > - out_be32 (emaclite.baseaddress + XEL_TSR_OFFSET, XEL_TSR_PROG_MAC_ADDR); > + out_be32 (dev->iobase + XEL_TSR_OFFSET, XEL_TSR_PROG_MAC_ADDR); > /* Wait for EMAC Lite to finish with the MAC address update */ > - while ((in_be32 (emaclite.baseaddress + XEL_TSR_OFFSET) & > + while ((in_be32 (dev->iobase + XEL_TSR_OFFSET) & > XEL_TSR_PROG_MAC_ADDR) != 0) ; I'll accept this for now as you are not adding this code new, but please note that all this should be cleaned up and converted into C structs instead of base + offset. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de There is, however, a strange, musty smell in the air that reminds me of something...hmm...yes...I've got it...there's a VMS nearby, or I'm a Blit. - Larry Wall in Configure from the perl distribution _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
