From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on archive.lwn.net X-Spam-Level: X-Spam-Status: No, score=-5.7 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by archive.lwn.net (Postfix) with ESMTP id 571937D2F0 for ; Fri, 30 Aug 2019 15:59:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727884AbfH3P7y (ORCPT ); Fri, 30 Aug 2019 11:59:54 -0400 Received: from verein.lst.de ([213.95.11.211]:56753 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727304AbfH3P7x (ORCPT ); Fri, 30 Aug 2019 11:59:53 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 50407227A81; Fri, 30 Aug 2019 17:59:49 +0200 (CEST) Date: Fri, 30 Aug 2019 17:59:49 +0200 From: Christoph Hellwig To: Geert Uytterhoeven Cc: Bartosz Golaszewski , Jonathan Corbet , Greg Kroah-Hartman , "Rafael J . Wysocki" , Alban Bedel , Linus Walleij , Arnd Bergmann , Geert Uytterhoeven , "open list:DOCUMENTATION" , Linux Kernel Mailing List , "open list:GPIO SUBSYSTEM" , Julia Lawall , Bartosz Golaszewski , Christoph Hellwig Subject: Re: [PATCH 0/9] drivers: add new variants of devm_platform_ioremap_resource() Message-ID: <20190830155949.GA26757@lst.de> References: <20190829143742.24726-1-brgl@bgdev.pl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org On Thu, Aug 29, 2019 at 04:48:36PM +0200, Geert Uytterhoeven wrote: > Hi Bartosz, > > On Thu, Aug 29, 2019 at 4:38 PM Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski > > > > The new devm_platform_ioremap_resource() helper has now been widely > > adopted and used in many drivers. Users of nocache and write-combined > > ioremap() variants could profit from the same code shrinkage. This > > series provides two new versions of devm_platform_ioremap_resource() > > and uses it in a few example drivers with the assumption that - just > > like was the case previously - a coccinelle script will be developed > > to ease the transition for others. > > Please be aware that the number of ioremap() variants is being > reduced, as some of them are redundant (e.g. ioremap() already creates > an uncached mapping, so ioremap_nocache() is not needed). > So less is better than more ;-) Yes. If I can get the ia64 and openrisc patch in I plan to send Linus a scripted removal of ioremap_nocache after -rc1. I already have a local patch for current mainline as of about two weeks ago.