[U-Boot] [PATCH V2 3/3] fs: add partition switch libary, implement ls and fsload commands

Marek Vasut marex at denx.de
Sat Oct 20 10:39:18 CEST 2012


Dear Tom Rini,

> On 10/19/12 12:26, Stephen Warren wrote:
> 
> [snip]
> 
> > Yes, I suppose I could:
> > 
> > * Add a bit-mask of legal filesystems as a parameter to
> > fs_set_blk_dev().
> > 
> > * Move the body of do_fsload() into some common called by
> > do_fsload(), do_ext2load(), do_fatload(), each passing in the
> > appropriate bit-mask, which gets passed down to fs_set_blk_dev().
> > 
> > That'd be easy, and probably entail only extremely minimal
> > code-size increases for an ext2-only or FAT-only build; just a few
> > bytes for a few more function calls. Sound like a plan?
> 
> Sounds good to me.  I bet you'll find some other clean-ups while
> you're in there.  Marek, any comments from the DM perspective here?

CCing pavel, block goo is on his slate.

Best regards,
Marek Vasut


More information about the U-Boot mailing list