From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753827AbcLLST1 (ORCPT <rfc822;w@1wt.eu>);
        Mon, 12 Dec 2016 13:19:27 -0500
Received: from mail-oi0-f68.google.com ([209.85.218.68]:34390 "EHLO
        mail-oi0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1752601AbcLLST0 (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 12 Dec 2016 13:19:26 -0500
MIME-Version: 1.0
In-Reply-To: <a09b8dbc-ca4b-000e-e5c2-5039119e99ba@gmail.com>
References: <1481479310-2698-1-git-send-email-arvind.yadav.cs@gmail.com> <a09b8dbc-ca4b-000e-e5c2-5039119e99ba@gmail.com>
From: Fabio Estevam <festevam@gmail.com>
Date: Mon, 12 Dec 2016 16:19:24 -0200
Message-ID: <CAOMZO5DsH_4oLOjWjuJH0Z_OD-TYU42b9MkGh7AJ6N=ODt=DTw@mail.gmail.com>
Subject: Re: [V1] mtd: devices: docg3:- Handle return value of devm_ioremap.
To: Marek Vasut <marek.vasut@gmail.com>
Cc: Arvind Yadav <arvind.yadav.cs@gmail.com>,
        Robert Jarzmik <robert.jarzmik@free.fr>,
        David Woodhouse <dwmw2@infradead.org>,
        Brian Norris <computersforpeace@gmail.com>,
        Boris Brezillon <boris.brezillon@free-electrons.com>,
        Richard Weinberger <richard@nod.at>,
        Cyrille Pitchen <cyrille.pitchen@atmel.com>,
        "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
        linux-kernel <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sun, Dec 11, 2016 at 5:15 PM, Marek Vasut <marek.vasut@gmail.com> wrote:

>>       ret = -ENOMEM;
>> +     base = devm_ioremap(dev, ress->start, DOC_IOSPACE_SIZE);
>> +     if (!base)
>> +             return ret;
>
> I think return -ENOMEM right away won't hurt here. Also, dev_err()
> explaining the failure would be nice to add.

There is no need to add dev_err() message as the mm core will complain
loudly if devm_ioremap() fails.