Message ID | 1480675876-19540-2-git-send-email-peda@axentia.se |
---|---|
State | New |
Headers | show
Return-Path: <linux-gpio-owner@vger.kernel.org> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3tVYGV48t6z9t1H for <incoming@patchwork.ozlabs.org>; Fri, 2 Dec 2016 23:23:30 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=axentiatech.onmicrosoft.com header.i=@axentiatech.onmicrosoft.com header.b="I2b/Sw7i"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754358AbcLBMX3 (ORCPT <rfc822;incoming@patchwork.ozlabs.org>); Fri, 2 Dec 2016 07:23:29 -0500 Received: from mail-db5eur01on0106.outbound.protection.outlook.com ([104.47.2.106]:60064 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753274AbcLBMX2 (ORCPT <rfc822;linux-gpio@vger.kernel.org>); Fri, 2 Dec 2016 07:23:28 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=axentiatech.onmicrosoft.com; s=selector1-axentia-se; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=6x7tkJDPzIsZjHYa2ykteAy4qhJa/ZmZ4lLH9KY2ufs=; b=I2b/Sw7i93f+yX183kiNe87FHS0XxuXAoxXXmn3EkLuWSa9WJHAc/moB+poavXPll2kx5pqpcp+r/9kgr61Sc6Ygr471fvhbU6mIBASOndMQYXBvLvfDf+QFvzJgzIKTkv4fgdDUgCoZIwDeqAb4rSY8YszNgB8afIh+w8HcHM0= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=peda@axentia.se; Received: from localhost.localdomain (217.210.101.82) by AM5PR0201MB2308.eurprd02.prod.outlook.com (10.169.243.7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.761.9; Fri, 2 Dec 2016 10:51:34 +0000 From: Peter Rosin <peda@axentia.se> To: Linus Walleij <linus.walleij@linaro.org> Cc: Peter Rosin <peda@axentia.se>, Andrey Smirnov <andrew.smirnov@gmail.com>, Neil Armstrong <narmstrong@baylibre.com>, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/3] pinctrl: sx150x: access the correct bits in the 4-bit regs of sx150[147] Date: Fri, 2 Dec 2016 11:51:14 +0100 Message-Id: <1480675876-19540-2-git-send-email-peda@axentia.se> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1480675876-19540-1-git-send-email-peda@axentia.se> References: <1480675876-19540-1-git-send-email-peda@axentia.se> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [217.210.101.82] X-ClientProxiedBy: HE1PR1001CA0018.EURPRD10.PROD.OUTLOOK.COM (10.171.99.28) To AM5PR0201MB2308.eurprd02.prod.outlook.com (10.169.243.7) X-MS-Office365-Filtering-Correlation-Id: fef0d149-f3e6-4c20-88fb-08d41aa12f9c X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:AM5PR0201MB2308; X-Microsoft-Exchange-Diagnostics: 1; AM5PR0201MB2308; 3:SyAOofFhR2eMI25ZIf4+sjwh+9VVqxQxRV5w9BOvLOLm4K/v+nrwud8ZlyX/i8nC5e0IR7p6wG0usPRMfWuf9J7YhYSdOrXbuDnZZD5vUlo0fWytNVEQPtzLS61pzgj5J3hoVnrQzICm3wBf43ToHLwwvowTP27gIgNvw34Gz6uaUHBJDIVrgNUp9+VxiPT1aLzVUp2tUWVHAWNiQcnR5lqSTBhSG/3cJP6tZ3puirQQj8CRLvaSYInRvJBdpvqGEuTtig9hiKnGwNNzPe81kQ== X-Microsoft-Exchange-Diagnostics: 1; AM5PR0201MB2308; 25:iHWoGoPbL6hjGbwaFeDTGfn0kYiCcvbySKujknkFRPkSG2UvKkzo2n7jU6JSook/Bf6wonBlO0dGNAI1OdySLZwsxbYlvvQtL0yqF9WCDUC0ydLYZEsFJNZGugYQlFdC1ydwIMoAoLfQv3Zc8D7i/6VXA7hU0gb33pZ/gzSF0Vr9f+zzXhlhxL2Dpvhbn0ZsyCU/RFAGftfMOlvTXNgwbKGVoSjrsdDN1npDc66qRGsRmiN64wMtrOF1UReWAx9Cj4lNlSqI5Nb+8+h5lwnCaXduaVzdRi5INpFphTAptyaWqyLkUzB4cWLBvUs5p7FryXPRYm6q0AvL5OLjbGR74hPLhcbMEA7q9EZ34zhVoX+EZfXuYoAY+lMRbZe3fLGPwsLiKK/l6PshNkmiA58T3XSLvsd8LNzsM3JG/ANM5liNoQWJjDX2fOUze/E2/Gih5ZY8O/kk67j5aDP3qG1JFpz60Gc4qM7KaFb3I0AY9E5qK0zJIBx/OukhGeQbhx/jqWJHqCx6+zyd222wwBuze6HCbkONLnecPemZNMEROZpf5Gsa5zz+061CoTX2rBocTpIQaCYWVHZiNWxwih/58MksW7i2d2fA989Uav+KYPhv8Y7LdrB3Sk49LGb/f8a7hYq8MhudWTjZLnj9nQb8ffoNKsBaaSZSwFN79jgl16sFVHW8Pt6rjgkWx1yKF1tIHfzwsCY693rS1UABszz2uFXm9jMJgu2LoBieP+YOdlB/HilSB1kOrngKe9YKx2sIq+M0Q4kAiqAh8D3JMv/HS6YQabPf9UAf2XeGui12B68= X-Microsoft-Exchange-Diagnostics: 1; AM5PR0201MB2308; 31:rm2SZ+joa+rqiRY+3rLEQv3LWcNYA6WRdqVIIAXAQ43APws5PeN3mp3xtvpsqLrV34J3c5FNTYV7xPzPBR3zGAu7l++JLu8T38OcuV0Lm0N871XLDmkQ74lG7lNDfCz7dQlbt65Iu2i33sMIQ8h7mc5jWaz5/8R8Ezzk3Sr0xaV1kP9z4woPSiMcaNJT+m7hZ9g8MNHt1dNsw5LxWQfU9TC3FNti/IqefSkpI1cPbu2cpQaVzQcFe1hScyqCeLlNKTvEzKgkzklyKaYlaM0gww==; 4:cFsKJ/VYHs+PQqxSVh0qnuH27CneEMNx7REdVdWN0JxjljDVh5On6AGl/+bXKy4JXNyCumw1ars8/M0LgN8PhHqrZeNa0IlJu+RQmvQmzmG9LSiVtpirBUr09z3pdHRBYuzEodSi6tv6l0vRRdD75k9L2UeLqHa7OuAHJmWc8WNtyV9KRm5pO7/P7Je/pT7OlXje9RRS9uxm0nSVoIGVnbHy+/OOhDnzq+kVoiQnXdllsvnynTMT8yR1K6Ic9IDjLQxrDt95seagSuCrdN7RD0cWPMFKLxL/vFrinIU3jcdE4XOB/Mz7CHWpWRyrVNml39y364ofTR+vAjaf8aXwUxjryfAssituRs1kLAln6KD8w8jpqZgGkGIDY/uqRWjsTzGdy0P+hzN/wxNNuqtotxpOpsw5hifqI/0VCJPdvt0f+DCndXNLilgYhgttSbSrwImbRDSNXvl1YYfPSKavvRn5rlc6BP18AmlOBXbXLG+nEst1yIR3ksFP0778KgjIGwV9h3HmasMFFOpkp9u70jWdaAiAiuF+0dq1/bbZ6YQk1JNK1VlmrZuYZhgy9JTD0HGloo1rRcwnnqY+NyaS4eF5zTYvWHEjV8vA/SeoyBXiXpnEM4Fh1tx819gc533d X-Microsoft-Antispam-PRVS: <AM5PR0201MB23089D92933B1AA96F6FD47BBC8E0@AM5PR0201MB2308.eurprd02.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046)(6041248)(20161123555025)(20161123562025)(20161123564025)(20161123560025)(2016111802025)(6043046)(6072148); SRVR:AM5PR0201MB2308; BCL:0; PCL:0; RULEID:; SRVR:AM5PR0201MB2308; X-Forefront-PRVS: 0144B30E41 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(4630300001)(6009001)(6069001)(7916002)(199003)(189002)(5660300001)(68736007)(2950100002)(106356001)(97736004)(189998001)(50986999)(105586002)(50466002)(5003940100001)(6666003)(42186005)(76176999)(6916009)(101416001)(48376002)(36756003)(33646002)(3846002)(6116002)(86362001)(39060400001)(47776003)(92566002)(733004)(7736002)(2906002)(7846002)(66066001)(305945005)(4326007)(74482002)(81156014)(8676002)(110136003)(81166006)(6486002)(39450400002)(38730400001)(39410400001)(50226002)(6506004)(6512004)(42262002); DIR:OUT; SFP:1102; SCL:1; SRVR:AM5PR0201MB2308; H:localhost.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: axentia.se does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AM5PR0201MB2308; 23:cRlj0iEEHrEQu84w/ZUe42KCRuNb1qXUBFaj0p8?= =?us-ascii?Q?6kNbBqdCtvqt7E6ZxfFp4TKYAFyk98QpQATQul754ECDm2dii26BsPPea+qG?= =?us-ascii?Q?IJDWKLF6GyVBqdc0eNOk5HE3UxNAPrSnQO7YODGzhXbV46VOVLP27pKkRELG?= =?us-ascii?Q?xGjodQ4ZBrR5zuLt6arXPdOncTCWP2Y8zSAofKISNxp+hvcx7LZ1RycborEU?= =?us-ascii?Q?iLFxKkIPrIR11EOO0AsWzgNRqBCbMbNtNhibGmQedIglDe1kE6iMrNp1CCSw?= =?us-ascii?Q?ZenHFKEzoSgja8NkZwzvRvUvAMXHqE+tB7F67KsKsujCPSUtFHFyuX7hCANT?= =?us-ascii?Q?hj8xVwFpIijbiURhZE5Lzeo4WXugO7PnEO9SN21ozUb0V6LjGWcs3NuW7cS3?= =?us-ascii?Q?ZgSFRSdgQuQd9LSVrbtbZ3nwOiJ/j78v2SUW3sD+En3awn/OypG3VyzCWYqh?= =?us-ascii?Q?9cQ+By7SNNyorzlj55gsD9MTZHRDm61iGKRSFBMDAQrZYnm0YxB0tC26NdOU?= =?us-ascii?Q?FRSgBYdkx+zTwaqVWaCy9Lhv69qc9zD68/z+1a7DjyBu7I2HJ5+xVUZm1D75?= =?us-ascii?Q?8vQYExc/O/sG1G27pbJOwJOH3Zo3NY9xfjSvRS314A2YPKpkNTiTB3EnGrml?= =?us-ascii?Q?KDgBeeq2uHMiFmms9Zu009t62idz80K4xfkh1PmM1LWaVDCcYvaeLJzhO1wg?= =?us-ascii?Q?D5Hg7NDVk0zbApguJEhkVXgkGSe/LxdmySqYeEu9FEXLfDq3gOz2StrdSFog?= =?us-ascii?Q?Af3gqd833QjlueQSssQFjpkSJxgHowEtvME+KiftsMU6Z+0OHMBs6Vu8hOEg?= =?us-ascii?Q?QkPAR0xPHPozYOLg1rwvik7H/+L10MRNmVJPLuJFWEicMBhFN86CxcLceR0r?= =?us-ascii?Q?8XRHPlBz2Czbz9A8IM7xzch9vXvT+KUVejZ4idQlXQHOjZPiH6cwMu5aBOqs?= =?us-ascii?Q?Btb8EmvjvEu7TTn3AcwTyQbHon8ee853UrlvfIHBJhQ/DRmW4t54mLSmERLk?= =?us-ascii?Q?lL07oTnunh4BcM8XgCLOJEFwCMSH26tTqgBBoQg4LsQwX9fAE2HuOKEoKtJm?= =?us-ascii?Q?tKGHwpSk6oM9Z16pzQdW7qg4SpidhAO4owen+XVuMrcJ2f0vmLDlZUcFwkzL?= =?us-ascii?Q?h6eAxc/xQkY1OzJ5tWvRvg3KGV2brxRDNMQwHk8rb9cTrjoLDpylKA7HckdX?= =?us-ascii?Q?EOa1tlwZjhaTKKUH45Hl2QKxp0UNZih3kwq79?= X-Microsoft-Exchange-Diagnostics: 1; AM5PR0201MB2308; 6:S4YASNn6tr29CekMAgjEq0Vn7LeZSd98G37eFlQ4TMHxCMnUf5ZBIwZqkczFxh4IulnvPUgkQLLUmTFPsoHPqUQXl29+Mk2j98WL61PRaT/53HrcC7eBmIYdSTI3uOKRgna/MeanUs8Kv2WF1TShZQH0VDwFShdZ5Lt0WINz6HfARVYJOOwbVfbUy0TEEpB+vQziZb4GuCvj3TdprN/umLfcUHgrBEvrrSc/duo3LFf+q4Mcad9sXO0DgffLW12zdfRRW/MpOPPZFT2nErsw3zZwCxXapGH/pk2Gy/B9Nu2QzI9SVqlDkZZMpbTj1aVUdekFJrThwehMwam958m85/VCUv/Wh0vtDVlovIAfM1CwFPpFCnLD1/hW4N02rwzg5IfpwDCbjBbes+C7wx6JmJ4abrvKRhv5le0/07aMdEOBAiVZC4MJNSFx0RG96dIX; 5:bdZnuaQJ8VKVF4VIzsBhICbO4ig/qWm548PYgzqgNKj+62MK8B9TNngCDdS7gzXzK/sfOYVRYGiixnuRI9sO4epI399yFOzIWHu4HSKlKX5lJaUri5OhBDAuICZtGlyKgmvmf9qE750I9DN3UE1QmA==; 24:/ephbE+4ICWjbGDy2/q83a3/n6QfPPGd1VZVgXzzds2B86Be5NvCnbvFLvi9DXMKfKwer5p1dxo7r+VN9xLz5eaaHhs//UnrdTae+/YA/Nc= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; AM5PR0201MB2308; 7:YoFxyHKg8OLeGOuSs82dg1SpKqea5Xq06fuNgo6MysECYhdc2pk/XoFPZa3S376D93WnuYycMmbEdHEhPh+Q52iy/f5hThwz+6fJdqlBTZv77uJ7iFOYNHbMonYYa/NAyU/SgliFrmLr8Dfvres63wWtV3IQVgJYs0mfiB16dYLyxUXT4VRJxBA1nQAkbOK4+8EUEPtQs/dA2fQdS9BOwLphfu6nCv40HxnE2mcazoZKI56u0iqrCx92JOlK6dz3KBi8x91Ca6ryazJjZGY8q9l3c4TC+1OU2TuIHD8kHUW/MprgtdINgGHaj92xhmtU1iSndruEpuX6oCIH8WULPAnk7HBQA7a0ZcN3HqTXJTra/jyhWpBVr4i77tSY/+MNSWo5t5m0o/ZgKJRP56eU+MhnOUUiq6suVzbaVrx+XH2/U/91vRBVL+PTS8dfKdgW8+HJHqvSMGacY0xEWx5kHA== X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Dec 2016 10:51:34.8520 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0201MB2308 Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk List-ID: <linux-gpio.vger.kernel.org> X-Mailing-List: linux-gpio@vger.kernel.org |
On Fri, Dec 2, 2016 at 11:51 AM, Peter Rosin <peda@axentia.se> wrote: > The code assumes 8-bit or 16-bit width registers, but three of the > chips (sx1501/sx1504/sx1507) are 4-bit. So, try to handle 4-bit chips as > well, they leave the high part of each register unused. > > Signed-off-by: Peter Rosin <peda@axentia.se> Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pinctrl/pinctrl-sx150x.c b/drivers/pinctrl/pinctrl-sx150x.c index 1a1c8b51a992..a121819ffc92 100644 --- a/drivers/pinctrl/pinctrl-sx150x.c +++ b/drivers/pinctrl/pinctrl-sx150x.c @@ -5,6 +5,7 @@ * Copyright (c) 2010, Code Aurora Forum. All rights reserved. * * Driver for Semtech SX150X I2C GPIO Expanders + * The handling of the 4-bit chips (SX1501/SX1504/SX1507) is untested. * * Author: Gregory Bean <gbean@codeaurora.org> * @@ -1088,7 +1089,7 @@ static int sx150x_regmap_reg_write(void *context, unsigned int reg, val = sx150x_maybe_swizzle(pctl, reg, val); - n = width - 8; + n = (width - 1) & ~7; do { const u8 byte = (val >> n) & 0xff;
The code assumes 8-bit or 16-bit width registers, but three of the chips (sx1501/sx1504/sx1507) are 4-bit. So, try to handle 4-bit chips as well, they leave the high part of each register unused. Signed-off-by: Peter Rosin <peda@axentia.se> --- drivers/pinctrl/pinctrl-sx150x.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)