greybus-dev October 2026

greybus-dev@lists.linaro.org

39 participants
60 discussions

[PATCH] staging: greybus: Add newlines to sysfs output

by Daniel West

From 6376e59cb4d0fa2d57ff7f4fb2ce6cdb8bb1d03a Mon Sep 17 00:00:00 2001 From: Daniel West <daniel.west.dev(a)gmail.com> Date: Tue, 27 Oct 2026 15:53:19 -0400 Subject: [PATCH] staging: greybus: Add newlines to sysfs output Add terminating newlines to the sysfs_emit attributes in the Greybus audio manager per warnings from checkpatch. WARNING: return sysfs_emit(...) formats should include a terminating newline Signed-off-by: Daniel West <daniel.west.dev(a)gmail.com> --- drivers/staging/greybus/audio_manager_module.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c index dc90cc2d2308..5737f2a32f5a 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -75,7 +75,7 @@ static void gb_audio_module_release(struct kobject *kobj) static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%s", module->desc.name); + return sysfs_emit(buf, "%s\n", module->desc.name); } static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = @@ -84,7 +84,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = static ssize_t gb_audio_module_vid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.vid); + return sysfs_emit(buf, "%d\n", module->desc.vid); } static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = @@ -93,7 +93,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = static ssize_t gb_audio_module_pid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.pid); + return sysfs_emit(buf, "%d\n", module->desc.pid); } static struct gb_audio_manager_module_attribute gb_audio_module_pid_attribute = @@ -103,7 +103,7 @@ static ssize_t gb_audio_module_intf_id_show(struct gb_audio_manager_module *modu struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.intf_id); + return sysfs_emit(buf, "%d\n", module->desc.intf_id); } static struct gb_audio_manager_module_attribute @@ -114,7 +114,7 @@ static ssize_t gb_audio_module_ip_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.ip_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.ip_devices); } static struct gb_audio_manager_module_attribute @@ -125,7 +125,7 @@ static ssize_t gb_audio_module_op_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.op_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.op_devices); } static struct gb_audio_manager_module_attribute -- 2.51.0

5 days, 1 hour

[PATCH] staging: greybus: light: use sysfs_emit() instead of sprintf()

by Ken

According to Documentation/filesystems/sysfs.rst, the show() callback should use sysfs_emit() instead of sprintf() or snprintf(). Convert fade_*_show() and color_show() to use sysfs_emit() for safer buffer handling and consistency with other greybus drivers. Signed-off-by: Ken <ken3009le(a)gmail.com> --- drivers/staging/greybus/light.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index cab02b5da..2689f9a75 100644 --- a/drivers/staging/greybus/light.c +++ b/drivers/staging/greybus/light.c @@ -173,7 +173,7 @@ static ssize_t fade_##__dir##_show(struct device *dev, \ struct led_classdev *cdev = dev_get_drvdata(dev); \ struct gb_channel *channel = get_channel_from_cdev(cdev); \ \ - return sprintf(buf, "%u\n", channel->fade_##__dir); \ + return sysfs_emit(buf, "%u\n", channel->fade_##__dir); \ } \ \ static ssize_t fade_##__dir##_store(struct device *dev, \ @@ -220,7 +220,7 @@ static ssize_t color_show(struct device *dev, struct device_attribute *attr, struct led_classdev *cdev = dev_get_drvdata(dev); struct gb_channel *channel = get_channel_from_cdev(cdev); - return sprintf(buf, "0x%08x\n", channel->color); + return sysfs_emit(buf, "0x%08x\n", channel->color); } static ssize_t color_store(struct device *dev, struct device_attribute *attr, -- 2.43.0

6 days, 21 hours

[PATCH] staging: greybus: light: use sysfs_emit() instead of sprintf()

by Kenzema

According to Documentation/filesystems/sysfs.rst, the show() callback should use sysfs_emit() instead of sprintf() or snprintf(). Convert fade_*_show() and color_show() to use sysfs_emit() for safer buffer handling and consistency with other greybus drivers. Signed-off-by: Kenzema <ken3009le(a)gmail.com> --- drivers/staging/greybus/light.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index cab02b5da..2689f9a75 100644 --- a/drivers/staging/greybus/light.c +++ b/drivers/staging/greybus/light.c @@ -173,7 +173,7 @@ static ssize_t fade_##__dir##_show(struct device *dev, \ struct led_classdev *cdev = dev_get_drvdata(dev); \ struct gb_channel *channel = get_channel_from_cdev(cdev); \ \ - return sprintf(buf, "%u\n", channel->fade_##__dir); \ + return sysfs_emit(buf, "%u\n", channel->fade_##__dir); \ } \ \ static ssize_t fade_##__dir##_store(struct device *dev, \ @@ -220,7 +220,7 @@ static ssize_t color_show(struct device *dev, struct device_attribute *attr, struct led_classdev *cdev = dev_get_drvdata(dev); struct gb_channel *channel = get_channel_from_cdev(cdev); - return sprintf(buf, "0x%08x\n", channel->color); + return sysfs_emit(buf, "0x%08x\n", channel->color); } static ssize_t color_store(struct device *dev, struct device_attribute *attr, -- 2.43.0

1 week, 3 days

[PATCH] staging: greybus: fix use-after-free in dump_module

by linfordWu

gb_audio_manager_dump_module() drops modules_rwsem before dereferencing the module. A concurrent gb_audio_manager_remove() can free the module between the lookup and the dump. Keep the read lock held while dumping the module so removal cannot free it during the access. Signed-off-by: linfordWu <linfordwu(a)163.com> --- drivers/staging/greybus/audio_manager.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/greybus/audio_manager.c b/drivers/staging/greybus/audio_manager.c index 118ada9b909..1b91acc084f 100644 --- a/drivers/staging/greybus/audio_manager.c +++ b/drivers/staging/greybus/audio_manager.c @@ -117,12 +117,13 @@ int gb_audio_manager_dump_module(int id) down_read(&modules_rwsem); module = gb_audio_manager_get_locked(id); - up_read(&modules_rwsem); - - if (!module) + if (!module) { + up_read(&modules_rwsem); return -EINVAL; + } gb_audio_manager_module_dump(module); + up_read(&modules_rwsem); return 0; } EXPORT_SYMBOL_GPL(gb_audio_manager_dump_module); -- 2.43.0

1 week, 3 days

[PATCH] staging: greybus: fix loopback Kconfig help

by linfordWu

The loopback driver help text describes the debug log class and names gb-log.ko. Update it to describe the loopback class and the actual module name, gb-loopback.ko. Signed-off-by: linfordWu <linfordwu(a)163.com> --- drivers/staging/greybus/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/Kconfig b/drivers/staging/greybus/Kconfig index e1d7ffdf4..3ff475500 100644 --- a/drivers/staging/greybus/Kconfig +++ b/drivers/staging/greybus/Kconfig @@ -75,10 +75,10 @@ config GREYBUS_LOOPBACK tristate "Greybus Loopback Class driver" help Select this option if you have a device that follows the - Greybus Debug Log Class specification. + Greybus Loopback Class specification. To compile this code as a module, chose M here: the module - will be called gb-log.ko + will be called gb-loopback.ko config GREYBUS_POWER tristate "Greybus Powersupply Class driver" -- 2.43.0

1 week, 4 days

[PATCH] staging: greybus: audio_manager: Add newline to sysfs output

by Maria Sheikh

Fix checkpatch warning: WARNING: return sysfs_emit(. . .) formats should include a terminating newline. Add "\n" to the sysfs_emit() format strings of the name, vid, pid, intf_id, ip_devices and op_devices attributes. Signed-off-by: Maria Sheikh <mariasheikh812(a)gmail.com> --- drivers/staging/greybus/audio_manager_module.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c index dc90cc2d2308..5737f2a32f5a 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -75,7 +75,7 @@ static void gb_audio_module_release(struct kobject *kobj) static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%s", module->desc.name); + return sysfs_emit(buf, "%s\n", module->desc.name); } static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = @@ -84,7 +84,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = static ssize_t gb_audio_module_vid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.vid); + return sysfs_emit(buf, "%d\n", module->desc.vid); } static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = @@ -93,7 +93,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = static ssize_t gb_audio_module_pid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.pid); + return sysfs_emit(buf, "%d\n", module->desc.pid); } static struct gb_audio_manager_module_attribute gb_audio_module_pid_attribute = @@ -103,7 +103,7 @@ static ssize_t gb_audio_module_intf_id_show(struct gb_audio_manager_module *modu struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.intf_id); + return sysfs_emit(buf, "%d\n", module->desc.intf_id); } static struct gb_audio_manager_module_attribute @@ -114,7 +114,7 @@ static ssize_t gb_audio_module_ip_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.ip_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.ip_devices); } static struct gb_audio_manager_module_attribute @@ -125,7 +125,7 @@ static ssize_t gb_audio_module_op_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.op_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.op_devices); } static struct gb_audio_manager_module_attribute -- 2.53.0

1 week, 4 days

[PATCH] staging: greybus: audio: add missing newline to sysfs_emit()

by Jose Nieto

The sysfs show() callbacks must emit a trailing newline. Commit 14c91651df34 ("staging: greybus: audio: Use sysfs_emit in show functions") converted them to sysfs_emit() but dropped the newline. Restore it. Signed-off-by: Jose Nieto <jnieto.alaber(a)gmail.com> --- drivers/staging/greybus/audio_manager_module.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c index dc90cc2d2308..5737f2a32f5a 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -75,7 +75,7 @@ static void gb_audio_module_release(struct kobject *kobj) static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%s", module->desc.name); + return sysfs_emit(buf, "%s\n", module->desc.name); } static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = @@ -84,7 +84,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = static ssize_t gb_audio_module_vid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.vid); + return sysfs_emit(buf, "%d\n", module->desc.vid); } static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = @@ -93,7 +93,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = static ssize_t gb_audio_module_pid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.pid); + return sysfs_emit(buf, "%d\n", module->desc.pid); } static struct gb_audio_manager_module_attribute gb_audio_module_pid_attribute = @@ -103,7 +103,7 @@ static ssize_t gb_audio_module_intf_id_show(struct gb_audio_manager_module *modu struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.intf_id); + return sysfs_emit(buf, "%d\n", module->desc.intf_id); } static struct gb_audio_manager_module_attribute @@ -114,7 +114,7 @@ static ssize_t gb_audio_module_ip_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.ip_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.ip_devices); } static struct gb_audio_manager_module_attribute @@ -125,7 +125,7 @@ static ssize_t gb_audio_module_op_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.op_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.op_devices); } static struct gb_audio_manager_module_attribute -- 2.47.3

1 week, 4 days

[PATCH] staging: greybus: audio_manager_module: add missing newline to sysfs_emit()

by Anghel Mihai-Alexandru

Documentation/filesystems/sysfs.rst recommends that single-value sysfs attributes end with a newline, consistent with other Greybus sysfs show() callbacks (see e.g. loopback.c). The name/vid/pid/intf_id/ ip_devices/op_devices attributes were missing it. No functional change. AI-assisted, Claude Sonnet 5. Signed-off-by: Anghel Mihai-Alexandru <mih(a)ianghel.com> --- drivers/staging/greybus/audio_manager_module.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c index dc90cc2d2308..5737f2a32f5a 100644 --- a/drivers/staging/greybus/audio_manager_module.c +++ b/drivers/staging/greybus/audio_manager_module.c @@ -75,7 +75,7 @@ static void gb_audio_module_release(struct kobject *kobj) static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%s", module->desc.name); + return sysfs_emit(buf, "%s\n", module->desc.name); } static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = @@ -84,7 +84,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_name_attribute = static ssize_t gb_audio_module_vid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.vid); + return sysfs_emit(buf, "%d\n", module->desc.vid); } static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = @@ -93,7 +93,7 @@ static struct gb_audio_manager_module_attribute gb_audio_module_vid_attribute = static ssize_t gb_audio_module_pid_show(struct gb_audio_manager_module *module, struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.pid); + return sysfs_emit(buf, "%d\n", module->desc.pid); } static struct gb_audio_manager_module_attribute gb_audio_module_pid_attribute = @@ -103,7 +103,7 @@ static ssize_t gb_audio_module_intf_id_show(struct gb_audio_manager_module *modu struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "%d", module->desc.intf_id); + return sysfs_emit(buf, "%d\n", module->desc.intf_id); } static struct gb_audio_manager_module_attribute @@ -114,7 +114,7 @@ static ssize_t gb_audio_module_ip_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.ip_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.ip_devices); } static struct gb_audio_manager_module_attribute @@ -125,7 +125,7 @@ static ssize_t gb_audio_module_op_devices_show(struct gb_audio_manager_module *m struct gb_audio_manager_module_attribute *attr, char *buf) { - return sysfs_emit(buf, "0x%X", module->desc.op_devices); + return sysfs_emit(buf, "0x%X\n", module->desc.op_devices); } static struct gb_audio_manager_module_attribute -- 2.54.0

2 weeks, 1 day

[PATCH v3] staging: greybus: Replace format specifiers

by pavankumaryalagada＠gmail.com

From: Yalagada Pavan Kumar <pavankumaryalagada(a)gmail.com> Use the appropriate format specifiers for signed and unsigned variables to match their types. Signed-off-by: Yalagada Pavan Kumar <pavankumaryalagada(a)gmail.com> --- v3: - Rechecked the format specifiers using additional compiler warning options and identified further mismatches that were missed in v1. - Corrected format specifiers for signed and unsigned variables, changing %d to %u or %u to %d as appropriate for the corresponding types in v2. - Included additional format specifier corrections in audio_codec.c, audio_topology.c, gbphy.c and loopback.c in v2. - In raw.c, removed unnecessary int casts from payload-size calculation and changed the corresponding format specifier to %zu. This improves consistency and readability, as suggested by Alex. - Omitted the Reviewed-by tag because additional changes have been made that have not yet been reviewed. v2: - Correct format specifiers for signed and unsigned variables. - Include additional format-specifiers corrections in audio_codec.c, audio_topology.c, gbphy.c and loopback.c. v1: https://lore.kernel.org/outreachy/237b20f5-3354-49d7-add8-71170d450306@lina… --- drivers/staging/greybus/audio_codec.c | 24 +++++++++--------- drivers/staging/greybus/audio_topology.c | 32 ++++++++++++------------ drivers/staging/greybus/bootrom.c | 2 +- drivers/staging/greybus/fw-download.c | 2 +- drivers/staging/greybus/fw-management.c | 2 +- drivers/staging/greybus/gbphy.c | 2 +- drivers/staging/greybus/gpio.c | 2 +- drivers/staging/greybus/light.c | 2 +- drivers/staging/greybus/log.c | 4 +-- drivers/staging/greybus/loopback.c | 6 ++--- drivers/staging/greybus/power_supply.c | 2 +- drivers/staging/greybus/raw.c | 6 ++--- drivers/staging/greybus/sdio.c | 4 +-- drivers/staging/greybus/spilib.c | 2 +- drivers/staging/greybus/uart.c | 4 +-- 15 files changed, 48 insertions(+), 48 deletions(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index 6daa4e706792..24b3317f7540 100644 --- a/drivers/staging/greybus/audio_codec.c +++ b/drivers/staging/greybus/audio_codec.c @@ -78,7 +78,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_STARTUP; - dev_dbg(module->dev, "Dynamic Register %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Register %u DAI\n", cportid); } /* hw_params */ @@ -95,7 +95,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_HWPARAMS; - dev_dbg(module->dev, "Dynamic hw_params %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic hw_params %u DAI\n", data_cport); } /* prepare */ @@ -116,7 +116,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_PREPARE; - dev_dbg(module->dev, "Dynamic prepare %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic prepare %u DAI\n", data_cport); } return 0; @@ -146,7 +146,7 @@ static int gbaudio_module_disable_tx(struct gbaudio_module_info *module, int id) "deactivate_tx failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic deactivate %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic deactivate %u DAI\n", data_cport); data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_HWPARAMS; } @@ -161,7 +161,7 @@ static int gbaudio_module_disable_tx(struct gbaudio_module_info *module, int id) "unregister_cport failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic Unregister %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Unregister %u DAI\n", cportid); data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_SHUTDOWN; } @@ -204,7 +204,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_STARTUP; - dev_dbg(module->dev, "Dynamic Register %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Register %u DAI\n", cportid); } /* hw_params */ @@ -221,7 +221,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_HWPARAMS; - dev_dbg(module->dev, "Dynamic hw_params %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic hw_params %u DAI\n", data_cport); } /* prepare */ @@ -243,7 +243,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_PREPARE; - dev_dbg(module->dev, "Dynamic prepare %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic prepare %u DAI\n", data_cport); } return 0; @@ -273,7 +273,7 @@ static int gbaudio_module_disable_rx(struct gbaudio_module_info *module, int id) "deactivate_rx failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic deactivate %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic deactivate %u DAI\n", data_cport); data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_HWPARAMS; } @@ -288,7 +288,7 @@ static int gbaudio_module_disable_rx(struct gbaudio_module_info *module, int id) "unregister_cport failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic Unregister %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Unregister %u DAI\n", cportid); data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_SHUTDOWN; } @@ -414,7 +414,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream, * is supported, validate params before configuring codec */ if (params_channels(hwparams) != 2) { - dev_err(dai->dev, "Invalid channel count:%d\n", + dev_err(dai->dev, "Invalid channel count:%u\n", params_channels(hwparams)); mutex_unlock(&codec->lock); return -EINVAL; @@ -422,7 +422,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream, channels = params_channels(hwparams); if (params_rate(hwparams) != 48000) { - dev_err(dai->dev, "Invalid sampling rate:%d\n", + dev_err(dai->dev, "Invalid sampling rate:%u\n", params_rate(hwparams)); mutex_unlock(&codec->lock); return -EINVAL; diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c index 76146f91cddc..cb05e235b1dc 100644 --- a/drivers/staging/greybus/audio_topology.c +++ b/drivers/staging/greybus/audio_topology.c @@ -199,7 +199,7 @@ static int gbcodec_mixer_ctl_info(struct snd_kcontrol *kcontrol, strscpy(uinfo->value.enumerated.name, name, sizeof(uinfo->value.enumerated.name)); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); break; } @@ -260,7 +260,7 @@ static int gbcodec_mixer_ctl_get(struct snd_kcontrol *kcontrol, le32_to_cpu(gbvalue.value.enumerated_item[1]); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); ret = -EINVAL; break; @@ -307,7 +307,7 @@ static int gbcodec_mixer_ctl_put(struct snd_kcontrol *kcontrol, cpu_to_le32(ucontrol->value.enumerated.item[1]); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); ret = -EINVAL; break; @@ -655,7 +655,7 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, return -ENOMEM; /* debug enum info */ - dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, + dev_dbg(gb->dev, "Max:%u, name_length:%u\n", gbe->items, le16_to_cpu(gb_enum->names_length)); for (i = 0; i < gbe->items; i++) dev_dbg(gb->dev, "src[%d]: %s\n", i, gbe->texts[i]); @@ -700,7 +700,7 @@ static int gbaudio_tplg_create_kcontrol(struct gbaudio_module_info *gb, return -EINVAL; } - dev_dbg(gb->dev, "%s:%d control created\n", ctl->name, ctl->id); + dev_dbg(gb->dev, "%s:%u control created\n", ctl->name, ctl->id); return ret; } @@ -861,7 +861,7 @@ static int gbaudio_tplg_create_enum_ctl(struct gbaudio_module_info *gb, return -ENOMEM; /* debug enum info */ - dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, + dev_dbg(gb->dev, "Max:%u, name_length:%u\n", gbe->items, le16_to_cpu(gb_enum->names_length)); for (i = 0; i < gbe->items; i++) dev_dbg(gb->dev, "src[%d]: %s\n", i, gbe->texts[i]); @@ -914,7 +914,7 @@ static int gbaudio_tplg_create_wcontrol(struct gbaudio_module_info *gb, return -EINVAL; } - dev_dbg(gb->dev, "%s:%d DAPM control created, ret:%d\n", ctl->name, + dev_dbg(gb->dev, "%s:%u DAPM control created, ret:%d\n", ctl->name, ctl->id, ret); return ret; } @@ -1022,7 +1022,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, ret = gbaudio_validate_kcontrol_count(w); if (ret) { - dev_err(module->dev, "Invalid kcontrol count=%d for %s\n", + dev_err(module->dev, "Invalid kcontrol count=%u for %s\n", w->ncontrols, w->name); return ret; } @@ -1044,7 +1044,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, curr); if (ret) { dev_err(module->dev, - "%s:%d type widget_ctl not supported\n", + "%s:%u type widget_ctl not supported\n", curr->name, curr->iface); goto error; } @@ -1123,7 +1123,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, } dw->name = w->name; - dev_dbg(module->dev, "%s: widget of type %d created\n", dw->name, + dev_dbg(module->dev, "%s: widget of type %u created\n", dw->name, dw->id); return 0; error: @@ -1155,7 +1155,7 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, ret = gbaudio_tplg_create_kcontrol(module, &dapm_kctls[i], curr); if (ret) { - dev_err(module->dev, "%s:%d type not supported\n", + dev_err(module->dev, "%s:%u type not supported\n", curr->name, curr->iface); goto error; } @@ -1192,7 +1192,7 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, } list_add(&control->list, &module->ctl_list); - dev_dbg(module->dev, "%d:%s created of type %d\n", curr->id, + dev_dbg(module->dev, "%u:%s created of type %u\n", curr->id, curr->name, curr->info.type); curr = (void *)curr + csize; } @@ -1228,7 +1228,7 @@ static int gbaudio_tplg_process_widgets(struct gbaudio_module_info *module, ret = gbaudio_tplg_create_widget(module, &dapm_widgets[i], curr, &w_size); if (ret) { - dev_err(module->dev, "%s:%d type not supported\n", + dev_err(module->dev, "%s:%u type not supported\n", curr->name, curr->type); goto error; } @@ -1278,7 +1278,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, dapm_routes->sink = gbaudio_map_widgetid(module, curr->destination_id); if (!dapm_routes->sink) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid sink\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid sink\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; @@ -1287,7 +1287,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, dapm_routes->source = gbaudio_map_widgetid(module, curr->source_id); if (!dapm_routes->source) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid source\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid source\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; @@ -1299,7 +1299,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, curr->index); if ((curr->control_id != GBAUDIO_INVALID_ID) && !dapm_routes->control) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid control\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid control\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; diff --git a/drivers/staging/greybus/bootrom.c b/drivers/staging/greybus/bootrom.c index 83921d90c322..fb1fbd3f7b99 100644 --- a/drivers/staging/greybus/bootrom.c +++ b/drivers/staging/greybus/bootrom.c @@ -222,7 +222,7 @@ static int gb_bootrom_firmware_size_request(struct gb_operation *op) size_response = op->response->payload; size_response->size = cpu_to_le32(bootrom->fw->size); - dev_dbg(dev, "%s: firmware size %d bytes\n", + dev_dbg(dev, "%s: firmware size %u bytes\n", __func__, size_response->size); unlock: diff --git a/drivers/staging/greybus/fw-download.c b/drivers/staging/greybus/fw-download.c index a0a683447b3d..9f7f68411418 100644 --- a/drivers/staging/greybus/fw-download.c +++ b/drivers/staging/greybus/fw-download.c @@ -148,7 +148,7 @@ static int exceeds_release_timeout(struct fw_request *fw_req) return 0; dev_err(fw_download->parent, - "Firmware download didn't finish in time, abort: %d\n", + "Firmware download didn't finish in time, abort: %u\n", fw_req->firmware_id); fw_req->timedout = true; diff --git a/drivers/staging/greybus/fw-management.c b/drivers/staging/greybus/fw-management.c index 5d01da6b6eb9..ee86ed9686c4 100644 --- a/drivers/staging/greybus/fw-management.c +++ b/drivers/staging/greybus/fw-management.c @@ -141,7 +141,7 @@ static int fw_mgmt_load_and_validate_operation(struct fw_mgmt *fw_mgmt, if (load_method != GB_FW_LOAD_METHOD_UNIPRO && load_method != GB_FW_LOAD_METHOD_INTERNAL) { dev_err(fw_mgmt->parent, - "invalid load-method (%d)\n", load_method); + "invalid load-method (%u)\n", load_method); return -EINVAL; } diff --git a/drivers/staging/greybus/gbphy.c b/drivers/staging/greybus/gbphy.c index bb9a5b538e6e..ac77b6a12ffe 100644 --- a/drivers/staging/greybus/gbphy.c +++ b/drivers/staging/greybus/gbphy.c @@ -80,7 +80,7 @@ static int gbphy_dev_uevent(const struct device *dev, struct kobj_uevent_env *en const struct gb_module *module = intf->module; const struct gb_host_device *hd = intf->hd; - if (add_uevent_var(env, "BUS=%u", hd->bus_id)) + if (add_uevent_var(env, "BUS=%d", hd->bus_id)) return -ENOMEM; if (add_uevent_var(env, "MODULE=%u", module->module_id)) return -ENOMEM; diff --git a/drivers/staging/greybus/gpio.c b/drivers/staging/greybus/gpio.c index 12185f7a982c..7616bcc23c6c 100644 --- a/drivers/staging/greybus/gpio.c +++ b/drivers/staging/greybus/gpio.c @@ -379,7 +379,7 @@ static int gb_gpio_request_handler(struct gb_operation *op) event = request->payload; if (event->which > ggc->line_max) { - dev_err(dev, "invalid hw irq: %d\n", event->which); + dev_err(dev, "invalid hw irq: %u\n", event->which); return -EINVAL; } diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index cab02b5da867..1fbd80523d4e 100644 --- a/drivers/staging/greybus/light.c +++ b/drivers/staging/greybus/light.c @@ -1221,7 +1221,7 @@ static int gb_lights_request_handler(struct gb_operation *op) if (light_id >= glights->lights_count || !glights->lights[light_id].ready) { - dev_err(dev, "Event received for unconfigured light id: %d\n", + dev_err(dev, "Event received for unconfigured light id: %u\n", light_id); return -EINVAL; } diff --git a/drivers/staging/greybus/log.c b/drivers/staging/greybus/log.c index 815d20ba9b6b..24f5d5cf21be 100644 --- a/drivers/staging/greybus/log.c +++ b/drivers/staging/greybus/log.c @@ -36,7 +36,7 @@ static int gb_log_request_handler(struct gb_operation *op) receive = op->request->payload; len = le16_to_cpu(receive->len); if (len != (op->request->payload_size - sizeof(*receive))) { - dev_err(dev, "log request wrong size %d vs %zu\n", len, + dev_err(dev, "log request wrong size %u vs %zu\n", len, (op->request->payload_size - sizeof(*receive))); return -EINVAL; } @@ -46,7 +46,7 @@ static int gb_log_request_handler(struct gb_operation *op) } if (len > GB_LOG_MAX_LEN) { - dev_err(dev, "log request too big: %d\n", len); + dev_err(dev, "log request too big: %u\n", len); return -EINVAL; } diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index efc69c10cffe..41a8b4d3b7bf 100644 --- a/drivers/staging/greybus/loopback.c +++ b/drivers/staging/greybus/loopback.c @@ -281,7 +281,7 @@ gb_dev_loopback_rw_attr(iteration_max, u); /* The current index of the for (i = 0; i < iteration_max; i++) loop */ gb_dev_loopback_ro_attr(iteration_count); /* A flag to indicate synchronous or asynchronous operations */ -gb_dev_loopback_rw_attr(async, u); +gb_dev_loopback_rw_attr(async, d); /* Timeout of an individual asynchronous request */ gb_dev_loopback_rw_attr(timeout, u); /* Maximum number of in-flight operations before back-off */ @@ -424,7 +424,7 @@ static void gb_loopback_async_operation_callback(struct gb_operation *operation) mutex_unlock(&gb->mutex); - dev_dbg(&gb->connection->bundle->dev, "complete operation %d\n", + dev_dbg(&gb->connection->bundle->dev, "complete operation %u\n", operation->id); /* Wake up waiters */ @@ -579,7 +579,7 @@ static int gb_loopback_async_transfer_complete(struct gb_loopback_async_operatio if (memcmp(request->data, response->data, len)) { dev_err(&gb->connection->bundle->dev, - "Loopback Data doesn't match operation id %d\n", + "Loopback Data doesn't match operation id %u\n", operation->id); retval = -EREMOTEIO; } else { diff --git a/drivers/staging/greybus/power_supply.c b/drivers/staging/greybus/power_supply.c index beefcbaf3681..1eeccf452eaf 100644 --- a/drivers/staging/greybus/power_supply.c +++ b/drivers/staging/greybus/power_supply.c @@ -1011,7 +1011,7 @@ static int gb_supplies_request_handler(struct gb_operation *op) if (psy_id >= supplies->supplies_count || !supplies->supply[psy_id].registered) { dev_err(&connection->bundle->dev, - "Event received for unconfigured power_supply id: %d\n", + "Event received for unconfigured power_supply id: %u\n", psy_id); ret = -EINVAL; goto out_unlock; diff --git a/drivers/staging/greybus/raw.c b/drivers/staging/greybus/raw.c index 4f1b3f4db404..2d23f6324639 100644 --- a/drivers/staging/greybus/raw.c +++ b/drivers/staging/greybus/raw.c @@ -111,9 +111,9 @@ static int gb_raw_request_handler(struct gb_operation *op) } receive = op->request->payload; len = le32_to_cpu(receive->len); - if (len != (int)(op->request->payload_size - sizeof(__le32))) { - dev_err(dev, "raw receive request wrong size %d vs %d\n", len, - (int)(op->request->payload_size - sizeof(__le32))); + if (len != (op->request->payload_size - sizeof(__le32))) { + dev_err(dev, "raw receive request wrong size %u vs %zu\n", len, + (op->request->payload_size - sizeof(__le32))); return -EINVAL; } if (len == 0) { diff --git a/drivers/staging/greybus/sdio.c b/drivers/staging/greybus/sdio.c index 3952f3d225db..773ec1f72407 100644 --- a/drivers/staging/greybus/sdio.c +++ b/drivers/staging/greybus/sdio.c @@ -294,7 +294,7 @@ static int _gb_sdio_send(struct gb_sdio_host *host, struct mmc_data *data, send_blksz = le16_to_cpu(response->data_blksz); if (len != send_blksz * send_blocks) { - dev_err(mmc_dev(host->mmc), "send: size received: %zu != %d\n", + dev_err(mmc_dev(host->mmc), "send: size received: %zu != %u\n", len, send_blksz * send_blocks); ret = -EINVAL; } @@ -340,7 +340,7 @@ static int _gb_sdio_recv(struct gb_sdio_host *host, struct mmc_data *data, recv_blksz = le16_to_cpu(response->data_blksz); if (len != recv_blksz * recv_blocks) { - dev_err(mmc_dev(host->mmc), "recv: size received: %d != %zu\n", + dev_err(mmc_dev(host->mmc), "recv: size received: %u != %zu\n", recv_blksz * recv_blocks, len); ret = -EINVAL; goto err_put_operation; diff --git a/drivers/staging/greybus/spilib.c b/drivers/staging/greybus/spilib.c index c1982f3fb931..6e34757d9051 100644 --- a/drivers/staging/greybus/spilib.c +++ b/drivers/staging/greybus/spilib.c @@ -540,7 +540,7 @@ int gb_spilib_master_init(struct gb_connection *connection, struct device *dev, for (i = 0; i < spi->num_chipselect; i++) { ret = gb_spi_setup_device(spi, i); if (ret < 0) { - dev_err(dev, "failed to allocate spi device %d: %d\n", + dev_err(dev, "failed to allocate spi device %u: %d\n", i, ret); goto exit_spi_unregister; } diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c index 2448fc9098d6..442aad685d33 100644 --- a/drivers/staging/greybus/uart.c +++ b/drivers/staging/greybus/uart.c @@ -176,7 +176,7 @@ static int gb_uart_receive_credits_handler(struct gb_operation *op) if (ret) { dev_err(&gb_tty->gbphy_dev->dev, - "invalid number of incoming credits: %d\n", + "invalid number of incoming credits: %u\n", incoming_credits); return ret; } @@ -300,7 +300,7 @@ static int send_break(struct gb_tty *gb_tty, u8 state) if ((state != 0) && (state != 1)) { dev_err(&gb_tty->gbphy_dev->dev, - "invalid break state of %d\n", state); + "invalid break state of %u\n", state); return -EINVAL; } -- 2.43.0

2 weeks, 2 days

[PATCH v2] staging: greybus: Replace format specifiers

by pavankumaryalagada＠gmail.com

From: Yalagada Pavan Kumar <pavankumaryalagada(a)gmail.com> Use the appropriate format specifiers for signed and unsigned variables to match their types. Signed-off-by: Yalagada Pavan Kumar <pavankumaryalagada(a)gmail.com> --- v2: - Correct format specifiers for signed and unsigned variables. - Include additional format-specifiers corrections in audio_codec.c, audio_topology.c, gbphy.c and loopback.c. v1: https://lore.kernel.org/outreachy/237b20f5-3354-49d7-add8-71170d450306@lina… --- drivers/staging/greybus/audio_codec.c | 24 +++++++++--------- drivers/staging/greybus/audio_topology.c | 32 ++++++++++++------------ drivers/staging/greybus/bootrom.c | 2 +- drivers/staging/greybus/fw-download.c | 2 +- drivers/staging/greybus/fw-management.c | 2 +- drivers/staging/greybus/gbphy.c | 2 +- drivers/staging/greybus/gpio.c | 2 +- drivers/staging/greybus/light.c | 2 +- drivers/staging/greybus/log.c | 4 +-- drivers/staging/greybus/loopback.c | 6 ++--- drivers/staging/greybus/power_supply.c | 2 +- drivers/staging/greybus/raw.c | 2 +- drivers/staging/greybus/sdio.c | 4 +-- drivers/staging/greybus/spilib.c | 2 +- drivers/staging/greybus/uart.c | 4 +-- 15 files changed, 46 insertions(+), 46 deletions(-) diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c index 6daa4e706792..24b3317f7540 100644 --- a/drivers/staging/greybus/audio_codec.c +++ b/drivers/staging/greybus/audio_codec.c @@ -78,7 +78,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_STARTUP; - dev_dbg(module->dev, "Dynamic Register %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Register %u DAI\n", cportid); } /* hw_params */ @@ -95,7 +95,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_HWPARAMS; - dev_dbg(module->dev, "Dynamic hw_params %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic hw_params %u DAI\n", data_cport); } /* prepare */ @@ -116,7 +116,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_PREPARE; - dev_dbg(module->dev, "Dynamic prepare %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic prepare %u DAI\n", data_cport); } return 0; @@ -146,7 +146,7 @@ static int gbaudio_module_disable_tx(struct gbaudio_module_info *module, int id) "deactivate_tx failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic deactivate %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic deactivate %u DAI\n", data_cport); data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_HWPARAMS; } @@ -161,7 +161,7 @@ static int gbaudio_module_disable_tx(struct gbaudio_module_info *module, int id) "unregister_cport failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic Unregister %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Unregister %u DAI\n", cportid); data->state[SNDRV_PCM_STREAM_PLAYBACK] = GBAUDIO_CODEC_SHUTDOWN; } @@ -204,7 +204,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_STARTUP; - dev_dbg(module->dev, "Dynamic Register %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Register %u DAI\n", cportid); } /* hw_params */ @@ -221,7 +221,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_HWPARAMS; - dev_dbg(module->dev, "Dynamic hw_params %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic hw_params %u DAI\n", data_cport); } /* prepare */ @@ -243,7 +243,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec, return ret; } data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_PREPARE; - dev_dbg(module->dev, "Dynamic prepare %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic prepare %u DAI\n", data_cport); } return 0; @@ -273,7 +273,7 @@ static int gbaudio_module_disable_rx(struct gbaudio_module_info *module, int id) "deactivate_rx failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic deactivate %d DAI\n", data_cport); + dev_dbg(module->dev, "Dynamic deactivate %u DAI\n", data_cport); data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_HWPARAMS; } @@ -288,7 +288,7 @@ static int gbaudio_module_disable_rx(struct gbaudio_module_info *module, int id) "unregister_cport failed:%d\n", ret); return ret; } - dev_dbg(module->dev, "Dynamic Unregister %d DAI\n", cportid); + dev_dbg(module->dev, "Dynamic Unregister %u DAI\n", cportid); data->state[SNDRV_PCM_STREAM_CAPTURE] = GBAUDIO_CODEC_SHUTDOWN; } @@ -414,7 +414,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream, * is supported, validate params before configuring codec */ if (params_channels(hwparams) != 2) { - dev_err(dai->dev, "Invalid channel count:%d\n", + dev_err(dai->dev, "Invalid channel count:%u\n", params_channels(hwparams)); mutex_unlock(&codec->lock); return -EINVAL; @@ -422,7 +422,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream, channels = params_channels(hwparams); if (params_rate(hwparams) != 48000) { - dev_err(dai->dev, "Invalid sampling rate:%d\n", + dev_err(dai->dev, "Invalid sampling rate:%u\n", params_rate(hwparams)); mutex_unlock(&codec->lock); return -EINVAL; diff --git a/drivers/staging/greybus/audio_topology.c b/drivers/staging/greybus/audio_topology.c index 76146f91cddc..cb05e235b1dc 100644 --- a/drivers/staging/greybus/audio_topology.c +++ b/drivers/staging/greybus/audio_topology.c @@ -199,7 +199,7 @@ static int gbcodec_mixer_ctl_info(struct snd_kcontrol *kcontrol, strscpy(uinfo->value.enumerated.name, name, sizeof(uinfo->value.enumerated.name)); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); break; } @@ -260,7 +260,7 @@ static int gbcodec_mixer_ctl_get(struct snd_kcontrol *kcontrol, le32_to_cpu(gbvalue.value.enumerated_item[1]); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); ret = -EINVAL; break; @@ -307,7 +307,7 @@ static int gbcodec_mixer_ctl_put(struct snd_kcontrol *kcontrol, cpu_to_le32(ucontrol->value.enumerated.item[1]); break; default: - dev_err(comp->dev, "Invalid type: %d for %s:kcontrol\n", + dev_err(comp->dev, "Invalid type: %u for %s:kcontrol\n", info->type, kcontrol->id.name); ret = -EINVAL; break; @@ -655,7 +655,7 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, return -ENOMEM; /* debug enum info */ - dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, + dev_dbg(gb->dev, "Max:%u, name_length:%u\n", gbe->items, le16_to_cpu(gb_enum->names_length)); for (i = 0; i < gbe->items; i++) dev_dbg(gb->dev, "src[%d]: %s\n", i, gbe->texts[i]); @@ -700,7 +700,7 @@ static int gbaudio_tplg_create_kcontrol(struct gbaudio_module_info *gb, return -EINVAL; } - dev_dbg(gb->dev, "%s:%d control created\n", ctl->name, ctl->id); + dev_dbg(gb->dev, "%s:%u control created\n", ctl->name, ctl->id); return ret; } @@ -861,7 +861,7 @@ static int gbaudio_tplg_create_enum_ctl(struct gbaudio_module_info *gb, return -ENOMEM; /* debug enum info */ - dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gbe->items, + dev_dbg(gb->dev, "Max:%u, name_length:%u\n", gbe->items, le16_to_cpu(gb_enum->names_length)); for (i = 0; i < gbe->items; i++) dev_dbg(gb->dev, "src[%d]: %s\n", i, gbe->texts[i]); @@ -914,7 +914,7 @@ static int gbaudio_tplg_create_wcontrol(struct gbaudio_module_info *gb, return -EINVAL; } - dev_dbg(gb->dev, "%s:%d DAPM control created, ret:%d\n", ctl->name, + dev_dbg(gb->dev, "%s:%u DAPM control created, ret:%d\n", ctl->name, ctl->id, ret); return ret; } @@ -1022,7 +1022,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, ret = gbaudio_validate_kcontrol_count(w); if (ret) { - dev_err(module->dev, "Invalid kcontrol count=%d for %s\n", + dev_err(module->dev, "Invalid kcontrol count=%u for %s\n", w->ncontrols, w->name); return ret; } @@ -1044,7 +1044,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, curr); if (ret) { dev_err(module->dev, - "%s:%d type widget_ctl not supported\n", + "%s:%u type widget_ctl not supported\n", curr->name, curr->iface); goto error; } @@ -1123,7 +1123,7 @@ static int gbaudio_tplg_create_widget(struct gbaudio_module_info *module, } dw->name = w->name; - dev_dbg(module->dev, "%s: widget of type %d created\n", dw->name, + dev_dbg(module->dev, "%s: widget of type %u created\n", dw->name, dw->id); return 0; error: @@ -1155,7 +1155,7 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, ret = gbaudio_tplg_create_kcontrol(module, &dapm_kctls[i], curr); if (ret) { - dev_err(module->dev, "%s:%d type not supported\n", + dev_err(module->dev, "%s:%u type not supported\n", curr->name, curr->iface); goto error; } @@ -1192,7 +1192,7 @@ static int gbaudio_tplg_process_kcontrols(struct gbaudio_module_info *module, } list_add(&control->list, &module->ctl_list); - dev_dbg(module->dev, "%d:%s created of type %d\n", curr->id, + dev_dbg(module->dev, "%u:%s created of type %u\n", curr->id, curr->name, curr->info.type); curr = (void *)curr + csize; } @@ -1228,7 +1228,7 @@ static int gbaudio_tplg_process_widgets(struct gbaudio_module_info *module, ret = gbaudio_tplg_create_widget(module, &dapm_widgets[i], curr, &w_size); if (ret) { - dev_err(module->dev, "%s:%d type not supported\n", + dev_err(module->dev, "%s:%u type not supported\n", curr->name, curr->type); goto error; } @@ -1278,7 +1278,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, dapm_routes->sink = gbaudio_map_widgetid(module, curr->destination_id); if (!dapm_routes->sink) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid sink\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid sink\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; @@ -1287,7 +1287,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, dapm_routes->source = gbaudio_map_widgetid(module, curr->source_id); if (!dapm_routes->source) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid source\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid source\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; @@ -1299,7 +1299,7 @@ static int gbaudio_tplg_process_routes(struct gbaudio_module_info *module, curr->index); if ((curr->control_id != GBAUDIO_INVALID_ID) && !dapm_routes->control) { - dev_err(module->dev, "%d:%d:%d:%d - Invalid control\n", + dev_err(module->dev, "%u:%u:%u:%u - Invalid control\n", curr->source_id, curr->destination_id, curr->control_id, curr->index); ret = -EINVAL; diff --git a/drivers/staging/greybus/bootrom.c b/drivers/staging/greybus/bootrom.c index 83921d90c322..fb1fbd3f7b99 100644 --- a/drivers/staging/greybus/bootrom.c +++ b/drivers/staging/greybus/bootrom.c @@ -222,7 +222,7 @@ static int gb_bootrom_firmware_size_request(struct gb_operation *op) size_response = op->response->payload; size_response->size = cpu_to_le32(bootrom->fw->size); - dev_dbg(dev, "%s: firmware size %d bytes\n", + dev_dbg(dev, "%s: firmware size %u bytes\n", __func__, size_response->size); unlock: diff --git a/drivers/staging/greybus/fw-download.c b/drivers/staging/greybus/fw-download.c index a0a683447b3d..9f7f68411418 100644 --- a/drivers/staging/greybus/fw-download.c +++ b/drivers/staging/greybus/fw-download.c @@ -148,7 +148,7 @@ static int exceeds_release_timeout(struct fw_request *fw_req) return 0; dev_err(fw_download->parent, - "Firmware download didn't finish in time, abort: %d\n", + "Firmware download didn't finish in time, abort: %u\n", fw_req->firmware_id); fw_req->timedout = true; diff --git a/drivers/staging/greybus/fw-management.c b/drivers/staging/greybus/fw-management.c index 5d01da6b6eb9..ee86ed9686c4 100644 --- a/drivers/staging/greybus/fw-management.c +++ b/drivers/staging/greybus/fw-management.c @@ -141,7 +141,7 @@ static int fw_mgmt_load_and_validate_operation(struct fw_mgmt *fw_mgmt, if (load_method != GB_FW_LOAD_METHOD_UNIPRO && load_method != GB_FW_LOAD_METHOD_INTERNAL) { dev_err(fw_mgmt->parent, - "invalid load-method (%d)\n", load_method); + "invalid load-method (%u)\n", load_method); return -EINVAL; } diff --git a/drivers/staging/greybus/gbphy.c b/drivers/staging/greybus/gbphy.c index bb9a5b538e6e..ac77b6a12ffe 100644 --- a/drivers/staging/greybus/gbphy.c +++ b/drivers/staging/greybus/gbphy.c @@ -80,7 +80,7 @@ static int gbphy_dev_uevent(const struct device *dev, struct kobj_uevent_env *en const struct gb_module *module = intf->module; const struct gb_host_device *hd = intf->hd; - if (add_uevent_var(env, "BUS=%u", hd->bus_id)) + if (add_uevent_var(env, "BUS=%d", hd->bus_id)) return -ENOMEM; if (add_uevent_var(env, "MODULE=%u", module->module_id)) return -ENOMEM; diff --git a/drivers/staging/greybus/gpio.c b/drivers/staging/greybus/gpio.c index 12185f7a982c..7616bcc23c6c 100644 --- a/drivers/staging/greybus/gpio.c +++ b/drivers/staging/greybus/gpio.c @@ -379,7 +379,7 @@ static int gb_gpio_request_handler(struct gb_operation *op) event = request->payload; if (event->which > ggc->line_max) { - dev_err(dev, "invalid hw irq: %d\n", event->which); + dev_err(dev, "invalid hw irq: %u\n", event->which); return -EINVAL; } diff --git a/drivers/staging/greybus/light.c b/drivers/staging/greybus/light.c index cab02b5da867..1fbd80523d4e 100644 --- a/drivers/staging/greybus/light.c +++ b/drivers/staging/greybus/light.c @@ -1221,7 +1221,7 @@ static int gb_lights_request_handler(struct gb_operation *op) if (light_id >= glights->lights_count || !glights->lights[light_id].ready) { - dev_err(dev, "Event received for unconfigured light id: %d\n", + dev_err(dev, "Event received for unconfigured light id: %u\n", light_id); return -EINVAL; } diff --git a/drivers/staging/greybus/log.c b/drivers/staging/greybus/log.c index 815d20ba9b6b..24f5d5cf21be 100644 --- a/drivers/staging/greybus/log.c +++ b/drivers/staging/greybus/log.c @@ -36,7 +36,7 @@ static int gb_log_request_handler(struct gb_operation *op) receive = op->request->payload; len = le16_to_cpu(receive->len); if (len != (op->request->payload_size - sizeof(*receive))) { - dev_err(dev, "log request wrong size %d vs %zu\n", len, + dev_err(dev, "log request wrong size %u vs %zu\n", len, (op->request->payload_size - sizeof(*receive))); return -EINVAL; } @@ -46,7 +46,7 @@ static int gb_log_request_handler(struct gb_operation *op) } if (len > GB_LOG_MAX_LEN) { - dev_err(dev, "log request too big: %d\n", len); + dev_err(dev, "log request too big: %u\n", len); return -EINVAL; } diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index efc69c10cffe..41a8b4d3b7bf 100644 --- a/drivers/staging/greybus/loopback.c +++ b/drivers/staging/greybus/loopback.c @@ -281,7 +281,7 @@ gb_dev_loopback_rw_attr(iteration_max, u); /* The current index of the for (i = 0; i < iteration_max; i++) loop */ gb_dev_loopback_ro_attr(iteration_count); /* A flag to indicate synchronous or asynchronous operations */ -gb_dev_loopback_rw_attr(async, u); +gb_dev_loopback_rw_attr(async, d); /* Timeout of an individual asynchronous request */ gb_dev_loopback_rw_attr(timeout, u); /* Maximum number of in-flight operations before back-off */ @@ -424,7 +424,7 @@ static void gb_loopback_async_operation_callback(struct gb_operation *operation) mutex_unlock(&gb->mutex); - dev_dbg(&gb->connection->bundle->dev, "complete operation %d\n", + dev_dbg(&gb->connection->bundle->dev, "complete operation %u\n", operation->id); /* Wake up waiters */ @@ -579,7 +579,7 @@ static int gb_loopback_async_transfer_complete(struct gb_loopback_async_operatio if (memcmp(request->data, response->data, len)) { dev_err(&gb->connection->bundle->dev, - "Loopback Data doesn't match operation id %d\n", + "Loopback Data doesn't match operation id %u\n", operation->id); retval = -EREMOTEIO; } else { diff --git a/drivers/staging/greybus/power_supply.c b/drivers/staging/greybus/power_supply.c index beefcbaf3681..1eeccf452eaf 100644 --- a/drivers/staging/greybus/power_supply.c +++ b/drivers/staging/greybus/power_supply.c @@ -1011,7 +1011,7 @@ static int gb_supplies_request_handler(struct gb_operation *op) if (psy_id >= supplies->supplies_count || !supplies->supply[psy_id].registered) { dev_err(&connection->bundle->dev, - "Event received for unconfigured power_supply id: %d\n", + "Event received for unconfigured power_supply id: %u\n", psy_id); ret = -EINVAL; goto out_unlock; diff --git a/drivers/staging/greybus/raw.c b/drivers/staging/greybus/raw.c index 4f1b3f4db404..2314496a858a 100644 --- a/drivers/staging/greybus/raw.c +++ b/drivers/staging/greybus/raw.c @@ -112,7 +112,7 @@ static int gb_raw_request_handler(struct gb_operation *op) receive = op->request->payload; len = le32_to_cpu(receive->len); if (len != (int)(op->request->payload_size - sizeof(__le32))) { - dev_err(dev, "raw receive request wrong size %d vs %d\n", len, + dev_err(dev, "raw receive request wrong size %u vs %d\n", len, (int)(op->request->payload_size - sizeof(__le32))); return -EINVAL; } diff --git a/drivers/staging/greybus/sdio.c b/drivers/staging/greybus/sdio.c index 3952f3d225db..773ec1f72407 100644 --- a/drivers/staging/greybus/sdio.c +++ b/drivers/staging/greybus/sdio.c @@ -294,7 +294,7 @@ static int _gb_sdio_send(struct gb_sdio_host *host, struct mmc_data *data, send_blksz = le16_to_cpu(response->data_blksz); if (len != send_blksz * send_blocks) { - dev_err(mmc_dev(host->mmc), "send: size received: %zu != %d\n", + dev_err(mmc_dev(host->mmc), "send: size received: %zu != %u\n", len, send_blksz * send_blocks); ret = -EINVAL; } @@ -340,7 +340,7 @@ static int _gb_sdio_recv(struct gb_sdio_host *host, struct mmc_data *data, recv_blksz = le16_to_cpu(response->data_blksz); if (len != recv_blksz * recv_blocks) { - dev_err(mmc_dev(host->mmc), "recv: size received: %d != %zu\n", + dev_err(mmc_dev(host->mmc), "recv: size received: %u != %zu\n", recv_blksz * recv_blocks, len); ret = -EINVAL; goto err_put_operation; diff --git a/drivers/staging/greybus/spilib.c b/drivers/staging/greybus/spilib.c index c1982f3fb931..6e34757d9051 100644 --- a/drivers/staging/greybus/spilib.c +++ b/drivers/staging/greybus/spilib.c @@ -540,7 +540,7 @@ int gb_spilib_master_init(struct gb_connection *connection, struct device *dev, for (i = 0; i < spi->num_chipselect; i++) { ret = gb_spi_setup_device(spi, i); if (ret < 0) { - dev_err(dev, "failed to allocate spi device %d: %d\n", + dev_err(dev, "failed to allocate spi device %u: %d\n", i, ret); goto exit_spi_unregister; } diff --git a/drivers/staging/greybus/uart.c b/drivers/staging/greybus/uart.c index 2448fc9098d6..442aad685d33 100644 --- a/drivers/staging/greybus/uart.c +++ b/drivers/staging/greybus/uart.c @@ -176,7 +176,7 @@ static int gb_uart_receive_credits_handler(struct gb_operation *op) if (ret) { dev_err(&gb_tty->gbphy_dev->dev, - "invalid number of incoming credits: %d\n", + "invalid number of incoming credits: %u\n", incoming_credits); return ret; } @@ -300,7 +300,7 @@ static int send_break(struct gb_tty *gb_tty, u8 state) if ((state != 0) && (state != 1)) { dev_err(&gb_tty->gbphy_dev->dev, - "invalid break state of %d\n", state); + "invalid break state of %u\n", state); return -EINVAL; } -- 2.43.0

2 weeks, 2 days

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

greybus-dev October 2026