diff options
author | Keith Short <keithshort@chromium.org> | 2022-04-11 17:31:34 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-04-12 18:17:33 +0000 |
commit | e308b1db1171d60e8695621fdf21d47495f157b8 (patch) | |
tree | a0c283cf8cfd650a3c56b48654265b423a5e6419 | |
parent | 73167b0439cd7a26e537328aa5402b7e27065e74 (diff) | |
download | chrome-ec-e308b1db1171d60e8695621fdf21d47495f157b8.tar.gz |
zephyr: docs: Update the doc template
Update the document template to reflect current best practices.
Specifically, move all related Kconfigs into a separate file and link to
the file only
BUG=b:207403708
BRANCH=none
TEST=View preview in gitiles
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I19e9b78124843e0c9828369464dc37889aeb5114
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3583046
Reviewed-by: Wai-Hong Tam <waihong@google.com>
-rw-r--r-- | docs/zephyr/zephyr_template.md | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/docs/zephyr/zephyr_template.md b/docs/zephyr/zephyr_template.md index f75883eec9..f36c9f90da 100644 --- a/docs/zephyr/zephyr_template.md +++ b/docs/zephyr/zephyr_template.md @@ -8,21 +8,19 @@ ## Kconfig Options -*List the Kconfig options that enable the feature and list any sub-configuration -options that control the behavior of the feature.* - -Kconfig Option | Default | Documentation -:------------------------------------- | :-----: | :------------ -`CONFIG_PLATFORM_EC_<option>` | y/n | [zephyr/Kconfig](../zephyr/Kconfig) - -Kconfig sub-option | Default | Documentation -:------------------------------------- | :-----: | :------------ -`CONFIG_PLATFORM_EC_<option>` | y/n | [zephyr/Kconfig](../zephyr/Kconfig) +*Link to the file providing all the Kconfig options related to the feature. If +the Kconfig options are not currently in a standalone file, consider moving the +related Kconfigs into their own file.* +*Example CL moving I2C related configs into a new file: https://crrev.com/c/3575081* *Note - Avoid documenting `CONFIG_` options in the markdown as the relevant -`Kconfig*` contains the authoritative definition. Link directly to the Kconfig -option in source like this: [I2C Passthru Restricted].* +`Kconfig*` contains the authoritative definition. If there is one main Kconfig +that must be enabled for the feature, mention it in this section. See the [I2C +documentation](zephyr_i2c.md#kconfig-options) for an example.* + +*If the `Kconfig` file does not provide sufficient help descriptions, please fix +them.* ## Devicetree Nodes @@ -44,6 +42,9 @@ compile and run. For many features, this can section can be empty.* *Provide any tips for testing and debugging the EC feature.* +*It's especially helpful to document EC console commands and `ectool` commands +from the AP in this section. Please provide example output.* + ## Example *Provide code snippets from a working board to walk the user through |