[U-Boot-Users] [PATCH 1/1] FAT Bare Partition Support

[Jerry Van Baren <gerald.vanbaren@ge.com>]

Michal Simek wrote:
> Hi Antonio
> 
> I am not responsible for this part of U-BOOT but your patch contain coding style
> violation.
> 
> Regards,
> Michal Simek
> 
>> Logic unit:
>> Purpose:    Add support for bare partitions (no partition table)
>> Author:     Antnoio R. Costa <antonio.costa <at> atmel.com>
>> Date  :     11 Jun 2008
>>
>> Status:
>> ~~~~~~
>> Some SD cards are not formatted with a partition table but with
>> just a bare partition at the beginnig of the memory.
>>
>> I modified get_partition_info_extended to call test_block_type
>> as done by print_partition_extended. In this way bare FAT partitions
>> are recognised. Now we need a test for Ext2.
>>
>> Signed-off-by: Antonio R. Costa <antonio.costa@atmel.com>
>>
>> diff --git a/disk/part_dos.c b/disk/part_dos.c
>> index 4d778ec..e5cc8aa 100644
>> --- a/disk/part_dos.c
>> +++ b/disk/part_dos.c

[snip]

>> +static inline int le16_to_int(unsigned char *le16)
>> +{
>> +    return ((le16[1] << 8) + le16[0]);
>> +}
>> +
> 
> this should be in header file

Please use <asm/byteorder.h> rather than making Yet Another Define. 
Note that the subdirectory asm gets symlinked to the appropriate 
arch-specific subdirectory and then the right endian munching is 
selected and it Just Works[tm].

PowerPC example:
<http://git.denx.de/?p=u-boot.git;a=blob;f=include/asm-ppc/byteorder.h;h=3f5bcf63a1f980eb0c5e95e458119f55f5855274;hb=HEAD>

[snip]

> Regards,
> Michal Simek

Thanks,
gvb