From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) (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 2293735C197 for ; Fri, 8 May 2026 19:16:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778267809; cv=none; b=QpfbZzwNTl7tlvkGw2vdJbrwC8nZFkUUOp3aGvargsQX2H0uwukXReW8zGi3qNtocUa0l7AFADjKLBcZx7iumL4Z5jJ/JoZD1jvT0SRq5p0zRUaslqHALf+tfXBBV0DkG8nuLDyw+/Zk9NxXP9YnMd7K5kI2y4HN0ldMDI6Zcz8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778267809; c=relaxed/simple; bh=xbtWBW+nQPnx2Cwu0N+CjfnUqwEUHF3QpzPNihT3Fck=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=herivEAkFFD34xXpf0vnUh25iJ9CquKgyG/lcNB9d1fnRl2anT2yc+bsPFlJraWgOg0hm3+OiOo9DXyJH0NFJ3tVkPJy0KY3y8m4+FaYIfoAqVc4+sCdaqYDmKwcclJRSobDKkUyst0HnHq974V4XKO1eP60jelfvTUNWA50WrA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=QBglkLWY; arc=none smtp.client-ip=209.85.221.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QBglkLWY" Received: by mail-wr1-f49.google.com with SMTP id ffacd0b85a97d-44c44af71f8so278986f8f.1 for ; Fri, 08 May 2026 12:16:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778267806; x=1778872606; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=xbtWBW+nQPnx2Cwu0N+CjfnUqwEUHF3QpzPNihT3Fck=; b=QBglkLWYyVARov93/PJjg4odO5l5QDfe3sJLQFsXT5/zUDTrV9bmsLI0nVCfpBDvfg Tg9t2GH5W4osfLkMjQWapP81y+gbrTr3Xw7UsWDyN+A8wxAIOLxZK6hKMcVIIpCsQlbV aUOHoBUDGQ407+A9or7gexBuiNbTBDEMAlrL6hYl3ebrC5qsRQTyIKdJu5o6jtflJ9KU Len+S/SjJabe+1dVaWLWkFg4qga5TPV3MmD/nKkux/O35ilj5l40mVMXiWLenS0HlLw1 DOhE9zPMVNi2N4yNKzve8PtM7zUktv2QG1UKD/LFh0ylCXSkvcyPa9KOUUllBBIeFAHH ztXQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778267806; x=1778872606; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=xbtWBW+nQPnx2Cwu0N+CjfnUqwEUHF3QpzPNihT3Fck=; b=JKBp/GIkQ8CXuxQ+qpjLRZDD/yZ1VR5Q4s6mtGOglOrXtt/vexu+WZ8Uf/cqQekmlt X2fdt+X/UbP2Ky72CFqFmzKsKK787NCw5MGX/ssZG/+V6lcqY3bYS/sM5cGHPwQLoCX5 3WSYsAah/fHFjA3hJfq2F3fCy5lzHD6K96/xDE5S80mmFjKP4FWINcGXAV+mUE35rIh9 kjDvdG3ryPN9xf5hUf6ePUlPKh3pGCdnQN+zJBWC4zl1ECEiIWqTjR/9LxldyUfTXNwC r6qSxsbpwF+f02Pw511qqt4FZvq9H1f7tS54LYwpRP/vscZkZrx0+wgQPGWXY+meK7ff dHxQ== X-Forwarded-Encrypted: i=1; AFNElJ9R2/3VZCzam4nZCXFxFDpt+uSoSnqPb2SgFcpeiE1t9yrdeAOKsgxyKgpRjcA0a0IcVBt97F2d80yhF4Q=@vger.kernel.org X-Gm-Message-State: AOJu0YzINO1uIYzpPFy/J7580Hck8p01+zA97/W0UfenV+H5HKbWZArM waT2O3iNS7+3F+bkUMVZMSm+bmzJNq2/YdOzZxR6rB1ZVS+a2OD5tUW5 X-Gm-Gg: AeBDietwjF0NNWbMQuXwVQLgZfZEBRxX9SZgyffDl0oxgT7Lj1L0+M0jf/32j5vJ6TI OL7Ia4epqjE5nssv7g54bnrxwzckZbwoXjvjr72QxiaUsi31bPrRkmvI7BE+fK5ijWnm/CD6rXE vR2iMtBqn2W/oXpQ8ZK6YMgRE9sSRWv8YHm9MjJJumwwlgufRrnDrZGsUmtcK/5k+QjbUDZmr9g j1oWI212zqYRz4S3V7CGlIBF3lbr6cPRDcBAwpAwFaTNn/d53P2tr/QbI2bRL1loK+Xqe2M/eCx P3f9hvTwrwj9028C04Q3anJIMyQHFMufwPLn4OOBnqJI8HCVOjiSFVpE1/IPx4+dmkASyy0eK2b ZhOTekF4mL1IcVCALphdPjLBThPM5u7Z8h/nR5pnnOMPOVcF5Fr2ugFhxSPPayEx9H8Tr2uuf6T hiI4Nbu+rA/6AOHj5z8U/Q2ZiunHTIEIOLaeabcWcIFMs= X-Received: by 2002:a05:600c:524a:b0:489:6c28:dbb4 with SMTP id 5b1f17b1804b1-48e5329e323mr96668075e9.5.1778267806320; Fri, 08 May 2026 12:16:46 -0700 (PDT) Received: from localhost.localdomain ([94.158.58.43]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48e6dd3ff54sm18446945e9.15.2026.05.08.12.16.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 May 2026 12:16:45 -0700 (PDT) From: Stepan Ionichev To: m32285159@gmail.com Cc: jic23@kernel.org, dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] iio: chemical: scd30: reject (response=NULL, size>0) in scd30_i2c_command() Date: Fri, 8 May 2026 23:16:12 +0500 Message-Id: <20260508181612.11176-1-sozdayvek@gmail.com> X-Mailer: git-send-email 2.33.0.windows.2 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Fri, 08 May 2026, Maxwell Doose wrote: > I guess we're still handling the cases where both response/rsp and > size are zero here below? > > Should add a comment showing that this handles this case but that's > just my personal nit. Not worth forcing a v3 so > > Acked-by: Maxwell Doose Thanks for the ack and the review! You're right -- the (NULL, 0) write-command path is still handled correctly by the existing rsp-NULL branch below; the new check only rejects the genuinely buggy (NULL, size>0) combination. Happy to add a short comment making that intent explicit if Jonathan would prefer a v3, otherwise I'll keep it noted for a future cleanup. Stepan