summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTristan Honscheid <honscheid@google.com>2023-02-13 18:55:10 -0700
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2023-02-14 11:31:34 +0000
commit42d83017fb78371f1ded1af599089daedf3a9a4d (patch)
treef4dc86bc75cdfe532c60bc2db7ce52fbfd81ac9a
parentb40c6970d3b788498a83de4f2828357e2ac3e279 (diff)
downloadchrome-ec-42d83017fb78371f1ded1af599089daedf3a9a4d.tar.gz
zephyr: Fix YAML files to conform with upstream style
cl:4177231 enabled YAML file linting for platform/ec/zephyr files based on upstream Zephyr's style but several of our YAML files do not currently conform. This CL has all the required corrections when running the linter on every YAML file. BUG=None TEST=util/zephyr_check_compliance.py modified to scan everything BRANCH=None Change-Id: I9dcf1d7f0c306028e4d6b368f20e88c882514169 Signed-off-by: Tristan Honscheid <honscheid@google.com> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4247159 Reviewed-by: Fabio Baltieri <fabiobaltieri@google.com> Commit-Queue: Fabio Baltieri <fabiobaltieri@google.com>
-rw-r--r--zephyr/dts/bindings/battery/bms-gf,cr50.yaml90
-rw-r--r--zephyr/dts/bindings/power/ap-pwrseq-sub-states.yaml6
-rw-r--r--zephyr/test/drivers/testcase.yaml7
3 files changed, 52 insertions, 51 deletions
diff --git a/zephyr/dts/bindings/battery/bms-gf,cr50.yaml b/zephyr/dts/bindings/battery/bms-gf,cr50.yaml
index 8175e48eb1..cd5cc8c05a 100644
--- a/zephyr/dts/bindings/battery/bms-gf,cr50.yaml
+++ b/zephyr/dts/bindings/battery/bms-gf,cr50.yaml
@@ -4,50 +4,50 @@ compatible: "bms-gf,cr50"
include: battery-smart.yaml
properties:
- enum-name:
- type: string
- default: "bms-gf,cr50"
+ enum-name:
+ type: string
+ default: "bms-gf,cr50"
- # Fuel gauge
- manuf_name:
- default: "BMS-GF"
- device_name:
- default: "CR50"
- ship_mode_reg_addr:
- default: 0x00
- ship_mode_reg_data:
- default: [ 0x0010, 0x0010 ]
- fet_mfgacc_support:
- default: 0
- fet_reg_addr:
- default: 0x43
- fet_reg_mask:
- default: 0x0001
- fet_disconnect_val:
- default: 0x0000
- fet_cfet_mask:
- default: 0x0002
- fet_cfet_off_val:
- default: 0x0000
+ # Fuel gauge
+ manuf_name:
+ default: "BMS-GF"
+ device_name:
+ default: "CR50"
+ ship_mode_reg_addr:
+ default: 0x00
+ ship_mode_reg_data:
+ default: [0x0010, 0x0010]
+ fet_mfgacc_support:
+ default: 0
+ fet_reg_addr:
+ default: 0x43
+ fet_reg_mask:
+ default: 0x0001
+ fet_disconnect_val:
+ default: 0x0000
+ fet_cfet_mask:
+ default: 0x0002
+ fet_cfet_off_val:
+ default: 0x0000
- # Battery info
- voltage_max:
- default: 17600
- voltage_normal:
- default: 15000
- voltage_min:
- default: 12000
- precharge_current:
- default: 256
- start_charging_min_c:
- default: 0
- start_charging_max_c:
- default: 50
- charging_min_c:
- default: 0
- charging_max_c:
- default: 60
- discharging_min_c:
- default: -20
- discharging_max_c:
- default: 60
+ # Battery info
+ voltage_max:
+ default: 17600
+ voltage_normal:
+ default: 15000
+ voltage_min:
+ default: 12000
+ precharge_current:
+ default: 256
+ start_charging_min_c:
+ default: 0
+ start_charging_max_c:
+ default: 50
+ charging_min_c:
+ default: 0
+ charging_max_c:
+ default: 60
+ discharging_min_c:
+ default: -20
+ discharging_max_c:
+ default: 60
diff --git a/zephyr/dts/bindings/power/ap-pwrseq-sub-states.yaml b/zephyr/dts/bindings/power/ap-pwrseq-sub-states.yaml
index f6570be9fb..8632b7e7b5 100644
--- a/zephyr/dts/bindings/power/ap-pwrseq-sub-states.yaml
+++ b/zephyr/dts/bindings/power/ap-pwrseq-sub-states.yaml
@@ -2,12 +2,10 @@
# SPDX-License-Identifier: Apache-2.0
description: |
- Define additional substates to be included in state machine.
+ Define additional substates to be included in state machine. Use corresponding
+ macros to provide action handlers.
compatible: "ap-pwrseq-sub-states"
-description: |
- String array of additional substates, use corresponding macros to provide
- action handlers.
properties:
chipset:
type: string-array
diff --git a/zephyr/test/drivers/testcase.yaml b/zephyr/test/drivers/testcase.yaml
index 807ce4b8bb..b3746f9556 100644
--- a/zephyr/test/drivers/testcase.yaml
+++ b/zephyr/test/drivers/testcase.yaml
@@ -200,7 +200,9 @@ tests:
- CONFIG_POWER_SEQUENCE_MOCK=y
- CONFIG_PLATFORM_EC_POWERSEQ_HOST_SLEEP=y
drivers.ps8xxx:
- extra_args: CONF_FILE="prj.conf;ps8xxx/prj.conf" DTC_OVERLAY_FILE="./boards/native_posix.overlay;./ps8xxx/usbc.dts"
+ extra_args: >
+ CONF_FILE="prj.conf;ps8xxx/prj.conf"
+ DTC_OVERLAY_FILE="./boards/native_posix.overlay;./ps8xxx/usbc.dts"
extra_configs:
- CONFIG_LINK_TEST_SUITE_PS8XXX=y
drivers.rt9490:
@@ -278,7 +280,8 @@ tests:
extra_configs:
- CONFIG_LINK_TEST_SUITE_USBC_CONSOLE_PD=y
drivers.usbc_console_pd_legacy:
- extra_args: DTC_OVERLAY_FILE="default/boards/native_posix.overlay;./usbc_console_pd/usbc_legacy.dts"
+ extra_args: >
+ DTC_OVERLAY_FILE="default/boards/native_posix.overlay;./usbc_console_pd/usbc_legacy.dts"
extra_configs:
- CONFIG_LINK_TEST_SUITE_USBC_CONSOLE_PD=y
- CONFIG_PLATFORM_EC_TCPC_INTERRUPT=n