<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/chrome-ec.git, branch release-R104-14909.B-main</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>prism: rgbkbd: Add rgb keyboard type field</title>
<updated>2022-07-25T18:38:48+00:00</updated>
<author>
<name>Parth Malkan</name>
<email>parthmalkan@google.com</email>
</author>
<published>2022-06-28T20:54:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=406ea8849be0450f47d5172d700bf345a467c667'/>
<id>406ea8849be0450f47d5172d700bf345a467c667</id>
<content type='text'>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: Idf4b9081e6510dcdbd4301d9d855aba2fe7e7499
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3732805
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit fa3766d2e4a13c228caf88c122eb6dfc30bd1818)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784557
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: Idf4b9081e6510dcdbd4301d9d855aba2fe7e7499
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3732805
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit fa3766d2e4a13c228caf88c122eb6dfc30bd1818)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784557
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>rgbkbd: Add get_config subcommand to rgbkbd host command</title>
<updated>2022-07-25T18:38:47+00:00</updated>
<author>
<name>Parth Malkan</name>
<email>parthmalkan@google.com</email>
</author>
<published>2022-06-28T19:53:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=047f56016cd2511810172f9eea03241e6997b0c0'/>
<id>047f56016cd2511810172f9eea03241e6997b0c0</id>
<content type='text'>
Add a response struct to rgbkbd host command and add GET_CONFIG as a
subcommand. This is used to return RGB keyboard type (number of zones
and LEDs) to the host.

BRANCH=none
BUG=b:232134905
TEST=Test on Taniks and Vell using ectool rgbkbd getconfig command

Cq-Depend: chromium:3784557, chromium:3784558
Cq-Depend: chromium:3784560
Change-Id: Ie5b18ed3be141f2a91326ff2a121649296ea1161
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3732802
Reviewed-by: Zhuohao Lee &lt;zhuohao@chromium.org&gt;
(cherry picked from commit 0fedae4fe40f63ceca8467a8b3603a32d2cae7ef)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784556
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a response struct to rgbkbd host command and add GET_CONFIG as a
subcommand. This is used to return RGB keyboard type (number of zones
and LEDs) to the host.

BRANCH=none
BUG=b:232134905
TEST=Test on Taniks and Vell using ectool rgbkbd getconfig command

Cq-Depend: chromium:3784557, chromium:3784558
Cq-Depend: chromium:3784560
Change-Id: Ie5b18ed3be141f2a91326ff2a121649296ea1161
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3732802
Reviewed-by: Zhuohao Lee &lt;zhuohao@chromium.org&gt;
(cherry picked from commit 0fedae4fe40f63ceca8467a8b3603a32d2cae7ef)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784556
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>taniks: rgbkbd: Add rgb keyboard type field</title>
<updated>2022-07-25T18:38:45+00:00</updated>
<author>
<name>Parth Malkan</name>
<email>parthmalkan@google.com</email>
</author>
<published>2022-06-28T21:01:08+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=030fcacad7a67a25777fffb4d267c4a3fb50b4c5'/>
<id>030fcacad7a67a25777fffb4d267c4a3fb50b4c5</id>
<content type='text'>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: I3fa2ebede9aa077661199072480d28a65e5d1206
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3733669
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit 7c733a4c9ebef218365c2c084259f94dea7aaedc)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784558
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: I3fa2ebede9aa077661199072480d28a65e5d1206
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3733669
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit 7c733a4c9ebef218365c2c084259f94dea7aaedc)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784558
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>osiris: rgbkbd: Add rgb keyboard type field</title>
<updated>2022-07-25T18:38:44+00:00</updated>
<author>
<name>Parth Malkan</name>
<email>parthmalkan@google.com</email>
</author>
<published>2022-06-28T20:59:51+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=6d80866f57a1222a164e2b9ce6a11c7f744e900f'/>
<id>6d80866f57a1222a164e2b9ce6a11c7f744e900f</id>
<content type='text'>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: I400f38f45469874f8ea9bf75f45135e3baa21764
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3733668
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit 7be23fa82e8bad96b0e6473f25b0f39273575ce3)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784560
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
rgbkbd_type describes number of zones and LEDs supported.

BRANCH=none
BUG=b:232134905
TEST=make -j buildall

Cq-Depend: chromium:3784556
Signed-off-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Change-Id: I400f38f45469874f8ea9bf75f45135e3baa21764
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3733668
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
(cherry picked from commit 7be23fa82e8bad96b0e6473f25b0f39273575ce3)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3784560
Tested-by: YH Lin &lt;yueherngl@chromium.org&gt;
Reviewed-by: YH Lin &lt;yueherngl@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>nereid: vbus measurement using charger ic ADC</title>
<updated>2022-06-09T05:50:22+00:00</updated>
<author>
<name>Deepti Deshatty</name>
<email>deepti.deshatty@intel.corp-partner.google.com</email>
</author>
<published>2022-05-27T06:40:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=4c1260e92d13969b8e5aabf79082e722a4617cea'/>
<id>4c1260e92d13969b8e5aabf79082e722a4617cea</id>
<content type='text'>
implement pd_check_vbus_level() to measure vbus precisely using
SM5803 vbus ADC similar to the CL:2406342

BUG=b:231419461
BRANCH=main
TEST=USBC dock is detected on Nereid port1

Change-Id: If418017256d0c34c683f1d84fb46423184cc7fa0
Signed-off-by: Deepti Deshatty &lt;deepti.deshatty@intel.corp-partner.google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3670785
Reviewed-by: Peter Marheine &lt;pmarheine@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
implement pd_check_vbus_level() to measure vbus precisely using
SM5803 vbus ADC similar to the CL:2406342

BUG=b:231419461
BRANCH=main
TEST=USBC dock is detected on Nereid port1

Change-Id: If418017256d0c34c683f1d84fb46423184cc7fa0
Signed-off-by: Deepti Deshatty &lt;deepti.deshatty@intel.corp-partner.google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3670785
Reviewed-by: Peter Marheine &lt;pmarheine@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>docs/fingerprint: Clean up formatting in fingerprint-factory-quick-guide.md</title>
<updated>2022-06-09T00:42:14+00:00</updated>
<author>
<name>Tom Hughes</name>
<email>tomhughes@chromium.org</email>
</author>
<published>2022-06-02T23:06:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=60032a8828801a316bdcb5f246ebb58da6e529f0'/>
<id>60032a8828801a316bdcb5f246ebb58da6e529f0</id>
<content type='text'>
BRANCH=none
BUG=b:234772776
TEST=view in gitiles

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I0a107a379367712d79a959dc40b4e6b215be8c00
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687544
Reviewed-by: Bobby Casey &lt;bobbycasey@google.com&gt;
Reviewed-by: Andrea Grandi &lt;agrandi@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
BRANCH=none
BUG=b:234772776
TEST=view in gitiles

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I0a107a379367712d79a959dc40b4e6b215be8c00
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3687544
Reviewed-by: Bobby Casey &lt;bobbycasey@google.com&gt;
Reviewed-by: Andrea Grandi &lt;agrandi@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>assert.h: Remove unnecessary extern</title>
<updated>2022-06-09T00:02:10+00:00</updated>
<author>
<name>Tom Hughes</name>
<email>tomhughes@chromium.org</email>
</author>
<published>2022-06-06T18:21:48+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=f40490f84a0a0d080e41f4fb9c954715c2696073'/>
<id>f40490f84a0a0d080e41f4fb9c954715c2696073</id>
<content type='text'>
Functions in C are extern by default; it's not necessary in a header
file. The extern combined with noreturn causes a compiler error when
building with C++:

./builtin/assert.h:33:8: error: an attribute list cannot appear here
extern noreturn void panic_assert_fail(const char *msg, const char *func,
       ^~~~~~~~

BRANCH=none
BUG=b:234181908
TEST=./util/compare_build.sh -b all -j 120
     =&gt; MATCH

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I7a2a86e57569bc90bb1ab93d5703217eabc042a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3691957
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Functions in C are extern by default; it's not necessary in a header
file. The extern combined with noreturn causes a compiler error when
building with C++:

./builtin/assert.h:33:8: error: an attribute list cannot appear here
extern noreturn void panic_assert_fail(const char *msg, const char *func,
       ^~~~~~~~

BRANCH=none
BUG=b:234181908
TEST=./util/compare_build.sh -b all -j 120
     =&gt; MATCH

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I7a2a86e57569bc90bb1ab93d5703217eabc042a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3691957
Reviewed-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>kblight: Initialize on AP startup</title>
<updated>2022-06-08T23:49:04+00:00</updated>
<author>
<name>Daisuke Nojiri</name>
<email>dnojiri@chromium.org</email>
</author>
<published>2022-06-07T18:29:45+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=74215f789c53e835461de7499155be39a7530163'/>
<id>74215f789c53e835461de7499155be39a7530163</id>
<content type='text'>
This patch makes keyboard backlight module initialize on chipset startup
instead of on reset. This will prevent LED drivers from being accessed
before they're powered.

BUG=b:233323599
BRANCH=None
TEST=Taniks

Change-Id: Ia57acdd57663c7bfa0dc3117cbc9ab6191d34c45
Signed-off-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3693054
Reviewed-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Tested-by: Yu-An Chen &lt;yu-an.chen@quanta.corp-partner.google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch makes keyboard backlight module initialize on chipset startup
instead of on reset. This will prevent LED drivers from being accessed
before they're powered.

BUG=b:233323599
BRANCH=None
TEST=Taniks

Change-Id: Ia57acdd57663c7bfa0dc3117cbc9ab6191d34c45
Signed-off-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3693054
Reviewed-by: Parth Malkan &lt;parthmalkan@google.com&gt;
Tested-by: Yu-An Chen &lt;yu-an.chen@quanta.corp-partner.google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "zephyr: defconfig: default to LEGACY_INCLUDE_PATH=n"</title>
<updated>2022-06-08T23:13:44+00:00</updated>
<author>
<name>Yuval Peress</name>
<email>peress@google.com</email>
</author>
<published>2022-06-01T20:20:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=8a7f9bb026ff1468ab435b4b3b7d02314d2d2d73'/>
<id>8a7f9bb026ff1468ab435b4b3b7d02314d2d2d73</id>
<content type='text'>
This reverts commit 4e012fd31e03507c4b254e49825e0dfbf263a375.

Reason for revert: Causes build issues in generated files

Original change's description:
&gt; zephyr: defconfig: default to LEGACY_INCLUDE_PATH=n
&gt;
&gt; Now that the code base compiles with LEGACY_INCLUDE_PATH=n, turn the
&gt; option to no by default so that we won't add any more non prefixed
&gt; include.
&gt;
&gt; BRANCH=none
&gt; BUG=none
&gt; TEST=cq dry run
&gt;
&gt; Signed-off-by: Fabio Baltieri &lt;fabiobaltieri@google.com&gt;
&gt; Change-Id: Ie95e79e659e755de869c8512d0bdfdc9523e1afa
&gt; Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3637580
&gt; Reviewed-by: Yuval Peress &lt;peress@google.com&gt;

Bug: none
Change-Id: I7541df82a963b81d983510cda58bb3a78c8156d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3683927
Commit-Queue: Al Semjonovs &lt;asemjonovs@google.com&gt;
Tested-by: Al Semjonovs &lt;asemjonovs@google.com&gt;
Reviewed-by: Fabio Baltieri &lt;fabiobaltieri@google.com&gt;
Auto-Submit: Yuval Peress &lt;peress@google.com&gt;
Reviewed-by: Keith Short &lt;keithshort@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reverts commit 4e012fd31e03507c4b254e49825e0dfbf263a375.

Reason for revert: Causes build issues in generated files

Original change's description:
&gt; zephyr: defconfig: default to LEGACY_INCLUDE_PATH=n
&gt;
&gt; Now that the code base compiles with LEGACY_INCLUDE_PATH=n, turn the
&gt; option to no by default so that we won't add any more non prefixed
&gt; include.
&gt;
&gt; BRANCH=none
&gt; BUG=none
&gt; TEST=cq dry run
&gt;
&gt; Signed-off-by: Fabio Baltieri &lt;fabiobaltieri@google.com&gt;
&gt; Change-Id: Ie95e79e659e755de869c8512d0bdfdc9523e1afa
&gt; Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3637580
&gt; Reviewed-by: Yuval Peress &lt;peress@google.com&gt;

Bug: none
Change-Id: I7541df82a963b81d983510cda58bb3a78c8156d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3683927
Commit-Queue: Al Semjonovs &lt;asemjonovs@google.com&gt;
Tested-by: Al Semjonovs &lt;asemjonovs@google.com&gt;
Reviewed-by: Fabio Baltieri &lt;fabiobaltieri@google.com&gt;
Auto-Submit: Yuval Peress &lt;peress@google.com&gt;
Reviewed-by: Keith Short &lt;keithshort@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>docs/fingerprint: Run mdformat</title>
<updated>2022-06-08T22:01:36+00:00</updated>
<author>
<name>Tom Hughes</name>
<email>tomhughes@chromium.org</email>
</author>
<published>2022-06-08T17:42:47+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/chrome-ec.git/commit/?id=f49eb6f8944670012c0d6875ce302a9241c628f8'/>
<id>f49eb6f8944670012c0d6875ce302a9241c628f8</id>
<content type='text'>
BRANCH=none
BUG=none
TEST=view in gitiles

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I20caf005a0692d7e1d89528c87bb49b7743b8199
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3696148
Reviewed-by: Andrea Grandi &lt;agrandi@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
BRANCH=none
BUG=none
TEST=view in gitiles

Signed-off-by: Tom Hughes &lt;tomhughes@chromium.org&gt;
Change-Id: I20caf005a0692d7e1d89528c87bb49b7743b8199
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3696148
Reviewed-by: Andrea Grandi &lt;agrandi@google.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
