From: Stuart Yoder <stuart.yoder@nxp.com>
To: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        Jose Rivera <german.rivera@nxp.com>
CC: "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "agraf@suse.de" <agraf@suse.de>, "arnd@arndb.de" <arnd@arndb.de>,
        Yang-Leo Li <leoyang.li@nxp.com>,
        =?utf-8?B?SG9yaWEgR2VhbnTEgw==?= <horia.geanta@freescale.com>
Subject: RE: [PATCH 13/14] staging: fsl-mc: return -EINVAL for all
 fsl_mc_portal_allocate() failures
Thread-Topic: [PATCH 13/14] staging: fsl-mc: return -EINVAL for all
 fsl_mc_portal_allocate() failures
Thread-Index: AQHRlBMTceGgL4DgbE+ZJfDJZPV7GJ+FIHmAgABFEuA=
Date: Mon, 11 Apr 2016 23:14:19 +0000
Message-ID: <HE1PR04MB16417E58F02248F63781398A8D940@HE1PR04MB1641.eurprd04.prod.outlook.com>
References: <1460393776-16500-1-git-send-email-stuart.yoder@nxp.com>
 <CALRxmdCJSswjVj+DfKD9kWAOL0mpStHB5nqRFBghr91XGCA5Yw@mail.gmail.com>
In-Reply-To: <CALRxmdCJSswjVj+DfKD9kWAOL0mpStHB5nqRFBghr91XGCA5Yw@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:23
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Apr 2016 23:14:19.6941
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR04MB1179
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u3C0ltBY006371
Content-Length: 1884
Lines: 50


> -----Original Message-----
> From: Stuart Yoder <stuart.yoder@nxp.com>
> Date: Mon, Apr 11, 2016 at 11:56 AM
> Subject: [PATCH 13/14] staging: fsl-mc: return -EINVAL for all
> fsl_mc_portal_allocate() failures
> To: gregkh@linuxfoundation.org, german.rivera@nxp.com
> Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
> agraf@suse.de, arnd@arndb.de, leoyang.li@nxp.com, Horia Geantă
> <horia.geanta@nxp.com>, Stuart Yoder <stuart.yoder@nxp.com>
> 
> 
> From: Horia Geantă <horia.geanta@nxp.com>
> 
> There are some error paths that allow for a NULL new_mc_io and err = 0
> return code. Return -EINVAL instead.
> 
> Signed-off-by: Horia Geantă <horia.geanta@nxp.com>
> Signed-off-by: Stuart Yoder <stuart.yoder@nxp.com>
> ---
>  drivers/staging/fsl-mc/bus/mc-allocator.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c
> b/drivers/staging/fsl-mc/bus/mc-allocator.c
> index 4676ba1..7ee71e7 100644
> --- a/drivers/staging/fsl-mc/bus/mc-allocator.c
> +++ b/drivers/staging/fsl-mc/bus/mc-allocator.c
> @@ -306,6 +306,7 @@ int __must_check fsl_mc_portal_allocate(struct
> fsl_mc_device *mc_dev,
>         if (error < 0)
>                 return error;
> 
> +       error = -EINVAL;
>         dpmcp_dev = resource->data;
>         if (WARN_ON(!dpmcp_dev))
>                 goto error_cleanup_resource;
> --
> 1.7.9.5

I had smtp mail server weirdness that led me to belive that the freescale.com
email addresses in this patch were causing the patch to not show up on the
mailing list, so I resent [PATCH 13/14] with the email address changed to
nxp.com.  Then both copies of [PATCH 13/14] showed up.  So please ignore
the duplicated email for this patch.  Both copies are identical and either
either freescale.com or nxp.com are ok for the email address.  nxp.com
version would be preferred I guess.

Thanks,
Stuart