Received: by 10.223.185.116 with SMTP id b49csp3232732wrg; Mon, 12 Feb 2018 23:08:34 -0800 (PST) X-Google-Smtp-Source: AH8x227aA00xHMAuC74YzRx0p7TScrhiL6ziq3y88VkeU1eP2fPdYJGGknYAS+iqTIBtk98pWWYv X-Received: by 10.99.156.17 with SMTP id f17mr244234pge.12.1518505713916; Mon, 12 Feb 2018 23:08:33 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1518505713; cv=none; d=google.com; s=arc-20160816; b=wrep1EENNM50XOsXzneArUZJ2Q6/RExtfdVGp1XSoZJwKDGOg4i4ny4bjofS9GB4AK XgiSTP6W1tkwWjGmKK1LkQPUvojx1d6F8kCzQf+JCeNf7RGkrqMVYYvPpGrrLKpWgId2 hFCv306tZ7bL9nw9GTXFO6+PEC5Qgfda1uMkj9EvU3rMFxI95CvkXTWcKI8pDOGttCNc gWJDSze/mCOYOmcN/WOM+6EvZjTSbyeu5/jKMCumrQLu9jeHqjyz5JcKETIKVUYNmGIv 1uBV0g80Wg6qV3Phdb5YYOLAFJ4rL62gzqyZO9md2KJf+RSwQ52TAXbHs4BMDLmm/j1q pARw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:user-agent:references :message-id:in-reply-to:subject:cc:to:from:date :arc-authentication-results; bh=jHFfNMgyZ7TRev6p5M6cLyOU48B3WNv4zuUpJ6nDUJw=; b=Hu2uSFODdzo5gHOMbU5nPqrkws2z9SSXXB8noffnouU8Zmtb9RqKQm7D+8/TrWceat 3wC2LLnaCEfp7/0Z+utXX+QCG1oeWiwMRP7EVASx0wbHmmlMfuAH5sJ3RdZL+oTAtqcq ErPivR+X7UML+hH4340WoucICGOTVDR1TicniPMxhRRKhYsFgrEF24qo2ZfhIPU6rNiW 9H7Qi+bHruBY2VAKvZvRA1F/4eZJXYd6agqxWbiKiBWjYBUpIb0snjn9Np0U+ndABP5o VFa4XSdJX+KJOgXsI4aulHQeI2MtG4Rp9Bbz9WkgT7x2vYIC+al1urCdpAY4B+C8nFGt vd1Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id bg11-v6si6950094plb.272.2018.02.12.23.08.06; Mon, 12 Feb 2018 23:08:33 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933481AbeBMHHE (ORCPT + 99 others); Tue, 13 Feb 2018 02:07:04 -0500 Received: from mail2-relais-roc.national.inria.fr ([192.134.164.83]:42321 "EHLO mail2-relais-roc.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933366AbeBMHHC (ORCPT ); Tue, 13 Feb 2018 02:07:02 -0500 X-IronPort-AV: E=Sophos;i="5.46,506,1511823600"; d="scan'208";a="313264434" Received: from abo-3-110-68.mrs.modulonet.fr (HELO [192.168.0.15]) ([85.68.110.3]) by mail2-relais-roc.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Feb 2018 08:07:00 +0100 Date: Tue, 13 Feb 2018 08:07:00 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Vinod Koul cc: sean.wang@mediatek.com, dan.j.williams@intel.com, robh+dt@kernel.org, mark.rutland@arm.com, dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Fengguang Wu Subject: Re: [PATCH v4 5/5] dmaengine: mediatek: fix zalloc-simple.cocci warnings In-Reply-To: <20180213064659.GD3666@localhost> Message-ID: References: <20180209125817.GB3666@localhost> <20180213064659.GD3666@localhost> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 13 Feb 2018, Vinod Koul wrote: > On Fri, Feb 09, 2018 at 03:55:58PM +0100, Julia Lawall wrote: > > > > > > - ring->txd = dma_alloc_coherent(hsdma2dev(hsdma), > > > > - pc->sz_ring, &ring->tphys, > > > > - GFP_ATOMIC | __GFP_ZERO); > > > > + ring->txd = dma_zalloc_coherent(hsdma2dev(hsdma), pc->sz_ring, > > > > + &ring->tphys, GFP_ATOMIC); > > > > if (!ring->txd) > > > > return -ENOMEM; > > > > > > > > - memset(ring->txd, 0, pc->sz_ring); > > > > > > This should be folded back.. > > > > Sorry, I'm not sure what you mean by "folded back". > > Since the original patch is also up for review in this series, it doesn't > help much to add a change and fix on top in a series. The fix should be > folded back into the original commit. IIRC Fengguang's bot also recommends > so.. Sure, no problem. Thanks. (I thought you meant folding or unfolding of a function definition). julia