From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6351AEB64DA for ; Thu, 20 Jul 2023 07:11:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=0qlAcnObbxaXDkiJiGAd87slSZgIoCh4XIL++vkrUww=; b=KecD/XFDWhJznbRuSCQS6VCnbN Y8B7Yzy1G7vpPnXZnriWErCOxd2sTJVXvW+aPl3TSY+kQDV8XFsn0gKMXV/VR1ZBtOoFjvGxthhsX F9jkvPNCf7geT2k/fMXviTeflmGsQq3CfwxvfDjh82TT5KNtliAtUfi2gmNaR8wgHDWdYcvAWfSmw E5uOVG+zNG0yFDkAanr84r2BauWWoECyeuWHnEWcRdBvbj5n0OKyi/ZwjkS1kan6NW0VwgVD8GwUq FX988ChHH9PhyQG6iIjq+09rTo0HEsetp3kCKw5Io2kp1T4cJl4tlL63OvJa4XY49ukC1teoR0ti6 T6NOFKQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qMNoj-00A57O-0K; Thu, 20 Jul 2023 07:11:17 +0000 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qMNoZ-00A4y2-2P for linux-arm-kernel@lists.infradead.org; Thu, 20 Jul 2023 07:11:14 +0000 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qMNo5-0006Kw-7e; Thu, 20 Jul 2023 09:10:37 +0200 Received: from [2a0a:edc0:0:900:1d::77] (helo=ptz.office.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.94.2) (envelope-from ) id 1qMNo2-000mWW-K5; Thu, 20 Jul 2023 09:10:34 +0200 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1qMNo1-006IQo-Re; Thu, 20 Jul 2023 09:10:33 +0200 Date: Thu, 20 Jul 2023 09:10:33 +0200 From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= To: Thierry Reding Subject: Re: [PATCH 00/10] pwm: Constistenly name pwm_chip variables "chip" Message-ID: <20230720071033.e72hjkpghf5ooe73@pengutronix.de> References: <20230714205623.2496590-1-u.kleine-koenig@pengutronix.de> MIME-Version: 1.0 In-Reply-To: X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-arm-kernel@lists.infradead.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230720_001107_794583_9650CAFF X-CRM114-Status: GOOD ( 22.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alexandre Belloni , Heiko Stuebner , Guenter Roeck , linux-riscv@lists.infradead.org, chrome-platform@lists.linux.dev, Florian Fainelli , linux-staging@lists.linux.dev, linux-rockchip@lists.infradead.org, Broadcom internal kernel review list , linux-pwm@vger.kernel.org, Ray Jui , Johan Hovold , greybus-dev@lists.linaro.org, Paul Walmsley , Benson Leung , linux-arm-kernel@lists.infradead.org, Alex Elder , Scott Branden , Greg Kroah-Hartman , Michael Walle , Palmer Dabbelt , kernel@pengutronix.de, Claudiu Beznea Content-Type: multipart/mixed; boundary="===============6476311511040171817==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6476311511040171817== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="icirkconcmfpuvh7" Content-Disposition: inline --icirkconcmfpuvh7 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hello Thierry, On Thu, Jul 20, 2023 at 08:48:11AM +0200, Thierry Reding wrote: > On Fri, Jul 14, 2023 at 10:56:13PM +0200, Uwe Kleine-K=F6nig wrote: > > Uwe Kleine-K=F6nig (10): > > pwm: Use a consistent name for pwm_chip pointers in the core > > pwm: atmel-hlcdc: Use consistent variable naming > > pwm: bcm-kona: Consistenly name pwm_chip variables "chip" > > pwm: crc: Consistenly name pwm_chip variables "chip" > > pwm: cros-ec: Consistenly name pwm_chip variables "chip" > > pwm: lp3943: Consistenly name pwm_chip variables "chip" > > pwm: rockchip: Consistenly name pwm_chip variables "chip" > > pwm: sifive: Consistenly name pwm_chip variables "chip" > > pwm: sl28cpld: Consistenly name pwm_chip variables "chip" > > staging: greybus: pwm: Consistenly name pwm_chip variables "chip" >=20 > This would've been much easier if it had been a single patch. Now I have > to either make you redo the whole series because you've misspelled PWM > or I have to go and update it myself in most of the above patches. Hint: > I'll do the latter. I guess you want to do s/pwm driver/PWM driver/? Fine for me, thanks. > There is really no reason to split this up into this many patches for > such a trivial change. Well, that's a subjective view. There are reasons to prefer several small patches over one big one, too. A small patch can be indiviually reviewed, so the "Reviewed-by: Alex Elder ..." tag only goes to the one change that he actually looked at and if later a fix to the sifive driver is to be backported to stable, the stable maintainers just pick the sifive one instead of one big patch. Did you skip the sl28cpld patch, or squash the fixup I sent in the reply to Michael Walle? Best regards and thanks, Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig | Industrial Linux Solutions | https://www.pengutronix.de/ | --icirkconcmfpuvh7 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmS43egACgkQj4D7WH0S /k4yxggAmA419LnNhUxGLLqAb6cq3dncldtmQs44Y7SOPuVmDMivbYjrghQCbQDx cwIq9JAASH/o55NbwiCBI6HBa7YDerPZyLNvpCcQ6r2QmX2uw18V/orXxcwCRohr yMtPxmgVImRc76bjnqpML3G0QIpJf0IbUKmZvkPIOBwj5c+rZQqsvjPcT4UQNWXf sWT3KXf2sJopeeWGHpThCgHnJuTEB/NdZkXhr2MwM8rJyafJBj91f+d8JQgVl5Pb MC250+Md3rZwtyDIO6yG1Ay2ViuXwSqvy7u57oGccS6zcU6ZjG63maJ3aUXSXe7i hClf7akfWvZhZdqYQwJ5WgqC9c3vEw== =PJgI -----END PGP SIGNATURE----- --icirkconcmfpuvh7-- --===============6476311511040171817== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6476311511040171817==--