I'm willing to help resolve those failed patches. I agreed to help MSI with this, so I will probably try to work on this tomorrow. I do want to understand the ramifications of these<br>two patches:<br><br>> Index: src/southbridge/broadcom/bcm5785/bcm5785_sata.c

> Index: src/devices/emulator/biosemu.c<div> before I commit them. I think that yhlu understands the broadcom and can comment on that, and Ollie and I can take a look at the biosemu change. </div> It is really wonderful to have MSI involved in LinuxBIOS!

We saw an MSI laptop at the LinuxBIOS summit and I for one really liked it. I wonder if we could look at an MSI laptop that is Turion-based for LinuxBIOS? I know that the Free Software Foundation would be very happy to see such a thing.

<br><br>thanks!<br><br>ron<br><br><br><div><span class="gmail_quote">On 10/26/06, <b class="gmail_sendername">Uwe Hermann</b> <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Hi, thanks a lot for you patch! It's great to see an MSI board supported! A few questions / remarks:  * What is the status of the code? Is the mainboard fully supported    (with all devices etc), or is this work in progress?

<br><br> * Who owns the copyright to the code? You personally, or MSI?<br>   Please add the usual GPL header to all files, see here<br>   <a href="http://www.linuxbios.org/Development_Guidelines#Common_License_Header">http://www.linuxbios.org/Development_Guidelines#Common_License_Header

</a><br>   for an example.<br>   For existing files which you copied and modified the orginal authors<br>   and license must also remain in the file, of course.<br><br> * The patch doesn't seem to apply (anymore?):<br><br>

patching file src/southbridge/broadcom/bcm5785/bcm5785_sata.c<br>Hunk #1 FAILED at 21.<br>Hunk #2 FAILED at 32.<br>2 out of 2 hunks FAILED -- saving rejects to file<br>src/southbridge/broadcom/bcm5785/bcm5785_sata.c.rej<br>

patching file src/devices/emulator/biosemu.c<br>Hunk #1 FAILED at 121.<br>1 out of 1 hunk FAILED -- saving rejects to file<br>src/devices/emulator/biosemu.c.rej<br><br><br>> Index: src/southbridge/broadcom/bcm5785/bcm5785_sata.c

<br>> Index: src/devices/emulator/biosemu.c<br><br>Are the changes to these two files generic, i.e. will they work for all<br>other mainboards LinuxBIOS supports, too? Or are they specific to<br>this board?<br><br><br>

> Property changes on: src/mainboard/msi/ms9185/Config.lb<br>> ___________________________________________________________________<br>> Name: svn:executable<br>>    + *<br><br>If I read this correctly you mark all files executable, which they

<br>probably should not be. I think these lines can be safely removed<br>from the patch.<br><br><br>> Index: src/mainboard/msi/ms9185/mainboard.c<br>[...]<br>> +#if CONFIG_CHIP_NAME == 1<br>> +struct chip_operations mainboard_msi_ms9185_ops = {

<br>> +       CHIP_NAME("MSI ms9185 mainboard")<br><br>I would make this read<br>  CHIP_NAME("MSI MS-9185 mainboard")<br>if my patch from<br><a href="http://www.linuxbios.org/pipermail/linuxbios/2006-October/016449.html">

http://www.linuxbios.org/pipermail/linuxbios/2006-October/016449.html</a><br>get committed.<br><br><br>Uwe.<br>--<br><a href="http://www.hermann-uwe.de">http://www.hermann-uwe.de</a>  | <a href="http://www.holsham-traders.de">

http://www.holsham-traders.de</a><br><a href="http://www.crazy-hacks.org">http://www.crazy-hacks.org</a> | <a href="http://www.unmaintained-free-software.org">http://www.unmaintained-free-software.org</a><br><br><br>-----BEGIN PGP SIGNATURE-----

<br>Version: GnuPG v1.4.5 (GNU/Linux)<br><br>iD8DBQFFQZmEXdVoV3jWIbQRAlz5AJ4iiGIdIYJ8EsVZno0MV86jAIo/2gCdG92v<br>vULBtfWcUL2y2gKPA3IVzPk=<br>=+6sE<br>-----END PGP SIGNATURE-----<br><br><br>--<br>linuxbios mailing list<br>

<a href="mailto:linuxbios@linuxbios.org">linuxbios@linuxbios.org</a><br><a href="http://www.openbios.org/mailman/listinfo/linuxbios">http://www.openbios.org/mailman/listinfo/linuxbios</a><br><br></blockquote></div><br>