From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qt1-f179.google.com (mail-qt1-f179.google.com [209.85.160.179]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0DB57352F86 for ; Tue, 12 May 2026 20:07:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778616461; cv=none; b=UcXl9C+bvGTzzTCW0tK9Oty+VDp3Po7rjag6cw1Hgb8M96JpiVyKcAVlY3J1SWxlabDjLAhSKqyrvT+dp+Srdn427LfAm5QwfPftxClk/eYFEvmf7e6dhfATow/MYFBRd8MzIiwUUZ7GHVDzu09zv0P0qsqoHW8xfktemOiJj0I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778616461; c=relaxed/simple; bh=K8No+YeWuFkZxveWkNDGdwNXEdXZViioPZHmyw0x7uA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=KzD2tGrdg8HDRD+Ozf3wH3HvmCwvxLHz+VAXBmSOjHNIBFFtU7LFnFUe+Otjy8cH9X93q2dv+8TTnWvC06qNBuBU+EtgBmk7DblYkwYlQ0M1guxxQqbIz2U6GT2WAzh9TKP8kEthBsZpiepW5aOHntFivf0pM83xOifcQ54NnxI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=ieee.org; spf=pass smtp.mailfrom=ieee.org; dkim=pass (1024-bit key) header.d=ieee.org header.i=@ieee.org header.b=VrS9Y3Ee; arc=none smtp.client-ip=209.85.160.179 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=ieee.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ieee.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ieee.org header.i=@ieee.org header.b="VrS9Y3Ee" Received: by mail-qt1-f179.google.com with SMTP id d75a77b69052e-50d75bfb259so40243481cf.1 for ; Tue, 12 May 2026 13:07:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ieee.org; s=google; t=1778616457; x=1779221257; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=5ixm4A9b1aqS3/lbS+x6VLWagRbgpxDZK7p6coKIV70=; b=VrS9Y3EeCjMiIQb7E4c05QmFWmG+ZIkHAYLGGzUgMSvOuEOSEHRbmYe/Vy3lWoJd5Z 5UJ9wkak7h+iR+N6Kk2IlxNc1LrEDOZKfRKe2tGa8U+lKSocBJNzrW7AQk3eyAixQMtW 7GS+fNY94WRWjGryrlusYM2v56GUN7AHQ2eEM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778616457; x=1779221257; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=5ixm4A9b1aqS3/lbS+x6VLWagRbgpxDZK7p6coKIV70=; b=Ogb/Q8J2u1+H2VezIiVmFqf5BQJmd1Dq5hkhSSFovp/QiRekPn1y3Fc3dJAZLkcFLE gG2RNzDpx6C8CzyPRzchDTwHdbYzCo68gfchDe9jT7nLyJHWh9M4qxKy3etR0mrqZg5P jlFN0VBEQfgblQS9bmhXtmxZkRcpnlsAWbALx+2wJcEN2ot3ekb7QecqZu8rlueo394O wv5SA3sHJ7Af4NG2q4cvM2zVyeAGGP98cEtv6Xgh/0bvBqWssICE+QcYwKfcKoN5D69t w9YmcnIzxCfs/YXeBy48Csq7fDWkDeMAZcfgZTOt9qoJlIpSvDJ9gVp9Wlg3Trluafld 9usA== X-Forwarded-Encrypted: i=1; AFNElJ8Aq+GYj7T0rXvTwsNBSHRzOnNjSx7TGisqd8ccF6stNQjEmRTM2zyeoi3pgoHe0W/lhlCHpneHrMRg0cY=@vger.kernel.org X-Gm-Message-State: AOJu0YxFdiw89uhwFeyqRTjpKyvNEWWplK/hghd2YtRcWR2yaBtkfP/p dnDVS2S0KxJEPOXg3lUe/GIbatVySlefC530fzw138TyL3JJ6sklTlw62WYuvH94yw== X-Gm-Gg: Acq92OFYsCX93JmE7Q74XOiOy6QTgsKGwRfN588fUF8SOEaO/BEGFBZ0uX3jTtJCMaI rtt/ehrvG8c4LovBFO0d63/UQJzauzx28WG5Crfotm4QxPn6CDUp8uyCGfj0wJnczLGhccB0vDH INahDSrr9/S0Z6rktroq8h33fIf5P+Zq8SPhSTNz2n58LXnY3c8iJhT1MoXUZXzRKkNnDO5zTwS F3tTT6eKK7UxMUnY39YhbWX4RGqqXXptkMBr4sz/VUD4nc0Qs/eAHjTkEiAztxlpLvRW/MC2arQ qeHZ4ic0wj1lf71yRgxXRzftV2MWcVlvLDs9KRqSgf/aUyAhN++eAW9fqN1KCP+qHcaCFfmnEQR nuw1pFS0hRZhAxKczgnGGzuYIis0bQlMUB2ylTVEIDCTELcy2nW+vACHFIE2/jhlsMXyM+xkkdQ 0eaxrDZbhTCnzlRYr7E+ttfOYMMZR6yNb1lNJRr2tP1s5xpJiFh6QuuUO2DA== X-Received: by 2002:a05:622a:114:b0:50d:8e2c:6a68 with SMTP id d75a77b69052e-5162f62ff6amr1586561cf.51.1778616456762; Tue, 12 May 2026 13:07:36 -0700 (PDT) Received: from [172.22.22.28] (c-75-72-117-212.hsd1.mn.comcast.net. [75.72.117.212]) by smtp.googlemail.com with ESMTPSA id d75a77b69052e-5148e82579fsm125729151cf.24.2026.05.12.13.07.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 12 May 2026 13:07:36 -0700 (PDT) Message-ID: <77358a41-b4cf-4c11-b751-86f024fd4b72@ieee.org> Date: Tue, 12 May 2026 15:07:34 -0500 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3] staging: greybus: audio: correct sscanf() return value check To: "Alexander A. Klimov" , Greg Kroah-Hartman Cc: Dan Carpenter , Vaibhav Agarwal , Mark Greer , Johan Hovold , Alex Elder , greybus-dev@lists.linaro.org, linux-staging@lists.linux.dev, Linux Kernel Mailing List , Pankaj Bharadiya References: <2026051243-factual-lyricism-6c54@gregkh> <381ed71b-1db1-45d3-a9be-bf2f8018a611@al2klimov.de> Content-Language: en-US From: Alex Elder In-Reply-To: <381ed71b-1db1-45d3-a9be-bf2f8018a611@al2klimov.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 5/12/26 2:15 PM, 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. > > Fixes: 49b9137a6002 ("staging: greybus: audio: remove redundant slot > field") > Signed-off-by: Alexander A. Klimov Looks good. Reviewed-by: Alex Elder > --- >  v2: added "Fixes:" to commit message >  v3: added all these v2/v3 lines here as requested by Greg KH >  v3: while on it, replaced title "Greybus audio protocols drivers:" ... >  v3: ... with "staging: greybus: audio:" > >  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);