diff options
| author | Evan Green <evgreen@chromium.org> | 2019-09-23 13:07:06 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-10-05 00:47:48 +0000 |
| commit | 0c2d7b648e1ea433aad8123538632d4edff6eb85 (patch) | |
| tree | 32b535dd6284c9008817a641b728a6784fdceb0c /driver | |
| parent | 3564b23531fc1814924f39e7ac97751d758c14db (diff) | |
| download | chrome-ec-0c2d7b648e1ea433aad8123538632d4edff6eb85.tar.gz | |
printf: Convert %l to %ll
In order to make our printf more standard, utilize %ll for long long
arguments, rather than %l. This does cost a little bit in flash space
for that extra l in a couple of places, but enables us to turn on
compile-time printf format checking.
For this commit only, the semantics are such that both %l and %ll
take 64-bit arguments. In the next commit, %l goes to its correct
behavior of taking a sizeof(long) argument.
BUG=chromium:984041
TEST=make -j buildall
BRANCH=none
Cq-Depend:chrome-internal:1863686,chrome-internal:1860161,chrome-internal:1914029
Change-Id: I18081b55a8dbf5ef8ec15fc499ca75e59d31da58
Signed-off-by: Evan Green <evgreen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1819652
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'driver')
| -rw-r--r-- | driver/touchpad_st.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/driver/touchpad_st.c b/driver/touchpad_st.c index d3381bd92c..a49ab2d7e2 100644 --- a/driver/touchpad_st.c +++ b/driver/touchpad_st.c @@ -544,8 +544,10 @@ static int st_tp_read_system_info(int reload) ST_TP_SHOW(chip0_ver); ST_TP_SHOW(scr_tx_len); ST_TP_SHOW(scr_rx_len); - ST_TP_SHOW(release_info); +#define ST_TP_SHOW64(attr) CPRINTS(#attr ": %04llx", system_info.attr) + ST_TP_SHOW64(release_info); #undef ST_TP_SHOW +#undef ST_TP_SHOW64 return ret; } @@ -1753,7 +1755,7 @@ static int st_tp_usb_set_interface(usb_uint alternate_setting, if (alternate_setting == 1) { if ((system_info.release_info & 0xFF) < ST_TP_MIN_HEATMAP_VERSION) { - CPRINTS("release version %04x doesn't support heatmap", + CPRINTS("release version %04llx doesn't support heatmap", system_info.release_info); /* Heatmap mode is not supported in this version. */ return -1; |