From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752214AbdEVGOy (ORCPT ); Mon, 22 May 2017 02:14:54 -0400 Received: from mout.web.de ([212.227.15.14]:50703 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751425AbdEVGOv (ORCPT ); Mon, 22 May 2017 02:14:51 -0400 Subject: [PATCH 2/2] ieee802154: ca8210: Delete an error message for a failed memory allocation in ca8210_skb_rx() From: SF Markus Elfring To: linuxdev@cascoda.com, linux-wpan@vger.kernel.org, netdev@vger.kernel.org, Alexander Aring , Harry Morris , Stefan Schmidt Cc: LKML , kernel-janitors@vger.kernel.org References: <55fddbc7-28c4-8866-67d3-ff1ff1a4eeb7@users.sourceforge.net> Message-ID: Date: Mon, 22 May 2017 08:14:38 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <55fddbc7-28c4-8866-67d3-ff1ff1a4eeb7@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:8SNqna2vS+bECEIl0lbWRCBP9yck5ESrmW2PdsU2YLPQm+zSDkH IituTo/hR3/hJ7xjRkHhPpANk9DU7pH48P8LqCsDsICtnWDXojymnSMZc3OMe9SqjGFrgrL oFPw5uV5IboF/nAj8ci2UyBPgezmaSGpWVmH1SSsAKOKlgq9DfwNMANsbC6pwpGn9TwuS0j 3AHA+avAgInecU89PDGZQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:i+XiX8y37h8=:gILEOrLB+xSOPCst+XHSjO /HxZF/0ogJO4CWHO0fT3lhXENtjhAYIf1ucMu2APnBPZGZ2lQc8Ct+xwkBga8DjF+Cq+6z63E xz4YzdMLHHh0EqAlhJGBMAyzS0JlPIJQbL/mMcztJQyVsT8EazuKSQksL2832/RUsQ8EoKWrP QSl0X1zSwJ8JWOiLsP1DRF3tYSsadI4N0mdwZuRIZSs3t85/72EbJSAtc0s29stHmzXLyNmHE G6si0xG3UX3nVtFUhWhBVI7fJIXWOWmckyNXrTBK5NRLaPxRCCoPFxwspD1aBcprm/UxLlBPi scXsI4iFEHZCAYnZKpEMmRjMgnT0xcqPj7nas354ixy3iokqFHxm05u10M05SluXicgjx0GBI NMIstX2bmcfa6vlA2E+TRtKkRPZ1QZ1K69UqdYuFeK4xjpIqv1bA9oakyt9mOH/ayjUQ4vdn1 WqqLv1wVJzPp14j5sx7AqauAnX2qTqNZhrTLQqZgYbVgVRQWe985Oi+R5F2ZvmCi2XAcygd8h Ug85IJR+/OCncV6RuoRT/RbvP1FUflXupEOj/W0kEzt6gVhmIBO3ipfaSeEiyMDC8UKMehcaz HG1zJ6SIy3dWJ0YEkwfwg8F9dDrhmTp+ek1YX5CREBzEBRooF7zUrZhpB8qS1lfCre6V6QIdq swOV2j3e6CAEHxLFA1KxCMrKze/n34Bi7+C62BPLU+p2mET5Uu137KdLe2WP4is1juYvB+jLa iuqvvrkea/lb3202gwX00015LYA1hdWg9wN0w7VARiMA6Uzaj2Yb3ax/jtlXRWukbbJFTdfz+ PenEcu5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Mon, 22 May 2017 08:03:17 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Signed-off-by: Markus Elfring --- drivers/net/ieee802154/ca8210.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/ieee802154/ca8210.c b/drivers/net/ieee802154/ca8210.c index 25ed11bb5ed3..f6df75e80a60 100644 --- a/drivers/net/ieee802154/ca8210.c +++ b/drivers/net/ieee802154/ca8210.c @@ -1808,10 +1808,9 @@ static int ca8210_skb_rx( /* Allocate mtu size buffer for every rx packet */ skb = dev_alloc_skb(IEEE802154_MTU + sizeof(hdr)); - if (!skb) { - dev_crit(&priv->spi->dev, "dev_alloc_skb failed\n"); + if (!skb) return -ENOMEM; - } + skb_reserve(skb, sizeof(hdr)); msdulen = data_ind[22]; /* msdu_length */ -- 2.13.0