From patchwork Tue Apr 8 01:40:00 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peng Fan X-Patchwork-Id: 879916 Received: from DB3PR0202CU003.outbound.protection.outlook.com (mail-northeuropeazon11011069.outbound.protection.outlook.com [52.101.65.69]) (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 AD8222744E; Tue, 8 Apr 2025 01:41:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.65.69 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744076506; cv=fail; b=iwdDvg1pxisAukwpmxe3wTTA/mWgYb9wiNQ7fJCbgHjnK4CmoztGJlN95fRD1xqnCKRBryYGdwR0tLKYV3IdYcUyukgfkVlcDuOSDWt4P2k3j3/DOakue6zpiL2BeeHKRXASpfAxaxo4rUEDHD+gkd/3A1k0Cf5II2nVaJ0PwFU= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744076506; c=relaxed/simple; bh=1o5x+KX0xgNVw5wDC+GMF01F18IYCuVvHuREM+xjPgQ=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=NiiiuVD+GwvefFjIxasfUCiVtDQUt4V3uurQwlVUHJq5iS6c1mdBJjl1qhxkhmNUsRk+xIG+QoGEYS8qaRnxRx3sWn7ZCbHbPhMKKZ4RpJkVCJ6O7WmHWM6+RJdYfYSvf8GQYglqaL1NHnEDSPNms5FsBJZI5X4G61XnKr902L0= 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=XfftPc0y; arc=fail smtp.client-ip=52.101.65.69 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="XfftPc0y" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Y0UNA3ajwcqNereOEx0/uFkVWEZhBxIN2R3ldmT/8XynxPCyLgCLgY6YkZJ6yOGjBEOzoy8SNL57p/MpLONwzJLw4RF8UCQqdy0w7aCDTzqkg9+A1bz6RKBpKJqRP432iq1Lm6y69fPmuRFG0F6hiv3vJwHdEHX6mT4CZ97XMfhqoW5e+59MYQ+hR5/qONWTFpTyukR56pIkXPAp6QRvu8uqxBG/ZrrUy6gwh8V02UQeodlUrCZBwJ7CFy6jSELdlmtaqoanu0KBEVif/4fpxITxPeqqZGztz4YMC6LXS112b/8OxkDDaSEZwt4Yt5BCzPru8Z6uY6cFrCWkAdP1HA== 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=OdoAPY/sUyxCMCaK082qQ7k8A2vCvUk1kF+c3PHcnjY=; b=RFCU6mv/HPcOkpuREz7jAcuD8tXP6Dey6C3SxRR0nLfyGi8qxXkN42B+Px+KgpCfMWCP5nZkJvLqNoJMFzmIARQyBAqtCf8BLMPcG8t/+SlP9xMSqbVE0gG7G1DgpPpRdpNj6l9+rbj8FIvdCTdH/yVJZqhZzTyFHlqIOUWZ+X5gcUXNbyuvcC1D8aNvcWlPCo5dHv4jqFRIKtWmyzEtKvntJtreTLy88GatJqu+mh715PjBjQup4wlTcZjVbs9Ed0eZR5+hdy2WO85KF1o6fGeAj2RxWwYHDb/gi7MvTW5LTQ3eVXh8W0uFOljFX3GQBoVhiZ718VuKyJEJqaDJag== 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=OdoAPY/sUyxCMCaK082qQ7k8A2vCvUk1kF+c3PHcnjY=; b=XfftPc0ygN0OmserHchAagFu7o/S72sAL+8ZQ1JpsAI7jJ/JJw+R0+KQ7IAmXeeWPlzvPjyf2R9+pgWsSz3zQ4oNXTmGfO0qHaLDMpQ5XMKP2MREsNwEpjvpse9YfpzUV4ob/kjR19GDI8AmMQiNeW2mYmjo/ZIU+0mxm+yqIif3uAayo/qZacxEHwYNhQV6yo5qPrvcEcTycrT8Te4Eg8rAlDkISpUoN3L6C20XKhtV6KjqViPIEaXOXLqi1xT+OqdQ0dcxtHh46HNcj5IY7CWtne3Tokvk64Toj+0etQ8IckOM3jdpxqbzSxkZFI3gi991HMKra+f7xV8Md7EuKQ== 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 GV1PR04MB10583.eurprd04.prod.outlook.com (2603:10a6:150:203::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8606.31; Tue, 8 Apr 2025 01:41:39 +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:39 +0000 From: "Peng Fan (OSS)" Date: Tue, 08 Apr 2025 09:40:00 +0800 Subject: [PATCH 4/7] ASoC: codec: cs42l56: Convert to GPIO descriptors Message-Id: <20250408-asoc-gpio-v1-4-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=6895; i=peng.fan@nxp.com; s=20230812; h=from:subject:message-id; bh=AFsqk6hLyIvdhwha/1DpMl3RBCtKKRrXwbMQN2/uW1c=; b=tiv9uoylB13VhzcyWY9lLkfx3geqSYjaAW7d/ip1lDEqLNBaGd+5DJn4BZZJtcvrkikyhrOmn Eu6ZV96+9fGByqsKyTxF+6rjuPIzALwkOKwzrY1M/IbSxKYpZvqp3Gj 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_|GV1PR04MB10583:EE_ X-MS-Office365-Filtering-Correlation-Id: d182951f-fc9c-4f41-ded1-08dd763e8171 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|376014|52116014|7416014|366016|1800799024|921020|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?ZFcnf3hQeQC2Qx00Auif7uPYojkPa3u?= =?utf-8?q?DnUyC2ibv71x1kllDScm1WYuRW5ndr4zGvEfoyzsRyXlFJquWbefgUvoEl4TUtc2O?= =?utf-8?q?Aflf3ruR/xQof3v1bCOzHGTcHcT1FIVajshYy0JGttxcV9AbFpRAb9dSp0MGHwPxQ?= =?utf-8?q?WHK25OVTbBQkewyGZqLcyPAPRP3KMt0hNCvACalTrT5EItTcd4+j/vCAmITr2amaM?= =?utf-8?q?/tA1OuipTL9hpsstTYlk4swk9jYYU85sLbha+VEif9EloYS4u3ahd6h2TAIKR27QK?= =?utf-8?q?nFzg+j6ExhSTs3f0oqJrX60PQbji5VbiYou/66zIFv/DnPeNxkCsRMefhFKRzfULa?= =?utf-8?q?FJTYPy9BXhiqjl+8gDjaffPQc5uKxxNorhTdCi0GFrfMRVgTuh0LUVJzF8YAIlhy4?= =?utf-8?q?6afSEk4H7zj6ejKBT1vVCWDucjPvl1a+HfTJd0Z9ErgKnSFmgUeCNT7BgG+h1CZwF?= =?utf-8?q?h22chwdRXFyIHASvfLBnQLy8txFzY/8ZPT7TvVaq7AdqhzoblY6BxvwKrs42RE2zV?= =?utf-8?q?43CkfjCRtPUthH1LdIppIzZUl/v1Zau4PVEwyH+0b2HRs+rqM6Ykpcd6iN9/6qjnw?= =?utf-8?q?69T7GeAFEpUGs6A1hWaK4dLoN8gjwVHrwaF0AnIEvweUv6+pAv7+D7cmy3WBTLQk8?= =?utf-8?q?4yIIfqXoyNTS2EaRJRUfDG7hpGRogtqbvXrpA2m4GoQHYo+gV8Hanev1FPw9CswGF?= =?utf-8?q?qV6C697g7csi9E3tnyxGoz+X2k0VNjiK+z0n89VQGzgh/AH2QUBC4CzfU/tofv8H6?= =?utf-8?q?sm7lKGzUkcjFgSAECA8cyPkCYDZFdTidCF59DL54g4LgSPOlGnSZdQGVkr2IIdtIV?= =?utf-8?q?U5gGYpWjJWTlVpG96CYQgkx+ocJMiK7warEycIWu2nWEdAESCF3KrJsDTwBvFK0EE?= =?utf-8?q?vCZrhlV4vjONzAn+4qpfwYmPgkyEehZiDz3MkIQaQwTQjjx42rrhq2j2AaCPV57H3?= =?utf-8?q?u/k6VLvEXAvfygBCGFC9q0DQqezoyJ/RWIbDfR2HXD6B1UnInVtW2ISNtGCMOjV0S?= =?utf-8?q?BlTte7yRpLBINL2qFzEB41p7u68tpOngECLls1xAVzLjQ9EaTPc4UwOMB6DKMFYL5?= =?utf-8?q?x7U8OsDM9vu3izMtMAh6Iu8R7XUH5NejjoZz0I50CbmTQ05/2MUyNu1UY/0xIfLQ8?= =?utf-8?q?8HXTkFm5tdsmHhIoj67llIyhEZRcGgUQAlMvdsaUlu+s55ZkYSZaZbOnl8Z/xaCRT?= =?utf-8?q?O3gCqg+G+NZhX56hvj+m6dbRDlXp8umPlFlu+8oMHhFhvy9AJWzsHoOCe8yWJLhSY?= =?utf-8?q?8Oy8MN3aPgmPlxl8zmOZjjCnPiHLaoDcUOdz0jEgGlm5gFxcMDkmX6Z7FYnc5Y62V?= =?utf-8?q?/ZIJJ9fLwGUV+g2nsu5WEn4ro1OuqW8j+mkUjYDiak0O5ofMSWQAbC7eP+UTin/HH?= =?utf-8?q?VwLtsPnNtPaBB0cAhboPKIoCApf5V9Iqw=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)(376014)(52116014)(7416014)(366016)(1800799024)(921020)(38350700014); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?5MbVybUXNLvN0BYDwe0nVz8NpByT?= =?utf-8?q?4z0iXKaDyPmOXJRCwcJn5PQaBobAm0KvPv3NeUYyU7BTFTKoVghGMCqsfXjQRyVky?= =?utf-8?q?jIzly2oukBntfdP8nOaoZA1vFYSvAUciroIMxt5JMFDOVI2FWnAqPkVAbjgftoELG?= =?utf-8?q?Rw2d1nkr5sky44gkKpx38jEvKJoyrnxsxjnnOGkf1t2kRY29v0TDrRfvpdtO4/K4j?= =?utf-8?q?nRDwXNgie9zEUAb4H+cDQOWlC0NRwYfBfwP79GPzH/OvaPrIs+yqEdTgq/v3MeFeB?= =?utf-8?q?Hw6oNGuTU/gi3uaBy0RGe9dN0xNlNITaqtPPAWRK5e5lm1a9MVgPXBEOLrcivujfz?= =?utf-8?q?+/x6X5hIhp9YKM1uJLnbFVExZgUxX+3hulALbCtdVoKX8zVpKOcDAwuzB0HpuEC2x?= =?utf-8?q?yBTgDiR3IuKabS6vk47mPWl/e464R2Dn0QCrPEbgsEt5baJ6S6hsgwfBrATrfZbvG?= =?utf-8?q?HnOVOjNAhJnMJHRYCg0sLawEeJKmngdMEQybjhUCSZLXcYIJucL50YhWJDGWcpw+S?= =?utf-8?q?zJLXJQpSRR/GRN4Of/f0oY3HEVZE5ojm8MZNLUkXdPKI45HS9ft7v0WHh28hHaQth?= =?utf-8?q?ituhlN9r09WXWPSG5PmYAugQsDxdXjrTLeTNJqDtrCCcP9R/CdN9HuX+qplrP+CH9?= =?utf-8?q?9zGZmQULOmDeJZ/8t48aIiBXUm9zApGk3U6zrU+DKFg+hKYSWOKvu70NVscTLNZ7q?= =?utf-8?q?1FXdahuTMZjst3h74Uhul8dun2AlJqurd/XyPYm8kfLCezPvhtAbUBbdWSMMeqaAm?= =?utf-8?q?0n9mrUG1S9poito1dAfS4tJLqExMPXIYKZUYeizqR1U6m429wRp7Ve21OOu8qqMDU?= =?utf-8?q?FHSOskooTw12tvGMuzYcfjX/EqDbgMaR/thZs/qhc3yLe9p8r40TnSe8TD/KIlftT?= =?utf-8?q?DSjhFy6SsDPPPj2OpvwnM6+4hy9Ce/TBT/f9V5CtVIP7R5XI+Jghhx53hjiRWK7rF?= =?utf-8?q?R1cNcLOw4GOSCXkyB6v4L/+RM5+YbRS4vhmDN3bDNAev+O8s6vGdRXnQwqvJw7YAL?= =?utf-8?q?/P+KYOwNW9E7bNCr7VvbNeG03ef1cHKV27kQHGA4U8c75wy/hzh/Gjcb4bV4k3Nn6?= =?utf-8?q?NhuI+Kj81jiHah+0O+H2xr+HYaH1dK2lGOAkjs/u99vXlqvpvlf93hAaxu2OB6gnT?= =?utf-8?q?OmBdTe14ZhyeqXUdUpyat+pR4B2aiRVZYoWJIgqPBlw/VT+rMnixBKdRmrb0hzfid?= =?utf-8?q?B8OvsBhhFAMcPMPNXr95hj1HDvW0MIQo50q0uq4/QtX8iAamhoUi71Nw1rYXwZlee?= =?utf-8?q?yNwDzehw6iUYmOH08d5rMmQ0bzc5s4qNqgHRSr28e6SwUULS2GD8+K0MAgFtc1niZ?= =?utf-8?q?zLTZy3v2E1LhVTrArovLvYSJSXRqNpFL3zn/bxZcYR8d27BMiRYo/FNcXo/OHnnib?= =?utf-8?q?7ZG/AcTXS4WU2lZCP1pSJOkpul0bqyR/PlemC5aIYe8hri9MKMOVvthJQrICdux6N?= =?utf-8?q?ywai09avILPdUq97GK1YJ7TSGxkL2UKYaiL+hpEXEj8n5LspNM/rdvDx3vl/ludaU?= =?utf-8?q?45+PWdF93JcZ?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: d182951f-fc9c-4f41-ded1-08dd763e8171 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:38.9329 (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: B0+xeXN7E0iIudnCJOvWPonw4AA2T7vjhsksycH+qQ9OHP/edipK6lMOfhYYv5uEMovgYklqxEjV7Xb+zULjbg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV1PR04MB10583 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 cs42l56_platform_data from sound/cs42l56.h to driver code - Drop sound/cs42l56.h because no user is creating the device using platform data Checking the current driver using legacy GPIO API, the nreset value is first output HIGH, then LOW, then HIGH. Checking the datasheet, nreset is should be held low after power on, when nreset is high, it starts to work. Since the driver has been here for quite long time and no complain on the nreset 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. There is no in-tree DTS has the device, so all should be fine. Signed-off-by: Peng Fan --- include/sound/cs42l56.h | 45 ----------------------- sound/soc/codecs/cs42l56.c | 91 +++++++++++++++++++++++++++++----------------- 2 files changed, 58 insertions(+), 78 deletions(-) diff --git a/include/sound/cs42l56.h b/include/sound/cs42l56.h deleted file mode 100644 index 62e9f7a3b414f6d1bcb651b22f7f8bd1f29b0eb3..0000000000000000000000000000000000000000 --- a/include/sound/cs42l56.h +++ /dev/null @@ -1,45 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0-only */ -/* - * linux/sound/cs42l56.h -- Platform data for CS42L56 - * - * Copyright (c) 2014 Cirrus Logic Inc. - */ - -#ifndef __CS42L56_H -#define __CS42L56_H - -struct cs42l56_platform_data { - - /* GPIO for Reset */ - unsigned int gpio_nreset; - - /* MICBIAS Level. Check datasheet Pg48 */ - unsigned int micbias_lvl; - - /* Analog Input 1A Reference 0=Single 1=Pseudo-Differential */ - unsigned int ain1a_ref_cfg; - - /* Analog Input 2A Reference 0=Single 1=Pseudo-Differential */ - unsigned int ain2a_ref_cfg; - - /* Analog Input 1B Reference 0=Single 1=Pseudo-Differential */ - unsigned int ain1b_ref_cfg; - - /* Analog Input 2B Reference 0=Single 1=Pseudo-Differential */ - unsigned int ain2b_ref_cfg; - - /* Charge Pump Freq. Check datasheet Pg62 */ - unsigned int chgfreq; - - /* HighPass Filter Right Channel Corner Frequency */ - unsigned int hpfb_freq; - - /* HighPass Filter Left Channel Corner Frequency */ - unsigned int hpfa_freq; - - /* Adaptive Power Control for LO/HP */ - unsigned int adaptive_pwr; - -}; - -#endif /* __CS42L56_H */ diff --git a/sound/soc/codecs/cs42l56.c b/sound/soc/codecs/cs42l56.c index aaf90c8b7339dc7d9fa469048a56f38dca1797cd..8223e22dd1fea68e746151e637b611100f4e1a6e 100644 --- a/sound/soc/codecs/cs42l56.c +++ b/sound/soc/codecs/cs42l56.c @@ -7,32 +7,66 @@ * 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 -#include #include -#include #include "cs42l56.h" #define CS42L56_NUM_SUPPLIES 3 + +struct cs42l56_platform_data { + + /* GPIO for Reset */ + struct gpio_desc *gpio_nreset; + + /* MICBIAS Level. Check datasheet Pg48 */ + unsigned int micbias_lvl; + + /* Analog Input 1A Reference 0=Single 1=Pseudo-Differential */ + unsigned int ain1a_ref_cfg; + + /* Analog Input 2A Reference 0=Single 1=Pseudo-Differential */ + unsigned int ain2a_ref_cfg; + + /* Analog Input 1B Reference 0=Single 1=Pseudo-Differential */ + unsigned int ain1b_ref_cfg; + + /* Analog Input 2B Reference 0=Single 1=Pseudo-Differential */ + unsigned int ain2b_ref_cfg; + + /* Charge Pump Freq. Check datasheet Pg62 */ + unsigned int chgfreq; + + /* HighPass Filter Right Channel Corner Frequency */ + unsigned int hpfb_freq; + + /* HighPass Filter Left Channel Corner Frequency */ + unsigned int hpfa_freq; + + /* Adaptive Power Control for LO/HP */ + unsigned int adaptive_pwr; + +}; + static const char *const cs42l56_supply_names[CS42L56_NUM_SUPPLIES] = { "VA", "VCP", @@ -1161,7 +1195,13 @@ static int cs42l56_handle_of_data(struct i2c_client *i2c_client, if (of_property_read_u32(np, "cirrus,hpf-left-freq", &val32) >= 0) pdata->hpfb_freq = val32; - pdata->gpio_nreset = of_get_named_gpio(np, "cirrus,gpio-nreset", 0); + pdata->gpio_nreset = devm_gpiod_get_optional(&i2c_client->dev, "cirrus,gpio-nreset", + GPIOD_OUT_LOW); + + if (IS_ERR(pdata->gpio_nreset)) + return PTR_ERR(pdata->gpio_nreset); + + gpiod_set_consumer_name(pdata->gpio_nreset, "CS42L56 /RST"); return 0; } @@ -1169,8 +1209,6 @@ static int cs42l56_handle_of_data(struct i2c_client *i2c_client, static int cs42l56_i2c_probe(struct i2c_client *i2c_client) { struct cs42l56_private *cs42l56; - struct cs42l56_platform_data *pdata = - dev_get_platdata(&i2c_client->dev); int ret, i; unsigned int devid; unsigned int alpha_rev, metal_rev; @@ -1188,28 +1226,15 @@ static int cs42l56_i2c_probe(struct i2c_client *i2c_client) return ret; } - if (pdata) { - cs42l56->pdata = *pdata; - } else { - if (i2c_client->dev.of_node) { - ret = cs42l56_handle_of_data(i2c_client, - &cs42l56->pdata); - if (ret != 0) - return ret; - } + if (i2c_client->dev.of_node) { + ret = cs42l56_handle_of_data(i2c_client, &cs42l56->pdata); + if (ret != 0) + return ret; } if (cs42l56->pdata.gpio_nreset) { - ret = gpio_request_one(cs42l56->pdata.gpio_nreset, - GPIOF_OUT_INIT_HIGH, "CS42L56 /RST"); - if (ret < 0) { - dev_err(&i2c_client->dev, - "Failed to request /RST %d: %d\n", - cs42l56->pdata.gpio_nreset, ret); - return ret; - } - gpio_set_value_cansleep(cs42l56->pdata.gpio_nreset, 0); - gpio_set_value_cansleep(cs42l56->pdata.gpio_nreset, 1); + gpiod_set_value_cansleep(cs42l56->pdata.gpio_nreset, 1); + gpiod_set_value_cansleep(cs42l56->pdata.gpio_nreset, 0); }