[U-Boot] [PATCH 1/2] fat: root directory cluster only makes sense for FAT32

Wolfgang Denk wd at denx.de
Sat Oct 1 21:52:11 CEST 2011


Dear Sergei Shtylyov,

In message <201108192332.34777.sshtylyov at ru.mvista.com> you wrote:
> The root directory cluster field only exists in a FAT32 boot sector, so the
> 'root_cluster' variable in do_fat_read() contains garbage in case of FAT12/16.
> Make it contain 0 instead as this is what is passed to get_vfatname() in that
> case anyway.
> 
> Signed-off-by: Sergei Shtylyov <sshtylyov at mvista.com>
> 
> ---
> Slight optimization atop of commit 3831530dcb7b71329c272ccd6181f8038b6a6dd0
> (VFAT: fix processing of scattered long file name entries).
> 
> The patch was generated atop of the below patches posted earlier:
> http://marc.info/?l=u-boot&m=131283284822789
> http://marc.info/?l=u-boot&m=131283289722891
> 
>  fs/fat/fat.c |   14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)

Applied, thanks.

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: wd at denx.de
Each team building another component has been using the  most  recent
tested  version  of the integrated system as a test bed for debugging
its piece. Their work will be set back by having that test bed change
under them. Of course it must. But the changes need to be  quantized.
Then  each  user  has periods of productive stability, interrupted by
bursts of test-bed change. This seems to be much less disruptive than
a constant rippling and trembling.
                     - Frederick Brooks Jr., "The Mythical Man Month"


More information about the U-Boot mailing list