Hi, <br><br>    I had fixing non-ascii characters in the comments.<br>    Thanks.<br><br>    Signed-off: Libra Li <<a href="http://libra.li/" target="_blank">libra.li</a>@<a href="http://technexion.com/" target="_blank">technexion.com</a>><br>
<br><br><div class="gmail_quote">2009/11/25 Bao, Zheng <span dir="ltr"><<a href="mailto:Zheng.Bao@amd.com">Zheng.Bao@amd.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I haven't apply your patch, but in your patch,<br>
line 140, 150, 159, 166, 175, 185, 194,<br>
have those chars.<br>
<br>
You can use the perl script to detect the non-ascii chars. I sense there should be another better way.<br>
<br>
#!/usr/bin/perl<br>
<br>
my($content, $len);<br>
<br>
open(FILE, "< tim5690_buzzer_2.patch") || die "Unable to open file small. <$!>\n";<br>
<br>
while( chomp($content = <FILE>) ) {<br>
    $len = length($content);<br>
    for( $i = 0; $i < $len; $i++ ) {<br>
        if( ord(substr($content, $i, 1)) > 127 )<br>
        {<br>
            print "$content\n";<br>
            last;<br>
        }<br>
    }<br>
}<br>
close(FILE);<br>
<br>
exit 0<br>
~<br>
<div><div></div><div class="h5"><br>
<br>
<br>
<br>
-----Original Message-----<br>
From: <a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a> [mailto:<a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a>] On Behalf Of Bao, Zheng<br>
Sent: Wednesday, November 25, 2009 11:32 AM<br>
To: Libra Li<br>
Cc: coreboot<br>
Subject: Re: [coreboot] [PATCH] The buzzer of TechNexion TIM-5690.<br>
<br>
There is some non-ascii characters in the comments. Please fix it.<br>
After that,<br>
<br>
Acked-by: Zheng Bao <<a href="mailto:zheng.bao@amd.com">zheng.bao@amd.com</a>><br>
<br>
Zheng<br>
________________________________________<br>
From: <a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a> [mailto:<a href="mailto:coreboot-bounces@coreboot.org">coreboot-bounces@coreboot.org</a>] On Behalf Of Libra Li<br>
Sent: Wednesday, November 25, 2009 9:46 AM<br>
To: Myles Watson<br>
Cc: coreboot<br>
Subject: Re: [coreboot] [PATCH] The buzzer of TechNexion TIM-5690.<br>
<br>
Hi, Myles, Coreboot,<br>
<br>
    I had change EARLY_STAGE into __PRE_RAM__. It's okay.<br>
    Thanks.<br>
<br>
    Signed-off: Libra Li <<a href="http://libra.li" target="_blank">libra.li</a>@<a href="http://technexion.com" target="_blank">technexion.com</a>><br>
<br>
2009/11/25 Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
2009/11/24 Libra Li <<a href="mailto:librali1977@gmail.com">librali1977@gmail.com</a>>:<br>
> Hi,<br>
><br>
>     This patch is buzzer function of TechNexion TIM-5690.<br>
>     That isn't everyone needs. So I don't put in public areas.<br>
>     Thanks.<br>
><br>
>     Signed-off: Libra Li <<a href="http://libra.li" target="_blank">libra.li</a>@<a href="http://technexion.com" target="_blank">technexion.com</a>><br>
Could you use __PRE_RAM__ instead of EARLY_STAGE?  Does it mean<br>
something different?<br>
<br>
Thanks,<br>
Myles<br>
<br>
<br>
<br>
</div></div><font color="#888888">--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
<br>
<br>
</font></blockquote></div><br>