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 3F28FD0E6C9 for ; Mon, 21 Oct 2024 08:31:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Type: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=06p4THjvYj3IGm+DaYomnlhbwcXxO2LQNOQV4XGwoHQ=; b=XW3dQ4Y7rjU3mJK3usV+o/0yKT IfQCum7dy45oX3RxIOrKlhwShwKYXiqeOGe2afUyGiMzZK/qw57NxCv/iKB2n4wbryfkl8Eh2Olzw DRJtz4Uv7r5FiRJzfi/XHjJEnC5jRadeEqOYlHN+PiNJeY8EFgNYmtM6o3OOiH4hf9nLeL+jhM21p gp/ifWFbogujCzQnqz44whIVP0KfLOtIG0BVfG1fGnxyF2yGeWADwOdypxtpb47DA9lPAQ+dIFVLB CWh/hSBGPucR0fyd7tMCmMyI5mgwXAn+vY2UlacjdEbMcraGWMnND5Pxe9ZOg18HRE1+sxK5WTzcr ZTRcut5w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t2nog-00000006YhX-2W8M; Mon, 21 Oct 2024 08:31:06 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t2mmr-00000006Mir-1heb for linux-arm-kernel@bombadil.infradead.org; Mon, 21 Oct 2024 07:25:09 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=06p4THjvYj3IGm+DaYomnlhbwcXxO2LQNOQV4XGwoHQ=; b=GZN1zF4SdSFGdohWaJWPC4lP28 mKGnCNoYnXOGvQ5/5A5IGCgN1YL2i4X+obY0tYI7RfdeWyj898GJDMAODlYphNYerb4UfsTT2tnno ylTFrKQRYjDMgBN+LK9kddTgxV+JyBJ5pbq2US3zn1CpJB382QN00Mv8H0JDBwGFFScEwJOL9h/QN v9N+O73Fatf/w/wNlGN3no8JZiKUKJF3yc9SiPjOfw5r44eHbt9S2t0r2AEP5H/4T7O+4ME8Sqqst x96tyZ0+2NvbZAP8CT7DxWu4kZvQdo8Y+KkUBciD0s/XShEQaTYWMcunpbIGA8J07NLgPb6IO/iVB kST+pQgw==; Received: from dfw.source.kernel.org ([139.178.84.217]) by desiato.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t2mmn-00000007qVu-1Isc for linux-arm-kernel@lists.infradead.org; Mon, 21 Oct 2024 07:25:07 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 7AF8A5C53BE; Mon, 21 Oct 2024 07:24:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E1E94C4CEC3; Mon, 21 Oct 2024 07:24:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1729495500; bh=Q/vHeCvRemXBDdF6JOsV/LJSic0sR7ukVpxi7m/3URA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bkTIECbvT/1ITopz/KTnSu6xS9L+g59e8y9t5fzgzv2o4c5Pkb8oSudj4pVnZHQcF Jr2C2FABZfw6oMfKZbFG9+nDpDLBafC+5iS/680Tk2ODH2+3R9R0iDjtr4dOIvQi4I h0UKZ17pBdik9LNNw8EDhjzXCJWg8VLlSFTlDkEg= Date: Mon, 21 Oct 2024 09:24:57 +0200 From: Greg KH To: Mihai.Sain@microchip.com Subject: Re: [RFC PATCH] tty: atmel_serial: Use devm_platform_ioremap_resource() Message-ID: <2024102148-stamp-isolation-ae90@gregkh> References: <20241021064321.3440-1-mihai.sain@microchip.com> <2024102151-showy-partake-a2aa@gregkh> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241021_082506_359390_C03343AB X-CRM114-Status: GOOD ( 18.70 ) 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@bootlin.com, claudiu.beznea@tuxon.dev, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, jirislaby@kernel.org, richard.genoud@bootlin.com, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Oct 21, 2024 at 07:16:31AM +0000, Mihai.Sain@microchip.com wrote: > > From: Greg KH > > Sent: Monday, October 21, 2024 10:00 AM > > To: Mihai Sain - M19926 > > Cc: richard.genoud@bootlin.com; jirislaby@kernel.org; Nicolas Ferre - M43238 > > ; alexandre.belloni@bootlin.com; > > claudiu.beznea@tuxon.dev; linux-kernel@vger.kernel.org; linux- > > serial@vger.kernel.org; linux-arm-kernel@lists.infradead.org > > Subject: Re: [RFC PATCH] tty: atmel_serial: Use > > devm_platform_ioremap_resource() > > > > On Mon, Oct 21, 2024 at 09:43:21AM +0300, Mihai Sain wrote: > > > Simplify the request port function by using a single call to > > > devm_platform_ioremap_resource(). > > > > > > This will also enhance the printing from /proc/iomem: > > > > > > cat /proc/iomem | grep flexcom ; cat /proc/iomem | grep serial > > > > > > f0004000-f00041ff : f0004000.flexcom flexcom@f0004000 > > > f8020000-f80201ff : f8020000.flexcom flexcom@f8020000 > > > f0004200-f00043ff : f0004200.serial serial@200 f8020200-f80203ff : > > > f8020200.serial serial@200 fffff200-fffff3ff : fffff200.serial > > > serial@fffff200 > > > > > > Signed-off-by: Mihai Sain > > > --- > > > drivers/tty/serial/atmel_serial.c | 12 +++--------- > > > 1 file changed, 3 insertions(+), 9 deletions(-) > > > > Why is this a rfc? What needs to be done before you want it merged? > > I used the rfc tag in order to get the feedback from you and all stakeholders. You need to ask for the feedback you are looking for, we don't know what you feel is wrong here. > If the modifications are OK for you I will send a new patch without rfc tag. As we are drowning in real patches to review, I almost always just ignore RFC patches as I know that means that the submitter feels there is still more work to be done. So please always take that in mind. Please ask for reviews from other developers in your company if you feel it is not ready for our review. thanks, greg k-h