From nobody Sun Jun 28 10:36:33 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 84CB4C433F5 for ; Fri, 11 Feb 2022 02:38:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347610AbiBKCiY (ORCPT ); Thu, 10 Feb 2022 21:38:24 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:59786 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1347581AbiBKCiU (ORCPT ); Thu, 10 Feb 2022 21:38:20 -0500 X-Greylist: delayed 61944 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Thu, 10 Feb 2022 18:38:19 PST Received: from smtpbgjp3.qq.com (smtpbgjp3.qq.com [54.92.39.34]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6885D55AD for ; Thu, 10 Feb 2022 18:38:19 -0800 (PST) X-QQ-mid: bizesmtp35t1644547068t7uhb17w Received: from localhost.localdomain (unknown [123.114.60.34]) by bizesmtp.qq.com (ESMTP) with id ; Fri, 11 Feb 2022 10:37:46 +0800 (CST) X-QQ-SSF: 01400000000000B0L000000A0000000 X-QQ-FEAT: ssdqv2V8ieI4r0S/p/YIMoSud1356wu151e25RfpCyjzft2pptpo8AIv5mrI4 Sz02K8iYTa8le6LOEpsZDZ4D68OqmW1R+ji4QFcTTcMOwgyou/aJtEJdMYQkBvNuIEZEfQ4 JBH8JtMoiX3Ty12FxOMHQoV4I/cpNxYCvFfr5lCrhGYHr2QBSBxRTW7pXNd/1OlXRTXLifR DOmy8sC8Bt/lwoFA1FHSULOrmbWX/cQRiG5hLK1UFMvwv3UuT3IdHAxMRcvG0Ux0vulkGDx NvcrA45edZnwni8P50Eto5c4WKfZaDkWNSrJJGSrxM5qVcDgEv7wqWVUj1hQ6srF2k3MI5C MXucQWSbw/f7uz/6OAh8OZJBdkp4w== X-QQ-GoodBg: 1 From: zhaoxiao To: thierry.reding@gmail.com, lee.jones@linaro.org Cc: u.kleine-koenig@pengutronix.de, linux-arm-kernel@lists.infradead.org, linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, zhaoxiao Subject: [PATCH v2] pwm: vt8500: Rename variable pointing to driver private data Date: Fri, 11 Feb 2022 10:37:41 +0800 Message-Id: <20220211023741.24061-1-zhaoxiao@uniontech.com> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-QQ-SENDSIZE: 520 Feedback-ID: bizesmtp:uniontech.com:qybgforeign:qybgforeign5 X-QQ-Bgrelay: 1 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Status quo is that variables of type struct vt8500_chip * are named "vt8500", "chip". Because usually only struct pwm_device * variables are named "pwm" and "chip" is usually used for variabled of type struct pwm_chip *. So consistently use the same and non-conflicting name "vt8500". Signed-off-by: zhaoxiao --- v2: Rename the "chip" to "vt8500" for variable of type struct vt8500_chip = *. drivers/pwm/pwm-vt8500.c | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/drivers/pwm/pwm-vt8500.c b/drivers/pwm/pwm-vt8500.c index 7170a315535b..f1ff9940b37c 100644 --- a/drivers/pwm/pwm-vt8500.c +++ b/drivers/pwm/pwm-vt8500.c @@ -235,7 +235,7 @@ MODULE_DEVICE_TABLE(of, vt8500_pwm_dt_ids); =20 static int vt8500_pwm_probe(struct platform_device *pdev) { - struct vt8500_chip *chip; + struct vt8500_chip *vt8500; struct device_node *np =3D pdev->dev.of_node; int ret; =20 @@ -244,48 +244,48 @@ static int vt8500_pwm_probe(struct platform_device *p= dev) return -EINVAL; } =20 - chip =3D devm_kzalloc(&pdev->dev, sizeof(*chip), GFP_KERNEL); - if (chip =3D=3D NULL) + vt8500 =3D devm_kzalloc(&pdev->dev, sizeof(*vt8500), GFP_KERNEL); + if (vt8500 =3D=3D NULL) return -ENOMEM; =20 - chip->chip.dev =3D &pdev->dev; - chip->chip.ops =3D &vt8500_pwm_ops; - chip->chip.npwm =3D VT8500_NR_PWMS; + vt8500->chip.dev =3D &pdev->dev; + vt8500->chip.ops =3D &vt8500_pwm_ops; + vt8500->chip.npwm =3D VT8500_NR_PWMS; =20 - chip->clk =3D devm_clk_get(&pdev->dev, NULL); - if (IS_ERR(chip->clk)) { + vt8500->clk =3D devm_clk_get(&pdev->dev, NULL); + if (IS_ERR(vt8500->clk)) { dev_err(&pdev->dev, "clock source not specified\n"); - return PTR_ERR(chip->clk); + return PTR_ERR(vt8500->clk); } =20 - chip->base =3D devm_platform_ioremap_resource(pdev, 0); - if (IS_ERR(chip->base)) - return PTR_ERR(chip->base); + vt8500->base =3D devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(vt8500->base)) + return PTR_ERR(vt8500->base); =20 - ret =3D clk_prepare(chip->clk); + ret =3D clk_prepare(vt8500->clk); if (ret < 0) { dev_err(&pdev->dev, "failed to prepare clock\n"); return ret; } =20 - ret =3D pwmchip_add(&chip->chip); + ret =3D pwmchip_add(&vt8500->chip); if (ret < 0) { dev_err(&pdev->dev, "failed to add PWM chip\n"); - clk_unprepare(chip->clk); + clk_unprepare(vt8500->clk); return ret; } =20 - platform_set_drvdata(pdev, chip); + platform_set_drvdata(pdev, vt8500); return ret; } =20 static int vt8500_pwm_remove(struct platform_device *pdev) { - struct vt8500_chip *chip =3D platform_get_drvdata(pdev); + struct vt8500_chip *vt8500 =3D platform_get_drvdata(pdev); =20 - pwmchip_remove(&chip->chip); + pwmchip_remove(&vt8500->chip); =20 - clk_unprepare(chip->clk); + clk_unprepare(vt8500->clk); =20 return 0; } --=20 2.20.1