summaryrefslogtreecommitdiff
path: root/board
Commit message (Collapse)AuthorAgeFilesLines
* Tomato: Update LED behaviorSue Chen2021-11-051-0/+2
| | | | | | | | | | | | | | | | Add led behavior for STATE_CHARGING_FULL_S5 and STATE_DISCHARGE_S0_BAT_LOW. BUG=b:199125941 BRANCH=none TEST=led behavior is correct in any state Signed-off-by: Sue Chen <sue.chen@quanta.corp-partner.google.com> Change-Id: Ie7767850d0659838d76e34b82ff6efcfd62f267c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246634 Reviewed-by: Ting Shen <phoenixshen@chromium.org> Commit-Queue: Ting Shen <phoenixshen@chromium.org> Tested-by: Ting Shen <phoenixshen@chromium.org>
* Todor: Free up flash spaceDiana Z2021-11-041-0/+3
| | | | | | | | | | | | | | Todor is nearly out of flash, so remove help strings. BRANCH=None BUG=None TEST=build, observe todor now has about 4.5k free flash Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I55e677de4419c9e1639f1dde87f4edf6feb4873b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262584 Commit-Queue: Keith Short <keithshort@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* Terrador: Free up flash spaceDiana Z2021-11-041-0/+1
| | | | | | | | | | | | | | Terrador is nearly out of flash space, so remove the help strings. BRANCH=None BUG=None TEST=build, observe terrador now has about 4.8k in free flash Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Ifd27849f20ec403018b9e5076921edb163bc2107 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262583 Commit-Queue: Keith Short <keithshort@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* Voema_npcx796fc: Free up flash spaceDiana Z2021-11-041-1/+4
| | | | | | | | | | | | | | Free up flash space for voema_npcx796fc by removing help strings. BRANCH=None BUG=None TEST=make -j buildall; observe there is now 4.5k free flash Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: Iffe897e8aa427f0e0ffbcbd8ecd25ca200961d05 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262582 Commit-Queue: Keith Short <keithshort@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* board/woomax,board/morphius: Fix return typeTom Hughes2021-11-034-8/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | clang warns: board/morphius/board.c:406:8: error: implicit conversion from enumeration type 'enum ioex_signal' to different enumeration type 'enum gpio_signal' [-Werror,-Wenum-conversion] ? IOEX_USB_C1_HPD_IN_DB ^~~~~~~~~~~~~~~~~~~~~ board/woomax/board.c:864:6: error: implicit conversion from enumeration type 'enum ioex_signal' to different enumeration type 'enum gpio_signal' [-Werror,-Wenum-conversion] : IOEX_USB_C1_HPD_IN_DB; ^~~~~~~~~~~~~~~~~~~~~ The only caller appears to be svdm_set_hpd_gpio() and svdm_get_hpd_gpio(), which correctly calls gpio_or_ioex_set_level() since the signal may be either gpio_signal or ioex_signal. Rename the board_usbc_port* function to indicate that it can return gpio or ioex. BRANCH=none BUG=b:172020503 TEST=make buildall TEST=CC=clang make BOARD=woomax TEST=CC=clang make BOARD=morphius Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I7be75c8a714ba103362b293413aadc6a92ab72c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3243008 Reviewed-by: Eric Yilun Lin <yllin@google.com>
* board/zinger: Disable CONFIG_DEBUG_PRINTFTom Hughes2021-11-031-2/+2
| | | | | | | | | | | | | | We're getting low on RO flash space, so disable CONFIG_DEBUG_PRINTF to free up more space. BRANCH=none BUG=b:172020503 TEST=make buildall Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I44345f076d26b8aab111f01081fe9bdfd0a69dee Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3253649 Reviewed-by: Edward Hill <ecgh@chromium.org>
* moonbuggy: modify LED pwm output typeBen Chen2021-11-033-21/+23
| | | | | | | | | | | | | | | | | Modify moonbuggy LED colors to red and blue. Also set the PWM configuration to active low. BUG=b:204504054 BRANCH=cros/main TEST=factory software PE test is PASS Change-Id: I9e79b007a8ab376b290ec441ffd3bb33338145e8 Signed-off-by: Ben Chen <ben.chen2@quanta.corp-partner.google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3255121 Reviewed-by: Joe Tessler <jrt@chromium.org> Reviewed-by: Matthew Ziegelbaum <ziegs@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Joe Tessler <jrt@chromium.org>
* brask: Enable recovery key to send event to OSZhuohao Lee2021-11-032-1/+2
| | | | | | | | | | | | | | | | | | In order to make the recovery key to trigger the sysrq and warm reset to the OS, we move the pin to the irq for receiving the button interrupt and then passing the key event to the OS via MKBP event. BUG=b:204512547 BRANCH=None TEST=1. Press/release recovery button shortly -> sysrq sent 2. Press/hold recovery button at least 4s -> warm reset Change-Id: I7d86851b12b4771a43985c2891c7e8b9299d32e4 Signed-off-by: Zhuohao Lee <zhuohao@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3251206 Reviewed-by: Andrew McRae <amcrae@google.com> Commit-Queue: Andrew McRae <amcrae@google.com>
* Dewatt: support 65w adapterSue Chen2021-11-033-0/+17
| | | | | | | | | | | | | Modify max power and max current to support 65w adapter. BUG=b:204386852 BRANCH=none TEST="make BOARD" pass for board dewatt, guybrush and nipperkin Signed-off-by: Sue Chen <sue.chen@quanta.corp-partner.google.com> Change-Id: Ic18ab3021e39913f2a69550fac269e91d3e83508 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3247739 Reviewed-by: Rob Barnes <robbarnes@google.com>
* beetley: Add motion sense config for BMI220 & ICM42607wizard2021-11-034-4/+148
| | | | | | | | | | | | | | | | | | Add BMI220 & ICM42607 base sensor config for second source BUG=b:204728251 BRANCH=dedede TEST=ectool 'motionsense' 1.Using ectool 'motionsense' verified lid angle goes from 0 to 360 2.swtiches totablet mode after crossing 200 threshold on re-work bmi220/icm42607 DUT Signed-off-by: wizard <shenhu5@huaqin.corp-partner.google.com> Change-Id: Ie5318f4c872136fcb4e120d25c6370500ae6c95e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3255467 Reviewed-by: Aseda Aboagye <aaboagye@chromium.org> Reviewed-by: jesen <wangganxiang@huaqin.corp-partner.google.com> Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
* hayato: Fix icm40608 will pending when gryo set offset value.Michael5 Chen12021-11-031-0/+1
| | | | | | | | | | | | | | | | | Because the configuration "icm426xx_base_gyro" miss setting "drv_data". It will cause icm40608 will pending when gyro set offset value. BUG=b:193752214 BRANCH=asurada TEST=manual 1. ectool motionsense offset 1 0 0 0 2. watch ectool motionsense => Check icm40608 sesnor. Signed-off-by: Michael5 Chen1 <michael5_chen1@pegatron.corp-partner.google.com> Change-Id: I895df941192c86a447c552f3872c9156d61bec80 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3256731 Reviewed-by: Ting Shen <phoenixshen@chromium.org> Reviewed-by: Eric Yilun Lin <yllin@google.com>
* taeko: modify keyboard backlight settingamber.chen2021-11-031-8/+6
| | | | | | | | | | | | | BUG=b:204737147 BRANCH=main TEST=make -j BOARD=taeko Signed-off-by: amber.chen <amber.chen@lcfc.corp-partner.google.com> Change-Id: Idce11dd6195c3ecefc3d4dfa043080b6a274c7a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3255474 Reviewed-by: Boris Mittelberg <bmbm@google.com> Reviewed-by: Parth Malkan <parthmalkan@google.com> Commit-Queue: Boris Mittelberg <bmbm@google.com>
* Homestar: Modify the BatteryParam registertongjian2021-11-021-2/+2
| | | | | | | | | | | | | | | | | The factory needs to read the battery CT number overstation, but fails to read it through Ectool BatteryParam, so the battery register needs to be modified. BRANCH=Trogdor BUG=b:201618156 TEST=make -j BOARD=homestar Verify build on SIT board Signed-off-by: tongjian <tongjian@huaqin.corp-partner.google.com> Change-Id: Ib53377d3127be9ce0bb9a93acbb6de8160f3bdae Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3195685 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org> Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
* brya: enable PSYS monitoringCaveh Jalali2021-11-021-0/+1
| | | | | | | | | | | | | | This enables PSYS (system power) monitoring at the charger. BRANCH=none BUG=b:195615830 TEST=psys values reported by dump_intel_rapl_consumption look reasonable Change-Id: I5dab0346fefb154c72b4141ed109257c54191244 Signed-off-by: Caveh Jalali <caveh@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3253077 Reviewed-by: Boris Mittelberg <bmbm@google.com> Reviewed-by: Ryan Lin <ryan.lin@intel.corp-partner.google.com>
* Mushu: Free up flash spaceWealian Liao2021-11-021-0/+1
| | | | | | | | | | | | | | Mushu is about 48 bytes, so free up 5600 in free flash space for it. BRANCH=None BUG=None TEST=make buildall with following CLs Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com> Change-Id: Ia7894b46abe301924376f46f14716a8e45f9a6d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3256723 Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* Fizz: Free up flash spaceWealian Liao2021-11-021-0/+3
| | | | | | | | | | | | | | | Fizz is about 1080 bytes(1k for public key), so free up 3900 bytes in free flash space for it. BRANCH=None BUG=None TEST=make buildall with following CLs Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com> Change-Id: I25af5330e749a1b8a86b0f0b35aa1400fe599d6c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3256722 Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* jinlon: Update fan table version for acousticDevin Lu2021-11-021-29/+29
| | | | | | | | | | | | | | This patch updates the fan table to improve acoustics while charging. BUG=b:195482344 BRANCH=firmware-hatch-12672.B TEST=On Jinlon. thermal team verified the fan steps worked as expected. Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: Iab85a39836f347f748a5adc0d04bd3e5392ddcca Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3225269 Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
* fizz: Remove `adc` console commandAseda Aboagye2021-11-011-0/+1
| | | | | | | | | | | | | | | | | Fizz is out of space, this commit just removes the ADC console command to free up more space. BUG=None BRANCH=None TEST=`make -j BOARD=fizz` verify that build succeeds. Signed-off-by: Aseda Aboagye <aaboagye@google.com> Change-Id: I32bd2ae5a7d262bb6ca3f3e5d34b5a2a59649198 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3255706 Tested-by: Aseda Aboagye <aaboagye@chromium.org> Auto-Submit: Aseda Aboagye <aaboagye@chromium.org> Reviewed-by: Diana Z <dzigterman@chromium.org> Commit-Queue: Diana Z <dzigterman@chromium.org>
* Cret: Change D-FET detect maskelmo_lan2021-10-301-40/+27
| | | | | | | | | | | | | | | | | | | | | | | | | Mfgacc_support will write OperationStatus(0x0054) to MANUFACTURER_ACCESS(0x00), and then read back from ALT_MANUFACTURER_ACCESS(0x44). However, batteries for cret, need to write ALT_MANUFACTURER_ACCESS instead of MANUFACTURER_ACCESS. Cret should read status directly from MANUFACTURER_ACCESS. This CL remove mfgacc_support, and modify D-FET mask bit. BUG=b:204285159 BRANCH=dedede TEST=Test on Cret, can boot up from battery cutoff, ec reboot via dc and ac+dc, charge and discharge. Signed-off-by: elmo_lan <elmo_lan@compal.corp-partner.google.com> Change-Id: Iaeb32fcecf851a1241ff2ea82e52cef3ecfdd9c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3251216 Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com> Reviewed-by: Aseda Aboagye <aaboagye@chromium.org> Tested-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com> Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
* brya: Enable VCONN control on C1 (ps8815)Caveh Jalali2021-10-291-1/+2
| | | | | | | | | | | | | | This enables TCPC based VCONN control for port C1 (ps8815). BRANCH=none BUG=b:204102039 TEST=with reset of patch stack, external monitor links up using active TBT4 cable Change-Id: I60482dfff657b53f533b6e72507776b3657a5ff5 Signed-off-by: Caveh Jalali <caveh@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3252411 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* scarlet: Disable verbose EC command in RO segmentNehemiah Dureus2021-10-291-0/+5
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save scarlet memory. Only applies to RO segment. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: Idcca6b200268d2f1a1eab7d92c138b1d544d334b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3252413 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* elm: Disable verbose output for EC commandNehemiah Dureus2021-10-291-0/+2
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save elm memory. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: I1776c917759f3b7db006b856b209631131c4062c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3252409 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* burnet: Disable verbose output for EC commandNehemiah Dureus2021-10-291-0/+3
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save burnet memory. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: I121b82cc936a2eaa2a6db31a366fabe974842bf2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3248344 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* kodama: Disable verbose output for EC commandNehemiah Dureus2021-10-291-0/+3
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save kodama memory. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: I1cb602f122bb7bc74f2d2c05f8bd0a7f0f8648bf Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3248343 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* kukui: Disable verbose EC command in RO segmentNehemiah Dureus2021-10-291-0/+4
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save kukui memory. Only applies to RO segment. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: I6b9229862dd7a1212e5864bda1a94068b740b400 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3248342 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* servo_v4: Disable verbose output for EC commandNehemiah Dureus2021-10-291-0/+1
| | | | | | | | | | | | | | | | | | | Compiles the less detailed but more memory efficient version of EC command `pd <port> srccaps`, to save servo_v4 memory. This is for a feature to make the EC power delivery command more verbose: c/3248345. BRANCH=None BUG=b:194402616 TEST=Built all boards (make -j buildall) Signed-off-by: Nehemiah Dureus <ndureus@google.com> Cq-Depend: chromium:3248345 Change-Id: If3ddb94513d08e829a4bdfa16f49ded40d084afe Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3248341 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* brya: Run TCPC1/PPC1 I2C at 1 MHz on newer boardsCaveh Jalali2021-10-291-3/+22
| | | | | | | | | | | | | | | | | | | This updates the TCPC1 and PPC1 (ps8815 daughterboard) I2C buses to operate at 1 MHz instead of 400 kHz. Boards before board ID 2 were not tuned for 1 MHz operation, so they will continue to operate at 400 kHz. BRANCH=none BUG=b:186189039,b:187764571,b:187764202 TEST=updated TCPC1 firmware using window scheme and FIFO scheme. TEST=verified I2C bus 4, 7 speed is 1000 on board ID 2 and at 400 when BOARD_ID_FAST_PLUS_CAPABLE is set to 9. Change-Id: I182cb6bc0d1be24d512ef14d16e2cb4347b7eb02 Signed-off-by: Caveh Jalali <caveh@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3181512 Reviewed-by: Boris Mittelberg <bmbm@google.com> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* Revert "kodama: drop kodama board"Alexander Hartl2021-10-299-0/+952
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3f50a1e35e093d3989dc27f340b4ce5fb7fca5ee. Reason for revert: The kukui-arc-r-postsubmit builder started failing after this change landed: https://buganizer.corp.google.com/issues/204520684 https://ci.chromium.org/p/chromeos/builders/postsubmit/kukui-arc-r-postsubmit The error is that the kodama board wasn't found. chromeos-ec-0.0.2-r11157: Makefile:25: *** unable to locate BOARD kodama. Stop. chromeos-ec-0.0.2-r11157: * ERROR: chromeos-base/chromeos-ec-0.0.2-r11157::chromiumos failed (compile phase): Original change's description: > kodama: drop kodama board > > Kodama is the tightest space board in kukui family, and we > have to constanly disable configs to free up space. > Since we don't really need the board in main branch as the > firmware branch has been created for a long time, just drop > this board on main branch. > > BUG=none > TEST=makd buildall > BRANCH=main > > Change-Id: I7f958ad7b97c191a8433d82987f8876add8c5dc3 > Signed-off-by: Eric Yilun Lin <yllin@chromium.org> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3252605 > Tested-by: Eric Yilun Lin <yllin@google.com> > Auto-Submit: Eric Yilun Lin <yllin@google.com> > Reviewed-by: Ting Shen <phoenixshen@chromium.org> > Commit-Queue: Ting Shen <phoenixshen@chromium.org> Bug: none Change-Id: I4d9b29a1c80b753a48763ee7cae42d3f6f25dccc Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3250679 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Alexander Hartl <alexanderhartl@google.com>
* cherry_scp: Enlarge vdec/venc/mdp stack sizeYunfei Dong2021-10-291-3/+3
| | | | | | | | | | | | | | | | Enlage vdec/venc/dmp stack size in case of open debug log fail. BRANCH=none BUG=b:184793035 TEST=make BOARD=cherry_scp Signed-off-by: Yunfei Dong <yunfei.dong@mediatek.com> Change-Id: I0ca61a7b00dc535f69fb6f37d9798795ac1636e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2928123 Reviewed-by: Rong Chang <rongchang@chromium.org> Reviewed-by: Yunfei Dong <yunfei.dong@mediatek.corp-partner.google.com> Commit-Queue: Tzung-Bi Shih <tzungbi@chromium.org> Tested-by: Tzung-Bi Shih <tzungbi@chromium.org>
* krabby: refactor board.c/baseboard.cTing Shen2021-10-295-13/+44
| | | | | | | | | | | | | | | | | | Move some functions which is likely to be use by zephyr ec outside of board/baseboard.c. BUG=b:202808130 TEST=1) make BOARD=krabby 2) zmake -D -l DEBUG configure -b zephyr/projects/corsola/krabby/ BRANCH=none Signed-off-by: Ting Shen <phoenixshen@google.com> Change-Id: I6ac4519bd9070cb8c54bce812dd62eaf8af21dfb Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246994 Auto-Submit: Ting Shen <phoenixshen@chromium.org> Reviewed-by: Eric Yilun Lin <yllin@google.com> Commit-Queue: Ting Shen <phoenixshen@chromium.org> Tested-by: Ting Shen <phoenixshen@chromium.org>
* kano: Initialize the vivaldi keyboard.David Huang2021-10-292-0/+28
| | | | | | | | | | | | | | | | | | | | Initialize the vivaldi keyboard. BUG=b:200692807 BRANCH=main TEST=manual 1. Scan all key. 2. Check action key function. 3. Check ALT + Volup + H. 4. Check ALT + Volup + R. 5. Check Refresh (F2) + powerbutton (EC reboot). 6. Check ESC + Refresh + powerbutton. Signed-off-by: David Huang <david.huang@quanta.corp-partner.google.com> Change-Id: I8b373bce021de2ef0f4a3b2d146f28d6b6be846f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3244681 Reviewed-by: Boris Mittelberg <bmbm@google.com> Commit-Queue: Boris Mittelberg <bmbm@google.com>
* kodama: drop kodama boardEric Yilun Lin2021-10-299-952/+0
| | | | | | | | | | | | | | | | | | | | Kodama is the tightest space board in kukui family, and we have to constanly disable configs to free up space. Since we don't really need the board in main branch as the firmware branch has been created for a long time, just drop this board on main branch. BUG=none TEST=makd buildall BRANCH=main Change-Id: I7f958ad7b97c191a8433d82987f8876add8c5dc3 Signed-off-by: Eric Yilun Lin <yllin@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3252605 Tested-by: Eric Yilun Lin <yllin@google.com> Auto-Submit: Eric Yilun Lin <yllin@google.com> Reviewed-by: Ting Shen <phoenixshen@chromium.org> Commit-Queue: Ting Shen <phoenixshen@chromium.org>
* brya: add PROCHOT assertion/de-assertion ratioWill Tsai2021-10-292-0/+2
| | | | | | | | | | | | | | | | | This adds a de-assertion ratio for the total power consumption to compare when adapter wattage greater or equal to 60W and the battery is not present(or the battery energy is less than 10%). BUG=b:198689488 b:198722634 BRANCH=none TEST=make -j BOARD=gimble Signed-off-by: Will Tsai <will_tsai@wistron.corp-partner.google.com> Change-Id: Ie7d6ffa8f30a1f0b0aec24055e8d823d1d3395b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3251823 Reviewed-by: caveh jalali <caveh@chromium.org> Commit-Queue: caveh jalali <caveh@chromium.org>
* brya: move prochot assertion/de-assertion ratio to variantWill Tsai2021-10-292-0/+20
| | | | | | | | | | | | | | | | Since each variant may have different prochot assertion/de-assertion configuration, the prochot assertion/de-assertion ratio setting will be moved from baseboard to variant. BUG=b:198689488 b:198722634 BRANCH=none TEST=make -j BOARD=gimble Signed-off-by: Will Tsai <will_tsai@wistron.corp-partner.google.com> Change-Id: I140e74e85fbec24e0e1759526d9c3b24152b68c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3247740 Reviewed-by: caveh jalali <caveh@chromium.org> Commit-Queue: caveh jalali <caveh@chromium.org>
* brya: Update stack sizes based on stack analyzerCaveh Jalali2021-10-298-97/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adjusts the task stack sizes on brya and variants. Many task stacks had to be increased, only CONSOLE could be reduced. Task stack size constants are added at the baseboard level for several tasks that tend to behave very similarly among variants. before: Task: CHARGER, Max size: 932, Allocated size: 928 Task: CHG_RAMP, Max size: 900, Allocated size: 672 Task: CHIPSET, Max size: 988, Allocated size: 928 Task: CONSOLE, Max size: 612, Allocated size: 928 Task: HOOKS, Max size: 580, Allocated size: 800 Task: HOSTCMD, Max size: 612, Allocated size: 800 Task: KEYPROTO, Max size: 660, Allocated size: 672 Task: KEYSCAN, Max size: 708, Allocated size: 672 Task: MOTIONSENSE, Max size: 796, Allocated size: 928 Task: PD_C0, Max size: 1052, Allocated size: 1056 Task: PD_C1, Max size: 1052, Allocated size: 1056 Task: PD_C2, Max size: 1052, Allocated size: 1056 Task: PD_INT_C0, Max size: 644, Allocated size: 672 Task: PD_INT_C1, Max size: 644, Allocated size: 672 Task: POWERBTN, Max size: 964, Allocated size: 800 Task: USB_CHG_P0, Max size: 508, Allocated size: 672 Task: USB_CHG_P1, Max size: 508, Allocated size: 672 Task: USB_CHG_P2, Max size: 508, Allocated size: 672 after: Task: CHARGER, Max size: 932, Allocated size: 1088 Task: CHG_RAMP, Max size: 900, Allocated size: 1088 Task: CHIPSET, Max size: 988, Allocated size: 1152 Task: CONSOLE, Max size: 612, Allocated size: 800 Task: HOOKS, Max size: 580, Allocated size: 800 Task: HOSTCMD, Max size: 612, Allocated size: 800 Task: KEYPROTO, Max size: 660, Allocated size: 800 Task: KEYSCAN, Max size: 708, Allocated size: 928 Task: MOTIONSENSE, Max size: 796, Allocated size: 928 Task: PD_C0, Max size: 1052, Allocated size: 1216 Task: PD_C1, Max size: 1052, Allocated size: 1216 Task: PD_C2, Max size: 1052, Allocated size: 1216 Task: PD_INT_C0, Max size: 644, Allocated size: 800 Task: PD_INT_C1, Max size: 644, Allocated size: 800 Task: POWERBTN, Max size: 964, Allocated size: 1088 Task: USB_CHG_P0, Max size: 508, Allocated size: 672 Task: USB_CHG_P1, Max size: 508, Allocated size: 672 Task: USB_CHG_P2, Max size: 508, Allocated size: 672 BRANCH=none BUG=b:204362187,b:204102039,b:204280744 TEST=brya EC no longer reboots due to stack overflow in PD task TEST=task command now reports 200 bytes of stack headroom for chipset TEST=boots on brya Change-Id: I081228da6d850bf69b431475b81025045c1c521d Signed-off-by: Caveh Jalali <caveh@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3248976 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* quackingstick: Add temp sensorSue Chen2021-10-283-1/+33
| | | | | | | | | | | | | | | | ADC6 (GPIO34) is connected to temp sensor. Enable ADC6 to read temperature. BUG=none BRANCH=trogdor TEST=EC console "temps" can read temperature. Signed-off-by: Sue Chen <sue.chen@quanta.corp-partner.google.com> Change-Id: I9600ea0442a5643ca65a9a484117f2ed47cdc17d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246876 Reviewed-by: Wai-Hong Tam <waihong@google.com> Reviewed-by: Bob Moragues <moragues@chromium.org> Commit-Queue: Bob Moragues <moragues@chromium.org>
* Beetley:Modify LED code parameter.wizard2021-10-281-1/+1
| | | | | | | | | | | | | | | The LED brightness test fail.Modify the problem code. BUG=b:204401300 BRANCH=dedede TEST=make BOARD=BLIPPER pass, and the test pass. Signed-off-by: Wizard Shen <shenhu5@huaqin.corp-partner.google.com> Change-Id: I1561779a6b0b142a00d61581adf2f07042d50a3a Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3250074 Reviewed-by: zhongtian wu <wuzhongtian@huaqin.corp-partner.google.com> Reviewed-by: Aseda Aboagye <aaboagye@chromium.org> Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
* cleanup: charger: Add BQ25710 sense resistor configsDeepti Deshatty2021-10-286-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | Defined different sense register configs for BQ25710. All the charger chip driver implementation uses common sense register configs i.e CONFIG_CHARGER_SENSE_RESISTOR and CONFIG_CHARGER_SENSE_RESISTOR_AC. When we enable a charger driver for a platform, it is expected that the platform define these sense register configs. But ADLRVP requires two different charger drivers i.e ISL9241 and BQ25720 to be enabled to support all the variant builds. Hence BQ25710 driver is changed to use different sense register configs so that the configs defined for ISL9241 are not affected. BRANCH=none TEST=make -j buildall has no issues Signed-off-by: Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com> Change-Id: If1e1422246e2e3a5cb628d9a37c23790502b5ca4 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3211773 Reviewed-by: Poornima Tom <poornima.tom@intel.com> Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org> Reviewed-by: caveh jalali <caveh@chromium.org> Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
* volteer: configure BC1.2 on type-C ports with DTstabilize-14312.B-mainPeter Marheine2021-10-281-0/+4
| | | | | | | | | | | | | This adopts the recently-added support for PI3USB9201 in Zephyr. BUG=b:202397628 TEST=builds BRANCH=None Change-Id: I87c898a4d1296090c723c2802df7cf3a27991aee Signed-off-by: Peter Marheine <pmarheine@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246636 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* kano: Change LED control behaviorDavid Huang2021-10-284-90/+60
| | | | | | | | | | | | | Change LED control from pwm to gpio. BUG=none BRANCH=master TEST=Check LED status in each state. Signed-off-by: David Huang <david.huang@quanta.corp-partner.google.com> Change-Id: I640380175fe505348df3c3fa532c392ffa5943d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3244810 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* kingoftown: Implement battery_get_vendor_param to read vendor paramsTommy Chung2021-10-282-0/+5
| | | | | | | | | | | | | | Add CONFIG_BATTERY_VENDOR_PARAM and define specific param regs to get battery CT code. BUG=none BRANCH=trogdor TEST=check `ectool batteryparam` cmd on kingoftown. Signed-off-by: Tommy Chung <tommy.chung@quanta.corp-partner.google.com> Change-Id: Ib98e1500ac8082568caf7097dec9f10a813da789 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3219978 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
* battery: Move battery get/set vendor param to common/battery.cTommy Chung2021-10-2819-162/+31
| | | | | | | | | | | | | | | Since the functions are copied by many boards, move them to common. The specific param regs are defined in each specific boards. BUG=b:203031618 BRANCH=none TEST=make BOARD=burnet/coachz/eve/homestar/kappa/mrbland/quackingstick/ wormdingler. Signed-off-by: Tommy Chung <tommy.chung@quanta.corp-partner.google.com> Change-Id: I77c070663ad3e800ec484bd21865c5d911a2c48b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3220718 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
* board/dirinboz,board/gumboz: Fix variable typeTom Hughes2021-10-272-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When building with clang, it warns: board/dirinboz/led.c:47:6: error: implicit conversion from enumeration type 'enum ioex_signal' to different enumeration type 'enum gpio_signal' [-Werror,-Wenum-conversion] IOEX_C1_CHARGER_LED_AMBER_DB); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ board/dirinboz/led.c:49:6: error: implicit conversion from enumeration type 'enum ioex_signal' to different enumeration type 'enum gpio_signal' [-Werror,-Wenum-conversion] IOEX_C1_CHARGER_LED_WHITE_DB); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ The led variables are used as parameters to gpio_or_ioex_set_level, which takes an int, so just use int so we don't have to cast between types. BRANCH=none BUG=b:172020503 TEST=none Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I16428fbc5ef49f497a3514ac56049e8bd0e40d53 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3243005 Reviewed-by: Diana Z <dzigterman@chromium.org>
* board/felwinter: Fix signal typeTom Hughes2021-10-271-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | When building with clang, it warns: board/felwinter/usbc_config.c:206:16: error: implicit conversion from enumeration type 'enum gpio_signal' to different enumeration type 'enum ioex_signal' [-Werror,-Wenum-conversion] rst_signal = GPIO_USB_C1_RT_RST_R_ODL; ~ ^~~~~~~~~~~~~~~~~~~~~~~~ When we are mixing gpio_signal and ioex_signal, we should be calling gpio_or_ioex_set_level, which will call gpio_set_level or ioex_set_level, depending on the signal type. BRANCH=none BUG=b:172020503 TEST=none Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: Id835a1a31d686bee128e0908d3916f36447f78ec Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3243006 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: YH Lin <yueherngl@chromium.org>
* board/metaknight: Fix compilation error when using clangTom Hughes2021-10-271-24/+14
| | | | | | | | | | | | | | | | | | | | | clang warns: board/metaknight/board.c:787:10: error: array index 1 is past the end of the array (which contains 1 element) [-Werror,-Warray-bounds] if (!(tcpc_config[1].flags & TCPC_FLAGS_TCPCI_REV2_0)) ^ ~ board/metaknight/board.c:741:1: note: array 'tcpc_config' declared here const struct tcpc_config_t tcpc_config[CONFIG_USB_PD_PORT_MAX_COUNT] = { ^ BRANCH=none BUG=b:172020503 TEST=CC=clang make BOARD=metaknight Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I510d6ec4343f258d734834f2d1e3b6760fca7761 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3238248 Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
* Cret: Add mfgacc_support for some batterieselmo_lan2021-10-271-0/+13
| | | | | | | | | | | | | | | | | TI support mfgacc_support. We will use mfgacc_support to detect dfet and cfet. BUG=b:204285159 BRANCH=dedede TEST=Test on Cret, can boot up from battery cutoff. Signed-off-by: elmo_lan <elmo_lan@compal.corp-partner.google.com> Change-Id: I0c6358176ded8756efa506db0a06285436ab3e46 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246885 Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com> Reviewed-by: Aseda Aboagye <aaboagye@chromium.org> Tested-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com> Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
* kingoftown: Swap left/right LEDTommy Chung2021-10-271-8/+8
| | | | | | | | | | | | | | | C0/C1 port is on left/right port respectively for kingoftown. Swap left/right LED to the correct port. BUG=b:202464175 BRANCH=trogdor TEST=make sure that "ectool led left/right auto/white/amber/off" correct. Signed-off-by: Tommy Chung <tommy.chung@quanta.corp-partner.google.com> Change-Id: I4f40ed76978b3b471a5c617d2dbf377146023c6b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3225270 Reviewed-by: Wai-Hong Tam <waihong@google.com>
* kingoftown: Update led configTommy Chung2021-10-271-7/+41
| | | | | | | | | | | | | | | | | | | | | | LED policies for kingoftown are defined in the following BUG tracker. This CL is updated to meet the policies. BUG=b:202464175 BRANCH=trogdor TEST=make sure that C0/C1 LED behaviors correct when 1. DC mode 2. DC mode when system suspend 3. AC mode when charging 4. AC mode when fully charging 5. AC mode when fully charging and system suspend 6. AC mode when battery is not present 7. AC mode and force idle. Signed-off-by: Tommy Chung <tommy.chung@quanta.corp-partner.google.com> Change-Id: I8e0eb03ba209194f2b47d6d32f64b44d1be1af1b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3199437 Reviewed-by: Wai-Hong Tam <waihong@google.com> Reviewed-by: Devin Lu <Devin.Lu@quantatw.com>
* SERVO_V4P1: Use IO-Expander driver specific flagsSam Hurst2021-10-271-2/+2
| | | | | | | | | | | | | | | | | | The TCA64XXA driver controls both the tca6416a and tca6424a io-expanders, so the IO-Expander driver specific flags are used to distinguish between the two. BUG=b:203907721 BRANCH=none TEST=ServoV4p1 flags are not clobbered and IO-Expanders are working Signed-off-by: Sam Hurst <shurst@google.org> Change-Id: I89335845f035aa18d39762cda8ede732e99df3ee Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3239015 Tested-by: Sam Hurst <shurst@google.com> Commit-Queue: caveh jalali <caveh@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* Scout: disable ALS interrupt on known-bad hardwareJoe Tessler2021-10-272-7/+35
| | | | | | | | | | | | | | | | | | | Prototype, Pre-EVT, and EVT board versions incorrectly use a 1.8V interrupt line, which sends a constant interrupt signal (active low) and eventually triggers a watchdog reset. This change ensures the interrupt remains disables for those known-bad board revision. BUG=b:203224828 TEST=Flash on EVT hardware; confirm no boot loop BRANCH=puff Signed-off-by: Joe Tessler <jrt@chromium.org> Change-Id: Ib959c197219b87024ef8fe2e01350b155ab95f44 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3246412 Commit-Queue: Jack Rosenthal <jrosenth@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
View Lorry Controller Status