On Sun, May 10, 2026 at 08:41:43PM +0200, Alexander A. Klimov wrote:
manager_sysfs_add_store() passes 6 pointers to sscanf(), but required latter to return 7 which always failed the operation. I corrected it to 6.
Signed-off-by: Alexander A. Klimov grandmaster@al2klimov.de
drivers/staging/greybus/audio_manager_sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/greybus/audio_manager_sysfs.c b/drivers/staging/greybus/audio_manager_sysfs.c index fcd518f954..ff323ca815 100644 --- a/drivers/staging/greybus/audio_manager_sysfs.c +++ b/drivers/staging/greybus/audio_manager_sysfs.c @@ -23,7 +23,7 @@ static ssize_t manager_sysfs_add_store(struct kobject *kobj, desc.name, &desc.vid, &desc.pid, &desc.intf_id, &desc.ip_devices, &desc.op_devices);
- if (num != 7)
if (num != 6) return -EINVAL;
num = gb_audio_manager_add(&desc);
-- 2.54.0
Cool, but was this tested? How has this ever worked?
thanks,
greg k-h