From patchwork Tue Apr 8 01:40:02 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Peng Fan \(OSS\)" X-Patchwork-Id: 879140 Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2065.outbound.protection.outlook.com [40.107.20.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5B78925EFB7; Tue, 8 Apr 2025 01:41:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.20.65 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744076512; cv=fail; b=joofBm+j28B/vV3wO9MVRXnJbAc3GjeZfuUIg5yS+2tGeekL9xng9m0Bucay3cbdyNMyy8ddiYjxw9L3FIDxPaGRhIChSweAeNBfknPEQoLRWDrSfQ2ajKf/X2t9rf7RgC6med2E8l5kSbkJ4svgeGoatkLpVV3rdPN7QZvoy3U= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744076512; c=relaxed/simple; bh=v+RPL3nHlGd7lMY5K/0g5UD3Yu9roR9UGEMa0enTg9A=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=piLl4pVlngfw8Ypnn0huQZm8BDZekcG/ngZ2vmQX92s2ZetVQM2/Qsd0CmpGZwfbVNcymTHuDnK+sQyMszOKcMsvzQ6S3Etj47Pf/dUSOlRXsCIhvc0zK2nS+FVUm1Fta1m2Nf0AyiFxGOOr12dFM+VRbhLdK7gMfF9jRbuj7LU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com; spf=pass smtp.mailfrom=oss.nxp.com; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b=vwRNNpQ/; arc=fail smtp.client-ip=40.107.20.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=NXP1.onmicrosoft.com header.i=@NXP1.onmicrosoft.com header.b="vwRNNpQ/" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=JJnwIR0sSJ0QOqAOemd4DJRVp0FHgOkDiT5GQXstGyQpyDbOBcAK8lczDkRoe5QfHK7mVd3O8Ty6Pxyp4YA9vZPaqZVGgamVDnTHkTNILjKlyZUcU96c9fkoyowsOWicu4DJMNy6GbsBnmAFMOkf9uipZKKItLplo4cmrBPWG6L7YiLHYUHqH/x7PLVOSwJYeUFe0mRCUPMrfGh9QhK+hsbOu2DK4w/0vjuJobOJOxTIYc0Bmo1MFmje7TVC+SkVlUzvNYcdDPgoRk2/U9Xdp6eYz07LeKv4sqaCfzYvTGNemc19UAHMZK2cHEaXaKxEWhZBZNExT+K2wZ47drXmtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=z3IIMIDbGoyGYwqxJqtSYGydirTJTdMneacc8sH8PYE=; b=QtwG3jjoeEjkmFn1yP6vgA3LQC9rzJlDgnPyP3zHyg+7GWNi4LPYuSxou4vWTYDGohBMROgDIN0j8hVxZvWAeegoJY6JFybZaxY8mZTE+ZWBzUW/78LGJC+LCjFcjwJlyE8rmP1L5FFmmeMBqemIleFMMaYihKe5pK1YbSqt7XRkyzRi2mjFE0NMqBFDQZ1JAQ5NvZDHKLci1kpChnu312udlNRxCT2z4BsKipzq9217W0gIKM7ISqBCfRwg1EibeKnn+emrEPrVtmQ6+gVg5VH30iB2XZN9jEBW4TinlsBdYNSsVHBMY0UJt9mJ5hvxE2nOJHDAsCCQqBTyddORfA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector1-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=z3IIMIDbGoyGYwqxJqtSYGydirTJTdMneacc8sH8PYE=; b=vwRNNpQ/Vm7KwVGoc1C0/iGOtIFw079m8xY36CMl0dIi/5LgqBC/M1q2dPOa6eIgMKwEmkaHYTEjHBAdkkc6XNh5ERe9jLy07AyxmNPjvRPi+F7NnnY84ub9UnC5ynh2IcZZmGx5wzfYs+Ks5zv45jZHVa9niKWFJX8vMp/3syRxQkUgXaHQ6kHAhgP8w/+pyBVyAPNzp95K4UHDxhoglHbieaVsSBwiQg+BRb+w8kYnlQ+WJB7kvAvWWPRJmaopMUTlJpjvnNfjCmSjSD/DJOp0Z7FnEz5oBIU45rDy+14GLnwCH+/wGBr0PIPzIYOEbLsCuwrII75zZxOAluYoKw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from PAXPR04MB8459.eurprd04.prod.outlook.com (2603:10a6:102:1da::15) by GV1PR04MB10199.eurprd04.prod.outlook.com (2603:10a6:150:1a8::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8606.34; Tue, 8 Apr 2025 01:41:48 +0000 Received: from PAXPR04MB8459.eurprd04.prod.outlook.com ([fe80::165a:30a2:5835:9630]) by PAXPR04MB8459.eurprd04.prod.outlook.com ([fe80::165a:30a2:5835:9630%6]) with mapi id 15.20.8606.033; Tue, 8 Apr 2025 01:41:48 +0000 From: "Peng Fan (OSS)" Date: Tue, 08 Apr 2025 09:40:02 +0800 Subject: [PATCH 6/7] ASoC: codec: cs42l52: Convert to GPIO descriptors Message-Id: <20250408-asoc-gpio-v1-6-c0db9d3fd6e9@nxp.com> References: <20250408-asoc-gpio-v1-0-c0db9d3fd6e9@nxp.com> In-Reply-To: <20250408-asoc-gpio-v1-0-c0db9d3fd6e9@nxp.com> To: Shenghao Ding , Kevin Lu , Baojun Xu , Jaroslav Kysela , Takashi Iwai , Liam Girdwood , Mark Brown , Linus Walleij , Bartosz Golaszewski , Peter Ujfalusi , David Rhodes , Richard Fitzgerald Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, patches@opensource.cirrus.com, Peng Fan X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1744076413; l=7472; i=peng.fan@nxp.com; s=20230812; h=from:subject:message-id; bh=qBgtNoCcAcvKxfr6iKprfrvHqR3mpPSjGlxZd+CHJAo=; b=r15niQXT/IuH0LfAZVap+mj4hUDUGvMPdQCZEGAAsNdabpSED9/z6nP3bheXwtxsAZZyQfmer ctgV4dvXE5RAi7WIRE1f8a7eCHwznMX2CiAR6BM0fcc1O0TdwVQARwZ X-Developer-Key: i=peng.fan@nxp.com; a=ed25519; pk=I4sJg7atIT1g63H7bb5lDRGR2gJW14RKDD0wFL8TT1g= X-ClientProxiedBy: SI2PR01CA0048.apcprd01.prod.exchangelabs.com (2603:1096:4:193::17) To PAXPR04MB8459.eurprd04.prod.outlook.com (2603:10a6:102:1da::15) Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB8459:EE_|GV1PR04MB10199:EE_ X-MS-Office365-Filtering-Correlation-Id: 9a9ce93a-4afa-44b4-09b5-08dd763e873b X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|366016|1800799024|376014|7416014|52116014|921020|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?T1MVD0n/wHLYvC0K7ZoXNuh+GJNCkbJ?= =?utf-8?q?a481Zb/aA+aJlXxAQgewY3R7X8vc+jipstwQOwpZ3Wq3kR4LyfA6KvicA26F9MlC2?= =?utf-8?q?4xqf8VBtKGwajPjNNVxbwemM5Pex91+8ZAnGpVEtlb8aKSwQGPyrC+8c767ZTPgfB?= =?utf-8?q?0q0449o7XLTqnSNLI26YqShPVcvv1IuxuQ77Pl+yWWkrtVwwzcbWeJSb7UOXePF6r?= =?utf-8?q?1PLLo+4+IoTJ5B+xOvLk3HChJIJWL9MQRWNBh0pOR3XCgdA9HbMYK/MCwp5blkxEa?= =?utf-8?q?6mBB91UNS38zXJ1FjOMr73SqJ8mK1WJgoJi7qCUEJdO+mmyNU5568qpwArEJ9qje2?= =?utf-8?q?g2ZOp52MMWbum+4pE1X58/E/ehyi1F0sWwI5f/ZayU/4E81eKOsfY9Uc06Aw/fiTf?= =?utf-8?q?GQWHVFbXEE6Tg3kuLVKvqwhJsOTN+3wgDsXXaDFPqSsVZ4e3jVDDFACD3YTGrYI3A?= =?utf-8?q?WC8zlTkerrpudqB9Fl83gCnjds6Sxi7TZmEmliRmliS6DN+oFCIdinVfChn6sA80J?= =?utf-8?q?OkKRPe7UQiD2TxBlkslbXUlySRtqkTni5o6JmB7hrH2uSPbFX0bP1Wd4CqHk458Ac?= =?utf-8?q?eqAv1TNdbBzPV1m+m5GdwugQLtU87fjfQiNKivlvU35dPwAU0bw1iyvOs0kU16dZi?= =?utf-8?q?FWuScj4GvbqacCIyHtRho3Q7+Wd7KGBrTaJgqdiB6uIMdixsEDvVOuT7lSdWkSuvi?= =?utf-8?q?+b830Ka5CbBL7kHnCSXRrRuaHU1DjElswFSHQmE5ZNa7ZvWVEz7KSozIuwHO24bRi?= =?utf-8?q?ok0ESrDKyjJD+HzyHqJqCGhjnYQFcdBILczV2ZVe75sRjOKBQXELcn5JmYMbTNcdv?= =?utf-8?q?R35oh9jqmRq8WOXE60NuXw4PSaYlE2c7qD7mBz00iz1coPWseOOvAOAMguJidnPKU?= =?utf-8?q?OFOrbAvt7ewXREQjEl7hwlnKCyAB0pT+/lHocbuC9jM9TS/ihNAzIwzCC1XwMGS4Q?= =?utf-8?q?vOA9Akl6h9zXzrEi4Uh1Q8EVzcY8HjJ7pUlqutNhDN/D27sbCRcn3JGjE0+0PQvQ2?= =?utf-8?q?a9JGwywrTNDchHjUu2ayaeioOeAi4Bn/DfJgfWB8+dFs/NcVNxBoNqAfox4HKMaYn?= =?utf-8?q?dVZBADzBWrNy8ofkXCU4XoIguCexiPY+mhFG9LZzaWU5x7t5XR1adW2o7c/vehIGD?= =?utf-8?q?gk/Wy0Z3u+WY9yjKU/2sClaGvc/JIXl/Nb9fsQBt3IVFxqjato93/pyfHAdSk9tM+?= =?utf-8?q?kR1VhNShmr7zpJZVISxY1kVUsPOkYE7MJR3pl4sYmIbzp9H2HMYTkiPSqp2TJYzoC?= =?utf-8?q?fBIXzmeVXvVzZt7BFcuh02P+nkHOaJvR8jOEGatPHDSXU9Y5b6M6rVJxPqVu+s31m?= =?utf-8?q?BvJUQeupkkBM7U3QZM8zvevK7EGcCrT880KCys2pLv8yFVfze9DhnM5ndnmD1Cbnj?= =?utf-8?q?Ive9M2GukurCzWYzDCfvxweO7X+WF9InA=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PAXPR04MB8459.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(366016)(1800799024)(376014)(7416014)(52116014)(921020)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?+qpT5RNp1hIrw1HKEIQu7zGFKTSF?= =?utf-8?q?4SVcYeB+1XQ/3pl6VJrJ+kf2t2yPMeU6+59rtwvY+n4ddlSgWJwxJgzZB/FHiYxr2?= =?utf-8?q?R5hgUsNrsg5EMWd09lXMXyIfb61wFmBdL72ILsoWRfVEFTv4LPTy+rbAWV3Nf0M+9?= =?utf-8?q?jL+cwQM54loxXyZ4Kcyuy8DCYRW91GrX6Kf1QfWaQkE4bGD3Q2tMVLiEqvjwpDl+q?= =?utf-8?q?MESC8PdXr/kE0yG1oYEyi1NKeGVayOBllLhsV2H2Eb3qJtMzIZrYSvZRRbUfACDP3?= =?utf-8?q?v8bphvZTX16jRaV1QGuti9ao7grUOsqxld5nn+E6jexN6F4cfGGKRjK7dk5As3NqT?= =?utf-8?q?lcmC/uq3mcwuBem0XbJXRFLWKmHAn4hcN4HK3TgrweShxTV5vx69+NI6QLTiLzC0i?= =?utf-8?q?jZbcBf7xc4rM04E1pkmnL6R1YOAAjadLUVdCB61jdgUoSeiTZzer8lQ3M+ygFDdF7?= =?utf-8?q?+PA5RFwzvQVaqII8ntRfz8cJR5Kqn2bhxwD1RHOorUQQOnDP8IOZsf6pAD2GzmFwV?= =?utf-8?q?hCt/wTAi/vBZ/KRwDtiO42eeAEh1N9lT1iRk5X3bjavOZmQSBwQ8MzNlX8tJZPIVi?= =?utf-8?q?UyF9N6Gz2UdDiVz875+Y929E4ofrjmTI56fKBWLd1Sm1nrb3QigepdpY+L7oRWce2?= =?utf-8?q?aQqT2hhSjhKQltWs9UpRlRIsVwTT/swElpMX3VftG3lixp6DukxyqR0ThUC8SrMXb?= =?utf-8?q?gqmLLoYar+ZqO0X510wj1iaQ2inZ029PNQEJ/rDg8AGFWqkHDWPKVaYzEFtJ8J3Uw?= =?utf-8?q?xFsg29kvxWzkpsUYRHyxIkXv/Tw6bPgVPbRdi6PUFtsDfyIMRjELpJu2wxO4to/u+?= =?utf-8?q?HHOiIJN8289c3Qilnep0io7eEVdtXvPRX1vXrp19AUUHee8BcuEOl7/IyrgG3kMnf?= =?utf-8?q?BnXydJa6joLndI2K3w3YJsUeZ1A+UhB7EEQQoNcDIMwXlXGVV+qB33Y7uCarP/FUD?= =?utf-8?q?Fu6Pz8yenTjIfcBoUATMWGeS5qr5UapjfmEodQrr/9+Aur3HUsyX9lKCOcgXkPwPr?= =?utf-8?q?X7/jTKhReVauWzgtg+dc2TNHQ0hlLcVetmckZcDN5WX6Yv/wqDMPd1t1Av6SrObBp?= =?utf-8?q?CkkTW5UIwpJqDc4bAK/3uV7DZ+Yxo5Jm2emSgprpu+cflORLOAK/gp4k/Eyv/H/95?= =?utf-8?q?iPcV73EeZ1h6NJDBe9ZSSAJLSUfFOXMncOFZ1qdhg1SBmRjfXZji1Qlbw0vsAnR9A?= =?utf-8?q?m1nptvhPeWl5dR4p3xXQk8enosKax63tCKDnqoRW8dqL7yzajSjjUJrAp/e/gEi4K?= =?utf-8?q?1WQ/BSeu8AscfbW2ZGovvhtr8eXmgMJXj1s3Yia9Aga6/IVuqs0PnSHuoYVt+BPDH?= =?utf-8?q?G7WCZhZeCZhu2ynjy7YjQOFN9Q9taEpy1SwfMRp2GGpKM0mjBlCAd/s7VBHy1YiYh?= =?utf-8?q?GnoPSJISFnWFwTC/phRi1lZGrmaEIPe3+EP/44JGpCm587sVkifJVvdbE0Ji68CWW?= =?utf-8?q?Jp6cDWPRxSRjYNz1UtUBCanVyLKXLiCQyqXtBX1w+hIEITHrUqvTjk0qAcuiDL+S2?= =?utf-8?q?O48TXR9rj0E/?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9a9ce93a-4afa-44b4-09b5-08dd763e873b X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB8459.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Apr 2025 01:41:48.6263 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: nQtYEECx/LHV1EjAxL14zy1nOAinZ0RorurqKPj8fHRLCsVlzHy0hSVw7pfv4dhQajwFd8QVNe54f0MzHRmQOg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV1PR04MB10199 From: Peng Fan of_gpio.h is deprecated, update the driver to use GPIO descriptors. - Use devm_gpiod_get_optional to get GPIO descriptor with default polarity GPIOD_OUT_LOW, set consumer name. - Use gpiod_set_value_cansleep to configure output value. While at here - reorder the included headers. - Move cs42l52_platform_data from sound/cs42l52.h to driver code - Drop sound/cs42l52.h because no user is creating the device using platform data Checking the current driver using legacy GPIO API, the reset value is first output HIGH, then LOW, then HIGH. Checking the datasheet, the device remains in Power-down state until RESET pin is brought high. Since the driver has been here for quite long time and no complain on the reset flow, still follow original flow when using GPIOD descriptors. Per datasheet, the DTS polarity should be GPIOD_ACTIVE_LOW. The binding example use value 0(GPIOD_ACTIVE_HIGH) which seems wrong. And the binding use reset-gpio as example, not same as driver using "cirrus,reset-gpio", and there is no in-tree DTS has the device, so all should be fine with this patch. Signed-off-by: Peng Fan --- include/sound/cs42l52.h | 29 ------------ sound/soc/codecs/cs42l52.c | 108 +++++++++++++++++++++++---------------------- 2 files changed, 56 insertions(+), 81 deletions(-) diff --git a/include/sound/cs42l52.h b/include/sound/cs42l52.h deleted file mode 100644 index c20649666abe5dcbbf628f6c2d1692d3e7190eeb..0000000000000000000000000000000000000000 --- a/include/sound/cs42l52.h +++ /dev/null @@ -1,29 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ -/* - * linux/sound/cs42l52.h -- Platform data for CS42L52 - * - * Copyright (c) 2012 Cirrus Logic Inc. - */ - -#ifndef __CS42L52_H -#define __CS42L52_H - -struct cs42l52_platform_data { - - /* MICBIAS Level. Check datasheet Pg48 */ - unsigned int micbias_lvl; - - /* MICA mode selection Differential or Single-ended */ - bool mica_diff_cfg; - - /* MICB mode selection Differential or Single-ended */ - bool micb_diff_cfg; - - /* Charge Pump Freq. Check datasheet Pg73 */ - unsigned int chgfreq; - - /* Reset GPIO */ - unsigned int reset_gpio; -}; - -#endif /* __CS42L52_H */ diff --git a/sound/soc/codecs/cs42l52.c b/sound/soc/codecs/cs42l52.c index cd3f92c19045ad32f1f0f0f1764390640d68eb10..e56faf6e45c2bd4303350ed088fb28f2b9d9c1e2 100644 --- a/sound/soc/codecs/cs42l52.c +++ b/sound/soc/codecs/cs42l52.c @@ -8,27 +8,26 @@ * Author: Brian Austin */ -#include -#include -#include -#include #include -#include -#include +#include #include +#include #include +#include +#include +#include +#include +#include #include #include #include -#include #include +#include #include #include #include #include -#include #include -#include #include "cs42l52.h" struct sp_config { @@ -36,6 +35,24 @@ struct sp_config { u32 srate; }; +struct cs42l52_platform_data { + + /* MICBIAS Level. Check datasheet Pg48 */ + unsigned int micbias_lvl; + + /* MICA mode selection Differential or Single-ended */ + bool mica_diff_cfg; + + /* MICB mode selection Differential or Single-ended */ + bool micb_diff_cfg; + + /* Charge Pump Freq. Check datasheet Pg73 */ + unsigned int chgfreq; + + /* Reset GPIO */ + struct gpio_desc *reset_gpio; +}; + struct cs42l52_private { struct regmap *regmap; struct snd_soc_component *component; @@ -1090,7 +1107,7 @@ static const struct regmap_config cs42l52_regmap = { static int cs42l52_i2c_probe(struct i2c_client *i2c_client) { struct cs42l52_private *cs42l52; - struct cs42l52_platform_data *pdata = dev_get_platdata(&i2c_client->dev); + struct cs42l52_platform_data *pdata; int ret; unsigned int devid; unsigned int reg; @@ -1107,50 +1124,44 @@ static int cs42l52_i2c_probe(struct i2c_client *i2c_client) dev_err(&i2c_client->dev, "regmap_init() failed: %d\n", ret); return ret; } - if (pdata) { - cs42l52->pdata = *pdata; - } else { - pdata = devm_kzalloc(&i2c_client->dev, sizeof(*pdata), + + pdata = devm_kzalloc(&i2c_client->dev, sizeof(*pdata), GFP_KERNEL); - if (!pdata) - return -ENOMEM; + if (!pdata) + return -ENOMEM; - if (i2c_client->dev.of_node) { - if (of_property_read_bool(i2c_client->dev.of_node, - "cirrus,mica-differential-cfg")) - pdata->mica_diff_cfg = true; + if (i2c_client->dev.of_node) { + if (of_property_read_bool(i2c_client->dev.of_node, + "cirrus,mica-differential-cfg")) + pdata->mica_diff_cfg = true; - if (of_property_read_bool(i2c_client->dev.of_node, - "cirrus,micb-differential-cfg")) - pdata->micb_diff_cfg = true; + if (of_property_read_bool(i2c_client->dev.of_node, + "cirrus,micb-differential-cfg")) + pdata->micb_diff_cfg = true; - if (of_property_read_u32(i2c_client->dev.of_node, - "cirrus,micbias-lvl", &val32) >= 0) - pdata->micbias_lvl = val32; + if (of_property_read_u32(i2c_client->dev.of_node, + "cirrus,micbias-lvl", &val32) >= 0) + pdata->micbias_lvl = val32; - if (of_property_read_u32(i2c_client->dev.of_node, - "cirrus,chgfreq-divisor", &val32) >= 0) - pdata->chgfreq = val32; + if (of_property_read_u32(i2c_client->dev.of_node, + "cirrus,chgfreq-divisor", &val32) >= 0) + pdata->chgfreq = val32; + + pdata->reset_gpio = devm_gpiod_get_optional(&i2c_client->dev, + "cirrus,reset", + GPIOD_OUT_LOW); + + if (IS_ERR(pdata->reset_gpio)) + return PTR_ERR(pdata->reset_gpio); + + gpiod_set_consumer_name(pdata->reset_gpio, "CS42L52 /RST"); - pdata->reset_gpio = - of_get_named_gpio(i2c_client->dev.of_node, - "cirrus,reset-gpio", 0); - } - cs42l52->pdata = *pdata; } + cs42l52->pdata = *pdata; if (cs42l52->pdata.reset_gpio) { - ret = devm_gpio_request_one(&i2c_client->dev, - cs42l52->pdata.reset_gpio, - GPIOF_OUT_INIT_HIGH, - "CS42L52 /RST"); - if (ret < 0) { - dev_err(&i2c_client->dev, "Failed to request /RST %d: %d\n", - cs42l52->pdata.reset_gpio, ret); - return ret; - } - gpio_set_value_cansleep(cs42l52->pdata.reset_gpio, 0); - gpio_set_value_cansleep(cs42l52->pdata.reset_gpio, 1); + gpiod_set_value_cansleep(cs42l52->pdata.reset_gpio, 1); + gpiod_set_value_cansleep(cs42l52->pdata.reset_gpio, 0); } i2c_set_clientdata(i2c_client, cs42l52); @@ -1214,18 +1225,11 @@ static const struct of_device_id cs42l52_of_match[] = { MODULE_DEVICE_TABLE(of, cs42l52_of_match); -static const struct i2c_device_id cs42l52_id[] = { - { "cs42l52" }, - { } -}; -MODULE_DEVICE_TABLE(i2c, cs42l52_id); - static struct i2c_driver cs42l52_i2c_driver = { .driver = { .name = "cs42l52", .of_match_table = cs42l52_of_match, }, - .id_table = cs42l52_id, .probe = cs42l52_i2c_probe, };