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 91666C77B7D for ; Sat, 20 May 2023 15:41:29 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=u4WpCfXlYxBMwK2VUCgjiJAi4oDYuLeMu8iQDQJJo9g=; b=ip6SYwwk/U5vGz fmMQshLShyJCsgK0ImwfryRKXJGEoIVDEkRIt0bYfz8dG/76jNyqR1p8JrQ97xdTMeNWul9/cdZQE Km7ca6WXyQXsSkufH2yL9m6Uf2qDG533U9asHNX4G5SCVEt5F4rtqTIkuUfC/AH7aiHEqbAhtUuwj 4xKgeVfAJwW1iZ/thpkZDcYMvVIo0egw3IbB5a918EQzMMQjERmBIBGHRKpFFe62N2kpDN1tRToLl TyBmGhTGvH4G9ioS0jJWfWAserRzM+vpzxVX4svCl3szDcGMV+or0Z3cRIkxumB3/10CE2nkrePye +nBcvtCTKngPUC4FW7sA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q0Ohf-001nQ2-0r; Sat, 20 May 2023 15:41:07 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q0OhY-001nOt-3A; Sat, 20 May 2023 15:41:05 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4E76B60AB4; Sat, 20 May 2023 15:41:00 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E5CF6C433D2; Sat, 20 May 2023 15:40:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1684597259; bh=9Ps1xvHyEdQpdoqJaEeCbsAna43Ebewm5CkBpwanu2U=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=oA1jURKODqVIMIR07QZUqj8P0EAwJYmfUC43ursKlC6SK3C3hlntNXzXTsh7MoDLE S4v38CrFsXmdYUg7WmsQFI+Mj11qBUdxNH3voE0X9zGRE+dA4k17WewdGBofvBOfjy FJjuHnp/WR6zIj/8XX/qNEsfSdGNVmQOewgZ/dEcaWI46Bj7mFXhksYykFVDObxS3A f/cdIdfOeA2AfTdliPQq+RzX0AIpsaQ1laI/45Ny8Lbh2STxYzUxyGHN+QwYKUGbnS 06ehIM6DmkW4aqDYhZ+4XZSkKMpdWLNT2Iq6bGg6kWSk1YhucU+WPEW9bvPWpJCLk4 cSMi10hTLRyrg== Date: Sat, 20 May 2023 16:57:07 +0100 From: Jonathan Cameron To: Shreeya Patel Cc: lars@metafoo.de, heiko@sntech.de, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, sebastian.reichel@collabora.com, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@collabora.com, gustavo.padovan@collabora.com, serge.broslavsky@collabora.com, Simon Xue Subject: Re: [PATCH 1/7] iio: adc: rockchip_saradc: Add support for RK3588 Message-ID: <20230520165707.695de520@jic23-huawei> In-Reply-To: <20230516230051.14846-2-shreeya.patel@collabora.com> References: <20230516230051.14846-1-shreeya.patel@collabora.com> <20230516230051.14846-2-shreeya.patel@collabora.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.37; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230520_084101_060116_4B7F08FB X-CRM114-Status: GOOD ( 15.98 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, 17 May 2023 04:30:45 +0530 Shreeya Patel wrote: > Refactor conversion operation to support rk3588 saradc and > add separate start, read, powerdown in respective hooks. > > Signed-off-by: Simon Xue > Signed-off-by: Shreeya Patel Hi Shreeya, This particular chain of tags has unclear authorship. If Simon wrote the driver, then the author (from address) of the patch should be Simon's. If it is a codeveloped situation look at how to use the co-developed-by tag Generally the nicest way to add new callbacks is to do them in their own patch. Then introduce new devices with different callbacks in a follow on patch. That means the first one can be reviewed to ensure there is no functional change separate from the new code in the following patch. > +static const struct rockchip_saradc_data rk3588_saradc_data = { > + .channels = rockchip_rk3588_saradc_iio_channels, > + .num_channels = ARRAY_SIZE(rockchip_rk3588_saradc_iio_channels), > + .clk_rate = 1000000, > + .start = rockchip_saradc_start_v2, As mentioned above, I'd like to see the changes for old devices done in patch 1 and the support for new device in a new patch. > + .read = rockchip_saradc_read_v2, > }; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel