From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH v7 1/7] of: Provide function to request and map memory Date: Fri, 06 Jun 2014 09:56:06 -0700 Message-ID: <5391F2A6.4040202@infradead.org> References: <1401989181-4712-1-git-send-email-matthias.bgg@gmail.com> <1401989181-4712-2-git-send-email-matthias.bgg@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Rob Herring , Matthias Brugger Cc: Mark Rutland , Andrew Lunn , "linux-doc@vger.kernel.org" , Thierry Reding , =?UTF-8?B?SGVpa28gU3TDvGJuZXI=?= , Russell King - ARM Linux , Arnd Bergmann , Daniel Lezcano , "linux-arm-kernel@lists.infradead.org" , Florian Vaussard , Sebastian Hesselbarth , "devicetree@vger.kernel.org" , Jason Cooper , Pawel Moll , Ian Campbell , Marc Zyngier , Rob Herring , Gregory Clement , Thomas Gleixner , Soren List-Id: devicetree@vger.kernel.org On 06/06/2014 08:19 AM, Rob Herring wrote: > On Thu, Jun 5, 2014 at 12:26 PM, Matthias Brugger > wrote: >> A call to of_iomap does not request the memory region. >> This patch adds the function of_io_request_and_map which requests >> the memory region before mapping it. >> >> Signed-off-by: Matthias Brugger >> --- >> drivers/of/address.c | 28 ++++++++++++++++++++++++++++ >> include/linux/of_address.h | 8 ++++++++ >> 2 files changed, 36 insertions(+) >> >> diff --git a/drivers/of/address.c b/drivers/of/address.c >> index cb4242a..c55b107 100644 >> --- a/drivers/of/address.c >> +++ b/drivers/of/address.c >> @@ -721,3 +721,31 @@ void __iomem *of_iomap(struct device_node *np, int index) >> return ioremap(res.start, resource_size(&res)); >> } >> EXPORT_SYMBOL(of_iomap); >> + >> +/** >> + * of_io_request_and_map - Requests a resource and maps the memory mapped IO >> + * for a given device_node > > I believe docbook requires this to be 1 line. No longer so. That was fixed a couple of years ago. -- ~Randy