From nobody Fri Dec 19 20:36:28 2025 Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.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 E299F207E01 for ; Thu, 9 Jan 2025 04:35:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397356; cv=none; b=HTsT8dUIs/Aimc2PIgAdsB6kCwgoSgonrbxWInqvDLDW6yPQ3bm5yJ7/IXq/BdrIMQpZRJwW2kHxPu6LyUqg0/hw24B4mi7ViBkS7WgO/6FWdY+fuSZI0/OeSko5p8fyJ6PBX/8yLCUxSzacNjeqRtFxmEK3r4Q8VB3hJzp9p8U= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397356; c=relaxed/simple; bh=28lfeP0elk+tC/FtwJpVo5ll99ssX0YY1tno5A6BThw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=hXt5ruE0MQboMOjgdGCjZTI7l+Z0IH/UNn2co+ZpfrWw1hTkKinZDvx0djpK8hjTHC2yRDFW5usvJ5EzxiiRaZlbW85OIzfqZ8DKidOhCjVdZljtIwVz7IViCNlOI7jEey7qNIchfuiVbf79+afjuqmWD7Em0idQ7WhetRsa34k= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=nuBlyRML; arc=none smtp.client-ip=209.85.167.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="nuBlyRML" Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-5401c52000fso483847e87.2 for ; Wed, 08 Jan 2025 20:35:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736397353; x=1737002153; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=eQ7Ey0366JcxuGYyhKacm3wyhGFkT3OGK/0czjEyhxY=; b=nuBlyRMLiarLwassnGCzdJAh+bF7sO3ZQjdH5C+6tpvQ18dH0SlFt61fFqn/Rapk16 I8+1zVMK3Gl2CuOaR8EmSUnamLbRGRT1Rqg3f25TgFy5BjfvF4qjQgg3h/d6E0KzM7cG g5gxbAZkJis2QPe5iHameL2NZP7zY1IiPnG+786wufRaS2T1NBf/00L22z34qj5yzmAJ 12/CRuJPOCOZtog1if5/kPw2gt/MpJTmKG7LRQJvPpqFyCrYDbM3Hl4/oDaOif4GOehN amSTVEub5WQzJl5ANEJDNe69YfhCivaL1OISwvJkk0PuJjA3XgRlNxAc4UBsbuLNKrRB DyAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736397353; x=1737002153; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=eQ7Ey0366JcxuGYyhKacm3wyhGFkT3OGK/0czjEyhxY=; b=nAl42ci5HKmdyGCCfuZGOzCSPtKf6mlJbGp7JGsxrUeDLnRKzrxTv/JcVF1kTaqCOx cBTeTOgkGJSTgaUj+xtZr9mwN3xQKbS5y1vq9hF0lL/RwWNkHVDK9KIN+tgKr6VgJqNT wiiRu2ISWVyy7/4yvEcFezRD3ldrO7m3e4hkzb0z+Ig9uCtVkWYOe5k67W95nWuQ9Dxl WelD3zoHGYAO1fB9jCkbotvgWHedNwfNKQy1p0c3FJMBAVOVRmSd2MYSWTP3XN4pVH/F j9xF8QzJLU7FcExavuSaiWcu4S8QdmxxYFeZPo4A/Db5OY9rHEWkVRl113luWoSBN123 +ajQ== X-Forwarded-Encrypted: i=1; AJvYcCW7PEmW0oskSvownqZUsX5qTZryAlPyxDdUXBMf47lsBthV2duOx62lFcrkEHT9uv2zOYZg4bFuLHTxX04=@vger.kernel.org X-Gm-Message-State: AOJu0YzSRCF1UuzA1teujDaz4dauAfO5iN6VyM9F2AwlP3dbdPcGQOfh VUgttA71Az7I3PQ7ze6bxvVSXrJFKD+zX8dtf561lNNNoN2PPN2HujF0jczi/fE= X-Gm-Gg: ASbGncuB2EZ7D2qQOIlWxtooeoqxa5TJKD98aCuqG5Lufboc1SohV0c2IzE7ty5n9ke pGKG0lWk8XjdMyDRP3AtjlFX4gsZk5Td04A0LFxOUa04r9Wri8SvhXweYS7/Fuo21NWIostvX5J w6iYZAHsqcl6/5hJDK8HQGRfvQtL/bt+VBCUrtc+4gqiHvdA3YfYLEy4H68tBtmnAMnqmxkpKRU vNISuyCdtHUEQ1eqoJmB6YQtDnV/3bL80saDg/Cpr+gFpZlCt6yp2/jHdRjQeZCtWsSf4x6X8Dv qMwzA8+InWp2SfZHRyyt8QK5 X-Google-Smtp-Source: AGHT+IG9P/BrUEzkdO3YjTFb4jKbXMe/fdiSuMkgWi8w/KxHUDnAYQ26vw2dH69lK6mbTLmORDlxjA== X-Received: by 2002:a05:6512:3192:b0:53e:335e:7756 with SMTP id 2adb3069b0e04-54284815d10mr1450834e87.46.1736397353005; Wed, 08 Jan 2025 20:35:53 -0800 (PST) Received: from [127.0.1.1] (2001-14ba-a0c3-3a00--782.rev.dnainternet.fi. [2001:14ba:a0c3:3a00::782]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5428bec06e1sm73866e87.191.2025.01.08.20.35.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jan 2025 20:35:52 -0800 (PST) From: Dmitry Baryshkov Date: Thu, 09 Jan 2025 06:35:45 +0200 Subject: [PATCH v4 1/5] nvmem: core: fix bit offsets of more than one byte Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-sar2130p-nvmem-v4-1-633739fe5f11@linaro.org> References: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> In-Reply-To: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> To: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Akhil P Oommen , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=2178; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=28lfeP0elk+tC/FtwJpVo5ll99ssX0YY1tno5A6BThw=; b=owEBbQKS/ZANAwAKARTbcu2+gGW4AcsmYgBnf1IjOYS2hG2FXUXbR7hTDfuUum1vNcR6hSs2u 0caXs+FDT2JAjMEAAEKAB0WIQRdB85SOKWMgfgVe+4U23LtvoBluAUCZ39SIwAKCRAU23LtvoBl uHVeD/9UumRB6poWTUCyfIZtKnmWF+AJy9MH2f3Do2WizXmDzFk4eUe+J4CIiqGvAIC4TXLRkkd EhK1+bETRA2IZKUvIjpwVj4j90Ee9DM6PKfH7UMIr3qH7nSibGnDoVeqF0zHs2OOT8fPzPIRkc1 NazeYfz2O0RsIRUyEkp/ExPbargr2iYPg+rT7/MLa2k52S0OQh+KcjJ2Py5gatggYzBHEC348gx eENbyNOmrHC4PqEBzf4eBh1caw8jcDacmQJo/ueWp9NNbIHaR2OX0jkH4JWZ2Z4zkOdXpa9s3Pg OFrIcGjW6sOrWVfDw/YtCEwLgrO2B2s+m6AkdrvTMjk9rzIi27FQ3cM9WA5doFyguC5fYCeM4Op WLtIbzjNawxE3XUJDfTKpRGQWyKHXuCojiQ5C6T277EbWjC61hwrw6jsedr8nbz0fvsEIH/OymX aXdFeGlFQXfqh8uNkYxsGxeyfeBqs+OohHDqmPQbzlySYG22kVKG9P2MEf9l1I+2UB9NbQyRJYA hYykNLoMVrIbCi9tR/9wLorq2KeKJbySZvU3sCOp7dWbfZyGgXRNmJgFxGfQaeXO9c7KsjAgc5s A5JXCguP2ou5fwaxuZXazHPl48xSkZCgJNT1h0+tqtr54nJz1ZwKyd1luMy+r73xXV909Tl9j4K FcoxU8hCJfgWanA== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A If the NVMEM specifies a stride to access data, reading particular cell might require bit offset that is bigger than one byte. Rework NVMEM core code to support bit offsets of more than 8 bits. Signed-off-by: Dmitry Baryshkov --- drivers/nvmem/core.c | 24 +++++++++++++++++------- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c index d6494dfc20a7324bde6415776dcabbb0bfdd334b..7fa85b0804db360035d7471002d= bf79658d5830b 100644 --- a/drivers/nvmem/core.c +++ b/drivers/nvmem/core.c @@ -834,7 +834,9 @@ static int nvmem_add_cells_from_dt(struct nvmem_device = *nvmem, struct device_nod if (addr && len =3D=3D (2 * sizeof(u32))) { info.bit_offset =3D be32_to_cpup(addr++); info.nbits =3D be32_to_cpup(addr); - if (info.bit_offset >=3D BITS_PER_BYTE || info.nbits < 1) { + if (info.bit_offset >=3D BITS_PER_BYTE * info.bytes || + info.nbits < 1 || + info.bit_offset + info.nbits > BITS_PER_BYTE * info.bytes) { dev_err(dev, "nvmem: invalid bits on %pOF\n", child); of_node_put(child); return -EINVAL; @@ -1627,21 +1629,29 @@ EXPORT_SYMBOL_GPL(nvmem_cell_put); static void nvmem_shift_read_buffer_in_place(struct nvmem_cell_entry *cell= , void *buf) { u8 *p, *b; - int i, extra, bit_offset =3D cell->bit_offset; + int i, extra, bytes_offset; + int bit_offset =3D cell->bit_offset; =20 p =3D b =3D buf; - if (bit_offset) { + + bytes_offset =3D bit_offset / BITS_PER_BYTE; + b +=3D bytes_offset; + bit_offset %=3D BITS_PER_BYTE; + + if (bit_offset % BITS_PER_BYTE) { /* First shift */ - *b++ >>=3D bit_offset; + *p =3D *b++ >> bit_offset; =20 /* setup rest of the bytes if any */ for (i =3D 1; i < cell->bytes; i++) { /* Get bits from next byte and shift them towards msb */ - *p |=3D *b << (BITS_PER_BYTE - bit_offset); + *p++ |=3D *b << (BITS_PER_BYTE - bit_offset); =20 - p =3D b; - *b++ >>=3D bit_offset; + *p =3D *b++ >> bit_offset; } + } else if (p !=3D b) { + memmove(p, b, cell->bytes - bytes_offset); + p +=3D cell->bytes - 1; } else { /* point to the msb */ p +=3D cell->bytes - 1; --=20 2.39.5 From nobody Fri Dec 19 20:36:28 2025 Received: from mail-lj1-f169.google.com (mail-lj1-f169.google.com [209.85.208.169]) (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 43DEF214207 for ; Thu, 9 Jan 2025 04:35:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.169 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397359; cv=none; b=LOKZnlaDDoKp/lG6wYsrU2Ev9+D+5IGU9HcXV1tzknv6VudZ4MF35Bxp3qDGiix4rFA05d8glGgVGbe3LXxmpcy2N5Z45xK40p6zRmslDV5Egr2G1PjOJb9+PWFDia2aGjGWDZbkqLkeP7fdA0SDxvoZ+7mW2qmi5fjZZh66+VE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397359; c=relaxed/simple; bh=2bskNibEbXqxfl2Ilmpmi+d9e6SD72hHz1dhkLTsXqw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=UFo2YRZ7sCGG09gDKoPWWKDSf8V830Uv0PxFLw5ajmK6akSnQzujNbxq6xh1Of1sXHuhgR0/NA6br0q0khDMITuuWLLC06lziIT+zRNI//eHuMJmaG1vHd8OXQ7UaLSIDxcjQlc7S3gOyY/IXFnGmHAb6tIZTjPBYsI7JtaK8JQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=R4tDzsiX; arc=none smtp.client-ip=209.85.208.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="R4tDzsiX" Received: by mail-lj1-f169.google.com with SMTP id 38308e7fff4ca-30034ad2ca3so3255651fa.1 for ; Wed, 08 Jan 2025 20:35:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736397355; x=1737002155; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=3aosplTLYhtpWTLaU5zR4iQWyyRW0YdWxTgdnKv+OWI=; b=R4tDzsiXiVcjWQKxwcpcuwZrpeVI0iiJvTQvP0gNdHs8m8TYq4b9CLao+x33dOEn1f u5kiV3grxKUVzRUQ+mj5zwoQAXd7hDeRJg5Y6MTVv4Lbz9JA95P0WdWEVtesj7B8b5cI H/cMZQ0OKv+Oz2HpjJ9n0SFhXs+NBw3jL3oT3BKcigg5Wgo1Mgkys9dPkiuU3WyPhmhn IuQHNUPXNgvJS5EdGmNLGT9u5rn2JupIBfBlDgO3wqh7pMURG1B9+d5edFlOIocDQXh8 2D0Xiq8RsYP8rDf8SEDZo62tho16CnW69KM2p2E8ouyE4AqUBy8eEYYbM9dhPxPXtYZO PJdw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736397355; x=1737002155; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=3aosplTLYhtpWTLaU5zR4iQWyyRW0YdWxTgdnKv+OWI=; b=dtLliaRdTwEUbKBby2jR9y/jfww7NuPbN/p15O77Gxrr0iTkiOJyldAwDAn60s8CTz 4KKC1N2TKSHkD+3951rXBxDop7y6bfWyf8DTsfKhaOqiHNmJvRseFGHgkbsoS7O6Mq/m xzEvAtlY2Y0lZPJ8fGqg5WOMZ+fdJWxJ/ep/eCmZjf1ppEqlKEXeCYXiN93wC75muZty 524acfcK8ApvNpKWmfZnztevjpbJJQM0qCIaJHFD8idFPvm+psx0oDaRP0AJiTTfzpEm aCGlD6qCqG9j6Xc+UmsU3aBNibvHX23n5T6GeyRz+8QEQL1ieAx64F8oDX1gKlTLGlIw AlhA== X-Forwarded-Encrypted: i=1; AJvYcCWMA+MslyH7pTCwqlBzE3QR1FSyfp5KGO/FFRFUlnnXsn9MGqePTDkWnjFkVMo6LSL7PU6RCT+k9bLwB+w=@vger.kernel.org X-Gm-Message-State: AOJu0YxlqOc1ufuAnIV7dPbvj1cc/i6VbYuy3gksF2i+bfPdZ7QUNlRJ kBEYeo8xSIcAfNUaJNlItax3DUg7n3tBF8h3a5a93ZBhliom7WSC6suEcfPdLbs= X-Gm-Gg: ASbGncuveTodTrse1nIrRev99pETnowhH4zPOgZfJE6zjPn/SvzPV1VM5tqx5pVLL8X xWDT796Z9vR5j3WkqS6zBhCX8BD6iyBmnhGgWQzVJPVjtppTdd3b7RAr4b/FmYmvDRZBuZLDkkY uszM6dLAAsLGh9JC4X149xmpxKJBSBUJ0rC6LT4aIRP/biWq4eeOz8ffvnStSlVKvxjOwRQKxkp P3uVoYrwFPUX9r7Hoy/ISsqgWXj+oCIerNdEx/QjYWkJDCeTWKgS34hc+Wu1SXD7xwIe81bFGUv FKIRuWea2EL9Q4Ln78UzNBdt X-Google-Smtp-Source: AGHT+IFurU+4fRGqLM1kG2UuFsjtR9sjwM9mtjisWHOKw7E/cc5bFihShsVmoDxew3v7KstsiFz9KA== X-Received: by 2002:a05:6512:e83:b0:541:1c49:26a with SMTP id 2adb3069b0e04-54284815d5amr1743053e87.43.1736397355392; Wed, 08 Jan 2025 20:35:55 -0800 (PST) Received: from [127.0.1.1] (2001-14ba-a0c3-3a00--782.rev.dnainternet.fi. [2001:14ba:a0c3:3a00::782]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5428bec06e1sm73866e87.191.2025.01.08.20.35.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jan 2025 20:35:54 -0800 (PST) From: Dmitry Baryshkov Date: Thu, 09 Jan 2025 06:35:46 +0200 Subject: [PATCH v4 2/5] nvmem: core: verify cell's raw_len Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-sar2130p-nvmem-v4-2-633739fe5f11@linaro.org> References: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> In-Reply-To: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> To: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Akhil P Oommen , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=930; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=2bskNibEbXqxfl2Ilmpmi+d9e6SD72hHz1dhkLTsXqw=; b=owEBbQKS/ZANAwAKARTbcu2+gGW4AcsmYgBnf1Ij5AW+SyAZq3O9yagXEl8p5JuqiY9aKUHLG RhBU+zo8rKJAjMEAAEKAB0WIQRdB85SOKWMgfgVe+4U23LtvoBluAUCZ39SIwAKCRAU23LtvoBl uNSdEACYA0FlOl7h1SRtSGwI1DheVcZK27ynUAsZws+LY5hBTSfVsSnavmFZpuWQxmXmnYwnqWF wIL45e9gI3wZJUt4+PAJAuQIpJe2Gpc5V7boWVMEZiAryCZsZ+miWj3w1EfhvSXu2gbSSK0OURo rFBZAz6bRY/2V9OiZmmLOr2fUlLNxuIIIXXW1yF8HddBS54LUKQxPoz3UIvDUAI5lGXez/X2JWL AmJjyp84ID6JLdNWOfrxRzjRL9/PCua/5dnzWQe+Yr6SMW97qZ+QMvfTK/xnVcA/yazIGF9h29j jFDz/cYQqtt9I0Qk+bdMfdHv1zX7euOEQNdJFDCRrGJdkzgClGYovR0SeIaCOdv+vBQSe7NhZiq rJfjfVJHr8RH4EuQaY7dMPZ3PqfTMnSbL4WJWFWra6WQchwUpd/MxWF8UI6UQjfWAwPuChEwdVQ z4R7UHXcLbqWTb6FoekugjEbAwLTZrHeqsXmLi9meHUq7WTbaTjCj7Nhea/doW0WTQvZLVABbWD QZ8kAVuJwEZXYNUgSS/TzFOovX1B8Y0S2pPJ5zJJldJfBgDsktM40QpdUTAcN0rEEwJwsPZZMW2 s0XCU9fL+tcK7bPER+AOb8OPDJO+mD20yOuVvxr5G1NEnSSv3+ae4Ak5YSnBhzUmhrvFxxWtuez 2Tuv0hX+VSiecGw== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Check that the NVMEM cell's raw_len is a aligned to word_size. Otherwise Otherwise drivers might face incomplete read while accessing the last part of the NVMEM cell. Signed-off-by: Dmitry Baryshkov Reported-by: Mark Brown Tested-by: Mark Brown --- drivers/nvmem/core.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c index 7fa85b0804db360035d7471002dbf79658d5830b..59885d8b6a19d01f50759f09d45= 3b092d1ec44bb 100644 --- a/drivers/nvmem/core.c +++ b/drivers/nvmem/core.c @@ -602,6 +602,14 @@ static int nvmem_cell_info_to_nvmem_cell_entry_nodup(s= truct nvmem_device *nvmem, return -EINVAL; } =20 + if (!IS_ALIGNED(cell->raw_len, nvmem->word_size)) { + dev_err(&nvmem->dev, + "cell %s raw len %zd unaligned to nvmem word size %d\n", + cell->name ?: "", cell->raw_len, + nvmem->word_size); + return -EINVAL; + } + return 0; } =20 --=20 2.39.5 From nobody Fri Dec 19 20:36:28 2025 Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (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 8D2D5214230 for ; Thu, 9 Jan 2025 04:35:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397361; cv=none; b=H2kGdbe6nEYNVTiiBMbZV9wzJe9eqQrNjoTqXMJkL27NvDMXPo0gUTygOK6ZBOYDi9G4iuXGXcSmJiY92aZtOl1g9S4TlJcNElOnHWkrOf5YCCiHJpazcV9NZNGqArxyWGsZ/w1kyPr7kIIJrgPgBnMx2sfRyduAU6rxPgCc4aI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397361; c=relaxed/simple; bh=KYi944Ng8PZyQOwpW6lInP7e9H63rwgPuemV1qM3aN0=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Ydsf1x3Uo9hdJPXQ2ptqH5i2CKaNnLg6oEWJYY1hOKp2eVeCnFdG2VRW17l8nCy9/5WQqUMnciH5JqwWaWOepcbn1lKtbnU6UK1tkG68ad0aWC1vOFMcXs/rylRejcsCvm0Tw+jOrF4kE9cvmWp0X/8I4H4zCcbJ68T8U9lVAdE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=tQFIJH4R; arc=none smtp.client-ip=209.85.167.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="tQFIJH4R" Received: by mail-lf1-f46.google.com with SMTP id 2adb3069b0e04-54252789365so577517e87.0 for ; Wed, 08 Jan 2025 20:35:59 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736397358; x=1737002158; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=jpXKuQIMgaAXMlwh5+Z08XtwvgtpNR7jrbvfEWHNUTA=; b=tQFIJH4RX5Dc+XQMqP3emA+DZzTKbRh0VysW8qMsLkPvFpiv1rbiBu/4YIbQ0yp06T LwZcD4YpqjAzFLaWjMMifj9GmwpfoshiDoABmmZSb231jRGFOUc4BOvE4np0f6YcGAYF HcvNgduPMC5DeA8aSNAuN8SI469KcQ4D9TLE+OZr2Haricn92adsmjoNvrkU/B4JiuI3 G/0dUNL8BkA+ckZppr8alUHcuNkfai4nlhfhgNMI70A9Q4guI/21rwA2Y9hrAqYZx2wE Z8vlcScLPJJMw/uWE7dvSnguvAEr/i4Xwt+sA4CiRxhj0qMcWN4X0AROPVUmo2xbeioj aU+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736397358; x=1737002158; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=jpXKuQIMgaAXMlwh5+Z08XtwvgtpNR7jrbvfEWHNUTA=; b=AmwnjaFMLZ3mLyEZfaEJZnqNKySlbMgDryzHft9ExIMQc3kB/QwmOZcxKl3hCihZ+c VOP3SLMcI0tUY0sujp9zSO/wa5CY3rG/y7IyBrmBdyUDIcoqt51b5tu1pZqhTClO9Wt4 o0tk7HYahyOCOXXaVm362GA/j9d6afXNHFr7SuHDa/2VffMOaS5YQ8/SmWT4rlI7YeWG Ne/cMyZ9NEacl30i/aFcQ1MUlZF1Nrl/FnwY9BBVM3+v9mRJ/U5CJ5UjokSWxaKKdoTX ig6IT4v3gzvJaECfnfCOu+k6pB7IcQcIBL8dPo3p49w6w8tHGQeEDPUzqHN5e1kRHkBC 8Gug== X-Forwarded-Encrypted: i=1; AJvYcCUdLej3HLb3z9qf4TFNY0xa3Eo5HxYUSHn1gjy/Su9ERgdOYgghoYS9sjQnN9lp/aRx9g18JRzoMWV7b5U=@vger.kernel.org X-Gm-Message-State: AOJu0Yx2ArJ4TDKNtleucLJqNUZ1p3eKFPovLJS2/7XvjudzkCcfrSyD k33P9thg84EPvHcPLiwtmM+4yJoaKYZ2cGvbx40OmVcNggN4GxKpwYiKG8nHPvE= X-Gm-Gg: ASbGncsNt4Oza7/V0vkbblr0a1AsmktpMVY7MhDR2psv+eOPEyxOhCcSIwoDowsPfQo Fvl26JkFnuy+p+v44Ii+9E2fVo+UpgMhI6bENkzErcKkbr6608ONW7wxumNkvvb4NM40L6YBWTP l1ObqeJomNlZFfoueFMl6TGRyQ06jjc4pRPNNJna4+13ctc93/tgGd3CcwWRyoggiBYlh8jKW5A R7WPgM9/mzwYRvR62snTOPWmVYlBLwcpprj/8duyCwZARy79wjf6eD7d2Ecf4JboZ1wtFrZdi/Z gKT3GPz353wxIIS7by51/k63 X-Google-Smtp-Source: AGHT+IHXvspid5uO57Ne/VYe04P/EGSy1AstqW/FydRu5FaZnAbabC9adB3YHkCwunnYN9qNMQViJw== X-Received: by 2002:a05:6512:10cd:b0:540:1c16:f2f7 with SMTP id 2adb3069b0e04-542845bdd1amr1369272e87.14.1736397357757; Wed, 08 Jan 2025 20:35:57 -0800 (PST) Received: from [127.0.1.1] (2001-14ba-a0c3-3a00--782.rev.dnainternet.fi. [2001:14ba:a0c3:3a00::782]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5428bec06e1sm73866e87.191.2025.01.08.20.35.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jan 2025 20:35:56 -0800 (PST) From: Dmitry Baryshkov Date: Thu, 09 Jan 2025 06:35:47 +0200 Subject: [PATCH v4 3/5] nvmem: core: update raw_len if the bit reading is required Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-sar2130p-nvmem-v4-3-633739fe5f11@linaro.org> References: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> In-Reply-To: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> To: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Akhil P Oommen , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=988; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=KYi944Ng8PZyQOwpW6lInP7e9H63rwgPuemV1qM3aN0=; b=owEBbQKS/ZANAwAKARTbcu2+gGW4AcsmYgBnf1IkutU4e8r41NLfSbrBUzZ8YhmPvzTOQ2v+k Ebwskduq6eJAjMEAAEKAB0WIQRdB85SOKWMgfgVe+4U23LtvoBluAUCZ39SJAAKCRAU23LtvoBl uEmUEACY4IBr5uLswWnHYFyhg1g9Jv/h+36nARGMzYvaAdq06riHGrGkSeNY15J6smUaQ8ho8i0 3ZBzap2sIOcnn3cHWhwki1KnoCzFY7R0T+xGkSY1fINkuWkpQ4CobR+61qdCU20r9DdtMbXyqIT BvofAxA5dr2/BtFd8nxf4C4SP3d7HqnxbLRu6zeVS0HDwBnVwkumtZqrKCBFcXiSjtq0OXrMO47 97LHuw9Ie9WlxyXW6tu51MIZZtS3+raU7spdjuhNBaVfxDUIpbH70Drv0oOyN3LxwQ1h07bM16a tGrSLdv4XM/U7d8uZf+/HqolAg3NeTIwtdp8rX3Ux8jiuYPWTnIiaTUR1rUaYlJj9NgIPGRx3Kl gIy/0Kf+guZjuGbEmcM9Dvwy4JE4Olbq805+R9ccpqm8GsjvCTprCV7Z2tcxUq0rK5aI3ShDCJf VIaVQKYqn85zZs+40w+/6VbZ3L7QLurh/fy+XXCKNArGfU2fRC/nK5Ziil1NglBR9DzWgRVe/jd q5GNyBuCwq3x4hsM/cp1gucxopI++vtoCboRHvqa6B3uL1XmFO4F3gTdnUD9B7Ob5HXFwefYsMo 7KxUlwOilzZLiOnxdcc0XwXgKE+xDnElwmv7oKOL6l7OIVKLMWTUlQ3iO49eTG5xfZ7CzmlUgbS 2QHgdnLPs1j8DPg== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A If NVMEM cell uses bit offset or specifies bit truncation, update raw_len manually (following the cell->bytes update), ensuring that the NVMEM access is still word-aligned. Signed-off-by: Dmitry Baryshkov --- drivers/nvmem/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c index 59885d8b6a19d01f50759f09d453b092d1ec44bb..d3c2e6917ed32ace30844faaab2= 7c41779b5bbf6 100644 --- a/drivers/nvmem/core.c +++ b/drivers/nvmem/core.c @@ -591,9 +591,11 @@ static int nvmem_cell_info_to_nvmem_cell_entry_nodup(s= truct nvmem_device *nvmem, cell->nbits =3D info->nbits; cell->np =3D info->np; =20 - if (cell->nbits) + if (cell->nbits) { cell->bytes =3D DIV_ROUND_UP(cell->nbits + cell->bit_offset, BITS_PER_BYTE); + cell->raw_len =3D ALIGN(cell->bytes, nvmem->word_size); + } =20 if (!IS_ALIGNED(cell->offset, nvmem->stride)) { dev_err(&nvmem->dev, --=20 2.39.5 From nobody Fri Dec 19 20:36:28 2025 Received: from mail-lf1-f46.google.com (mail-lf1-f46.google.com [209.85.167.46]) (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 0DC832144B9 for ; Thu, 9 Jan 2025 04:36:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397363; cv=none; b=mKzuOm0RSK/xgoK8koprkgRk3PLfucNPwPXLHvX47zp+yhuBPMXlwfenqEaYkG2n94R4aVBbGovs7LYDgrBCsOXM2gmZKqssXHmUS7U8djDYZEVI3PbR5Pnv07e2bMXkaJEfSLtyhH0ZUSwNUykJPKQqUVUTM7Dm6L3ShzjuEdY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397363; c=relaxed/simple; bh=Weli6Ud1pnhnVMQNhXbarzN29phyH4B6NUF7IDp6kk8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=rmIv5Z84XGE8tJo2EXqPJ+gsubzk8GTwne3B7nOi5/SONq5oZ8y5nc5EVOjWbWcFQ8kjnBw0wFDNnafEPkZ8bA6637Atab/Bmc9uph3ZZ9dVK5jUW+w+llzjdGY0TAQA4RClqnGOrOf5+jqcUpPKg6xN3omYTPo6CugK1bFbqM8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=cDnBa5KN; arc=none smtp.client-ip=209.85.167.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="cDnBa5KN" Received: by mail-lf1-f46.google.com with SMTP id 2adb3069b0e04-5401bd6ccadso506062e87.2 for ; Wed, 08 Jan 2025 20:36:01 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736397360; x=1737002160; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=lOzuf/LkISp9KTJoorQzqPXBIv79BHB/w9fYq6gMhqs=; b=cDnBa5KN9Z0q3M7Vf2CiIOCBxwkmQg/Z9fMDEhiQBW4ZJ0/4H8i0Txh5OID1ibvoqM VlVyljmnVf/0v+VSK0IrgD6Jy0PBq84Vo/5mvMdRA0O7HUY2mxr6cIcKVJEVxXZCjT7o mx+3qQW6QPL3cadZ3RJfPgp7OHLbBIy+J+zTu2F2dmmJq/INtCbt5yU4H7gkuvpRXQHj /AsQLuVlPxFKmWusu7sFP5uyyjBwQPJhuY4SdRJEARtxpGdZhmCHlqQIl9Wa5dlafrRa IWebD6rZKAJDyHrEDziImCh3HoJZrss+4A9JFY5sKFgixNBv3HgFMZmZ+fdKPR5IlsNx f/aQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736397360; x=1737002160; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=lOzuf/LkISp9KTJoorQzqPXBIv79BHB/w9fYq6gMhqs=; b=cvwxw3E6swlvZtqyIoV68VN8vKs4vgQXDWSP2YoW2y88r11PepIvCFQcvpC1DD+7C2 wAHFl2zxqdEHjZEsh3qDqZ86p0B1IB14m09qZCaOcEX4ljt6ow/6Rp/nQPVd0krU1Dmy +hiYy9KgytT+/mMY7A4jRMo3ulvG3zyy+OAvDdEckOjvnE14kTbJceMOCD5dyvFRdFos RismAwS6yqJjJTryW70pM9BWP3bITJC3RtJSSop7lk+2Blvv6dpfwQAfF/vz724fW/cW qj7zt9Aqxb8EMEr/7c+88t5iv5/KJ3qEzRiI0AqOzoFNWr4n7ueqXZqFSCy24Z+SeTd3 c5FA== X-Forwarded-Encrypted: i=1; AJvYcCVxWenWEATAy8A4T1Yd53Bl5xbJLxgXflkJHJzWOUyt3FE+WSIW8q3bEKTixyio8H1u2fA64JqWrs79PT0=@vger.kernel.org X-Gm-Message-State: AOJu0YxTNIG31mLLnz5CdRO9KyXxUlXk8AjNKf5P71CxpmjkOCcg5rfK E9Z07cAoCh/NCwKzb9qkE5hF58v5x1AYqKpskb+CziUZo0fhoKTcbVROXBNIqqs= X-Gm-Gg: ASbGnctgNIMPQQT2p5FqVOVhFJlwnDFMFxkHHJ305xAaxeLrp4VvFnd16csczuakjpP lOHGefbKDy6l0F+QX6lDESPcYHGSodX1Oxz+vNPWc1YFLtWZA3Jucvw82qlzgTorW8qTMJE3dst xAH8Pi5SDmLqd5Rv53eE+IowvFI8sUZkqhQk+UhBbflyoB2wkRFlSQ+z8Ofw+5B4YSt98ZoJB9/ Se0PH0pgRaT/ljaXRl2qG+4VOTDvnhmbqc7Qvu9W35RJyH8SyqGebGxJd6fghWORJf+QZ70J1oW mtWRoiS25rrZJ868OLh/9Hk/ X-Google-Smtp-Source: AGHT+IHs3R+5B1+LBttE+o7iirut3LHPZTq8vF+yRZe/KNbZaBYwfnKQ9/7DTDyxEbPDhjk7vysLQQ== X-Received: by 2002:a05:6512:690:b0:53d:e5c0:b9bc with SMTP id 2adb3069b0e04-542845c05f5mr1136490e87.50.1736397360155; Wed, 08 Jan 2025 20:36:00 -0800 (PST) Received: from [127.0.1.1] (2001-14ba-a0c3-3a00--782.rev.dnainternet.fi. [2001:14ba:a0c3:3a00::782]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5428bec06e1sm73866e87.191.2025.01.08.20.35.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jan 2025 20:35:58 -0800 (PST) From: Dmitry Baryshkov Date: Thu, 09 Jan 2025 06:35:48 +0200 Subject: [PATCH v4 4/5] nvmem: qfprom: switch to 4-byte aligned reads Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-sar2130p-nvmem-v4-4-633739fe5f11@linaro.org> References: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> In-Reply-To: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> To: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Akhil P Oommen , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=2302; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=Weli6Ud1pnhnVMQNhXbarzN29phyH4B6NUF7IDp6kk8=; b=owEBbQKS/ZANAwAKARTbcu2+gGW4AcsmYgBnf1IkACTJK3+cbEVvQiLITbU/iuCdV9QIf923Z +4gR8kMz82JAjMEAAEKAB0WIQRdB85SOKWMgfgVe+4U23LtvoBluAUCZ39SJAAKCRAU23LtvoBl uCbOD/9y3o9U+3IKjSZHvf0PVEnBIh5ablg36IJ6zcIvPqMEWb9e/IrHB9i/hxb+Ndi5oOxY96A a+o0PchHna8jfEfQOdlO4sX1ZFDR4rXEN9vlq3js96XRsbeyE6Qj0uD+4Z6wkmDmi0Ux19uAGkd zaHC8mfrito3xE9Rke3bivnJz620TW0eTAWBHOiICk7NjV4KpVXYK6b1vpLoiAAT60JDdihI69A ZaYdPtj61kHUVrhD0+Z1WNixvu4pfropDa7qA1gSpdn+GChXJONzjKrq2SZCPkhRmhcm6SRJ1vc mQTQMILV9ZQzc3Apqn82tZKXwQ/0amgx2fOxQngAPQW1YsloCmN8Zq/PY19mtrDH7uwMIE8h9PC nzhkXRumB61iQVYiKbZ1MzHPv1xGX8oB8zzT2XuLBNDnF2anv6GG9Km2ucqC7tCaYXSu2lQBejB hJkVsmm3JNVwiGkNQ4GnU3FdWly0eLEItkFC9IGM61wTITSCa7QV5dDpaYaJpl7lzYz61oABBct 5RTQkxajL9rDkpdZ0vFEycIpfQbWVGs3KDZ3J/sprYSJlZAljWVrgqpkvHstnAZpVqzN+PQMtTp SX3834e1v1is/rP1Lndu+/hQR8MOEUnyBMlCRXbrhqBSMZdCR3DNZpsHFl/X8ElvGlOrTLMKA7u xF5aUi3mvdjawDw== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A All platforms since Snapdragon 8 Gen1 (SM8450) require using 4-byte reads to access QFPROM data. While older platforms were more than happy with 1-byte reads, change the qfprom driver to use 4-byte reads for all the platforms. Specify stride and word size of 4 bytes. To retain compatibility with the existing DT and to simplify porting data from vendor kernels, use fixup_dt_cell_info in order to bump alignment requirements. Signed-off-by: Dmitry Baryshkov --- drivers/nvmem/qfprom.c | 26 ++++++++++++++++++++------ 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/drivers/nvmem/qfprom.c b/drivers/nvmem/qfprom.c index 116a39e804c70b4a0374f8ea3ac6ba1dd612109d..a872c640b8c5a558da9ea00e380= 4c904f8987247 100644 --- a/drivers/nvmem/qfprom.c +++ b/drivers/nvmem/qfprom.c @@ -321,19 +321,32 @@ static int qfprom_reg_read(void *context, unsigned int reg, void *_val, size_t bytes) { struct qfprom_priv *priv =3D context; - u8 *val =3D _val; - int i =3D 0, words =3D bytes; + u32 *val =3D _val; void __iomem *base =3D priv->qfpcorrected; + int words =3D DIV_ROUND_UP(bytes, sizeof(u32)); + int i; =20 if (read_raw_data && priv->qfpraw) base =3D priv->qfpraw; =20 - while (words--) - *val++ =3D readb(base + reg + i++); + for (i =3D 0; i < words; i++) + *val++ =3D readl(base + reg + i * sizeof(u32)); =20 return 0; } =20 +/* Align reads to word boundary */ +static void qfprom_fixup_dt_cell_info(struct nvmem_device *nvmem, + struct nvmem_cell_info *cell) +{ + unsigned int byte_offset =3D cell->offset % sizeof(u32); + + cell->bit_offset +=3D byte_offset * BITS_PER_BYTE; + cell->offset -=3D byte_offset; + if (byte_offset && !cell->nbits) + cell->nbits =3D cell->bytes * BITS_PER_BYTE; +} + static void qfprom_runtime_disable(void *data) { pm_runtime_disable(data); @@ -358,10 +371,11 @@ static int qfprom_probe(struct platform_device *pdev) struct nvmem_config econfig =3D { .name =3D "qfprom", .add_legacy_fixed_of_cells =3D true, - .stride =3D 1, - .word_size =3D 1, + .stride =3D 4, + .word_size =3D 4, .id =3D NVMEM_DEVID_AUTO, .reg_read =3D qfprom_reg_read, + .fixup_dt_cell_info =3D qfprom_fixup_dt_cell_info, }; struct device *dev =3D &pdev->dev; struct resource *res; --=20 2.39.5 From nobody Fri Dec 19 20:36:28 2025 Received: from mail-lf1-f51.google.com (mail-lf1-f51.google.com [209.85.167.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 8B2A92147EA for ; Thu, 9 Jan 2025 04:36:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397366; cv=none; b=hmV3/mcDw+an4fikDuq2y9e7FYlpmnspXO/yLF5RbzXKorh552ZPiRBGU5CzfbEkDMcX12Qkr5aXztGyYg3tQKr72t4L0N50TsOuOoJqxveL5N1FzecdQ9voJWHf8mCFgROVltLQgAzj3I2f90zexg66MhnXSDHpLremoZ5a0KI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736397366; c=relaxed/simple; bh=bFoUxd13xqGzTL8i0Uke1VWxv8nz0uckVKpDL5fYKp4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=lazKRYmjMRq05M293xNzH3rnlydQnsFIcqpTpoZ3T3fEymgFNPTn4CRDG/kNrIWgJK6p5YIfaSnvCuAT/3/pZNPKl2Ptp2JF/KtbQ6y6PIugtRM0PY7Rsqo0VDSVoCExEu88uDczg7UiM65oqL9bmlXaHVRwVB/7i8L/m90J7MA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=Pj0tpkKH; arc=none smtp.client-ip=209.85.167.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Pj0tpkKH" Received: by mail-lf1-f51.google.com with SMTP id 2adb3069b0e04-53e399e3310so519182e87.1 for ; Wed, 08 Jan 2025 20:36:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736397362; x=1737002162; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=tBnHSa/ei6D2pomsNrU1v/8n3usXWfC08yobzI09Jro=; b=Pj0tpkKHWmRfq9IupCpqdtxOrU0ZJnk2tNicleGk3isY33BEPZ132l/2UyX9xB4K1f TAMOUjd++y+/sG1gxLgptuQiGOP3nT91i+ZGa+cPcQGGNg3KrU6+4nuZx8svAPveyWNY BUNWYn1aYTlR7/5wsngTB6teLEqyUq55jOL1XurX5i9o1hNKatMQXFpib4RkgRpa6Qmx pqmevoBglaIgbeQG23sNyGOVcw0mbMU7ytuR8DZvVxXhNC0uOn2zgJ4f2qipC1a2ofOq c9UlfbyIBuHMNzDFuP9YyzxeJuT7TLeZs/eUOe5QKpBqvYq3uRmOg2FzWhyn7aclciyy U/6g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736397362; x=1737002162; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=tBnHSa/ei6D2pomsNrU1v/8n3usXWfC08yobzI09Jro=; b=IA3TXLus121hvJHjHqbInQM0Gu4Dp4kzqaCcbbwBdnxL+2Sem3s9i7Ley1O/X0xHU0 ihKX5PZBqu7JKDMG/L1d+jHesd7JdiFJ+5KuDGG6Zw5YHkG9da3ydWjTHUgbX3B2ZE0b VJwc40/Z2F83AwFSgYg5Qo5G8ihNI5sQPH0wZP8No8tJuvq/Rk9oiaT/OHJk0gahnMux /VlL9eY3xOUfEmLYlrA/8QyZiFXJ9WPvme1gLMhcgU6Y6Jmda+ytqBOgQjMHoDvcR5Vi MG0g2P3D7xeff3TT59GwFsNATI7pi3mGljajGG7j5I78i8ERQ9V60wL9zmYlyxEwYGxc cEjw== X-Forwarded-Encrypted: i=1; AJvYcCXNRDAs3gSnDDUuXFoeNOdLWdFhOuA6vTQW+d3TxRWT1cPjp4YP5Whn+3W0UF3dkJK18Vlda0kcODzVrMg=@vger.kernel.org X-Gm-Message-State: AOJu0Yz1zbIgul+fxBhQvJBzll8QqAcSyyV2zoT41i63ZG9DxnCE0A+C Umj6E/a1SePgZX9HajA1+6k2/nCYAy4s//RFF/HPc4P2knadwznLWgHLtmwVXJY= X-Gm-Gg: ASbGnctq/Pv0gasN7gzTuPXs+Fd/k+uegvr81f5Zyi66wOWLlf1EgTs9rSfqeDZQFyl Wl+w+GvQlzlwBuF74leNfQEVQOQS0qsDabPtEBDKix66Ynrx5BIemSMs1vGD0h+iOR+7P7n3yG2 ocnCQvDapL7aHfgcTTA5N+ejZ/+AnTy9QEATtohAJvYBzXWjFQq3XpVGpuf3ebNCU79svZka6yW kfsiKAMu37nxJ5PlwGDvQyXwJATXf8OsV99S4hstrB6kcZ2krgU7wTRT6alRnLBPw9HWy6+2Eov 9ZhRSZ3MbWQoDg6C9gi4PFxV X-Google-Smtp-Source: AGHT+IHNpliUbCzShTHU3YELA4A2kOZ8c3cc6bdarZT0RN/YgBassMDYaqQENO1wTf98MuMXuQ9xwg== X-Received: by 2002:a05:6512:220b:b0:540:1dca:52bb with SMTP id 2adb3069b0e04-542845c52a3mr1607860e87.28.1736397362546; Wed, 08 Jan 2025 20:36:02 -0800 (PST) Received: from [127.0.1.1] (2001-14ba-a0c3-3a00--782.rev.dnainternet.fi. [2001:14ba:a0c3:3a00::782]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5428bec06e1sm73866e87.191.2025.01.08.20.36.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Jan 2025 20:36:01 -0800 (PST) From: Dmitry Baryshkov Date: Thu, 09 Jan 2025 06:35:49 +0200 Subject: [PATCH v4 5/5] dt-bindings: nvmem: qcom,qfprom: Add SAR2130P compatible Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250109-sar2130p-nvmem-v4-5-633739fe5f11@linaro.org> References: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> In-Reply-To: <20250109-sar2130p-nvmem-v4-0-633739fe5f11@linaro.org> To: Srinivas Kandagatla , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Akhil P Oommen , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=866; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=bFoUxd13xqGzTL8i0Uke1VWxv8nz0uckVKpDL5fYKp4=; b=owEBbQKS/ZANAwAKARTbcu2+gGW4AcsmYgBnf1IkEzuENDcUEcO0yq8IH9HJSBlZNlZ96P5un s4G2pMI+vyJAjMEAAEKAB0WIQRdB85SOKWMgfgVe+4U23LtvoBluAUCZ39SJAAKCRAU23LtvoBl uK/UEACfZKDyD2ZRlKrK0nQ7XUI4Xg0II/U+NjC//mQGfymuOwLzxtzekZ76kaNWGnUtcnbmvIQ 7byw02C76jPzUi+g9MLHcFllLdEYRX2o4JTveJo5ZfOTNGe7bkVdJpsAcFr5rcLUhdUsmL3ex+k Olj3xZ4gGDGy2c6RNiM5EdLfIwgeO795TpSK1UanTuE2vi2OJz9ZCDjrczsw5Fw3hS/8R+NGMby XDnv9a9WIEdZdx6QX+CPF2Z2Bffg1pELrm5lupfLMakBoqKfBeVxx+Aul9/xDxPWctGoSsg2puz +0Sn4RdLOCv7UX1M7kNR1HXlyhnAvxgw31cGrGXqgXhrrY+WTqChJf6LqK6VZi8+H0jdIFTQfX8 kWEgJt0sOj/K3PKEZoTYxshnv+nQNxAS8H5isSDoybOqQlqDdFNpoqXtVvg9W4cNysNZbBtpqHc dw5/PQVJEJn8nS4vzkR1txVQ0oKV0rfEdMtNQG8LRNud8nxqwjolYU4HAChN6xj6fC8SdVpHlQG n+Wg6zpQ25LJmvkqFnwSxl6D3lO0gWkwsfU8oj2z2HLzm1HBMbuS3p4nc2aPLP9JhFIol4/FcxG e+2IAnE8NAPgaIR61l04tbHdNlQyAKVPJe3jLerCltmW3lTRROUHo7tLftXKGvkHnucY+Rw5Oi1 cKfwlasmdca2hRw== X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Document compatible for the QFPROM on SAR2130P platform. Signed-off-by: Dmitry Baryshkov --- Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml b/Doc= umentation/devicetree/bindings/nvmem/qcom,qfprom.yaml index 80845c722ae46611c722effeaaf014a0caf76e4a..9755b31946bf9d4c1055a993145= d06c274b61a37 100644 --- a/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml +++ b/Documentation/devicetree/bindings/nvmem/qcom,qfprom.yaml @@ -32,6 +32,7 @@ properties: - qcom,msm8998-qfprom - qcom,qcm2290-qfprom - qcom,qcs404-qfprom + - qcom,sar2130p-qfprom - qcom,sc7180-qfprom - qcom,sc7280-qfprom - qcom,sc8280xp-qfprom --=20 2.39.5