[U-Boot-Users] [PATCH] for Xilinx's ml300 board

sean chang sean.chang at xilinx.com
Wed Jun 30 02:41:35 CEST 2004


Hi,

Sorry for the mess.  Apparently I missed a step in cleaning up the
spacing for 
my patch.  Please ignore this patch and I'll submit another one soon.

Andrew,  your points are well taken.  I was planning on doing this in
different 
steps but it doesn't seem like a good idea after all.  I'll separate out
the 
non-standard environment functions from IIC support and re-submit the
patch.

Further comments/suggestions are welcomed.

Thanks,
Sean


Andrew May wrote:
> 
> On Tue, Jun 29, 2004 at 09:06:51AM -0700, Sean Chang wrote:
> > Hi Wolfgang,
> >
> > This is the first time I am submitting a patch, so if there is any problem
> > please let me know and I'll be more than happy to fix it.
> >
> > This patch is applied against the u-boot 1.1.1 release.  Basically it adds
> > support for the ml300 board to read out its environment information stored
> > on the EEPROM.
> 
> I would think the more important feature is that IIC support is added.
> I also wouldn't skip the fact that the OS independent code has changes as
> well.
> There are also a lot of tab to spaces conversions, that hide the real
> changes.
> 
> It would be up to Wolfgang if he wants the stuff redone, but I would like
> to see a more complete change log. like
> 
> CHANGELOG:
> Patch by Sean Chang, 28 Jun 2004:
>   Updates to Virtex-II Pro/ml300 board
>   OS independent code to Xilinx EDK 6.2
>   Added IIC support (with non-standard environment functions)
>   Add support for ml300 board to read out its environment
>   information stored on the EEPROM.
>   Convert Xilinx style environment vars to U-boot style vars.
> 
> The non-standard env stuff should really be moved to a separate file so
> people can use just the IIC support without pulling that garbage in.
> 
> The whole xparameters.h file is really annoying. It makes the IPIF code
> impossible to use on more than one board type without a recompile. That
> may be OK for U-boot but it is horrible in the Linux kernel.
> 
> These things really need some ()'s.
> +#define XIIC_CR_REG_OFFSET   0x00+XIIC_REG_OFFSET   /* Control Register   */
> 
> -------------------------------------------------------
> This SF.Net email sponsored by Black Hat Briefings & Training.
> Attend Black Hat Briefings & Training, Las Vegas July 24-29 -
> digital self defense, top technical experts, no vendor pitches,
> unmatched networking opportunities. Visit www.blackhat.com
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users




More information about the U-Boot mailing list