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 X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E585CC2BA2B for ; Sun, 12 Apr 2020 10:23:42 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 962D42072D for ; Sun, 12 Apr 2020 10:23:42 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="VPfx7RW0"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="J32CbtzR" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 962D42072D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject: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=32StzhXGwHrbinF6Z9EoVhQbOWStmnPog+Z36aG2Pqg=; b=VPfx7RW0JADn/Y rzn/Dm7m2k1OgvKPEy/4Q3CvVQl2ZQpgIZ3pWDIBnabFC0cTVhU7QamxVHCQtWm2RzBm1HZLjgAwG bRPn+BBLeYMd3b3tlRNEokGrx1h3emnoPWYwO4+pjlJOmkoaAnfKS3BFMtGPYTbbacrk+WgKhbnRP /TJmMEJJ/+L+m9ymDtU0aQjBC3PUrjAk4dfE9yilaLnU93QIX+mvB5JVOjDmpnLfrnWXju6dWrx85 Jv3PiqExKRRHEbBCC3bg3SvqVk0RnyUM8VegQpYQT6l87iqB6HEc9JBZx8x2sEvXnfKJh2nWhYu95 /nN2Abz/aEbgSxy7y6eQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jNZm6-0008LE-Ud; Sun, 12 Apr 2020 10:23:38 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jNZm4-0008Kv-7m for linux-arm-kernel@lists.infradead.org; Sun, 12 Apr 2020 10:23:37 +0000 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 88B5620708; Sun, 12 Apr 2020 10:23:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586687015; bh=frxqGb3ZeCMDj12BXCJ72kJINTywrbpgCn5KIN0PuA0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=J32CbtzRqUlmzA2j0TeJNpopZUA16VYAb9xoJe9+93Ya/Fk9yeXi/bhXnXjcBo7uV ME+L9z+fT+Zr3++Y5BWtxsZi1dQhSIzE4ZmeSpmL27XPS9Ygmc8iyX/UNrwiuCVyk3 XGvVL9DiMgJme4Tl9XfxJ7uWaZkP58tNnM8HUGEg= Date: Sun, 12 Apr 2020 11:23:30 +0100 From: Jonathan Cameron To: Andy Shevchenko Subject: Re: [PATCH] iio: adc: at91-adc: Use devm_platform_ioremap_resource Message-ID: <20200412112330.39527f21@archlinux> In-Reply-To: References: <20200409151125.32677-1-aishwaryarj100@gmail.com> <20200409155936.GV3628@piout.net> <20200410112236.GX3628@piout.net> X-Mailer: Claws Mail 3.17.5 (GTK+ 2.24.32; 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-20200412_032336_318026_FFA9DB8A X-CRM114-Status: GOOD ( 25.14 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alexandre Belloni , Lars-Peter Clausen , Kefeng Wang , linux-iio , Linux Kernel Mailing List , Stephen Boyd , Ludovic Desroches , linux-arm Mailing List , Peter Meerwald-Stadler , Hartmut Knaack , Thomas Gleixner , Aishwarya R , Allison Randal Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 10 Apr 2020 19:37:25 +0300 Andy Shevchenko wrote: > On Fri, Apr 10, 2020 at 2:22 PM Alexandre Belloni > wrote: > > On 10/04/2020 13:55:26+0300, Andy Shevchenko wrote: > > > On Thu, Apr 9, 2020 at 7:00 PM Alexandre Belloni > > > wrote: > > > > On 09/04/2020 20:41:23+0530, Aishwarya R wrote: > > > > > Use the helper function that wraps the calls to > > > > > platform_get_resource() and devm_ioremap_resource() > > > > > together. > > > > > > > Please elaborate the actual value of doing that. > > > > > > Please, elaborate actual value of not doing that. > > > > > > Yes, I know that you are p* off of these changes, but why you not > > > going further and forbid all clean ups we are doing in the code? > > > > > > To the point. Above change is reducing code base and showing the new > > > comers modern APIs to use. > > > The value of doing it is to reduce the code size by 16 bytes. The same > > people doing that will actively ruin that by adding error string for > > error that will never ever happen. > > I don't see it in the patch we are discussing, so, not an argument. > > > Also, the commit message is definitively lacking. A good commit message > > would say that the patch has been generated using coccinelle, that no > > testing has been done and that no thought has been given. > > That's I agree with. > But somebody need to teach people how to do it better. > > > It would definitively make sense to send one patch per subsystem instead > > of having 475 different patches each changing only one location. > > Depends on the maintainer and subsystem. This is arguable argument. > In my subsystems (let's forget about PDx86, where one per subsystem in > principle is not working by nature of the subsystem, but consider > others I'm maintaining) I prefer to have a possibility to track > changes independently on driver basis. Likewise. I often request patches are broken out. > > > The whole "let's let newcomers fix trivial bugs" thing is definitively > > not working and it is not leading to an increase of the number of useful > > reviewers and contributors > > Semi-agree. That people can be self-organized into a reviewer gang and > try to learn together. > In any case, as for maintainers, the task has not only technical > aspects, but mentoring as well. > It would be interesting to see if we do have examples of people who have started with cleanup patches and moved on to more interesting things. Some of the outreach programs that start with simple patches as a means of getting familiar with the tools have led to more significant improvements but those are somewhat of a special case. I'd rather have hundreds of people send me these sorts of patches a year than loose one potential long term contributor. Jonathan p.s. Aishwarya - You've been a bit unlucky in a sense here running into the latest instance of a very long term debate! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel