<p>Shelley Chen has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/23182">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">google/fizz: Dynamically set PsysPl2 MSR if using type-C charger<br><br>If using type-C charger, then PsysPl2 may be lower than barrel jack<br>value of 90W, so need to override value to the max power of type-C<br>charger.<br><br>BUG=b:71594855<br>BRANCH=None<br>TEST=Make sure that PsysPL2 value set to 60W with zinger, but 90W<br>     when using proper barrel jack adapter on and i7.<br><br>Change-Id: If955b9af0e23f47719f001f1d73ec37113937cea<br>Signed-off-by: Shelley Chen <shchen@chromium.org><br>---<br>M src/mainboard/google/fizz/mainboard.c<br>1 file changed, 24 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/23182/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/google/fizz/mainboard.c b/src/mainboard/google/fizz/mainboard.c</span><br><span>index 98c0078..82c3153 100644</span><br><span>--- a/src/mainboard/google/fizz/mainboard.c</span><br><span>+++ b/src/mainboard/google/fizz/mainboard.c</span><br><span>@@ -31,6 +31,7 @@</span><br><span> #define FIZZ_SKU_ID_I7_U42 0x4</span><br><span> #define FIZZ_PL2_I7_U42    44</span><br><span> #define FIZZ_PL2_OTHERS    29</span><br><span style="color: hsl(120, 100%, 40%);">+#define FIZZ_PSYSPL2_ALL   90</span><br><span> /*</span><br><span>  * For type-C chargers, set PL2 to 90% of max power to account for</span><br><span>  * cable loss and FET Rdson loss in the path from the source.</span><br><span>@@ -122,6 +123,28 @@</span><br><span>     return watts;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/*</span><br><span style="color: hsl(120, 100%, 40%);">+ * mainboard_get_psyspl2</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * @return value PsysPl2 should be set to</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Check if charger is USB C.  If so, set to the max value.</span><br><span style="color: hsl(120, 100%, 40%);">+ * Otherwise, set PL2 based on sku id.</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+static u32 mainboard_get_psyspl2(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+  enum usb_chg_type type;</span><br><span style="color: hsl(120, 100%, 40%);">+       u32 watts;</span><br><span style="color: hsl(120, 100%, 40%);">+    /* If type C charger, set to max Watts of charger */</span><br><span style="color: hsl(120, 100%, 40%);">+  int rv = google_chromeec_get_usb_pd_power_info(&type, &watts);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      /* If we can't get charger info or not PD charger, assume barrel jack */</span><br><span style="color: hsl(120, 100%, 40%);">+  if (rv != 0 || type != USB_CHG_TYPE_PD)</span><br><span style="color: hsl(120, 100%, 40%);">+               watts = FIZZ_PSYSPL2_ALL;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   return watts;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> static uint8_t board_oem_id(void)</span><br><span> {</span><br><span>         static int id = -1;</span><br><span>@@ -211,6 +234,7 @@</span><br><span>    config_t *conf = root->chip_info;</span><br><span> </span><br><span>     conf->tdp_pl2_override = mainboard_get_pl2();</span><br><span style="color: hsl(120, 100%, 40%);">+      conf->tdp_psyspl2 = mainboard_get_psyspl2();</span><br><span> </span><br><span>  set_bj_adapter_limit();</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/23182">change 23182</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/23182"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: If955b9af0e23f47719f001f1d73ec37113937cea </div>
<div style="display:none"> Gerrit-Change-Number: 23182 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Shelley Chen <shchen@google.com> </div>