<br><br><div class="gmail_quote">On Mon, Oct 19, 2009 at 6:15 PM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
See patch.<br>
<br>
I'm happy for any comments. Also, I'm not sure how all this<br>
hard_reset() stuff is meant to work. Many southbridges have _two_<br>
implementations of hard_reset(). Why? Some only have one.<br></blockquote><div><br>> Index: src/southbridge/amd/amd8111/amd8111_reset.c<br>> ===================================================================<br>> --- src/southbridge/amd/amd8111/amd8111_reset.c    (Revision 4813)<br>
> +++ src/southbridge/amd/amd8111/amd8111_reset.c    (Arbeitskopie)<br>> @@ -54,6 +54,7 @@<br>>  #include "../../../northbridge/amd/amdk8/reset_test.c"<br>>  <br>>  <br>> +/* FIXME: There's another implementation in amd8111_early_ctrl.c. Why? */<br>
>  void hard_reset(void)<br>>  {<br>>      device_t dev;<br>> Index: src/southbridge/amd/amd8111/amd8111_early_ctrl.c<br>> ===================================================================<br>> --- src/southbridge/amd/amd8111/amd8111_early_ctrl.c    (Revision 4813)<br>
> +++ src/southbridge/amd/amd8111/amd8111_early_ctrl.c    (Arbeitskopie)<br>> @@ -35,6 +35,7 @@<br>>      enable_cf9_x(sbbusn, sbdn);<br>>  }<br>>  <br>> +/* FIXME: There's another implementation in amd8111_reset.c. Why? */<br>
>  static void hard_reset(void)<br>>  {<br>>          set_bios_reset();<br>> <br>CAR code has its own implementation of many functions: printk, die, HT initialization, ...<br><br>The early files are inlcuded into the cache_as_ram_auto.c, and the other ones are used with coreboot_ram.<br>
<br>>  - Drop hard_reset() dummy from asus/a8v-e_se and asus/m2v-mx_se.<br>>    As mentioned by ruik on IRC, the soft_reset() function already does a<br>>    hard reset, so this function is not needed.<br><br>I think it's a bug to have soft_reset() do a hard reset.<br>
<br>Thanks,<br>Myles<br><br></div></div>