<br><br><div class="gmail_quote">On Thu, Jun 2, 2011 at 2:12 AM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Mon, 30 May 2011 11:59:57 -0400<br>
<div class="im">Steven Zakulec <<a href="mailto:spzakulec@gmail.com">spzakulec@gmail.com</a>> wrote:<br>
<br>
</div><div class="im">> Hi, this is the latest version of the patch, taking into account Peter's<br>
> request to me to change the struct to uint16_t and the values to milliwatts.<br>
> Signed-off-by: Steven Zakulec <<a href="mailto:spzakulec@gmail.com">spzakulec@gmail.com</a>><br>
<br>
</div>hello steven (what a nice name! ;)<br>
<br>
your patch does not compile:<br>
flashchips.c:1335: error: expected expression before ‘,’ token<br>
flashchips.c:1373: error: expected expression before ‘,’ token<br>
<br>
if you look at it, it is quite obvious ;) probably some copy & paste<br>
mistake or regex going berserk.<br>
<br>
since you have to revise it anyway i have a few nitpicking comments<br>
and two important notes to make:<br>
<br>
> Index: flash.h<br>
> ===================================================================<br>
> --- flash.h   (revision 1322)<br>
> +++ flash.h   (working copy)<br>
> @@ -145,6 +145,10 @@<br>
<div class="im">>       int (*unlock) (struct flashchip *flash);<br>
>       int (*write) (struct flashchip *flash, uint8_t *buf, int start, int len);<br>
>       int (*read) (struct flashchip *flash, uint8_t *buf, int start, int len);<br>
</div>> +     struct {<br>
> +             uint16_t min;<br>
> +             uint16_t max;<br>
> +     }voltage;<br>
}<space>voltage; is our usual style afaics<br>
<br>
> Index: flashchips.c<br>
> ===================================================================<br>
> --- flashchips.c      (revision 1322)<br>
<div class="im">> +++ flashchips.c      (working copy)<br>
> @@ -54,6 +54,7 @@<br>
>        * .unlock              = Chip unlock function<br>
>        * .write               = Chip write function<br>
>        * .read                = Chip read function<br>
</div>> +      * .voltage             = Voltage min and max range in decivolt<br>
this is no longer correct.<br>
<br>
>       {<br>
> @@ -3285,6 +3379,7 @@<br>
>               },<br>
>               .write          = write_m29f400bt,<br>
>               .read           = read_memmapped,<br>
> +             .voltage        = {4750, 5250}, /*5.0V +-5% for -55 model, +-10% for rest */<br>
<space> after /*<br>
<br>
>       {<br>
> @@ -3483,6 +3583,7 @@<br>
<div class="im">>               },<br>
>               .write          = write_82802ab,<br>
>               .read           = read_memmapped,<br>
</div>> +             .voltage        = {4500, 5500}, /*5.0V +-10% read, 12V fast program & erase- +-5% standard, +-10% option */<br>
>       },<br>
<space> after /*<br>
<br>
>       {<br>
> @@ -6653,6 +6841,7 @@<br>
<div class="im">>               },<br>
>               .write          = write_jedec_1,<br>
>               .read           = read_memmapped,<br>
</div>> +             .voltage        = {4500, 5500}, /* Datasheet says some are only 4.75-5.25 V */<br>
>       },<br>
i havent looked at the datasheet, but if so then we should use the<br>
tightest range i.e. 4750, 5250 imho.<br>
<br>
>       {<br>
> @@ -6684,6 +6873,7 @@<br>
<div class="im">>               },<br>
>               .write          = write_jedec_1,<br>
>               .read           = read_memmapped,<br>
</div>> +             .voltage        = {4500, 5500}, /* Datasheet says some are only 4.75-5.25 V */<br>
>       },<br>
same<br>
<br>
looks good to me otherwise, thanks a lot!<br>
i have not checked the datasheets or the spreadsheet you made<br>
though.</decline of responsibility> ;)<br>
<font color="#888888"><br>
--<br>
Kind regards/Mit freundlichen Grüßen, Stefan Tauner<br>
</font></blockquote></div><br>All issues fixed.<br>