| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|