[U-Boot] [PATCH][repost] bin_dep.sh Support

Prafulla Wadaskar prafulla at marvell.com
Tue Jul 14 19:04:14 CEST 2009


 

> -----Original Message-----
> From: Detlev Zundel [mailto:dzu at denx.de] 
> Sent: Tuesday, July 14, 2009 5:09 PM
> To: Prafulla Wadaskar
> Cc: Wolfgang Denk; Manas Saksena; Ronen Shitrit; Nicolas 
> Pitre; u-boot at lists.denx.de; Ashish Karkare; Prabhanjan 
> Sarnaik; Lennert Buijtenhek
> Subject: Re: [U-Boot] [PATCH][repost] bin_dep.sh Support
> 
> Hi Prafulla,
> 
> > Then I feel what Jean has suggested is the only way to go 
> ahead with 
> > kirkwood use case I will go ahead with this.
> >
> > Ref:
> >> > we can simply add a new target
> >> > u-boot.kwd
> >> > which will do somthing like this
> >> > 
> >> > $(obj)u-boot.kwd:	$(obj)u-boot.bin
> >> > 	$(do_image) $(obj)u-boot.kwd $(obj)u-boot.bin 
> >> > $(obj)board/$(BOARDDIR)/sdramregs.txt
> 
> I think this looks good.
> 
> > Second issue:
> > I will have to add a tool i.e. do_image (to be renamed) 
> which will create u-boot.kwd.
> > this tool is specific to Kirkwood only, I am going to add this in 
> > ./tools/ folder (as per Jean's feedback earlier), is this okay?
> 
> The plan as such is ok, although you will probably need to 
> rewrite the latest version of do_image posted to this ML[1] 
> completely.  Starting out to read the code, I was tempted to 
> post some specific remarks about the code, but quickly 
> realized that this is not a good attempt to move further here.
> 
> The code really needs to be rewritten in a clean fashion 
> using common programming metaphors, style and library 
> functions (i.e. use getopt(3), etc.).
> 
Dear Detlev
Thanks for your feedback
Yes I am doing the same code cleanup.
I am planning to remove all command line interface and the configuration needed will be provided using CONFIGs. This will make code size smaller and sdramregs.txt a standard c header file.
What do you think?

If you have further inputs those are welcomed... :-)

Thanks and regards..
Prafulla . . . 

> 
> [1] http://article.gmane.org/gmane.comp.boot-loaders.u-boot/57251
> 
> --
> Of course my password is the same as my pet's name My macaw's 
> name was Q47pY!3 and I change it every 90 days
>                         -- Trevor Linton
> --
> DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de
> 


More information about the U-Boot mailing list