From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f177.google.com (mail-pg1-f177.google.com [209.85.215.177]) (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 3BDA919E968 for ; Fri, 1 May 2026 01:30:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777599054; cv=none; b=J+OTOtkHsQ/IZvHI/T/mUMkCmoKA9bIYlW1+8dReJCHWOy0MrRNpCwKW/SCrS795tJ+LVT8pqi3jr+8o7CQJcPK2sGPCj6QcjnbdEecQbIhXIRSFBD7g92aLYN9ii1dWCPvksQiRo6zJv7cihkXCy8X2smEcpm0Fv+4hOdCMNnU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777599054; c=relaxed/simple; bh=2I/2yDgmZOODXBhkIxHPsuuNJbD+rAxj+nnAQ+GVSCI=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=hWS2I3/53czpldQp2GoFPG51vv5bm2ahaCSZs7Q92QOYI21vOfD8MSecN8egOCmFJzju9D8AwfTkaZ9/8CjqmDUkBDASC7cOGNbGysxCgJdGhiwfatXzY9Vo9sJj1gF5Sw860AJ7J8ksRq6AsIHmHuuosNFNmkoZeQtRn07DJeY= 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=S9wrBmm7; arc=none smtp.client-ip=209.85.215.177 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="S9wrBmm7" Received: by mail-pg1-f177.google.com with SMTP id 41be03b00d2f7-c7b9f54d3deso928276a12.1 for ; Thu, 30 Apr 2026 18:30:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777599053; x=1778203853; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=/n6JYUDzgGxuCjwV/yEzfnH2hleELeafJ0aAhqmVVIU=; b=S9wrBmm7kgQ/aYb6TPvWSsbXyuN85paCiJEDntlmDEHoBQXvH9HGJe5CCDOD9OYH9k 8f4sH+/UB7KCci+VirnGLbgwAUUFdYeJS8s79VQsFLrzq2JJefo+BVyG/C3xQnFvQHoy xyGi05Jw7FnfOQn5OZ9kYle+VQ8zfieK9Dr1WKLqXkO1J/Z33wfZQ3TehAjJtLiM4FeZ LV6TWXOsXQZjpZdXcpJwhPfR+CzcX6325Qm95P7ad2DEUNpyNQ89HHZOADFsWb8pM9CF KpR+bq96yD0xmnSlhl3G+glWbKWyxbzoVOkpHdMxfN8W4p+UATpQ/r1hCH2KDybSwdiY wxOQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777599053; x=1778203853; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=/n6JYUDzgGxuCjwV/yEzfnH2hleELeafJ0aAhqmVVIU=; b=BDo8+0e3wKtP9j2VljP1MKaF0vhmobdBQq2/Frgmgl0IB2HqLnmb1quiz6rEqgizkY 3KYgdTRR6DDEmFp1Xy3gpGzSfHWl62YhhpBUwfO3I+FY7S4gHXrBTe98+PnxsZ9Lod5H JxHptIPucaFN4QWRMNZ1C8T6xVJEtLbeUNLEQfEDijiQ4qWXrMlLOrVLp/aISlsjSBLW hnrd+ovSNImh7V9TzlNz5o9tv7kWBu0DEcwvpjC3Qh0g+1fT2W0xo0/174iuRJ4RHI0D KHMiE+Ww9MGPcMg6N3DcNogkuAUqlg05BVplyGecIV1AbmMgf+Z99srln3Z8vXNlrPs1 B49g== X-Forwarded-Encrypted: i=1; AFNElJ9XTk+vOCfGkJaEt0OeCjcw3mKode2rzq8uBpKqkv+7Q+RbA7NwKaySz6pdbq/XgSAeZa3Vjvy34t5KEWM=@vger.kernel.org X-Gm-Message-State: AOJu0YzX37uDZGMrbGp2WstJ0eJ/Zd6Bjiw8/hlCvY2OxvX+XPOwE1pr KIAiM1W896nXT12ax2ujq0ZXC7vFPP3I2ZZkJ2QJDwOo9X4mr4JlkRYN X-Gm-Gg: AeBDieuPQh0tgUTGYx5c20dRbixalTyq83DgH17akdl9UqZZJefHlMnnfG6neF8X2p8 W60ygD4HsxpobFfFjjWMfuSGvyNCkL8i2CxHK5k7oCgCpuMS5epe99PHbEoruka9FYUmx+EIOX0 VmHGew987wi4fUKPfVUhlz/OYHZwXDXcSSxHrw6x/niWyU8NtX95jvij/T2orXV91PL7Z/UyS59 mx7J+YB69TEgKBRYx0Gv+GVLc5zfcP+8ibeLKNQp/LnV2Ss0HZr10ttf34p30Qg3tIGD2Xr0oP7 RQkS/+7jopRwTIx5glSKvMVHQhUycqD7wb6aYWMrLXZllTzyWXSYxppaxStql5q2JpsIunNBoND EZResKIzKHXE5Cq2wh7t4sz/pl4CLDHTARsCBxQZyjcbKWWcc7tLUyYq+jFTO1l3Gn+6CPB+G8i R+xgrn0NzH0jrd6MZFqMMzMOlGjI7m3svaftB19JvkpFeVgaWvkMHjiXvXUqIrU8RFFdizXFsAr hmwvA== X-Received: by 2002:a05:6a00:1749:b0:82f:355a:857e with SMTP id d2e1a72fcca58-834fdc801bdmr6395680b3a.47.1777599052531; Thu, 30 Apr 2026 18:30:52 -0700 (PDT) Received: from tabrez-VivoBook-ASUSLaptop-X513UA-KM513UA.. ([116.72.73.183]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-835158a8432sm1014523b3a.20.2026.04.30.18.30.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 30 Apr 2026 18:30:51 -0700 (PDT) From: Tabrez Ahmed To: Guenter Roeck Cc: linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org, shuah@kernel.org, me@brighamcampbell.com, Tabrez Ahmed Subject: [PATCH v4 0/3] hwmon: (ads7871) Fix endianness and modernize driver Date: Fri, 1 May 2026 07:00:40 +0530 Message-ID: <20260501013044.22756-1-tabreztalks@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit This series addresses several issues in the ads7871 driver. It fixes an architecture-dependent endianness bug in the 16-bit register read logic, migrates the driver to the modern hwmon_device_register_with_info() API, and moves the SPI transfer buffer into the driver's private data structure to ensure DMA safety. Note: I do not have access to the physical ADS7871 hardware. This series has been compile-tested only. Changes in v4: - Fixed an additional Big-Endian hazard where passing a pointer to a 32-bit 'int' for a 1-byte SPI write transmitted the wrong command byte. Refactored to use a dedicated 'u8' transmit variable. Changes in v3: - Added Patch 1 to fix the pre-existing endianness bug in 16-bit reads, originally flagged by Sashiko AI. - Fixed multiple formatting and alignment issues caught by checkpatch --strict, as requested by Guenter Roeck. - Added "While at it, fix checkpatch violations" to Patch 2 commit message. Changes in v2: - Dropped custom mutex in favor of native hwmon core serialization. - Split API migration and DMA fix into separate, logical patches. - Corrected output scaling and sign extension to meet hwmon ABI. Tabrez Ahmed (3): hwmon: (ads7871) Fix endianness bug in 16-bit register reads hwmon: (ads7871) Convert to hwmon_device_register_with_info hwmon: (ads7871) Use DMA-safe buffer for SPI writes drivers/hwmon/ads7871.c | 121 +++++++++++++++++++++++----------------- 1 file changed, 71 insertions(+), 50 deletions(-) -- 2.43.0