summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* kinox: Initial EC imagestabilize-quickfix-14526.91.B-mainstabilize-14526.89.B-mainstabilize-14526.84.B-mainstabilize-14526.73.B-mainstabilize-14526.67.B-mainstabilize-14526.57.B-mainrelease-R100-14526.B-mainDtrain Hsu2022-02-1713-0/+1610
| | | | | | | | | | | | | | | | | | Create the initial EC image for the kinox variant by copying the brask reference board EC files into a new directory named for the variant. (Auto-Generated by create_initial_ec_image.sh version 1.5.0). BUG=b:215049181 BRANCH=None TEST=make BOARD=kinox Signed-off-by: Dtrain Hsu <dtrain_hsu@compal.corp-partner.google.com> Change-Id: I604216664a0efc690ba9cd2c0c071a0e4650b228 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3448054 Reviewed-by: Zhuohao Lee <zhuohao@chromium.org> Commit-Queue: Zhuohao Lee <zhuohao@chromium.org>
* adlrvp: remove pull up set for EC servo uart linesDeepti Deshatty2022-02-171-1/+1
| | | | | | | | | | | | | | | Removed GPIO_PULL_UP set for the EC servo uart lines. With this change suzy-q EC logs Tx works fine on rvp. BUG=none BRANCH=none TEST=Verified Micro servo and SuzyQ EC logs on ADL-N Rvp Signed-off-by: Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com> Change-Id: Id2b6be2e8d30776f42d25d353b5545e294d8f71c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3282978 Reviewed-by: Vijay P Hiremath <vijay.p.hiremath@intel.com> Reviewed-by: caveh jalali <caveh@chromium.org>
* rt9490: implement dump_registersTing Shen2022-02-171-0/+34
| | | | | | | | | | | | | | | Support printing some status registers and ADCs for HW testing. BUG=b:194967754 TEST=manually run `charger dump` BRANCH=none Signed-off-by: Ting Shen <phoenixshen@google.com> Change-Id: Ic493907fe1983af974b9f7d835982944c33cdab2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3351726 Reviewed-by: Eric Yilun Lin <yllin@google.com> Commit-Queue: Ting Shen <phoenixshen@chromium.org> Tested-by: Ting Shen <phoenixshen@chromium.org>
* zephyr: Create Kconfig.pd_console_cmdSam Hurst2022-02-172-50/+60
| | | | | | | | | | | | | | Split PD Console CMD Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_console_cmd. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: If9bea6a2bc623a6d00e26be30962d518ae93e990 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445496 Reviewed-by: Keith Short <keithshort@chromium.org>
* nipperkin: update gpio enable/disable sequenceZick Wei2022-02-171-2/+2
| | | | | | | | | | | | | | | | We found a leakage during enable HDMI retimer, this CL update HDMI retimer enable sequence to prevent leakage. BUG=b:218975937 BRANCH=none TEST=make BOARD=nipperkin Signed-off-by: Zick Wei <zick.wei@quanta.corp-partner.google.com> Change-Id: I9513027291ef545350c0d768b9e1aa0741e170ea Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3467377 Reviewed-by: Rob Barnes <robbarnes@google.com> Reviewed-by: Diana Z <dzigterman@chromium.org>
* redrix: Add SSFC field for 2nd LID sensorDevin Lu2022-02-175-0/+111
| | | | | | | | | | | | | This patch adds SSFC field to support the 2nd LID sensor. BUG=b:219450750 BRANCH=none TEST=On Redix. Set SSFC and make sure the LID sensor works normally. Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: I5435f3d8c44745c7e53d6e8838ed31b6d798d9ef Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459595 Reviewed-by: caveh jalali <caveh@chromium.org>
* baseboard/brya: Add SSFC entryDevin Lu2022-02-172-0/+12
| | | | | | | | | | | | | This patch adds SSFC entry for board variants. BUG=b:219450750 BRANCH=none TEST=buildall Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: I15790834cd9a4f6b9740c097aa2de8f71e628871 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459594 Reviewed-by: caveh jalali <caveh@chromium.org>
* vell: Set battery current Prochot# thresholdDevin Lu2022-02-171-0/+7
| | | | | | | | | | | | | | Vell battery over-discharging current is 10.5A. This patch set battery current Prochot# threshold to 10.5A. BUG=b:216391312 BRANCH=none TEST=On Vell. Check the DC PROCHOT threshold is correct. Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: Ia91c61c8ff7705dec0ebf1f8c8eb8938a5ee56c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3450002 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* vell: Change battery/AC current sense valueDevin Lu2022-02-171-2/+2
| | | | | | | | | | | BUG=b:216391312 BRANCH=none TEST=On Vell. Battery charging current ramp to 6.4A. Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: Ie3805b5214b9ecf977e4911201504376948a9f1c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3450001 Reviewed-by: Boris Mittelberg <bmbm@google.com>
* nissa: correct pin definition for PP1050 monitoringPeter Marheine2022-02-174-7/+12
| | | | | | | | | | | | | | | | | This pin is used for analog monitoring of PP1050_PROC generated by the SoC, not as a power-good input. An entry is added to the named-adc enumeration because PP1050_PROC has not been used before and a legacy enum value is required even though the code using this analog channel will be Zephyr-first. BUG=b:203446068,b:219634193 TEST=EC_VSENSE_PP1050_PROD channel appears in `adc` output on Nivviks BRANCH=none Signed-off-by: Peter Marheine <pmarheine@chromium.org> Change-Id: I26e0d7d8e3dbe0a2643b789cc5c6891ada710062 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3463101 Reviewed-by: Andrew McRae <amcrae@google.com>
* zmake: Add some python type hintsJeremy Bettis2022-02-171-60/+61
| | | | | | | | | | | | | | | | | | Some project references were not auto-completing in VSCode, so I added type hints to help. BRANCH=None BUG=None TEST=None Signed-off-by: Jeremy Bettis <jbettis@google.com> Change-Id: I0819151f2053e0daf1012874beb72c112e7a80ef Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3462632 Reviewed-by: Abe Levkoy <alevkoy@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Commit-Queue: Jeremy Bettis <jbettis@chromium.org> Auto-Submit: Jeremy Bettis <jbettis@chromium.org> Tested-by: Jeremy Bettis <jbettis@chromium.org>
* zephyr: Fix broken tests when coverage is enabledJeremy Bettis2022-02-171-1/+1
| | | | | | | | | | | | | | | | | | | | | While zmake testall was passing, zmake coverage was not. Probably because of branch coverage calculation, if an if statement includes both a const portion and a non-const portion, the const portion must be first, or it won't get optimized out. BRANCH=None BUG=b:219862697 TEST=zmake -j8 coverage build/zcoverage Signed-off-by: Jeremy Bettis <jbettis@google.com> Change-Id: I97c9aeab737702541b94b7867e02f943beb559c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3470261 Commit-Queue: Jeremy Bettis <jbettis@chromium.org> Tested-by: Jeremy Bettis <jbettis@chromium.org> Auto-Submit: Jeremy Bettis <jbettis@chromium.org> Reviewed-by: Al Semjonovs <asemjonovs@google.com> Commit-Queue: Al Semjonovs <asemjonovs@google.com>
* driver/isl9241: Correct charger informationDevin Lu2022-02-162-6/+11
| | | | | | | | | | | | | | | This patch re-scale the maximum/minimum/step value base current sense value resistor. BUG=b:216391312 BRANCH=none TEST=buildall Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: Ic6426aa416964f2a369ecddb87ce74722527abd3 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3450000 Reviewed-by: caveh jalali <caveh@chromium.org> Commit-Queue: caveh jalali <caveh@chromium.org>
* pazquel: Change PVC mode of DA9313 to improve high frequency noiseYunlong Jia2022-02-162-0/+33
| | | | | | | | | | | | | | | | | | | | Our team found the source of high frequency noise coming from the DA9313. Change the DA9313 PVC mode as follows: S0 -> S3 : PVC operates in auto frequency mode S3 -> S0 : PVC operates in fixed frequency mode and no more high frequency noise heard BUG=b:218973914 BRANCH=trogdor TEST=test again and no more high frequency noise heard Signed-off-by: Yunlong Jia <yunlong.jia@ecs.corp-partner.google.com> Change-Id: If6ef403ea80dc1a006fd259d58ced7373b39c0ca Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3455503 Reviewed-by: Bob Moragues <moragues@chromium.org> Reviewed-by: Wai-Hong Tam <waihong@google.com> Commit-Queue: Wai-Hong Tam <waihong@google.com>
* homestar/mrbland/wormdingler: Add support PS8755Siyu Qin2022-02-163-2/+7
| | | | | | | | | | | | | | | | Import PS8755 as the 2rd source for TCPC chip. BUG=b:218786366 BRANCH=trogdor TEST=make BOARD=homestar/mrbland/wormdingler -j Signed-off-by: Siyu Qin <qinsiyu@huaqin.corp-partner.google.com> Change-Id: I01185737e55bff93cea65fe0667344841231d30b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3463103 Reviewed-by: wen zhang <zhangwen6@huaqin.corp-partner.google.com> Reviewed-by: Bob Moragues <moragues@chromium.org> Reviewed-by: Wai-Hong Tam <waihong@google.com> Commit-Queue: Wai-Hong Tam <waihong@google.com>
* Skyrim: implement battery parameterOwen_Ou2022-02-164-0/+62
| | | | | | | | | | | | | | | | | | | Setup battery parameter for skyrim. BUG=b:219670453 BRANCH=none TEST=1. zmake configure -b skyrim 2. ec can read battery info(ec log) 3. battery can charge & discharge 4. battery can cutoff Signed-off-by: Owen_Ou <Owen_Ou@compal.corp-partner.google.com> Change-Id: I94d522e75b464c55cd7b71a5e7e68493513822c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3463105 Tested-by: Owen Ou <owen_ou@compal.corp-partner.google.com> Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com> Reviewed-by: Diana Z <dzigterman@chromium.org> Commit-Queue: Diana Z <dzigterman@chromium.org>
* Skyrim: Enable RSMRST delayDiana Z2022-02-161-0/+1
| | | | | | | | | | | | | | We need to delay setting SOC_RSMRST_L high by 10ms after EN_PWR_S5 goes high, so enable CONFIG_PLATFORM_EC_POWERSEQ_RSMRST_DELAY. BRANCH=None BUG=b:218867363 TEST=on skyrim, observe correct T1 timing Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: If7a3794ebacced590d9439ba2fb60dbf2dbc3ed5 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3456685 Reviewed-by: Keith Short <keithshort@chromium.org>
* Skyrim: Remove pull-up on 6 axis interruptDiana Z2022-02-163-3/+6
| | | | | | | | | | | | | | | Remove this pull-up as it's causing leakage. Boards should be reworked with an external pull-up for this interrupt line to work. BRANCH=None BUG=b:218536811 TEST=verify no leakage is seen on skyrim Signed-off-by: Diana Z <dzigterman@chromium.org> Change-Id: I959d0aee627d5dbef6c0283376955192734aeb09 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3453165 Tested-by: Charlie Costakis <ccostakis@google.com> Reviewed-by: Rob Barnes <robbarnes@google.com>
* zephyr: Create Kconfig.pd_vbus_detectionSam Hurst2022-02-162-29/+39
| | | | | | | | | | | | | | Split USBA Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_vbus_detection. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: I263b7f8998dc28e1c19c860830a47301ef66a578 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3446460 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Create Kconfig.pd_dischargeSam Hurst2022-02-162-33/+43
| | | | | | | | | | | | | | Split PD VBUS Discharge Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_discharge. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: I52536f70d6a0d833a0d0980f361adf0cabd6c1f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445459 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Disable eeprom simulator for driver testsAndrew McRae2022-02-161-0/+1
| | | | | | | | | | | | | | | | The EEPROM simulator (enabled by default on posix builds) creates a file called eeprom.bin in the current directory. BUG=b:219832922 TEST=zmake configure --test test-drivers BRANCH=none Signed-off-by: Andrew McRae <amcrae@google.com> Change-Id: I2bea3fd27d58b08ae37a45dd6a09dacebbd3861a Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466679 Reviewed-by: Aaron Massey <aaronmassey@google.com> Reviewed-by: Jeremy Bettis <jbettis@chromium.org> Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
* Reland "zephyr: Remove unused enum-name entries"Andrew McRae2022-02-1612-92/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 0f6242e8bd99e0b1fe4d2f15e95bf48eb7daa516. Someone added a new reference to one of the removed enums. Reason for revert: Fixed build. BUG=b:216466985 TEST=zmake testall BRANCH=none Original change's description: > Revert "zephyr: Remove unused enum-name entries" > > This reverts commit 112bc3616b55421af4ae5003c542e9a425128650. > > Reason for revert: suspected at fault for postsubmit failures > https://ci.chromium.org/p/chromeos/builders/postsubmit/firmware-zephyr-postsubmit/14846 > > Original change's description: > > zephyr: Remove unused enum-name entries > > > > After removal of the unsupported projects, > > take another run through the enum-names to > > remove the unused entries. > > > > BUG=b:216466985 > > TEST=zmake testall > > BRANCH=none > > > > Signed-off-by: Andrew McRae <amcrae@google.com> > > Change-Id: Id652c4fda7dd64a59b3776bb52640413a31e4e3e > > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3446945 > > Reviewed-by: Tristan Honscheid <honscheid@google.com> > > Reviewed-by: Keith Short <keithshort@chromium.org> > > Commit-Queue: Keith Short <keithshort@chromium.org> > > Bug: b:216466985 > Change-Id: I97e4ba659b15a8860d408ec46a6629a8d292b05e > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3465516 > Commit-Queue: Jack Rosenthal <jrosenth@chromium.org> > Tested-by: Jack Rosenthal <jrosenth@chromium.org> > Auto-Submit: Jack Rosenthal <jrosenth@chromium.org> > Reviewed-by: Keith Short <keithshort@chromium.org> > Commit-Queue: Keith Short <keithshort@chromium.org> Bug: b:216466985 Change-Id: I51284582ad8e17ab2016b65f8a31944702d70579 Signed-off-by: Andrew McRae <amcrae@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466674 Reviewed-by: Dawid Niedzwiecki <dawidn@google.com> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
* agah: update gpioZick Wei2022-02-162-8/+14
| | | | | | | | | | | | | | Update EC gpio follow schematic. BUG=b:210970640 BRANCH=none TEST=make BOARD=agah Signed-off-by: Zick Wei <zick.wei@quanta.corp-partner.google.com> Change-Id: If1c1ff66c8a16d9f395fb9662006768bc4cc0495 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3361699 Reviewed-by: Devin Lu <Devin.Lu@quantatw.com> Reviewed-by: caveh jalali <caveh@chromium.org>
* agah: config USBC0 port and remove dbZick Wei2022-02-169-230/+66
| | | | | | | | | | | | | | | | | | This patch update USBC0 setting: TCPC RT1716 with PPC SYV682, and remove daughter board to fit agah HW design. Will update another CL for USBC1 after function verified. BUG=b:210970640 BRANCH=none TEST=make sure USBC0 can get 60W power through servoV4 with 65W adapter. Change-Id: Ibdc1d2e4dd86a3b8383ac33622d8ca038862b721 Signed-off-by: Zick Wei <zick.wei@quanta.corp-partner.google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3360328 Reviewed-by: Devin Lu <Devin.Lu@quantatw.com> Reviewed-by: caveh jalali <caveh@chromium.org>
* herobrine: battery: Remove '_npcx9' suffix in commentMatthias Kaehlcke2022-02-161-3/+3
| | | | | | | | | | | | | | | | The project for herobrine has been renamed from 'herobrine_npcx9' to just 'herobrine', also remove the '_npcx9' suffix from a comment in the battery info for herobrine. BRANCH=None BUG=b:215771396 TEST=zmake configure -b herobrine Change-Id: Id7d701d02fb8e5163e669c345dc71c4b07d9b32e Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3444681 Reviewed-by: Wai-Hong Tam <waihong@google.com> Reviewed-by: Bob Moragues <moragues@chromium.org>
* herobrine: Rename project to 'herobrine' (was 'herobrine_npcx9')Matthias Kaehlcke2022-02-1617-12/+12
| | | | | | | | | | | | | | | | The NPCX9 is the only EC chip that will be used by herobrine, rename the project from 'herobrine_npcx9' to 'herobrine'. BRANCH=None BUG=b:215771396 TEST=zmake configure -b herobrine Cq-Depend: chrome-internal:4546687 Change-Id: I206a65f253ce38d1b263a13df6ace90c90a0e450 Signed-off-by: Matthias Kaehlcke <mka@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459518 Reviewed-by: Bob Moragues <moragues@chromium.org> Reviewed-by: Wai-Hong Tam <waihong@google.com>
* adlrvp: configure board specific battery type firstDeepti Deshatty2022-02-161-3/+3
| | | | | | | | | | | | | | | | | | | | board_pre_task_i2c_peripheral_init() api selects board specific battery type and charger information. Charger configure is called first and later configure battery type is called. Charger configuration uses battery type information, hence battery type needs to be configured before calling charger configuration. BUG=none BRANCH=none TEST=Verified battery fuctionality on ADL-N rvp board. Change-Id: Ie0f54fc2675721ea3f38f675eca446a32da49cf7 Signed-off-by: Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459596 Reviewed-by: Vijay P Hiremath <vijay.p.hiremath@intel.com> Tested-by: Deepti Deshatty <deepti.deshatty@intel.com> Reviewed-by: caveh jalali <caveh@chromium.org>
* zephyr: Create Kconfig.pd_frsSam Hurst2022-02-162-41/+51
| | | | | | | | | | | | | | Split PD FRS Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_frs. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: If20792ef6fe8148e77db20caa72f916086db4a81 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445453 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Create Kconfig.pd_meas_vbusSam Hurst2022-02-162-35/+45
| | | | | | | | | | | | | | Split PD Measure VBUS Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_meas_vbus. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: Ic6587cfc6d419f5b9d61969d1fd78fd8efb9b019 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445452 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Create Kconfig.pd_int_sharedSam Hurst2022-02-162-39/+49
| | | | | | | | | | | | | | Split PD Interrupt Shared Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.pd_int_shared. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: I29164219ca8a6cf520d918d21168b928b015fb3f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445325 Reviewed-by: Keith Short <keithshort@chromium.org>
* nissa: implement hibernate for nereidPeter Marheine2022-02-163-23/+46
| | | | | | | | | | | | | | | The board_hibernate() functions are moved out of the usbc files since hibernation isn't directly related to USB or charging. BUG=b:219808597 TEST=`hibernate` on console hibernates nereid & nivviks BRANCH=none Signed-off-by: Peter Marheine <pmarheine@chromium.org> Change-Id: I1ff3470906e7757e62e802b9712e3f4d2616a5e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466682 Reviewed-by: Andrew McRae <amcrae@google.com> Commit-Queue: Andrew McRae <amcrae@google.com>
* volmar: Modify USB C1 i2c port speedYu-An Chen2022-02-161-17/+0
| | | | | | | | | | | | | | | Modify the I2C frequency of I2C_PORT_USB_C1_TCPC and I2C_PORT_USB_C1_PPC from 400 kHz to 1000 kHz and disable C1_PPC dynamic speed. BUG=b:211891086 BRANCH=brya TEST=check pd 1 working Signed-off-by: Yu-An Chen <yu-an.chen@quanta.corp-partner.google.com> Change-Id: Ia6e20ce5ce1064add91de021f00121cf70bcfc96 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459601 Reviewed-by: caveh jalali <caveh@chromium.org> Commit-Queue: caveh jalali <caveh@chromium.org>
* zephyr: test: Verify invoke typec status host cmdAaron Massey2022-02-161-2/+15
| | | | | | | | | | | | | | To facilitate assertion failure isolation, add a test verifying only that the typec status host command can be successfully invoked. BRANCH=none BUG=b:189954415 TEST=zmake configure --test test-drivers Signed-off-by: Aaron Massey <aaronmassey@google.com> Change-Id: I685144a6c123b24b6931783f1e0f8afc7408fd4b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3462646 Reviewed-by: Keith Short <keithshort@chromium.org>
* zephyr: Create Kconfig.retimerSam Hurst2022-02-162-76/+85
| | | | | | | | | | | | | | Split Retimer Kconfig options from Kconfig.usbc and place in a separate file named Kconfig.retimer. BUG=b:194432779 TEST=Verified Kconfig options with menuconfig BRANCH=main Signed-off-by: Sam Hurst <shurst@google.com> Change-Id: I17fd3d1d41a456d0d9c00a536533f5bfb3d0326f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3445146 Reviewed-by: Keith Short <keithshort@chromium.org>
* nissa: update and regenerate GPIOsPeter Marheine2022-02-163-13/+13
| | | | | | | | | | | | | | | This fixes the default state for the power button GPIO (which was default-asserted) and updates the generated device tree files to reflect the current version of the pinmap CSV. BUG=b:203446068 TEST=zmake build nereid BRANCH=none Signed-off-by: Peter Marheine <pmarheine@chromium.org> Change-Id: I243ef557d0d2f19e8129257818a263ada16dfa89 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3461049 Reviewed-by: Andrew McRae <amcrae@google.com>
* zephyr: npcx: shi: configure power down bitJun Lin2022-02-161-0/+18
| | | | | | | | | | | | | | | | | | | | This CL improves the power consumption by: * CLear the power down bit for SHI module when SHI is enable. * Set the power down bit for SHI module when SHI is disabled. BUG=b:219388463 BRANCH=none TEST=pass "zmake testall --clobber" TEST=Check power down bit is set when the system at G3. TEST=Check power down bit is cleard when the system at S0. Signed-off-by: Jun Lin <CHLin56@nuvoton.com> Change-Id: Ib9b0f0b744221aef15791383a71bb7303ff39d44 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3461574 Tested-by: CH Lin <chlin56@nuvoton.com> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org> Commit-Queue: CH Lin <chlin56@nuvoton.com>
* qcom: Catch the unexpected case that POWER_GOOD up again after lostWai-Hong Tam2022-02-161-7/+19
| | | | | | | | | | | | | | | | | | When the POWER_GOOD was lost, it sets the shutdown_from_on flag. We don't expect the POWER_GOOD up again. However, a hardware bug makes this happen. Catch this unexpected case and go straight to turn off the switchcap. BRANCH=None BUG=b:218878427 TEST=Typed "shutdown -P now" in userspace. Saw the EC warning message: [41.599700 Warning: POWER_GOOD up again after lost] Change-Id: Ieadffd5347f46070336cc608af2ed3cbc20c612c Signed-off-by: Wai-Hong Tam <waihong@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466417 Reviewed-by: Alexandru Stan <amstan@chromium.org> Commit-Queue: Alexandru Stan <amstan@chromium.org>
* qcom: Unconditionally disable AP_RST_L interrupt on power offWai-Hong Tam2022-02-161-5/+6
| | | | | | | | | | | | | | | | Move disabling the AP_RST_L interrupt out of the if clause. The if clause won't be entered if the shutdown is triggered by POWER GOOD lost. BRANCH=None BUG=b:218878427 TEST=Triggered the power off sequence from the AP side. Checked the AP_RST_L interrupt disabled. Change-Id: I69c22a44f6fa42ec4397b52a7a2d9d025c2de5b4 Signed-off-by: Wai-Hong Tam <waihong@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466416 Reviewed-by: Alexandru Stan <amstan@chromium.org>
* brya: increase prochot release threshold when adapter < 60wScott Chao2022-02-162-2/+2
| | | | | | | | | | | | | | | | | | Set de-assert threshold to as same as adapter rating. We used 90% of adapter rating as de-assert point, but system loading was hard to reach that low when battery < 10%. BUG=b:212359724, b:212363905 BRANCH=none TEST=Test 45w adapter on primus. Change-Id: I6e99cc17299d5a2fbd07f0040f1335150a3de295 Signed-off-by: Scott Chao <scott_chao@wistron.corp-partner.google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3460762 Reviewed-by: caveh jalali <caveh@chromium.org> Tested-by: caveh jalali <caveh@chromium.org> Reviewed-by: Boris Mittelberg <bmbm@google.com> Commit-Queue: Boris Mittelberg <bmbm@google.com>
* TCPMv2: Data Reset to exit modesAbe Levkoy2022-02-151-4/+23
| | | | | | | | | | | | | | | Try to Data Reset when exiting alternate modes or USB4. For alternate modes still try Exit Mode if Data Reset isn't supported. BUG=b:209625351 TEST=Attach USB4 hub; ectool typeccontrol 1 0; observe PD traffic BRANCH=none Cq-Depend: chromium:3465516 Signed-off-by: Abe Levkoy <alevkoy@chromium.org> Change-Id: Icfc8af8b6d7d47313ce1d1659db47fe3ad2d330c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3436451 Reviewed-by: Diana Z <dzigterman@chromium.org>
* volteer: Disable several console commandsAbe Levkoy2022-02-151-2/+12
| | | | | | | | | | | | | | | | Save ~1084 bytes of flash space. Some of these config items were not taking up any flash space presently, but disable them explicitly to avoid looking them up next time volteer runs out of space. BUG=none TEST=make buildall BRANCH=none Cq-Depend: chromium:3465516 Signed-off-by: Abe Levkoy <alevkoy@chromium.org> Change-Id: Ic15fd4ddc8d36ac50499920a058ec61babea9f3c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459739 Reviewed-by: Keith Short <keithshort@chromium.org>
* guybrush: Enable apthrottle console commandRob Barnes2022-02-151-0/+1
| | | | | | | | | | | | | | | Enable apthrottle/CONFIG_CMD_APTHROTTLE to help debug prochot issues. BUG=b:219589838 BRANCH=guybruhs TEST=apthrottle on nipperkin Change-Id: Ic8a41d334163595c7845099b0004aa5a01fb285e Signed-off-by: Rob Barnes <robbarnes@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3464455 Reviewed-by: Raul Rangel <rrangel@chromium.org> Reviewed-by: Diana Z <dzigterman@chromium.org> Reviewed-by: Jonathon Murphy <jpmurphy@chromium.org>
* stm32: Add the DFU Boot ManagerBrian J. Nemec2022-02-1523-2/+366
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the DFU Boot Manager to control the switch between the DFU and main application region. The DFU Bootmanager performs self checks when the application boots. If any of the following conditions are true it will jump into DFU: * The system backup registers contain a 'Jump to DFU' request. * The start of the RW region is erased indicating the prior firmware update has failed. * An optional check can be enabled to record the number of consecutive unexpected reboots which will be useful for developer test images to avoid locked systems. Testing requires enabling multiple configuration settings set in the follower CL which enables it on Servo_v4. dfu-util version 0.9 lacks the 'leave' parameter required to automatically jump out, systems using this would need to be power cycled to leave DFU. BRANCH=servo BUG=b:217955677 TEST=Enabled the DFU configuration on a Servo_v4. Verified that the runtime identifier is correctly recognized by dfu-util and pydfu.py implementations of the DFU host protocol. Verified firmware updates via the DFU protocol function correctly. TEST=sudo dfu-util -a 0 -s 0x08000000:leave -e -D ec.bin Verified that dfu-util can program the application region. Signed-off-by: Brian Nemec <bnemec@chromium.org> Change-Id: Ie535efc91318244574949542c03efa0b5973cbe5 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3443960 Tested-by: Brian Nemec <bnemec@google.com> Reviewed-by: Sam Hurst <shurst@google.com> Commit-Queue: Brian Nemec <bnemec@google.com>
* Revert "zephyr: Remove unused enum-name entries"Jack Rosenthal2022-02-1511-1/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 112bc3616b55421af4ae5003c542e9a425128650. Reason for revert: suspected at fault for postsubmit failures https://ci.chromium.org/p/chromeos/builders/postsubmit/firmware-zephyr-postsubmit/14846 Original change's description: > zephyr: Remove unused enum-name entries > > After removal of the unsupported projects, > take another run through the enum-names to > remove the unused entries. > > BUG=b:216466985 > TEST=zmake testall > BRANCH=none > > Signed-off-by: Andrew McRae <amcrae@google.com> > Change-Id: Id652c4fda7dd64a59b3776bb52640413a31e4e3e > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3446945 > Reviewed-by: Tristan Honscheid <honscheid@google.com> > Reviewed-by: Keith Short <keithshort@chromium.org> > Commit-Queue: Keith Short <keithshort@chromium.org> Bug: b:216466985 Change-Id: I97e4ba659b15a8860d408ec46a6629a8d292b05e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3465516 Commit-Queue: Jack Rosenthal <jrosenth@chromium.org> Tested-by: Jack Rosenthal <jrosenth@chromium.org> Auto-Submit: Jack Rosenthal <jrosenth@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* docs: Set BOARD in quick build instructionsBobby Casey2022-02-151-0/+13
| | | | | | | | | | | | | | | | | | | | Step 3 of the instructions in the "Building" section indicate to run a make command that includes `make BOARD=${BOARD}`. This assumes that ${BOARD} is set properly but the instructions to not specify to do so. The first paragraph does link to documents that cover this but then go on to state that you can safely ignore them. Fix by adding a step to specify the BOARD env var, including a brief description of where to find this value and a safe default. BUG=b:219081299 BRANCH=none TEST=view in gitiles Signed-off-by: Bobby Casey <bobbycasey@google.com> Change-Id: I9b0a7f9b6b64ff2b1bb5844c5fa750514ecf6d1b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3460062 Reviewed-by: Tom Hughes <tomhughes@chromium.org>
* zephyr: Include common/system.c in posix alsoJeremy Bettis2022-02-155-83/+42
| | | | | | | | | | | | | | | | | | | | | | | | Change arch posix to also include common/system.c This required removing some mocked functions from ztest_system.c, adding others, and making some of them test_mockable for tests that already provided them. There was one test scenario that had to be removed, as it's not clear how to simulate late jump to rw with the real system.c code. Added a TODO to come back and fix that test. BRANCH=None BUG=b:214256453,b:219520539 TEST=zmake testall Signed-off-by: Jeremy Bettis <jbettis@google.com> Change-Id: I59be918f95b3e1048c87b6cd504ae48627fd84c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3451273 Tested-by: Jeremy Bettis <jbettis@chromium.org> Auto-Submit: Jeremy Bettis <jbettis@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* zephyr: Remove unused enum-name entriesAndrew McRae2022-02-1511-91/+1
| | | | | | | | | | | | | | | | | After removal of the unsupported projects, take another run through the enum-names to remove the unused entries. BUG=b:216466985 TEST=zmake testall BRANCH=none Signed-off-by: Andrew McRae <amcrae@google.com> Change-Id: Id652c4fda7dd64a59b3776bb52640413a31e4e3e Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3446945 Reviewed-by: Tristan Honscheid <honscheid@google.com> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Keith Short <keithshort@chromium.org>
* parasa: Config keyboard configruation for parasaMichael5 Chen12022-02-152-3/+34
| | | | | | | | | | | | | | | Config keyboard configuration for parasa. BUG=b:215961769 BRANCH=dedede TEST=Manual Check keyboard scan by evtest. Signed-off-by: Michael5 Chen1 <michael5_chen1@pegatron.corp-partner.google.com> Change-Id: I5fa31c451b215e69f3198b672b298b52d82f154f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3459586 Reviewed-by: Shou-Chieh Hsu <shouchieh@chromium.org> Commit-Queue: Shou-Chieh Hsu <shouchieh@chromium.org>
* driver/tcs3400: Set timestamp when emulating interruptGwendal Grignou2022-02-151-18/+13
| | | | | | | | | | | | | | | | | | | | | | | Use the same interrupt handler for wire or emulated interrupt, so we are sure to timestamp the samples. BUG=b:218890983,b:129419982 TEST=make -j buildall Check on kukui/krane that tast test hardware.SensorIioserviceHard passes. It would fail previously with error like: "error during validation: too much delta between samples for lid cros-ec-gyro at index 174(1m52.310435376s): got 14.894431ms; want <= 7.5ms" BRANCH=kukui,asurada Fixes: commit ae22b703601 ("driver: add an option to emulate irq event in TCS3400") Signed-off-by: Gwendal Grignou <gwendal@chromium.org> Change-Id: I9f4deaa214abe6e8ac7a2aa6b6071a02565f8825 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3454215 Reviewed-by: Ting Shen <phoenixshen@chromium.org>
* anahera: Enable PPC controlled FRSDevin Lu2022-02-151-0/+2
| | | | | | | | | | | | | BRANCH=none BUG=b:218754289 TEST=buildall TEST=Manually, With HDMI dock + AC on dock then remove AC. HDMI stay on. Signed-off-by: Devin Lu <Devin.Lu@quantatw.com> Change-Id: I7cd129e1c398c4432761afce76b473591d3b758f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3455105 Reviewed-by: caveh jalali <caveh@chromium.org>