linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] platform/chrome: cros_ec_trace: Match trace commands with EC commands
@ 2019-10-11 13:39 Enric Balletbo i Serra
  2019-10-14 18:10 ` Raul Rangel
  0 siblings, 1 reply; 2+ messages in thread
From: Enric Balletbo i Serra @ 2019-10-11 13:39 UTC (permalink / raw)
  To: linux-kernel
  Cc: Collabora Kernel ML, groeck, bleung, dtor, gwendal, tzungbi, rrangel

There are some EC commands that are not included yet as trace commands,
in order to get all the traces for the all supported commands match the
commands accordingly.

Note that a change, adding or removing an EC command, should be
reflected in the cros_ec_trace.c file in order to avoid mismatches
again.

The list of current commands is generated using the following script:

 sed -n 's/^#define \(EC_CMD_[[:alnum:]_]*\)\s.*/\tTRACE_SYMBOL(\1), \\/p' \
        include/linux/platform_data/cros_ec_commands.h

Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
---

 drivers/platform/chrome/cros_ec_trace.c | 73 ++++++++++++++++++++++---
 1 file changed, 66 insertions(+), 7 deletions(-)

diff --git a/drivers/platform/chrome/cros_ec_trace.c b/drivers/platform/chrome/cros_ec_trace.c
index 6f80ff4532ae..8e93cccfbcf5 100644
--- a/drivers/platform/chrome/cros_ec_trace.c
+++ b/drivers/platform/chrome/cros_ec_trace.c
@@ -8,6 +8,11 @@
 // Generate the list using the following script:
 // sed -n 's/^#define \(EC_CMD_[[:alnum:]_]*\)\s.*/\tTRACE_SYMBOL(\1), \\/p' include/linux/platform_data/cros_ec_commands.h
 #define EC_CMDS \
+	TRACE_SYMBOL(EC_CMD_ACPI_READ), \
+	TRACE_SYMBOL(EC_CMD_ACPI_WRITE), \
+	TRACE_SYMBOL(EC_CMD_ACPI_BURST_ENABLE), \
+	TRACE_SYMBOL(EC_CMD_ACPI_BURST_DISABLE), \
+	TRACE_SYMBOL(EC_CMD_ACPI_QUERY_EVENT), \
 	TRACE_SYMBOL(EC_CMD_PROTO_VERSION), \
 	TRACE_SYMBOL(EC_CMD_HELLO), \
 	TRACE_SYMBOL(EC_CMD_GET_VERSION), \
@@ -22,6 +27,8 @@
 	TRACE_SYMBOL(EC_CMD_GET_PROTOCOL_INFO), \
 	TRACE_SYMBOL(EC_CMD_GSV_PAUSE_IN_S5), \
 	TRACE_SYMBOL(EC_CMD_GET_FEATURES), \
+	TRACE_SYMBOL(EC_CMD_GET_SKU_ID), \
+	TRACE_SYMBOL(EC_CMD_SET_SKU_ID), \
 	TRACE_SYMBOL(EC_CMD_FLASH_INFO), \
 	TRACE_SYMBOL(EC_CMD_FLASH_READ), \
 	TRACE_SYMBOL(EC_CMD_FLASH_WRITE), \
@@ -29,6 +36,8 @@
 	TRACE_SYMBOL(EC_CMD_FLASH_PROTECT), \
 	TRACE_SYMBOL(EC_CMD_FLASH_REGION_INFO), \
 	TRACE_SYMBOL(EC_CMD_VBNV_CONTEXT), \
+	TRACE_SYMBOL(EC_CMD_FLASH_SPI_INFO), \
+	TRACE_SYMBOL(EC_CMD_FLASH_SELECT), \
 	TRACE_SYMBOL(EC_CMD_PWM_GET_FAN_TARGET_RPM), \
 	TRACE_SYMBOL(EC_CMD_PWM_SET_FAN_TARGET_RPM), \
 	TRACE_SYMBOL(EC_CMD_PWM_GET_KEYBOARD_BACKLIGHT), \
@@ -40,6 +49,8 @@
 	TRACE_SYMBOL(EC_CMD_LED_CONTROL), \
 	TRACE_SYMBOL(EC_CMD_VBOOT_HASH), \
 	TRACE_SYMBOL(EC_CMD_MOTION_SENSE_CMD), \
+	TRACE_SYMBOL(EC_CMD_FORCE_LID_OPEN), \
+	TRACE_SYMBOL(EC_CMD_CONFIG_POWER_BUTTON), \
 	TRACE_SYMBOL(EC_CMD_USB_CHARGE_SET_MODE), \
 	TRACE_SYMBOL(EC_CMD_PSTORE_INFO), \
 	TRACE_SYMBOL(EC_CMD_PSTORE_READ), \
@@ -50,6 +61,9 @@
 	TRACE_SYMBOL(EC_CMD_RTC_SET_ALARM), \
 	TRACE_SYMBOL(EC_CMD_PORT80_LAST_BOOT), \
 	TRACE_SYMBOL(EC_CMD_PORT80_READ), \
+	TRACE_SYMBOL(EC_CMD_VSTORE_INFO), \
+	TRACE_SYMBOL(EC_CMD_VSTORE_READ), \
+	TRACE_SYMBOL(EC_CMD_VSTORE_WRITE), \
 	TRACE_SYMBOL(EC_CMD_THERMAL_SET_THRESHOLD), \
 	TRACE_SYMBOL(EC_CMD_THERMAL_GET_THRESHOLD), \
 	TRACE_SYMBOL(EC_CMD_THERMAL_AUTO_FAN_CTRL), \
@@ -59,10 +73,12 @@
 	TRACE_SYMBOL(EC_CMD_MKBP_STATE), \
 	TRACE_SYMBOL(EC_CMD_MKBP_INFO), \
 	TRACE_SYMBOL(EC_CMD_MKBP_SIMULATE_KEY), \
+	TRACE_SYMBOL(EC_CMD_GET_KEYBOARD_ID), \
 	TRACE_SYMBOL(EC_CMD_MKBP_SET_CONFIG), \
 	TRACE_SYMBOL(EC_CMD_MKBP_GET_CONFIG), \
 	TRACE_SYMBOL(EC_CMD_KEYSCAN_SEQ_CTRL), \
 	TRACE_SYMBOL(EC_CMD_GET_NEXT_EVENT), \
+	TRACE_SYMBOL(EC_CMD_KEYBOARD_FACTORY_TEST), \
 	TRACE_SYMBOL(EC_CMD_TEMP_SENSOR_GET_INFO), \
 	TRACE_SYMBOL(EC_CMD_HOST_EVENT_GET_B), \
 	TRACE_SYMBOL(EC_CMD_HOST_EVENT_GET_SMI_MASK), \
@@ -73,6 +89,7 @@
 	TRACE_SYMBOL(EC_CMD_HOST_EVENT_CLEAR), \
 	TRACE_SYMBOL(EC_CMD_HOST_EVENT_SET_WAKE_MASK), \
 	TRACE_SYMBOL(EC_CMD_HOST_EVENT_CLEAR_B), \
+	TRACE_SYMBOL(EC_CMD_HOST_EVENT), \
 	TRACE_SYMBOL(EC_CMD_SWITCH_ENABLE_BKLIGHT), \
 	TRACE_SYMBOL(EC_CMD_SWITCH_ENABLE_WIRELESS), \
 	TRACE_SYMBOL(EC_CMD_GPIO_SET), \
@@ -92,33 +109,75 @@
 	TRACE_SYMBOL(EC_CMD_CHARGE_STATE), \
 	TRACE_SYMBOL(EC_CMD_CHARGE_CURRENT_LIMIT), \
 	TRACE_SYMBOL(EC_CMD_EXTERNAL_POWER_LIMIT), \
+	TRACE_SYMBOL(EC_CMD_OVERRIDE_DEDICATED_CHARGER_LIMIT), \
+	TRACE_SYMBOL(EC_CMD_HIBERNATION_DELAY), \
 	TRACE_SYMBOL(EC_CMD_HOST_SLEEP_EVENT), \
+	TRACE_SYMBOL(EC_CMD_DEVICE_EVENT), \
 	TRACE_SYMBOL(EC_CMD_SB_READ_WORD), \
 	TRACE_SYMBOL(EC_CMD_SB_WRITE_WORD), \
 	TRACE_SYMBOL(EC_CMD_SB_READ_BLOCK), \
 	TRACE_SYMBOL(EC_CMD_SB_WRITE_BLOCK), \
 	TRACE_SYMBOL(EC_CMD_BATTERY_VENDOR_PARAM), \
-	TRACE_SYMBOL(EC_CMD_CODEC_I2S), \
-	TRACE_SYMBOL(EC_CMD_REBOOT_EC), \
-	TRACE_SYMBOL(EC_CMD_GET_PANIC_INFO), \
-	TRACE_SYMBOL(EC_CMD_ACPI_READ), \
-	TRACE_SYMBOL(EC_CMD_ACPI_WRITE), \
-	TRACE_SYMBOL(EC_CMD_ACPI_QUERY_EVENT), \
+	TRACE_SYMBOL(EC_CMD_SB_FW_UPDATE), \
+	TRACE_SYMBOL(EC_CMD_ENTERING_MODE), \
+	TRACE_SYMBOL(EC_CMD_I2C_PASSTHRU_PROTECT), \
 	TRACE_SYMBOL(EC_CMD_CEC_WRITE_MSG), \
 	TRACE_SYMBOL(EC_CMD_CEC_SET), \
 	TRACE_SYMBOL(EC_CMD_CEC_GET), \
+	TRACE_SYMBOL(EC_CMD_CODEC_I2S), \
+	TRACE_SYMBOL(EC_CMD_REBOOT_EC), \
+	TRACE_SYMBOL(EC_CMD_GET_PANIC_INFO), \
 	TRACE_SYMBOL(EC_CMD_REBOOT), \
 	TRACE_SYMBOL(EC_CMD_RESEND_RESPONSE), \
 	TRACE_SYMBOL(EC_CMD_VERSION0), \
 	TRACE_SYMBOL(EC_CMD_PD_EXCHANGE_STATUS), \
+	TRACE_SYMBOL(EC_CMD_PD_HOST_EVENT_STATUS), \
 	TRACE_SYMBOL(EC_CMD_USB_PD_CONTROL), \
 	TRACE_SYMBOL(EC_CMD_USB_PD_PORTS), \
 	TRACE_SYMBOL(EC_CMD_USB_PD_POWER_INFO), \
 	TRACE_SYMBOL(EC_CMD_CHARGE_PORT_COUNT), \
+	TRACE_SYMBOL(EC_CMD_USB_PD_FW_UPDATE), \
+	TRACE_SYMBOL(EC_CMD_USB_PD_RW_HASH_ENTRY), \
+	TRACE_SYMBOL(EC_CMD_USB_PD_DEV_INFO), \
 	TRACE_SYMBOL(EC_CMD_USB_PD_DISCOVERY), \
 	TRACE_SYMBOL(EC_CMD_PD_CHARGE_PORT_OVERRIDE), \
 	TRACE_SYMBOL(EC_CMD_PD_GET_LOG_ENTRY), \
-	TRACE_SYMBOL(EC_CMD_USB_PD_MUX_INFO)
+	TRACE_SYMBOL(EC_CMD_USB_PD_GET_AMODE), \
+	TRACE_SYMBOL(EC_CMD_USB_PD_SET_AMODE), \
+	TRACE_SYMBOL(EC_CMD_PD_WRITE_LOG_ENTRY), \
+	TRACE_SYMBOL(EC_CMD_PD_CONTROL), \
+	TRACE_SYMBOL(EC_CMD_USB_PD_MUX_INFO), \
+	TRACE_SYMBOL(EC_CMD_PD_CHIP_INFO), \
+	TRACE_SYMBOL(EC_CMD_RWSIG_CHECK_STATUS), \
+	TRACE_SYMBOL(EC_CMD_RWSIG_ACTION), \
+	TRACE_SYMBOL(EC_CMD_EFS_VERIFY), \
+	TRACE_SYMBOL(EC_CMD_GET_CROS_BOARD_INFO), \
+	TRACE_SYMBOL(EC_CMD_SET_CROS_BOARD_INFO), \
+	TRACE_SYMBOL(EC_CMD_GET_UPTIME_INFO), \
+	TRACE_SYMBOL(EC_CMD_ADD_ENTROPY), \
+	TRACE_SYMBOL(EC_CMD_ADC_READ), \
+	TRACE_SYMBOL(EC_CMD_ROLLBACK_INFO), \
+	TRACE_SYMBOL(EC_CMD_AP_RESET), \
+	TRACE_SYMBOL(EC_CMD_CR51_BASE), \
+	TRACE_SYMBOL(EC_CMD_CR51_LAST), \
+	TRACE_SYMBOL(EC_CMD_FP_PASSTHRU), \
+	TRACE_SYMBOL(EC_CMD_FP_MODE), \
+	TRACE_SYMBOL(EC_CMD_FP_INFO), \
+	TRACE_SYMBOL(EC_CMD_FP_FRAME), \
+	TRACE_SYMBOL(EC_CMD_FP_TEMPLATE), \
+	TRACE_SYMBOL(EC_CMD_FP_CONTEXT), \
+	TRACE_SYMBOL(EC_CMD_FP_STATS), \
+	TRACE_SYMBOL(EC_CMD_FP_SEED), \
+	TRACE_SYMBOL(EC_CMD_FP_ENC_STATUS), \
+	TRACE_SYMBOL(EC_CMD_TP_SELF_TEST), \
+	TRACE_SYMBOL(EC_CMD_TP_FRAME_INFO), \
+	TRACE_SYMBOL(EC_CMD_TP_FRAME_SNAPSHOT), \
+	TRACE_SYMBOL(EC_CMD_TP_FRAME_GET), \
+	TRACE_SYMBOL(EC_CMD_BATTERY_GET_STATIC), \
+	TRACE_SYMBOL(EC_CMD_BATTERY_GET_DYNAMIC), \
+	TRACE_SYMBOL(EC_CMD_CHARGER_CONTROL), \
+	TRACE_SYMBOL(EC_CMD_BOARD_SPECIFIC_BASE), \
+	TRACE_SYMBOL(EC_CMD_BOARD_SPECIFIC_LAST)
 
 #define CREATE_TRACE_POINTS
 #include "cros_ec_trace.h"
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] platform/chrome: cros_ec_trace: Match trace commands with EC commands
  2019-10-11 13:39 [PATCH] platform/chrome: cros_ec_trace: Match trace commands with EC commands Enric Balletbo i Serra
@ 2019-10-14 18:10 ` Raul Rangel
  0 siblings, 0 replies; 2+ messages in thread
From: Raul Rangel @ 2019-10-14 18:10 UTC (permalink / raw)
  To: Enric Balletbo i Serra
  Cc: linux-kernel, Collabora Kernel ML, Guenter Roeck, Benson Leung,
	Dmitry Torokhov, gwendal, Tzung-Bi Shih

On Fri, Oct 11, 2019 at 7:39 AM Enric Balletbo i Serra
<enric.balletbo@collabora.com> wrote:
>
> There are some EC commands that are not included yet as trace commands,
> in order to get all the traces for the all supported commands match the
> commands accordingly.
>
> Note that a change, adding or removing an EC command, should be
> reflected in the cros_ec_trace.c file in order to avoid mismatches
> again.
>
> The list of current commands is generated using the following script:
>
>  sed -n 's/^#define \(EC_CMD_[[:alnum:]_]*\)\s.*/\tTRACE_SYMBOL(\1), \\/p' \
>         include/linux/platform_data/cros_ec_commands.h
>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
>
>  drivers/platform/chrome/cros_ec_trace.c | 73 ++++++++++++++++++++++---
>  1 file changed, 66 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_trace.c b/drivers/platform/chrome/cros_ec_trace.c
> index 6f80ff4532ae..8e93cccfbcf5 100644
> --- a/drivers/platform/chrome/cros_ec_trace.c
> +++ b/drivers/platform/chrome/cros_ec_trace.c
> @@ -8,6 +8,11 @@
>  // Generate the list using the following script:
>  // sed -n 's/^#define \(EC_CMD_[[:alnum:]_]*\)\s.*/\tTRACE_SYMBOL(\1), \\/p' include/linux/platform_data/cros_ec_commands.h
>  #define EC_CMDS \
> +       TRACE_SYMBOL(EC_CMD_ACPI_READ), \
> +       TRACE_SYMBOL(EC_CMD_ACPI_WRITE), \
> +       TRACE_SYMBOL(EC_CMD_ACPI_BURST_ENABLE), \
> +       TRACE_SYMBOL(EC_CMD_ACPI_BURST_DISABLE), \
> +       TRACE_SYMBOL(EC_CMD_ACPI_QUERY_EVENT), \
>         TRACE_SYMBOL(EC_CMD_PROTO_VERSION), \
>         TRACE_SYMBOL(EC_CMD_HELLO), \
>         TRACE_SYMBOL(EC_CMD_GET_VERSION), \
> @@ -22,6 +27,8 @@
>         TRACE_SYMBOL(EC_CMD_GET_PROTOCOL_INFO), \
>         TRACE_SYMBOL(EC_CMD_GSV_PAUSE_IN_S5), \
>         TRACE_SYMBOL(EC_CMD_GET_FEATURES), \
> +       TRACE_SYMBOL(EC_CMD_GET_SKU_ID), \
> +       TRACE_SYMBOL(EC_CMD_SET_SKU_ID), \
>         TRACE_SYMBOL(EC_CMD_FLASH_INFO), \
>         TRACE_SYMBOL(EC_CMD_FLASH_READ), \
>         TRACE_SYMBOL(EC_CMD_FLASH_WRITE), \
> @@ -29,6 +36,8 @@
>         TRACE_SYMBOL(EC_CMD_FLASH_PROTECT), \
>         TRACE_SYMBOL(EC_CMD_FLASH_REGION_INFO), \
>         TRACE_SYMBOL(EC_CMD_VBNV_CONTEXT), \
> +       TRACE_SYMBOL(EC_CMD_FLASH_SPI_INFO), \
> +       TRACE_SYMBOL(EC_CMD_FLASH_SELECT), \
>         TRACE_SYMBOL(EC_CMD_PWM_GET_FAN_TARGET_RPM), \
>         TRACE_SYMBOL(EC_CMD_PWM_SET_FAN_TARGET_RPM), \
>         TRACE_SYMBOL(EC_CMD_PWM_GET_KEYBOARD_BACKLIGHT), \
> @@ -40,6 +49,8 @@
>         TRACE_SYMBOL(EC_CMD_LED_CONTROL), \
>         TRACE_SYMBOL(EC_CMD_VBOOT_HASH), \
>         TRACE_SYMBOL(EC_CMD_MOTION_SENSE_CMD), \
> +       TRACE_SYMBOL(EC_CMD_FORCE_LID_OPEN), \
> +       TRACE_SYMBOL(EC_CMD_CONFIG_POWER_BUTTON), \
>         TRACE_SYMBOL(EC_CMD_USB_CHARGE_SET_MODE), \
>         TRACE_SYMBOL(EC_CMD_PSTORE_INFO), \
>         TRACE_SYMBOL(EC_CMD_PSTORE_READ), \
> @@ -50,6 +61,9 @@
>         TRACE_SYMBOL(EC_CMD_RTC_SET_ALARM), \
>         TRACE_SYMBOL(EC_CMD_PORT80_LAST_BOOT), \
>         TRACE_SYMBOL(EC_CMD_PORT80_READ), \
> +       TRACE_SYMBOL(EC_CMD_VSTORE_INFO), \
> +       TRACE_SYMBOL(EC_CMD_VSTORE_READ), \
> +       TRACE_SYMBOL(EC_CMD_VSTORE_WRITE), \
>         TRACE_SYMBOL(EC_CMD_THERMAL_SET_THRESHOLD), \
>         TRACE_SYMBOL(EC_CMD_THERMAL_GET_THRESHOLD), \
>         TRACE_SYMBOL(EC_CMD_THERMAL_AUTO_FAN_CTRL), \
> @@ -59,10 +73,12 @@
>         TRACE_SYMBOL(EC_CMD_MKBP_STATE), \
>         TRACE_SYMBOL(EC_CMD_MKBP_INFO), \
>         TRACE_SYMBOL(EC_CMD_MKBP_SIMULATE_KEY), \
> +       TRACE_SYMBOL(EC_CMD_GET_KEYBOARD_ID), \
>         TRACE_SYMBOL(EC_CMD_MKBP_SET_CONFIG), \
>         TRACE_SYMBOL(EC_CMD_MKBP_GET_CONFIG), \
>         TRACE_SYMBOL(EC_CMD_KEYSCAN_SEQ_CTRL), \
>         TRACE_SYMBOL(EC_CMD_GET_NEXT_EVENT), \
> +       TRACE_SYMBOL(EC_CMD_KEYBOARD_FACTORY_TEST), \
>         TRACE_SYMBOL(EC_CMD_TEMP_SENSOR_GET_INFO), \
>         TRACE_SYMBOL(EC_CMD_HOST_EVENT_GET_B), \
>         TRACE_SYMBOL(EC_CMD_HOST_EVENT_GET_SMI_MASK), \
> @@ -73,6 +89,7 @@
>         TRACE_SYMBOL(EC_CMD_HOST_EVENT_CLEAR), \
>         TRACE_SYMBOL(EC_CMD_HOST_EVENT_SET_WAKE_MASK), \
>         TRACE_SYMBOL(EC_CMD_HOST_EVENT_CLEAR_B), \
> +       TRACE_SYMBOL(EC_CMD_HOST_EVENT), \
>         TRACE_SYMBOL(EC_CMD_SWITCH_ENABLE_BKLIGHT), \
>         TRACE_SYMBOL(EC_CMD_SWITCH_ENABLE_WIRELESS), \
>         TRACE_SYMBOL(EC_CMD_GPIO_SET), \
> @@ -92,33 +109,75 @@
>         TRACE_SYMBOL(EC_CMD_CHARGE_STATE), \
>         TRACE_SYMBOL(EC_CMD_CHARGE_CURRENT_LIMIT), \
>         TRACE_SYMBOL(EC_CMD_EXTERNAL_POWER_LIMIT), \
> +       TRACE_SYMBOL(EC_CMD_OVERRIDE_DEDICATED_CHARGER_LIMIT), \
> +       TRACE_SYMBOL(EC_CMD_HIBERNATION_DELAY), \
>         TRACE_SYMBOL(EC_CMD_HOST_SLEEP_EVENT), \
> +       TRACE_SYMBOL(EC_CMD_DEVICE_EVENT), \
>         TRACE_SYMBOL(EC_CMD_SB_READ_WORD), \
>         TRACE_SYMBOL(EC_CMD_SB_WRITE_WORD), \
>         TRACE_SYMBOL(EC_CMD_SB_READ_BLOCK), \
>         TRACE_SYMBOL(EC_CMD_SB_WRITE_BLOCK), \
>         TRACE_SYMBOL(EC_CMD_BATTERY_VENDOR_PARAM), \
> -       TRACE_SYMBOL(EC_CMD_CODEC_I2S), \
> -       TRACE_SYMBOL(EC_CMD_REBOOT_EC), \
> -       TRACE_SYMBOL(EC_CMD_GET_PANIC_INFO), \
> -       TRACE_SYMBOL(EC_CMD_ACPI_READ), \
> -       TRACE_SYMBOL(EC_CMD_ACPI_WRITE), \
> -       TRACE_SYMBOL(EC_CMD_ACPI_QUERY_EVENT), \
> +       TRACE_SYMBOL(EC_CMD_SB_FW_UPDATE), \
> +       TRACE_SYMBOL(EC_CMD_ENTERING_MODE), \
> +       TRACE_SYMBOL(EC_CMD_I2C_PASSTHRU_PROTECT), \
>         TRACE_SYMBOL(EC_CMD_CEC_WRITE_MSG), \
>         TRACE_SYMBOL(EC_CMD_CEC_SET), \
>         TRACE_SYMBOL(EC_CMD_CEC_GET), \
> +       TRACE_SYMBOL(EC_CMD_CODEC_I2S), \
> +       TRACE_SYMBOL(EC_CMD_REBOOT_EC), \
> +       TRACE_SYMBOL(EC_CMD_GET_PANIC_INFO), \
>         TRACE_SYMBOL(EC_CMD_REBOOT), \
>         TRACE_SYMBOL(EC_CMD_RESEND_RESPONSE), \
>         TRACE_SYMBOL(EC_CMD_VERSION0), \
>         TRACE_SYMBOL(EC_CMD_PD_EXCHANGE_STATUS), \
> +       TRACE_SYMBOL(EC_CMD_PD_HOST_EVENT_STATUS), \
>         TRACE_SYMBOL(EC_CMD_USB_PD_CONTROL), \
>         TRACE_SYMBOL(EC_CMD_USB_PD_PORTS), \
>         TRACE_SYMBOL(EC_CMD_USB_PD_POWER_INFO), \
>         TRACE_SYMBOL(EC_CMD_CHARGE_PORT_COUNT), \
> +       TRACE_SYMBOL(EC_CMD_USB_PD_FW_UPDATE), \
> +       TRACE_SYMBOL(EC_CMD_USB_PD_RW_HASH_ENTRY), \
> +       TRACE_SYMBOL(EC_CMD_USB_PD_DEV_INFO), \
>         TRACE_SYMBOL(EC_CMD_USB_PD_DISCOVERY), \
>         TRACE_SYMBOL(EC_CMD_PD_CHARGE_PORT_OVERRIDE), \
>         TRACE_SYMBOL(EC_CMD_PD_GET_LOG_ENTRY), \
> -       TRACE_SYMBOL(EC_CMD_USB_PD_MUX_INFO)
> +       TRACE_SYMBOL(EC_CMD_USB_PD_GET_AMODE), \
> +       TRACE_SYMBOL(EC_CMD_USB_PD_SET_AMODE), \
> +       TRACE_SYMBOL(EC_CMD_PD_WRITE_LOG_ENTRY), \
> +       TRACE_SYMBOL(EC_CMD_PD_CONTROL), \
> +       TRACE_SYMBOL(EC_CMD_USB_PD_MUX_INFO), \
> +       TRACE_SYMBOL(EC_CMD_PD_CHIP_INFO), \
> +       TRACE_SYMBOL(EC_CMD_RWSIG_CHECK_STATUS), \
> +       TRACE_SYMBOL(EC_CMD_RWSIG_ACTION), \
> +       TRACE_SYMBOL(EC_CMD_EFS_VERIFY), \
> +       TRACE_SYMBOL(EC_CMD_GET_CROS_BOARD_INFO), \
> +       TRACE_SYMBOL(EC_CMD_SET_CROS_BOARD_INFO), \
> +       TRACE_SYMBOL(EC_CMD_GET_UPTIME_INFO), \
> +       TRACE_SYMBOL(EC_CMD_ADD_ENTROPY), \
> +       TRACE_SYMBOL(EC_CMD_ADC_READ), \
> +       TRACE_SYMBOL(EC_CMD_ROLLBACK_INFO), \
> +       TRACE_SYMBOL(EC_CMD_AP_RESET), \
> +       TRACE_SYMBOL(EC_CMD_CR51_BASE), \
> +       TRACE_SYMBOL(EC_CMD_CR51_LAST), \
> +       TRACE_SYMBOL(EC_CMD_FP_PASSTHRU), \
> +       TRACE_SYMBOL(EC_CMD_FP_MODE), \
> +       TRACE_SYMBOL(EC_CMD_FP_INFO), \
> +       TRACE_SYMBOL(EC_CMD_FP_FRAME), \
> +       TRACE_SYMBOL(EC_CMD_FP_TEMPLATE), \
> +       TRACE_SYMBOL(EC_CMD_FP_CONTEXT), \
> +       TRACE_SYMBOL(EC_CMD_FP_STATS), \
> +       TRACE_SYMBOL(EC_CMD_FP_SEED), \
> +       TRACE_SYMBOL(EC_CMD_FP_ENC_STATUS), \
> +       TRACE_SYMBOL(EC_CMD_TP_SELF_TEST), \
> +       TRACE_SYMBOL(EC_CMD_TP_FRAME_INFO), \
> +       TRACE_SYMBOL(EC_CMD_TP_FRAME_SNAPSHOT), \
> +       TRACE_SYMBOL(EC_CMD_TP_FRAME_GET), \
> +       TRACE_SYMBOL(EC_CMD_BATTERY_GET_STATIC), \
> +       TRACE_SYMBOL(EC_CMD_BATTERY_GET_DYNAMIC), \
> +       TRACE_SYMBOL(EC_CMD_CHARGER_CONTROL), \
> +       TRACE_SYMBOL(EC_CMD_BOARD_SPECIFIC_BASE), \
> +       TRACE_SYMBOL(EC_CMD_BOARD_SPECIFIC_LAST)
>
>  #define CREATE_TRACE_POINTS
>  #include "cros_ec_trace.h"
> --
> 2.20.1
>

Acked-by: Raul E Rangel <rrangel@chromium.org>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-10-14 18:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-11 13:39 [PATCH] platform/chrome: cros_ec_trace: Match trace commands with EC commands Enric Balletbo i Serra
2019-10-14 18:10 ` Raul Rangel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).