summaryrefslogtreecommitdiff
path: root/zephyr
Commit message (Collapse)AuthorAgeFilesLines
* Revert "zephyr: Update hooks shim to match ECOS"Jack Rosenthal2021-01-082-12/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e1224e693a33269866c1d8e1af44b325a7f69a72. Reason for revert: chumped CL is responsible for CQ failures Original change's description: > zephyr: Update hooks shim to match ECOS > > At present the hooks declaration in Zephyr doesn't use const but does > use static. This makes it incompatible with ECOS, even if it might be > more strictly correct. > > Update the hook to fix this so that we can build the missing USB code. > > BUG=b:175434113 > BRANCH=none > TEST=make BOARD=volteer > build zephyr on volteer > > Change-Id: Ib8decd11aa2adf85bb760965b0794c54854c41c7 > Signed-off-by: Simon Glass <sjg@chromium.org> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2611895 > Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Bug: b:175434113 Change-Id: I25ef8436ae3a8fc839ce73c82dc04089ce9bac4e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2617116 Reviewed-by: Yuval Peress <peress@chromium.org> Reviewed-by: Simon Glass <sjg@chromium.org> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Yuval Peress <peress@chromium.org> Commit-Queue: Simon Glass <sjg@chromium.org> Tested-by: Jack Rosenthal <jrosenth@chromium.org> Auto-Submit: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Update hooks shim to match ECOSSimon Glass2021-01-072-7/+12
| | | | | | | | | | | | | | | | | | At present the hooks declaration in Zephyr doesn't use const but does use static. This makes it incompatible with ECOS, even if it might be more strictly correct. Update the hook to fix this so that we can build the missing USB code. BUG=b:175434113 BRANCH=none TEST=make BOARD=volteer build zephyr on volteer Change-Id: Ib8decd11aa2adf85bb760965b0794c54854c41c7 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2611895 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* Zephyr: Add Kconfigs for RUNTIME_CONFIGSimon Glass2021-01-072-1/+51
| | | | | | | | | | | | | | | | Add support for USB_PD_TCPC_RUNTIME_CONFIG and USB_MUX_RUNTIME_CONFIG so that we get the correct type for tcpc_config[] and usb_mux[]. BUG=b:175434113 BRANCH=none TEST=make BOARD=volteer -j30 With a zephyr-chrome CL, build volteer on zephyr Signed-off-by: Simon Glass <sjg@chromium.org> Cq-Depend: chromium:2607401 Change-Id: Ic4c982f5d697cd6b61d83f2f12657220a5325094 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2607508 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Add Kconfig for USB PIDSimon Glass2021-01-062-0/+16
| | | | | | | | | | | | | | | | This is the USB product ID and is needed for reporting identity information to connected USB devices. Add a Kconfig for it. BUG=b:175434113 BRANCH=none TEST=with zephyr-chrome CL, build on volteer with full EC usb_pd_policy.c Cq-Depend: chromium:2607398 Change-Id: I71aa1b9110dbe526a31555b16723e31e57c314e9 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2607505 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: Update Kconfig for the remaining console commandsSimon Glass2021-01-061-9/+34
| | | | | | | | | | | | | | | | Use the same format for these existing commands and add some help so that it is clear what they do. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check that each command looks correct Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I146e22f05349d7b2d803ef7879c72420bdc62ce4 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2607456 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: Update USB-C Kconfig for claritySimon Glass2021-01-061-30/+43
| | | | | | | | | | | | | | | | | | Two of the options have a lot of dependencies, so use menuconfig for these. Also move dependencies together in the file to avoid confusion. Drop the 'Enable' language since it just slows down reading of the available options. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the options read a bit better Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I8629ffbe22a1b0d35114b625b01559f167b8b45b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2605814 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Use clearer prompts for config optionsSimon Glass2021-01-061-31/+70
| | | | | | | | | | | | | | | | | | | | | | | At present the useless word 'Enable' appears in lots of the options, making it harder to read what is actually important, the thing being enabled. Also some options have pretty poor help, which does not set a good example for the future. Tidy up these things. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the options read a bit better Change-Id: I3f4c2d21a52e9776b85c27a72701cf3a7e584728 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606578 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Yuval Peress <peress@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Make HAS_TASK_HOSTCMD a Kconfig optionSimon Glass2021-01-063-12/+22
| | | | | | | | | | | | | | | | | | | | This option is implied so we may as well create it. Set it to be visible (and automatically enabled) only if PLATFORM_EC_HOSTCMD is enabled. This mostly mimics the current behaviour. Also select the hostcmd task automatically if PLATFORM_EC_HOSTCMD is enabled, since that feature cannot work without the task. If PLATFORM_EC_HOSTCMD is not enabled there is no need for the task. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the operation of HAS_TASK_HOSTCMD and PLATFORM_EC_HOSTCMD Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I837c6b30bfc9746a17a697b9ee94a7870a01fb56 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606577 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Connect HAS_TASK_CHARGER and PLATFORM_EC_BATTERYSimon Glass2021-01-062-0/+2
| | | | | | | | | | | | | | | | | | | | | These options are linked in that we cannot have one without the other. Select the charger task automatically if PLATFORM_EC_BATTERY is enabled, since that feature cannot work without the task. If PLATFORM_EC_BATTERY is not enabled there is no need for the task. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the operation of HAS_TASK_CHARGER and PLATFORM_EC_BATTERY Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I14ada8716053649ac13ea12a7b06c2ae1702df42 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606576 Reviewed-by: Yuval Peress <peress@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Connect HAS_TASK_POWERBTN, PLATFORM_EC_POWER_BUTTONSimon Glass2021-01-062-1/+3
| | | | | | | | | | | | | | | | | | | | | These options are linked in that we cannot have one without the other. Select the powerbtn task automatically if PLATFORM_EC_POWER_BUTTON is enabled, since that feature cannot work without the task. If PLATFORM_EC_POWER_BUTTON is not enabled there is no need for the task. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the operation of HAS_TASK_POWERBTN and PLATFORM_EC_POWER_BUTTON Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I3e6bb2dccb58009ad4f8c4931dbcb43665327a8b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606575 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Yuval Peress <peress@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Make HAS_TASK_KEYSCAN a Kconfig optionSimon Glass2021-01-063-9/+28
| | | | | | | | | | | | | | | | | | | | | This option is implied so we may as well create it. Set it to be visible (and automatically enabled) only if PLATFORM_EC_KEYBOARD is enabled. This mostly mimics the current behaviour. Also select the keyproto task automatically if PLATFORM_EC_KEYBOARD_PROTOCOL_8042 is enabled, since that feature cannot work without the task. If PLATFORM_EC_KEYBOARD_PROTOCOL_8042 is not enabled there is no need for the task. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the operation of HAS_TASK_KEYSCAN and PLATFORM_EC_KEYBOARD Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I576f2ee3560a307e00d11bc6b8b6c04acb81dd24 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606574 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Make HAS_TASK_CHIPSET a Kconfig optionSimon Glass2021-01-063-10/+28
| | | | | | | | | | | | | | | | This option is implied so we may as well create it. Set it to be visible (and automatically enabled) only if PLATFORM_EC_POWERSEQ is enabled. This mostly mimics the current behaviour. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig Check the operation of HAS_TASK_CHIPSET and PLATFORM_EC_POWERSEQ Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I7dc698c15fa65eb16ec40724a06b67d7e4ddc27d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606573 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: add a fake shared memory implementation for posix-ecJack Rosenthal2021-01-061-0/+5
| | | | | | | | | | | | | | | | | | | | | | CL:2575207 was not tested against posix-ec and caused build issues. posix-ec does not have the _image_ram_end symbol available. This adds a new configuration for a fake 1MB shared memory option, and enables it for posix architecture by default. Note that if this CL (or an alternative fix) cannot be landed in a timely fashion, CL:2575207 and any CLs which depend on it should be reverted so we can enable the CQ again and stop stepping on our own toes. BUG=b:176828988 BRANCH=none TEST=run zephyr-chrome firmware_builder.py Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I1d828f480c6a5da0b162a124c8a5a62ae8afb444 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2611985 Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: usbc: disable by default for devices without a batteryJack Rosenthal2021-01-061-1/+1
| | | | | | | | | | | | | | | | | | | | For devices without a battery (e.g., posix-ec), this code has compile issues. Note that it's still possible to have devices with USB-C but no battery (e.g., Chromeboxes), so this is not a strict "depends on". If we make Chromeboxes in the future with Zephyr then we can either set PLATFORM_EC_USBC manually or re-evaluate (e.g., "default y if CHROMEBOX") BUG=b:176828988 BRANCH=none TEST=less compile errors for posix-ec Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I0fe76199bd75b93c907fd79f571f5951c350b06f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2611984 Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: disable flash storageJack Rosenthal2021-01-061-1/+1
| | | | | | | | | | | | | | This code fails to compile (see b/176828988). Disable by default until this code can be compiled. BUG=b:176828988 BRANCH=none TEST=less compile errors Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I051a7491d88194a212ece0843b7b76611998886c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2611823 Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: Update naming and help for tasksSimon Glass2021-01-051-28/+36
| | | | | | | | | | | | | | | | | | | | | The name is currently too verbose and the help is too brief. Update to make this easier to read and more useful. Drop the task name from 'Stack size' since it is implied and just makes things harder to follow. Also put the tasks into alphabetical order. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig See that the new task options appear with help Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I7514cc730e1a33c60388eb4a0f0ce2c800056499 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606572 Reviewed-by: Yuval Peress <peress@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: add ACPI handlerWealian Liao2020-12-313-0/+59
| | | | | | | | | | | | | | | | | | Add ACPI handler, which will handle the Host data from the ACPI I/O port for X86 AP. For the Zephyr shim, the ACPI handler is invoked by ESPI_PERIPHERAL_HOST_IO event in ESPI_BUS_PERIPHERAL_NOTIFICATION eSPI event. BRANCH=none BUG=b:175217186 TEST=build & boot EC on volteer Cq-Depend: chromium:2603204 Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com> Change-Id: Ic73b113bf863d10c2a9531174aba7b85018986f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2594757 Reviewed-by: Simon Glass <sjg@chromium.org> Commit-Queue: Simon Glass <sjg@chromium.org>
* zephyr: add host command handlerWealian Liao2020-12-313-0/+75
| | | | | | | | | | | | | | | | | | | | | | | The CL include the following: 1. Add host command handler. For the Zephyr shim, the host command handler is invoked by ESPI_PERIPHERAL_EC_HOST_CMD event in ESPI_BUS_PERIPHERAL_NOTIFICATION eSPI event. 2. Add system_reset() empty function for host command build pass in shim/ship/npcx/system.c BRANCH=none BUG=b:175217186 TEST=Get the host command & show on the console for volteer. e.g. [HC 0x0d] [HC 0x0d err 1] Cq-Depend: chromium:2603204 Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com> Change-Id: Ifd70b9a73c3a2404530182cfc4ce5f3ce4038b49 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2594756 Reviewed-by: Simon Glass <sjg@chromium.org> Commit-Queue: Simon Glass <sjg@chromium.org>
* zephyr: shim lpc_update_host_event_status() & lpc_get_memmap_range()Wealian Liao2020-12-311-6/+128
| | | | | | | | | | | | | | | | | | | | The CL include the following: 1. Shim lpc_update_host_event_status() & lpc_get_memmap_range() that needed by host command. 2. Most of the hardware initial functions in lpc_init() move to Zephyr repository eSPI init(espi_npcx_init()). The remaining initial functions which don't implement in Zephyr move to host_command_init HOOK_INIT function. BRANCH=none BUG=b:175217186 TEST=build & boot EC on volteer Signed-off-by: Wealian Liao <whliao@nuvoton.corp-partner.google.com> Change-Id: I519a9587eee1b1d709931600b89b0ab314af4e22 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2594755 Reviewed-by: Simon Glass <sjg@chromium.org> Commit-Queue: Simon Glass <sjg@chromium.org>
* zephyr: improve shimming for unit testsPaul Fagerburg2020-12-314-4/+78
| | | | | | | | | | | | | | | | | | | | | | | * Move the Ztest-related definitions out of test_util and into a shim version of that header. * Add definitions for EC unit tests declaring additional tasks so the task shim will pick them up correctly. * Copy the prng functions into the shimmed test_util. BUG=None BRANCH=None TEST=`make clobber ; make buildall -j ; make runhosttests` Then build the unit tests that have been ported to Ztest, e.g. `zmake configure --test -B build/base32 zephyr/test/base32` for `base32`, `crc`, `hooks`, `i2c`,and `tasks` Signed-off-by: Paul Fagerburg <pfagerburg@google.com> Change-Id: Ia2873a4c2028fcb6960e80f683189def2f3a228d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2605231 Tested-by: Paul Fagerburg <pfagerburg@chromium.org> Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org> Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org> Auto-Submit: Paul Fagerburg <pfagerburg@chromium.org>
* zephyr: Convert the sub-Kconfig files into menusSimon Glass2020-12-304-5/+9
| | | | | | | | | | | | | | | | | | | Use a menu for each of these since it produces a tidier top-level view for the EC. Put the function as the first word to make it easier to follow. For battery and USB-C, use menuconfig so that it works the same way as power sequencing. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig See that the menus appear Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I57e58f94aeaa0783ca69b0c948b7760c2e42adbf Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606571 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Rename the top-level KconfigSimon Glass2020-12-301-2/+7
| | | | | | | | | | | | | | | | | | Update PLATFORM_EC to refer to the shim since that is really what it is brining in. Once we actually have code on the Zephyr side then perhaps we can introduce a distinction here, but that is not the current goal. Also use Chromium OS EC consistently. BUG=b:176449230 BRANCH=none TEST=ninja -C /tmp/z/vol/build-ro menuconfig See that the naming appears Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I11aefa5ba224b22a5bb91c4a4ed146ad5b0c500c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2606570 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: shim the flash accessJunLin2020-12-293-7/+330
| | | | | | | | | | | | | | | | | Implement the flash_physical* APIs by wrapping the cros_flash device driver APIs in zephyr-chrome BRANCH=none BUG=b:174874876 TEST=pass build with related patches. TEST=combine with related patches, use console commands to test all the implemented flash APIs. Signed-off-by: Simon Glass <sjg@chromium.org> Signed-off-by: JunLin <CHLin56@nuvoton.com> Cq-Depend: chromium:2601823, chromium:2598212 Change-Id: I72543fe6290e24b74f7c6bf133599e577c529f08 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2600809
* zephyr: Add the beginnings of a shim for flash accessSimon Glass2020-12-296-0/+158
| | | | | | | | | | | | | | Add myriad Kconfig options to make the flash code compile. This does not yet do anything useful, as we need to connect up to the Zephyr flash device (and need support for this in the npcx too). BUG=b:174873770 BRANCH=none TEST=build for volteer Change-Id: Ib4bed4cdd39cdf33a1d27b39aadb89df491941b3 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575208
* zephyr: Add basic support for shared memorySimon Glass2020-12-294-1/+28
| | | | | | | | | | | | | | | | | | | | | | | On the EC, shared memory is at the top of RAM, above all linker symbols that are located in RAM. Add a simple shim for this. At present it does enough to compile, but does not actually work. Yuval probably knows what should be done here. Add a Kconfig for the 'shmem' command. BUG=b:174873770 BRANCH=none TEST=build and boot on volteer (nothing useful happens) 20-12-04 15:22:49.267 shmem 20-12-04 15:22:50.011 Size:-537684993 20-12-04 15:22:50.011 Used: 0 20-12-04 15:22:50.011 Max: 0 Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I465c8519476e7c4a4cafba2f06630675f396c252 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575207 Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: tasks: Fix collision of shimmed_tasks.hYuval Peress2020-12-284-3/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The issue here was initially found in https://crrev.com/c/2602440 where it was difficult (also not guaranteed to work) to declare custom tasks for tests. The issue was that in zephyr/test/tasks/ the file shimmed_tasks.h was used which is intended to replace zephyr/shim/include/shimmed_tasks.h. So far we've been lucky in that the build was resolving this to the test instance, but this is not guaranteed. This change does the following: 1. If running CONFIG_ZTEST then we add a new CONFIG_HAS_TEST_TASKS option that is defaulted to 'n'. 2. If CONFIG_HAS_TEST_TASKS is set to 'y', then we'll try to include shimmed_test_tasks.h which should be defined by the test and added somewhere in the include path (see zephyr/test/tasks/ for example). 3. We can now define custom tasks for tests. Note that in zephyr we can just use normal tasks via CROS_EC_TASK and don't need to use a special macro for test tasks. BRANCH=none BUG=b:172240633 TEST=zmake testall Signed-off-by: Yuval Peress <peress@chromium.org> Change-Id: Ib99c8ec08923a7e958e045d45d1aa2ec5e841ed5 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2603458 Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org> Commit-Queue: Paul Fagerburg <pfagerburg@chromium.org>
* zephyr: Add initial USB-C supportSimon Glass2020-12-244-0/+587
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This includes a variety of things to get volteer to build with some amount of USB-C support. It seems to charge the battery and prevent booting when the battery is too low. Much work remains. In particular: - check the Kconfig help and add missing help - possibly add a TCPM Kconfig option for grouping - BCI1.2 event handler - TCPMv2 event generator BUG=b:175434113, b:176171475 BRANCH=none TEST=build and run on volteer: 20-12-11 17:43:12.344 uart:~$ tcpci_dump 0 20-12-11 17:44:45.493 VENDOR_ID (0x00) = 0x0451 20-12-11 17:44:45.493 PRODUCT_ID (0x02) = 0x0422 20-12-11 17:44:45.493 BCD_DEV (0x04) = 0x0100 20-12-11 17:44:45.504 TC_REV (0x06) = 0x0011 20-12-11 17:44:45.504 PD_REV (0x08) = 0x2011 20-12-11 17:44:45.504 PD_INT_REV (0x0a) = 0x1010 20-12-11 17:44:45.515 ALERT (0x10) = 0x000f 20-12-11 17:44:45.515 ALERT_MASK (0x12) = 0x007f 20-12-11 17:44:45.526 POWER_STATUS_MASK (0x14) = 0x04 20-12-11 17:44:45.526 FAULT_STATUS_MASK (0x15) = 0x7f 20-12-11 17:44:45.526 EXT_STATUS_MASK (0x16) = 0x00 20-12-11 17:44:45.536 ALERT_EXTENDED_MASK (0x17) = 0x00 20-12-11 17:44:45.536 CONFIG_STD_OUTPUT (0x18) = 0x60 20-12-11 17:44:45.536 TCPC_CTRL (0x19) = 0x11 20-12-11 17:44:45.547 ROLE_CTRL (0x1a) = 0x1a 20-12-11 17:44:45.547 FAULT_CTRL (0x1b) = 0x06 20-12-11 17:44:45.558 POWER_CTRL (0x1c) = 0x70 20-12-11 17:44:45.558 CC_STATUS (0x1d) = 0x13 20-12-11 17:44:45.558 POWER_STATUS (0x1e) = 0x8c 20-12-11 17:44:45.569 FAULT_STATUS (0x1f) = 0x00 20-12-11 17:44:45.569 EXT_STATUS (0x20) = 0x00 20-12-11 17:44:45.580 ALERT_EXT (0x21) = 0x00 20-12-11 17:44:45.580 DEV_CAP_1 (0x24) = 0x1e98 20-12-11 17:44:45.580 DEV_CAP_2 (0x26) = 0x00c5 20-12-11 17:44:45.592 STD_INPUT_CAP (0x28) = 0x00 20-12-11 17:44:45.592 STD_OUTPUT_CAP (0x29) = 0x00 20-12-11 17:44:45.592 CONFIG_EXT_1 (0x2a) = 0x00 20-12-11 17:44:45.602 MSG_HDR_INFO (0x2e) = 0x02 20-12-11 17:44:45.602 RX_DETECT (0x2f) = 0x00 20-12-11 17:44:45.613 RX_BYTE_CNT (0x30) = 0x00 20-12-11 17:44:45.613 RX_BUF_FRAME_TYPE (0x31) = 0x00 20-12-11 17:44:45.613 TRANSMIT (0x50) = 0x00 20-12-11 17:44:45.624 VBUS_VOLTAGE (0x70) = 0x0000 20-12-11 17:44:45.624 VBUS_SINK_DISCONNECT_THRESH (0x72) = 0x0000 20-12-11 17:44:45.633 VBUS_STOP_DISCHARGE_THRESH (0x74) = 0x0000 20-12-11 17:44:45.633 VBUS_VOLTAGE_ALARM_HI_CFG (0x76) = 0x0000 20-12-11 17:44:45.633 VBUS_VOLTAGE_ALARM_LO_CFG (0x78) = 0x0000 20-12-11 17:44:49.544 uart:~$ ppc_dump 0 20-12-11 17:45:09.838 FUNC_SET1 [50h] = 0x0b 20-12-11 17:45:09.838 FUNC_SET2 [51h] = 0x18 20-12-11 17:45:09.838 FUNC_SET3 [52h] = 0x6a 20-12-11 17:45:09.838 FUNC_SET4 [53h] = 0xfe 20-12-11 17:45:09.838 FUNC_SET5 [54h] = 0x37 20-12-11 17:45:09.838 FUNC_SET6 [55h] = 0xc1 20-12-11 17:45:09.847 FUNC_SET7 [56h] = 0x70 20-12-11 17:45:09.847 FUNC_SET8 [57h] = 0xbd 20-12-11 17:45:09.847 FUNC_SET9 [58h] = 0x34 20-12-11 17:45:09.847 FUNC_SET10 [59h] = 0x70 20-12-11 17:45:09.847 FUNC_SET11 [5ah] = 0x24 20-12-11 17:45:09.858 FUNC_SET12 [5bh] = 0x70 20-12-11 17:45:09.858 INT_STATUS_REG1 [2fh] = 0x00 20-12-11 17:45:09.858 INT_STATUS_REG2 [30h] = 0x80 20-12-11 17:45:09.858 INT_STATUS_REG3 [31h] = 0x09 20-12-11 17:45:09.858 INT_STATUS_REG4 [32h] = 0x08 20-12-11 17:45:09.862 INT_TRIP_RISE_REG1 [20h] = 0x80 20-12-11 17:45:09.862 INT_TRIP_RISE_REG2 [21h] = 0x00 20-12-11 17:45:09.874 INT_TRIP_RISE_REG3 [22h] = 0x05 20-12-11 17:45:09.874 INT_TRIP_FALL_REG1 [23h] = 0x80 20-12-11 17:45:09.874 INT_TRIP_FALL_REG2 [24h] = 0x00 20-12-11 17:45:09.879 INT_TRIP_FALL_REG3 [25h] = 0x05 20-12-11 17:45:09.879 INT_MASK_RISE_REG1 [26h] = 0xef 20-12-11 17:45:09.879 INT_MASK_RISE_REG2 [27h] = 0xe1 20-12-11 17:45:09.892 INT_MASK_RISE_REG3 [28h] = 0xff 20-12-11 17:45:09.892 INT_MASK_FALL_REG1 [29h] = 0xff 20-12-11 17:45:09.892 INT_MASK_FALL_REG2 [2ah] = 0xef 20-12-11 17:45:09.892 INT_MASK_FALL_REG3 [2bh] = 0x8f ... 20-12-11 18:21:17.576 battery 20-12-11 18:21:17.933 Status: 0x0080 INIT 20-12-11 18:21:17.933 Param flags:00000003 20-12-11 18:21:17.933 Temp: 0x0b64 = %.1d K (%.1d C) 20-12-11 18:21:17.933 V: 0x2aa8 = 10920 mV 20-12-11 18:21:17.933 V-desired: 0x3390 = 13200 mV 20-12-11 18:21:17.944 I: 0x00bd = 189 mA(CHG) 20-12-11 18:21:17.944 I-desired: 0x0a19 = 2585 mA 20-12-11 18:21:17.944 Charging: Allowed 20-12-11 18:21:17.944 Charge: 3 % 20-12-11 18:21:17.948 Manuf: LG 20-12-11 18:21:17.948 Device: AC17A8 20-12-11 18:21:17.953 Chem: LIO 20-12-11 18:21:17.960 Serial: 0xb754 20-12-11 18:21:17.960 V-design: 0x2d1e = 11550 mV 20-12-11 18:21:17.965 Mode: 0x6001 20-12-11 18:21:17.965 Abs charge:3 % 20-12-11 18:21:17.974 Remaining: 139 mAh 20-12-11 18:21:17.974 Cap-full: 5093 mAh (4991 mAh with 98 % compensation) 20-12-11 18:21:17.983 Display: 0.0 % 20-12-11 18:21:17.983 Design: 5360 mAh 20-12-11 18:21:17.993 Time-full: 26h:21 20-12-11 18:21:17.993 Empty: 0h:0 Cq-Depend: chromium:2585918 Change-Id: I0d852169c9cbbc70603944b70f4708111abd5b42 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2587225 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Enable the charger taskSimon Glass2020-12-243-0/+27
| | | | | | | | | | | | | | | This supports monitoring and charging the battery. Enable it for Zephyr. BUG=b:175248556 BRANCH=none TEST=build and run zephyr on volteer No visible change, except more code is compiled in Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I21ff976ba2f5de6e8f069e30028770eda408041a Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2600229 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Enable building of the battery fuel gaugeSimon Glass2020-12-243-1/+17
| | | | | | | | | | | | | | Bring in this file needed for monitoring a battery. BUG=b:175248556 BRANCH=none TEST=build and run zephyr on volteer No visible change, except more code is compiled in Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I49a3960c62153ad0ea5c5f8121d0aa0d35e7ffe4 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2600228 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Add a stub for system_is_locked()Simon Glass2020-12-231-0/+11
| | | | | | | | | | | | | | Add stubs for this and system_get_image_copy(). These should be implemented when we have the underlying functionality available. BUG=b:176171847 BRANCH=none TEST=build on zephyr Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: Iddee3b94cdb9e93a52718e49aea2750f77f5ea2e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2601141 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Add support for battery presenceSimon Glass2020-12-232-5/+43
| | | | | | | | | | | | | | | Add the battery-presence settings so that a battery can be detected. BUG=b:175248556 BRANCH=none TEST=build and run zephyr on volteer No visible change, except more code is compiled in Cq-Depend: chromium:2599491 Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I9a7086e6df67f7e9fe3aedb6a15f891e71b58d49 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2600337 Reviewed-by: Keith Short <keithshort@chromium.org>
* tcpm: Move tcpm.h into an include directorySimon Glass2020-12-221-1/+2
| | | | | | | | | | | | | | | | | | This header file is used from quite a few files, relying on the EC build system to find includes in the driver/tcpm directory. For Zephyr we don't want to add that as an include. It makes more sense for header files to be in an include directory, so move it and fix up the users. BUG=b:175434113 BRANCH=none TEST=build Zephyr and ECOS on volteer Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I5851914b1a7d3fdc1ba911c0fbe9046afbaf6f5d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2597985 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Fix missing GPIO header in keyboard_rawSimon Glass2020-12-171-0/+1
| | | | | | | | | | | | | Fix the warning about npcx_get_gpio_dev() not being declared. BUG=b:174871569 BRANCH=none TEST=build for zephyr, see the warning gone Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: Ic0ef6e85baf884fa2c0d0a09044fd1da606e4bb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2595221 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Enable isl9241 for VolteerSimon Glass2020-12-172-0/+9
| | | | | | | | | | | | | | | Add configs and build rules to enable this driver. A few issues are left to clean up later. BUG=b:175248556 BRANCH=none TEST=build for volteer Change-Id: If2c5427d016ae8cddcc25926668011aa11673091 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2589562 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zmake: change projects to chroot-supported toolchainsJack Rosenthal2020-12-165-5/+5
| | | | | | | | | | | | | | | | | Switch all zmake.yaml files to chroot-supported toolchains (chroot llvm for these files) so that we can run "zmake testall" without grabbing the Zephyr SDK. BUG=b:175248887 BRANCH=none TEST=zmake testall Cq-Depend: chromium:2590992 Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I6c57ca734c18bdcc159b185709a3dd22a52a8b52 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2591628 Commit-Queue: Simon Glass <sjg@chromium.org> Reviewed-by: Simon Glass <sjg@chromium.org>
* zephyr: put project(...) after find_package(Zephyr)Jack Rosenthal2020-12-155-5/+5
| | | | | | | | | | | | | | | | When including the Zephyr source, this will change the values of CMAKE_C_COMPILER and CMAKE_CXX_COMPILER, causing cmake to execute again having forgotten about some of the command (since these are cached variables). BUG=b:175248887 BRANCH=none TEST=zmake testall, no "cached variables have changed" from cmake Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I7eb1004060b0ba51d8c336dfe507ad747abe80ee Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2591627 Reviewed-by: Simon Glass <sjg@chromium.org>
* zephyr: clear a few compiler warningsJack Rosenthal2020-12-151-0/+1
| | | | | | | | | | | | BUG=b:174871569 BRANCH=none TEST=compile, observe less warnings Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: Id45ebc740344ec9261e28454a966e22a1cd6f629 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2591788 Commit-Queue: Keith Short <keithshort@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: fix misspeling in eSPI shimDawid Niedzwiecki2020-12-151-1/+1
| | | | | | | | | | | | | | Use CONFIG_PLATFORM_EC_POWERSEQ instead of CONFIG_PLATFORM_EC_POWERseQ. BUG=b:171815541 BRANCH=none TEST=volteer power sequencing to S0 Signed-off-by: Dawid Niedzwiecki <dn@semihalf.com> Change-Id: If61717da8e0d80e1d55ea3d7f627105909c1650e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2592491 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
* Reland "zephyr: Add battery and smart battery options"Keith Short2020-12-144-0/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a reland of ac4f512e60af42efe693e6955c110a7908090e56 Original change's description: > zephyr: Add battery and smart battery options > > Shim in battery and smart battery support. > > BUG=b:175248556 > BRANCH=none > TEST=boot on volteer, run 'battery' command. > TEST=run 'ninja menuconfig' and check KConfig help text. > 20-12-10 20:08:12.778 battery > 20-12-10 20:08:13.509 Status: 0x02c0 DCHG INIT RC > 20-12-10 20:08:13.529 Param flags:00000003 > 20-12-10 20:08:13.541 Temp: 0x0b63 = %.1d K (%.1d C) > 20-12-10 20:08:13.541 V: 0x2a1e = 10782 mV > 20-12-10 20:08:13.541 V-desired: 0x3390 = 13200 mV > 20-12-10 20:08:13.541 I: 0x0000 = 0 mA > 20-12-10 20:08:13.550 I-desired: 0x0a19 = 2585 mA > 20-12-10 20:08:13.550 Charging: Allowed > 20-12-10 20:08:13.550 Charge: 0 % > 20-12-10 20:08:13.550 Manuf: LG > 20-12-10 20:08:13.555 Device: AC17A8 > 20-12-10 20:08:13.573 Chem: LIO > 20-12-10 20:08:13.573 Serial: 0xb754 > 20-12-10 20:08:13.573 V-design: 0x2d1e = 11550 mV > 20-12-10 20:08:13.573 Mode: 0x6001 > 20-12-10 20:08:13.573 Abs charge:0 % > 20-12-10 20:08:13.573 Remaining: 0 mAh > 20-12-10 20:08:13.577 Cap-full: 4932 mAh (4833 mAh with 98 % compensation) > 20-12-10 20:08:13.585 Design: 5360 mAh > 20-12-10 20:08:13.594 Time-full: 0h:0 > 20-12-10 20:08:13.594 Empty: 0h:0 > > Change-Id: Ie782e75ee4027ab2a5c6a0ae7f4ad81e9c360711 > Signed-off-by: Simon Glass <sjg@chromium.org> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575199 Bug: b:175248556 TEST=make buildall, which fails with: /opt/coreboot-sdk/lib/gcc/nds32le-elf/8.3.0/../../../../nds32le-elf/bin/ld: __flash_dma_size < IT83XX_ILM_BLOCK_SIZE BRANCH=none Change-Id: I318fb13ba91485cd7d876b2294f20d748f47e0fd Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2585931 Tested-by: Simon Glass <sjg@chromium.org> Auto-Submit: Simon Glass <sjg@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
* task_set_event: remove the wait argumentDawid Niedzwiecki2020-12-142-8/+7
| | | | | | | | | | | | | | | | | | | | There is an option in the task_set_event function which force the calling task to wait for an event. However, the option is never used thus remove it. This also will help in the Zephyr migration process. BUG=b:172360521 BRANCH=none TEST=make buildall Signed-off-by: Dawid Niedzwiecki <dn@semihalf.com> Change-Id: Ic152fd3d6862d487bcc0024c48d136556c0b81bc Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2521599 Reviewed-by: Jett Rink <jettrink@chromium.org> Reviewed-by: Tom Hughes <tomhughes@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
* Revert "zephyr: Add battery and smart battery options"Eric Yilun Lin2020-12-143-73/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ac4f512e60af42efe693e6955c110a7908090e56. Reason for revert: This CL stops chipset_task forever, revert this to boot AP. Original change's description: > zephyr: Add battery and smart battery options > > Shim in battery and smart battery support. > > BUG=b:175248556 > BRANCH=none > TEST=boot on volteer, run 'battery' command. > TEST=run 'ninja menuconfig' and check KConfig help text. > 20-12-10 20:08:12.778 battery > 20-12-10 20:08:13.509 Status: 0x02c0 DCHG INIT RC > 20-12-10 20:08:13.529 Param flags:00000003 > 20-12-10 20:08:13.541 Temp: 0x0b63 = %.1d K (%.1d C) > 20-12-10 20:08:13.541 V: 0x2a1e = 10782 mV > 20-12-10 20:08:13.541 V-desired: 0x3390 = 13200 mV > 20-12-10 20:08:13.541 I: 0x0000 = 0 mA > 20-12-10 20:08:13.550 I-desired: 0x0a19 = 2585 mA > 20-12-10 20:08:13.550 Charging: Allowed > 20-12-10 20:08:13.550 Charge: 0 % > 20-12-10 20:08:13.550 Manuf: LG > 20-12-10 20:08:13.555 Device: AC17A8 > 20-12-10 20:08:13.573 Chem: LIO > 20-12-10 20:08:13.573 Serial: 0xb754 > 20-12-10 20:08:13.573 V-design: 0x2d1e = 11550 mV > 20-12-10 20:08:13.573 Mode: 0x6001 > 20-12-10 20:08:13.573 Abs charge:0 % > 20-12-10 20:08:13.573 Remaining: 0 mAh > 20-12-10 20:08:13.577 Cap-full: 4932 mAh (4833 mAh with 98 % compensation) > 20-12-10 20:08:13.585 Design: 5360 mAh > 20-12-10 20:08:13.594 Time-full: 0h:0 > 20-12-10 20:08:13.594 Empty: 0h:0 > > Change-Id: Ie782e75ee4027ab2a5c6a0ae7f4ad81e9c360711 > Signed-off-by: Simon Glass <sjg@chromium.org> > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575199 Bug: b:175248556 Change-Id: I66086d8fee46d5fd02d26938468a7d76dab71c6e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2589140 Reviewed-by: Eric Yilun Lin <yllin@chromium.org> Commit-Queue: Eric Yilun Lin <yllin@chromium.org> Tested-by: Eric Yilun Lin <yllin@chromium.org>
* zephyr: Update the I2C tableSimon Glass2020-12-111-7/+2
| | | | | | | | | | | | | | | | | | | | | | | | Something has happened since this code was written and now it doesn't work. The current code checks the status of each of the named-i2c-ports nodes, which always seems to return 0, then ends up searching for a device called "". Obviously this code did work and should work, but I'm uploading this just because it makes the battery command work for me: On the other hand, I don't think we need to check the status. After all, we are searching for a device by its node label and if the node is disabled it will not have a device anyway. BUG=b:175248556 BRANCH=none TEST=verify with debugging that the i2c_devices[] array is now inited correctly Signed-off-by: Simon Glass <sjg@chromium.org> Change-Id: I709421be86027e0e6166a7658fe59ba68783cbe7 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2586614 Reviewed-by: Yuval Peress <peress@chromium.org>
* zephyr: Add battery and smart battery optionsKeith Short2020-12-113-0/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Shim in battery and smart battery support. BUG=b:175248556 BRANCH=none TEST=boot on volteer, run 'battery' command. TEST=run 'ninja menuconfig' and check KConfig help text. 20-12-10 20:08:12.778 battery 20-12-10 20:08:13.509 Status: 0x02c0 DCHG INIT RC 20-12-10 20:08:13.529 Param flags:00000003 20-12-10 20:08:13.541 Temp: 0x0b63 = %.1d K (%.1d C) 20-12-10 20:08:13.541 V: 0x2a1e = 10782 mV 20-12-10 20:08:13.541 V-desired: 0x3390 = 13200 mV 20-12-10 20:08:13.541 I: 0x0000 = 0 mA 20-12-10 20:08:13.550 I-desired: 0x0a19 = 2585 mA 20-12-10 20:08:13.550 Charging: Allowed 20-12-10 20:08:13.550 Charge: 0 % 20-12-10 20:08:13.550 Manuf: LG 20-12-10 20:08:13.555 Device: AC17A8 20-12-10 20:08:13.573 Chem: LIO 20-12-10 20:08:13.573 Serial: 0xb754 20-12-10 20:08:13.573 V-design: 0x2d1e = 11550 mV 20-12-10 20:08:13.573 Mode: 0x6001 20-12-10 20:08:13.573 Abs charge:0 % 20-12-10 20:08:13.573 Remaining: 0 mAh 20-12-10 20:08:13.577 Cap-full: 4932 mAh (4833 mAh with 98 % compensation) 20-12-10 20:08:13.585 Design: 5360 mAh 20-12-10 20:08:13.594 Time-full: 0h:0 20-12-10 20:08:13.594 Empty: 0h:0 Change-Id: Ie782e75ee4027ab2a5c6a0ae7f4ad81e9c360711 Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575199
* zephyr: Fix 'edefines' typoSimon Glass2020-12-111-1/+1
| | | | | | | | | | | | | | Fix a typo that crept into the Kconfig. BRANCH=none BUG=none TEST=build for volteer Change-Id: I57634da26ca94aba3b80d5f4c10d75b250a9ea2c Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2585064 Reviewed-by: Keith Short <keithshort@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: Sort file list by config optionKeith Short2020-12-101-10/+17
| | | | | | | | | | | | | | Sort the cmake file list by the config option name. BRANCH=none BUG=none TEST=build and boot on volteer Signed-off-by: Keith Short <keithshort@chromium.org> Change-Id: I0c66524b990bcc24ac3d19b260daf7ff070861cb Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2582279 Commit-Queue: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass <sjg@chromium.org>
* zephyr: remove dependency on chip specific code for npcxYuval Peress2020-12-108-83/+50
| | | | | | | | | | | | | BRANCH=none BUG=b:175249000 TEST=zmake testall Cq-Depend: chromium:2583272 Signed-off-by: Yuval Peress <peress@chromium.org> Change-Id: Ie50e56df60e060c3741013912ab46d807fc5e417 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2582819 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: enable CONFIG_HOSTCMD_ESPIJack Rosenthal2020-12-092-0/+24
| | | | | | | | | | | | | | | | | | This enables CONFIG_HOSTCMD_ESPI when we have ESPI and host commands enabled (the case with volteer). There were two functions stubbed out that we will leave to Nuvoton to implement. BUG=b:175217186,b:172678200 BRANCH=none TEST=boot volteer Signed-off-by: Jack Rosenthal <jrosenth@chromium.org> Change-Id: I9b6a8c7c60d1a1018292a207a61c583fd171e546 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2582445 Commit-Queue: Simon Glass <sjg@chromium.org> Reviewed-by: Simon Glass <sjg@chromium.org>
* zephyr: fix build for keyboard shimYuval Peress2020-12-056-14/+121
| | | | | | | | | | | | | | | | | | | | | See platform/ec/common/build.mk for reference. The use of keyboard_scan.c should only be allowed when HAS_TASK_KEYSCAN is defined. As such, we should always set HAS_TASK_KEYSCAN to 1 if CONFIG_PLATFORM_EC_KEYBOARD is defined. Further, CROS_EC_TASK_LIST was defined per project and it doesn't need to be, we can instead define it once for all boards depending on which tasks are set to 1 in shimmed_tasks.h Note that shimmed_tasks.h can still be used in tests. BRANCH=none BUG=none TEST=zmake testall Cq-Depend: chromium:2566421 Signed-off-by: Yuval Peress <peress@chromium.org> Change-Id: Id0ed49dd49e3c4eb2ac23184cf943c91dcd261eb Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2567560
* zephyr: Sort the options a bitSimon Glass2020-12-053-112/+121
| | | | | | | | | | | | | | | | | | | We have quite a few options already and plan to add more. Sort them to make it easier to see what is there. The I2C enum isn't really connected with the basic CONFIG munging done in config_chip.h so move it to i2c.h instead. Fix the 'keybaord' typo while we are here. BRANCH=none BUG=none TEST=build and boot on volteer Change-Id: Id03fe44ff859736f1e2f86231ca1e8d9dc4c50af Signed-off-by: Simon Glass <sjg@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2575198 Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* zephyr: add CBI module to zephyr optionsJett Rink2020-12-043-0/+40
| | | | | | | | | | | | | | Shim in CBI module and necessary dependencies. BRANCH=none BUG=b:168032342,b:168032589 TEST=type `cbi` on volteer Signed-off-by: Jett Rink <jettrink@chromium.org> Change-Id: Ic08f472f8d52cd4521b78f819ea22e337cccc6f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2572736 Reviewed-by: Simon Glass <sjg@chromium.org> Commit-Queue: Simon Glass <sjg@chromium.org>
View Lorry Controller Status