<br><br><div class="gmail_quote">2008/6/16 Jerry Van Baren &lt;<a href="mailto:gerald.vanbaren@ge.com">gerald.vanbaren@ge.com</a>&gt;:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">Michal Simek wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Antonio<br>
<br>
I am not responsible for this part of U-BOOT but your patch contain coding style<br>
violation.<br>
<br>
Regards,<br>
Michal Simek<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Logic unit:<br>
Purpose: &nbsp; &nbsp;Add support for bare partitions (no partition table)<br>
Author: &nbsp; &nbsp; Antnoio R. Costa &lt;antonio.costa &lt;at&gt; <a href="http://atmel.com" target="_blank">atmel.com</a>&gt;<br>
Date &nbsp;: &nbsp; &nbsp; 11 Jun 2008<br>
<br>
Status:<br>
~~~~~~<br>
Some SD cards are not formatted with a partition table but with<br>
just a bare partition at the beginnig of the memory.<br>
<br>
I modified get_partition_info_extended to call test_block_type<br>
as done by print_partition_extended. In this way bare FAT partitions<br>
are recognised. Now we need a test for Ext2.<br>
<br>
Signed-off-by: Antonio R. Costa &lt;<a href="mailto:antonio.costa@atmel.com" target="_blank">antonio.costa@atmel.com</a>&gt;<br>
<br>
diff --git a/disk/part_dos.c b/disk/part_dos.c<br>
index 4d778ec..e5cc8aa 100644<br>
--- a/disk/part_dos.c<br>
+++ b/disk/part_dos.c<br>
</blockquote></blockquote>
<br></div>
[snip]<div class="Ih2E3d"><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

+static inline int le16_to_int(unsigned char *le16)<br>
+{<br>
+ &nbsp; &nbsp;return ((le16[1] &lt;&lt; 8) + le16[0]);<br>
+}<br>
+<br>
</blockquote>
<br>
this should be in header file<br>
</blockquote>
<br></div>
Please use &lt;asm/byteorder.h&gt; 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].<br>

<br>
PowerPC example:<br>
&lt;<a href="http://git.denx.de/?p=u-boot.git;a=blob;f=include/asm-ppc/byteorder.h;h=3f5bcf63a1f980eb0c5e95e458119f55f5855274;hb=HEAD" target="_blank">http://git.denx.de/?p=u-boot.git;a=blob;f=include/asm-ppc/byteorder.h;h=3f5bcf63a1f980eb0c5e95e458119f55f5855274;hb=HEAD</a>&gt;<br>

</blockquote><div><br><br>Telling the truth it seems that I&#39;ve used this function during debug in fact at a deeper look it seems it disappeared from the code a part the definition.<br>Ok I&#39;m going to fix the problem and re-submit.<br>
<br>In these cases is there a special manner to format the subject to make an explicit reference to a patched patch :) ?<br><br>Regards,<br>Antonio<br>&nbsp;</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
[snip]<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Regards,<br>
Michal Simek<br>
</blockquote>
<br>
Thanks,<br>
gvb<br>
</blockquote></div><br>