<div dir="ltr"><div>As I understand the code, KGPE-d16 doesn't use AGESA part (nor any?). <br><br></div>Any reason why this value would be defaulting to enabled for whole sb700 dependents?<br><br>--- a/src/vendorcode/amd/cimx/sb700/SBTYPE.h<br>+++ b/src/vendorcode/amd/cimx/sb700/SBTYPE.h<br>@@ -133,7 +133,7 @@ typedef struct _AMDSBCFG<br>         UINT32  SataPortMultCap :1;                     //6, 0:OFF   1:ON<br>         UINT32  SataReserved    :2;                     //8:7, Reserved<br>         UINT32  SataClkAutoOff  :1;                     //9, AutoClockOff for IDE modes 0:Disabled, 1:Enabled<br>-        UINT32  SataIdeCombinedMode     :1;             //10, SataIDECombinedMode 0:Disabled, 1:Enabled<br>+        UINT32  SataIdeCombinedMode     :0;             //10, SataIDECombinedMode 0:Disabled, 1:Enabled<br>         UINT32  SataIdeCombMdPriSecOpt:1;               //11, Combined Mode, SATA as primary or secondary 0:primary 1:secondary<br>         UINT32  SataReserved1   :6;                     //17:12, Not used currently<br>         UINT32  SataEspPort     :6;                     //23:18 SATA port is external accessiable on a signal only connector (eSATA:)<br><br><div><br><div class="gmail_quote"><div dir="ltr">Le jeu. 2 nov. 2017 à 09:33, Peter Stuge <<a href="mailto:peter@stuge.se">peter@stuge.se</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thierry Laurion wrote:<br>
> ENABLE_IDE_COMBINED_MODE available for sp800 but not for sp700, for ewhich<br>
> sp5100 is derived from:<br>
..<br>
> Suggested Workaround<br>
> Disable combined mode by setting a platform BIOS callback option to CIMx<br>
> called "SataIdeCombinedMode" to 0.<br>
..<br>
> Is there something i'm missing? Is it possible to disable combined<br>
> sata mode for sb700 from coreboot?<br>
<br>
SB700 mainboard support seems copypasted rather than engineered. The<br>
sustainable solution is to move sb700_cfg.c from src/mainboard/*/<br>
to src/southbridge/amd/cimx/sb700/ and hook the configuration in that<br>
file into Kconfig.<br>
<br>
Until someone does that, you could indeed change the assignment of<br>
that option by looking for<br>
<br>
SataIdeCombinedMode<br>
<br>
in the source code, if a sb700_cimx_config() function is called for<br>
this board - but that might not be the case if the board port chose<br>
not to use that part of AGESA.<br>
<br>
<br>
//Peter<br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="https://mail.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://mail.coreboot.org/mailman/listinfo/coreboot</a><br>
</blockquote></div></div></div>