<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/chrome-ec.git, branch release-R36-5841.B</title>
<subtitle>review.coreboot.org: chrome-ec.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/'/>
<entry>
<title>ec_uartd: bugfix for buffer pointer on partial write to FD.</title>
<updated>2014-05-09T22:04:12+00:00</updated>
<author>
<name>Todd Broch</name>
<email>tbroch@chromium.org</email>
</author>
<published>2014-05-08T23:42:46+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=479aa8b1bc46190fbef4fbcd2da514fe6a935004'/>
<id>479aa8b1bc46190fbef4fbcd2da514fe6a935004</id>
<content type='text'>
Previously I neglected to increment the buffer pointer which would
allow double writing of same data and exclude tail if write didn't
complete in one call.

Also fixing bug with EAGAIN/EWOULDBLOCK to retry in those cases.
Previously we'd just have just failed for those errno values.

Signed-off-by: Todd Broch &lt;tbroch@chromium.org&gt;

BRANCH=none
BUG=chromium:371147
TEST=still compiles

Change-Id: I30dbe56b2a4c735c487464349786a9db430130a8
Reviewed-on: https://chromium-review.googlesource.com/199052
Reviewed-by: Vincent Palatin &lt;vpalatin@chromium.org&gt;
Commit-Queue: Todd Broch &lt;tbroch@chromium.org&gt;
Tested-by: Todd Broch &lt;tbroch@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Previously I neglected to increment the buffer pointer which would
allow double writing of same data and exclude tail if write didn't
complete in one call.

Also fixing bug with EAGAIN/EWOULDBLOCK to retry in those cases.
Previously we'd just have just failed for those errno values.

Signed-off-by: Todd Broch &lt;tbroch@chromium.org&gt;

BRANCH=none
BUG=chromium:371147
TEST=still compiles

Change-Id: I30dbe56b2a4c735c487464349786a9db430130a8
Reviewed-on: https://chromium-review.googlesource.com/199052
Reviewed-by: Vincent Palatin &lt;vpalatin@chromium.org&gt;
Commit-Queue: Todd Broch &lt;tbroch@chromium.org&gt;
Tested-by: Todd Broch &lt;tbroch@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tegra: don't set auto_power_on if SYSJUMP</title>
<updated>2014-05-09T08:26:22+00:00</updated>
<author>
<name>Louis Yung-Chieh Lo</name>
<email>yjlou@chromium.org</email>
</author>
<published>2014-05-08T21:17:15+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=b5f3455a98ac2efad01228c01d25b05e9bddf52a'/>
<id>b5f3455a98ac2efad01228c01d25b05e9bddf52a</id>
<content type='text'>
The auto_power_on is set 1 unexpectedly while EC jumps. This has a
side effect that would turn on the AP unexpectedly after "power off".
See comment 43 of issue 28249

BUG=chrome-os-partner:28249
BRANCH=tot,nyan
TEST=on nyan:
&gt; reboot
&gt; sysinfo
// If EC is in RO, "sysjump RW"
&gt; power off
// The AP keeps off.

Change-Id: I3c06e99383c06af7cd6c17dd65040e20f06d8e73
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198941
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The auto_power_on is set 1 unexpectedly while EC jumps. This has a
side effect that would turn on the AP unexpectedly after "power off".
See comment 43 of issue 28249

BUG=chrome-os-partner:28249
BRANCH=tot,nyan
TEST=on nyan:
&gt; reboot
&gt; sysinfo
// If EC is in RO, "sysjump RW"
&gt; power off
// The AP keeps off.

Change-Id: I3c06e99383c06af7cd6c17dd65040e20f06d8e73
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198941
</pre>
</div>
</content>
</entry>
<entry>
<title>tegra: waits for battery stable before booting AP.</title>
<updated>2014-05-09T08:26:13+00:00</updated>
<author>
<name>Louis Yung-Chieh Lo</name>
<email>yjlou@chromium.org</email>
</author>
<published>2014-05-06T17:50:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=863f661c6f6235525bcffc2b4b50a270128922fa'/>
<id>863f661c6f6235525bcffc2b4b50a270128922fa</id>
<content type='text'>
This is a follow-up for CL 197990 and 197991. See issue tracker for
more details.

BUG=chrome-os-partner:28289
BRANCH=tot,nyan
TEST=Copy from https://chromium-review.googlesource.com/#/c/197990
Going battery shipmode and plug-in AC, See booting and expect EC log:
    "[... battery initialized]"
Disconnect battery, and plug-in and see booting and expect EC log:
    "[... battery not responding]"

Change-Id: Ic218414c21057a971b478ec60e406f1eb69dd379
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198412
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is a follow-up for CL 197990 and 197991. See issue tracker for
more details.

BUG=chrome-os-partner:28289
BRANCH=tot,nyan
TEST=Copy from https://chromium-review.googlesource.com/#/c/197990
Going battery shipmode and plug-in AC, See booting and expect EC log:
    "[... battery initialized]"
Disconnect battery, and plug-in and see booting and expect EC log:
    "[... battery not responding]"

Change-Id: Ic218414c21057a971b478ec60e406f1eb69dd379
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198412
</pre>
</div>
</content>
</entry>
<entry>
<title>tegra: reorder the "AP running" and "XPSHOLD seen".</title>
<updated>2014-05-09T04:45:41+00:00</updated>
<author>
<name>Louis Yung-Chieh Lo</name>
<email>yjlou@chromium.org</email>
</author>
<published>2014-05-07T23:28:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=73b8292d76612263b0666606357609483b435f42'/>
<id>73b8292d76612263b0666606357609483b435f42</id>
<content type='text'>
FAFT expects the "AP running" shows first, then "XPSHOLD seen".

BUG=chrome-os-partner:28316
BRANCH=tot,nyan
TEST=build only

Change-Id: Ic3e9d68b7727d62d3ccc2c037e7547f0276c2eae
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198766
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
FAFT expects the "AP running" shows first, then "XPSHOLD seen".

BUG=chrome-os-partner:28316
BRANCH=tot,nyan
TEST=build only

Change-Id: Ic3e9d68b7727d62d3ccc2c037e7547f0276c2eae
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198766
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tegra: fixed a corner case that AP_OFF flag is not cleared.</title>
<updated>2014-05-09T04:45:19+00:00</updated>
<author>
<name>Louis Yung-Chieh Lo</name>
<email>yjlou@chromium.org</email>
</author>
<published>2014-05-07T00:12:52+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=2cff588c1aaf05db05d07a4325881dffe24db761'/>
<id>2cff588c1aaf05db05d07a4325881dffe24db761</id>
<content type='text'>
If we follow the TEST steps below, the power state machine in AP
and EC were out of sync -- due to the un-clear bit and wrong
initial power state.

BUG=chrome-os-partner:24835
BRANCH=tot,nyan
TEST=on big.
&gt; reboot
&gt; power off      // De-assert XPSHOLD
&gt; reboot ap-off
&gt; sysinfo        // This reset flags does NOT contain "ap-off".

&gt; power on
% ectool reboot_ec RW // The following message is NOT observed.
                      // "system is on, but RESET_FLAG_AP_OFF is on".
&gt; power               // This should show the AP is "on".

// ensure everything still works.
&gt; reboot ap-off      // AP keeps off.
&gt; reboot             // AP is on.

Change-Id: I51afed7201d16ebcd889ad12a7af90026591cc2d
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198587
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If we follow the TEST steps below, the power state machine in AP
and EC were out of sync -- due to the un-clear bit and wrong
initial power state.

BUG=chrome-os-partner:24835
BRANCH=tot,nyan
TEST=on big.
&gt; reboot
&gt; power off      // De-assert XPSHOLD
&gt; reboot ap-off
&gt; sysinfo        // This reset flags does NOT contain "ap-off".

&gt; power on
% ectool reboot_ec RW // The following message is NOT observed.
                      // "system is on, but RESET_FLAG_AP_OFF is on".
&gt; power               // This should show the AP is "on".

// ensure everything still works.
&gt; reboot ap-off      // AP keeps off.
&gt; reboot             // AP is on.

Change-Id: I51afed7201d16ebcd889ad12a7af90026591cc2d
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198587
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>stm32: Re-enable spi transfers after a sysjump</title>
<updated>2014-05-09T04:41:58+00:00</updated>
<author>
<name>Randall Spangler</name>
<email>rspangler@chromium.org</email>
</author>
<published>2014-05-08T22:35:34+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=6803674335d88f7ce1b3f61b2b19b88f55464116'/>
<id>6803674335d88f7ce1b3f61b2b19b88f55464116</id>
<content type='text'>
If we sysjump to EC-RW, that sets enabled=0.  enabled is only set back
to 1 when the chipset resumes.  But on an AP-requested sysjump, the
chipset is already on, and so the resume hook never gets called.

So, in spi init, check if the AP is already on.  If it is, enable spi
transfers right away.

This probably also affects Pit/Pi.  That may need an additional fix in
power/gaia.c, if it returns an incorrect chipset state after a sysjump
(I didn't test that.)

BUG=chrome-os-partner:28249
BRANCH=nyan,pit
TEST=Power+Refresh boot system with RONormal disabled, so that the AP
     tells the EC to jump to EC-RW.  Confirm EC communication still works
     after that.

Change-Id: I588ef6d841040cf05d5527f645f122d5708b16ad
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198869
Reviewed-by: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
Commit-Queue: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Tested-by: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Tested-by: Gabe Black &lt;gabeblack@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If we sysjump to EC-RW, that sets enabled=0.  enabled is only set back
to 1 when the chipset resumes.  But on an AP-requested sysjump, the
chipset is already on, and so the resume hook never gets called.

So, in spi init, check if the AP is already on.  If it is, enable spi
transfers right away.

This probably also affects Pit/Pi.  That may need an additional fix in
power/gaia.c, if it returns an incorrect chipset state after a sysjump
(I didn't test that.)

BUG=chrome-os-partner:28249
BRANCH=nyan,pit
TEST=Power+Refresh boot system with RONormal disabled, so that the AP
     tells the EC to jump to EC-RW.  Confirm EC communication still works
     after that.

Change-Id: I588ef6d841040cf05d5527f645f122d5708b16ad
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198869
Reviewed-by: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
Commit-Queue: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Tested-by: Yung-chieh Lo &lt;yjlou@chromium.org&gt;
Tested-by: Gabe Black &lt;gabeblack@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tegra: fix bug that apshutdown cannot shut down the AP.</title>
<updated>2014-05-09T02:49:44+00:00</updated>
<author>
<name>Louis Yung-Chieh Lo</name>
<email>yjlou@chromium.org</email>
</author>
<published>2014-05-07T21:58:58+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=315631c871c3977b9f109081410f9a6651d55298'/>
<id>315631c871c3977b9f109081410f9a6651d55298</id>
<content type='text'>
The power_request variable is not clear in apshutdown case. Thus,
the AP comes up again by the "power up" request. Clear this variable
only when apshutdown command to avoid affecting the regular power
states.

BUG=chrome-os-partner:28621
BRANCH=tot,nyan
TEST=see below
&gt; reboot  // or other methods to turn on the AP.
&gt; power on
&gt; apshutdown
// The AP should remain off.

Change-Id: Icfc8a50db95aba656168f5b10dabd9443d75338d
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198770
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The power_request variable is not clear in apshutdown case. Thus,
the AP comes up again by the "power up" request. Clear this variable
only when apshutdown command to avoid affecting the regular power
states.

BUG=chrome-os-partner:28621
BRANCH=tot,nyan
TEST=see below
&gt; reboot  // or other methods to turn on the AP.
&gt; power on
&gt; apshutdown
// The AP should remain off.

Change-Id: Icfc8a50db95aba656168f5b10dabd9443d75338d
Signed-off-by: Louis Yung-Chieh Lo &lt;yjlou@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/198770
Reviewed-by: David Hendricks &lt;dhendrix@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Wait for battery boot-up</title>
<updated>2014-05-08T08:35:41+00:00</updated>
<author>
<name>Wonjoon Lee</name>
<email>woojoo.lee@samsung.com</email>
</author>
<published>2014-05-02T10:34:49+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=da5b8fa6c79807a8ea69258f43f60f742fb9246d'/>
<id>da5b8fa6c79807a8ea69258f43f60f742fb9246d</id>
<content type='text'>
Some battery uses clock stretching feature, and this could disturb
PMU communication before battery going stable.
AP does not know and will attempt PMU setting, and could get fail
For various battery indicates usually 1s for stable
(even if it is much less in real world 200ms~700ms)
Let's checking 'battery is ready' when first pump-up power.

BUG=chrome-os-partner:28289
TEST=Going battery shipmode and plug-in AC, See booting and EC log
Disconnect battery, and plug-in and see booting and EC log

Change-Id: I9b62266132d5322366265afe03adbe0db1f9ae75
Signed-off-by: Wonjoon Lee &lt;woojoo.lee@samsung.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/197991
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Tested-by: Doug Anderson &lt;dianders@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Some battery uses clock stretching feature, and this could disturb
PMU communication before battery going stable.
AP does not know and will attempt PMU setting, and could get fail
For various battery indicates usually 1s for stable
(even if it is much less in real world 200ms~700ms)
Let's checking 'battery is ready' when first pump-up power.

BUG=chrome-os-partner:28289
TEST=Going battery shipmode and plug-in AC, See booting and EC log
Disconnect battery, and plug-in and see booting and EC log

Change-Id: I9b62266132d5322366265afe03adbe0db1f9ae75
Signed-off-by: Wonjoon Lee &lt;woojoo.lee@samsung.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/197991
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Tested-by: Doug Anderson &lt;dianders@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Adding waiting function when battery boot-up</title>
<updated>2014-05-07T21:43:04+00:00</updated>
<author>
<name>Wonjoon Lee</name>
<email>woojoo.lee@samsung.com</email>
</author>
<published>2014-05-02T10:24:04+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=0a166c0467c45d5df90263d00b359b6a2cae8bc6'/>
<id>0a166c0467c45d5df90263d00b359b6a2cae8bc6</id>
<content type='text'>
Some battery uses clock stretching feature, and this could disturb
PMU communication before battery going stable.
AP does not know and will attempt PMU setting, and could get fail
For various battery indicates usually 1s for stable
(even if it is much less in real world 200ms~700ms)
Let's checking 'battery is ready' when first pump-up power.

BRANCH=ToT
BUG=chrome-os-partner:28289
TEST=Going battery shipmode and plug-in AC, See booting and EC log
Disconnect battery, and plug-in and see booting and EC log

Change-Id: Idd8ae2ab4ec164b11fe67413bbf647cad18bc481
Signed-off-by: Wonjoon Lee &lt;woojoo.lee@samsung.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/197990
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Doug Anderson &lt;dianders@chromium.org&gt;
Tested-by: Doug Anderson &lt;dianders@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Some battery uses clock stretching feature, and this could disturb
PMU communication before battery going stable.
AP does not know and will attempt PMU setting, and could get fail
For various battery indicates usually 1s for stable
(even if it is much less in real world 200ms~700ms)
Let's checking 'battery is ready' when first pump-up power.

BRANCH=ToT
BUG=chrome-os-partner:28289
TEST=Going battery shipmode and plug-in AC, See booting and EC log
Disconnect battery, and plug-in and see booting and EC log

Change-Id: Idd8ae2ab4ec164b11fe67413bbf647cad18bc481
Signed-off-by: Wonjoon Lee &lt;woojoo.lee@samsung.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/197990
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Doug Anderson &lt;dianders@chromium.org&gt;
Tested-by: Doug Anderson &lt;dianders@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>CHROMIUM: ec: Add Port 80 history access from ectools</title>
<updated>2014-05-07T03:41:27+00:00</updated>
<author>
<name>Wenkai Du</name>
<email>wenkai.du@intel.com</email>
</author>
<published>2014-05-02T16:03:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=372317af34b2def08bd22e7be1552402dd6e456f'/>
<id>372317af34b2def08bd22e7be1552402dd6e456f</id>
<content type='text'>
While debugging reboot issue, it was difficult to get POST code from failing
boards. Currently POST code is only accessible from EC console. Not all boards
are fitted with servo board.

This patch adds Port 80 history access from ectool. Reuse command code 0x48,
EC_CMD_PORT80_LAST_BOOT with version 1.

Signed-off-by: Wenkai Du &lt;wenkai.du@intel.com&gt;

BUG=chrome-os-partner:28514
BRANCH=rambi
TEST=manually test on rambi to confirm port 80 history match EC console

Change-Id: If204d8fb457d8d8d18055f8282a406a35c03305e
Reviewed-on: https://chromium-review.googlesource.com/198012
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-by: Wenkai Du &lt;wenkai.du@intel.com&gt;
Commit-Queue: Wenkai Du &lt;wenkai.du@intel.com&gt;
Tested-by: Wenkai Du &lt;wenkai.du@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
While debugging reboot issue, it was difficult to get POST code from failing
boards. Currently POST code is only accessible from EC console. Not all boards
are fitted with servo board.

This patch adds Port 80 history access from ectool. Reuse command code 0x48,
EC_CMD_PORT80_LAST_BOOT with version 1.

Signed-off-by: Wenkai Du &lt;wenkai.du@intel.com&gt;

BUG=chrome-os-partner:28514
BRANCH=rambi
TEST=manually test on rambi to confirm port 80 history match EC console

Change-Id: If204d8fb457d8d8d18055f8282a406a35c03305e
Reviewed-on: https://chromium-review.googlesource.com/198012
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-by: Wenkai Du &lt;wenkai.du@intel.com&gt;
Commit-Queue: Wenkai Du &lt;wenkai.du@intel.com&gt;
Tested-by: Wenkai Du &lt;wenkai.du@intel.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
