From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oo1-f51.google.com (mail-oo1-f51.google.com [209.85.161.51]) (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 4EEAB212D68 for ; Thu, 20 Feb 2025 17:32:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740072736; cv=none; b=mxqt8x7D9Qr1XlsOqUmVo16J/3rHx9PnwspXu3JTYBws9QJikRLgoMNst/heOW+tWBI7FptdvbQwiV0ELGls8MLnd+E0wm49ThxbQSvcCZzVuo5N+HPzWeph8wD+I0uALejng+KJbc0KOZDh8XDiu+pkQWibhcXD7OjtADE+NGs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740072736; c=relaxed/simple; bh=3sYO6hLTbz1Yx4jhh2H9VLVNmtvcj0Yg/VTm0rvs6nk=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZD5ZM9hZGeAsSnVs+uiS9r0jMgzbiRsAdPdcqbwZ4NdCdd6PzH2rN86xFYhzx4HQrswEbRs++RPng4TR03Fum6MF9teWGypYEw24Mm9Tmpm67zBp9oQL7Sd1rIMGybBEnWphlnTnBMEUpQbXOw7OHWpBjeRJqitoNG/Gu78WBPw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=nbJmFXiM; arc=none smtp.client-ip=209.85.161.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="nbJmFXiM" Received: by mail-oo1-f51.google.com with SMTP id 006d021491bc7-5fcd686fe0fso734166eaf.3 for ; Thu, 20 Feb 2025 09:32:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1740072733; x=1740677533; 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=AIc+j8TxBrdPej0mz926UOoeIMB5TisIsEjQp/BF3M8=; b=nbJmFXiMhYuFOf79oDrqVL0kUcFacWOkUYIIIVtNWcAouw843PQE6H6udff9h7i+gU K4eFJbkxtrv33IsMJtypin5N4rSk0sdRI+ttHI/mxSDO4A0KnN4PzZ/P3HpBfceM97AN ZSFW8VtWZMDA1DKxgTQ0KjPIWp5/P1usyYosKLxCjuVu4/4KU9tdMgYQyt7hqqGP2/JS XniHexCZveNbWFfTk4e5SAcDRtuwuZtXbx1TAY8A889gGMqIMoOECdNpUlX6+oR1TmrQ ncy6dQ1Cxw4daDyqeZsTLbhTWMqIFz3GX9XN+ELoQoHLWvGmr2inL/N9EN9OVTsJ+xkM tzOA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740072733; x=1740677533; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=AIc+j8TxBrdPej0mz926UOoeIMB5TisIsEjQp/BF3M8=; b=tiPGFabD7gFFQTnOCUZuke32Jk06o5BE/SLlce6aYJbzlcBiVrM/HMkis3rIq3LHJb iyVAaIgFDCp+nh7rzhjDsBwg+Q1qtOQLQQElSFp6/yVf6wQ4WVFAk54pjfGHchXxCGnN JzVgJ14ShAqT3+fvToClWKc5+u2OmdOVz5QZeCjdqmjO7ybiyZG0ngQ5kLRg46lMCbv5 hj9dkqVlcMazjLwW2EUh3z1O78Lm+LjUzqEcYR0cyLV/5Me0US1b9UyoT+dGr4/DPgSl P0jYxIx3bmBu5Z8CG/XrEU8Bkx78/xoZV8OP8kBdp/bWA11o1WBmvxmU98v5YHoZrRo2 EcUA== X-Forwarded-Encrypted: i=1; AJvYcCVhHCNNYcKB91dwXqFWG9l+LQIBhkNchTQrE5S6PSBgBguwJKux3K4Foumf4E5y5tSznrdGiefc89E=@vger.kernel.org X-Gm-Message-State: AOJu0YzNzPZgRWc67LD1xaBQHKBxtesNY529o+++SnQKNDmFlwLgKgEO q/fkUaFc+LHdrxYDk74To+vkADXOHOGQh3+6PGWKCdZj+jSUxAQVbg0sNxm49hI= X-Gm-Gg: ASbGncsSE77vos9ItmpTRBOaptimdUi6YWn1flkXvAei/RjrYpGvV8EA1BrkSGn0YDC v3oH4NyM5fxorDD09GGm5S3I7izxmViMHmBH26yjqD8z3Wsagnz7p2wIkSF4LiIaNQO3ZhRhrl5 mIBmhb6+EdVFAgYhm+TEo8r2AQ+0qJs5HhCTx8tISpaauuBmYNbJVntOO7VmSIfQp/yC+9I1FJ7 MxrLoWJCNBhG/0zOPoS2cTUNuiL18zgPALfMwn6sr5I671UD+sCD/+bXM9l+cKr1Gy88/LQgY0R k7QSFLdKIU9smjd214499Oa3FEhIg25C/bIEttUa/z2bVNxarT0x X-Google-Smtp-Source: AGHT+IHr8T5reKexqMOxlmPqVf7GUavXfHMshpa/XldJ83QUP4dtM6xNCp5f26J8xaA/wL6S5nPJuQ== X-Received: by 2002:a05:6820:260d:b0:5fc:92b3:2b03 with SMTP id 006d021491bc7-5fd1949807fmr168082eaf.1.1740072733243; Thu, 20 Feb 2025 09:32:13 -0800 (PST) Received: from [192.168.0.142] (ip98-183-112-25.ok.ok.cox.net. [98.183.112.25]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7271f7c5ba8sm2752929a34.32.2025.02.20.09.32.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 20 Feb 2025 09:32:12 -0800 (PST) Message-ID: <0084eef7-3831-4e62-acf1-6c2dc0e15dd1@baylibre.com> Date: Thu, 20 Feb 2025 11:32:10 -0600 Precedence: bulk X-Mailing-List: linux-mmc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 05/15] bus: ts-nbus: use bitmap_get_value8() To: Simon Horman Cc: Linus Walleij , Bartosz Golaszewski , Andy Shevchenko , Geert Uytterhoeven , Lars-Peter Clausen , Michael Hennerich , Jonathan Cameron , Ulf Hansson , Peter Rosin , Andrew Lunn , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Vinod Koul , Kishon Vijay Abraham I , =?UTF-8?Q?Nuno_S=C3=A1?= , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, linux-mmc@vger.kernel.org, netdev@vger.kernel.org, linux-phy@lists.infradead.org, linux-sound@vger.kernel.org, Andy Shevchenko References: <20250210-gpio-set-array-helper-v3-0-d6a673674da8@baylibre.com> <20250210-gpio-set-array-helper-v3-5-d6a673674da8@baylibre.com> <20250220101742.GR1615191@kernel.org> Content-Language: en-US From: David Lechner In-Reply-To: <20250220101742.GR1615191@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 2/20/25 4:17 AM, Simon Horman wrote: > On Mon, Feb 10, 2025 at 04:33:31PM -0600, David Lechner wrote: >> Use bitmap_get_value8() instead of accessing the bitmap directly. >> >> Accessing the bitmap directly is not considered good practice. We now >> have a helper function that can be used instead, so let's use it. >> >> Suggested-by: Andy Shevchenko > u> Signed-off-by: David Lechner >> --- >> drivers/bus/ts-nbus.c | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/bus/ts-nbus.c b/drivers/bus/ts-nbus.c >> index b4c9308caf0647a3261071d9527fffce77784af2..beac67f3b820377f8bb1fc4f4ee77e15ee240834 100644 >> --- a/drivers/bus/ts-nbus.c >> +++ b/drivers/bus/ts-nbus.c >> @@ -10,6 +10,7 @@ >> * TS-4600 SoM. >> */ >> >> +#include >> #include >> #include >> #include >> @@ -107,7 +108,7 @@ static void ts_nbus_reset_bus(struct ts_nbus *ts_nbus) >> { >> DECLARE_BITMAP(values, 8); >> >> - values[0] = 0; >> + bitmap_set_value8(values, byte, 0); > > Hi David, > > byte doesn't appear to exist in the scope of this function. > > I tried this: > > bitmap_set_value8(values, 0, 8); > > But when compiling with GCC 14.2.0 I see warnings that values > is used uninitialised - bitmap_set_value8() appears to rely on > it being so. Ah yes, I see the problem (I don't think this driver compiles with allmodconfig so the compiler didn't catch it for me). > > CC drivers/bus/ts-nbus.o > In file included from drivers/bus/ts-nbus.c:13: > In function ‘bitmap_write’, > inlined from ‘ts_nbus_reset_bus’ at drivers/bus/ts-nbus.c:111:2: > ./include/linux/bitmap.h:818:12: error: ‘values’ is used uninitialized [-Werror=uninitialized] > 818 | map[index] &= (fit ? (~(mask << offset)) : ~BITMAP_FIRST_WORD_MASK(start)); > | ~~~^~~~~~~ > In file included from ./include/linux/kasan-checks.h:5, > from ./include/asm-generic/rwonce.h:26, > from ./arch/x86/include/generated/asm/rwonce.h:1, > from ./include/linux/compiler.h:344, > from ./include/linux/build_bug.h:5, > from ./include/linux/bits.h:22, > from ./include/linux/bitops.h:6, > from ./include/linux/bitmap.h:8: > drivers/bus/ts-nbus.c: In function ‘ts_nbus_reset_bus’: > drivers/bus/ts-nbus.c:109:24: note: ‘values’ declared here > 109 | DECLARE_BITMAP(values, 8); > | ^~~~~~ > ./include/linux/types.h:11:23: note: in definition of macro ‘DECLARE_BITMAP’ > 11 | unsigned long name[BITS_TO_LONGS(bits)] > | ^~~~ > > >> >> gpiod_multi_set_value_cansleep(ts_nbus->data, values); >> gpiod_set_value_cansleep(ts_nbus->csn, 0); >> @@ -151,7 +152,7 @@ static void ts_nbus_write_byte(struct ts_nbus *ts_nbus, u8 byte) >> { >> DECLARE_BITMAP(values, 8); We can fix by zero-initialing the bitmap. DECLARE_BITMAP(values, 8) = { }; Would you like me to send a new version of the patch? >> >> - values[0] = byte; >> + bitmap_set_value8(values, byte, 8); >> >> gpiod_multi_set_value_cansleep(ts_nbus->data, values); >> } >> >> -- >> 2.43.0 >>