Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp1351539pxb; Sun, 21 Feb 2021 22:30:46 -0800 (PST) X-Google-Smtp-Source: ABdhPJx3d01dFCCOPPXurJShBGQlpFNH3noNu1y/1yyEIclTReGUFyHtxUGwvh6VoyDBKemBBXkV X-Received: by 2002:a17:906:9b86:: with SMTP id dd6mr19957476ejc.516.1613975446299; Sun, 21 Feb 2021 22:30:46 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1613975446; cv=none; d=google.com; s=arc-20160816; b=whYeGMDeJ/SVLTIQCLUD2LoHjvyi6q5/2IGK5ABRk+jYe/N0VpnCqlxjPtkP7Xl0ee SfyKpfJYgNQ00Pa+x/8kL2jvZ4IU7LsgyvxFG0cNVthxcOlaS+nSXSe21l3VC2IyR9bN c95GUyGdi1HNMvJpSVOv0mVbzrAs4KwbMdMedLgdvrOa+9YiSEwKM2w4crkn+ktOI/Yo bVClZRKZfZpmpfssMg64rg6m3UUxVOLPiHmp9iBsYU40lhPU5yILbfNrBp9b2nvRluTW kw15p5DqRTYCVsPXnlJm67gGLIAYqo2H9eL0S8kS48+NzuAbfeIft3c32mtj0UB1Ym4s iXHA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=9rROBaHXtAdoPay+/wxFmTz+pB912Bh3fUYAYRMbSiU=; b=0LFAXnTgnUQAc+IGQnom8EpZ0Ym2uuv6aW5iajGI6ElIoWLrLnWilQ2KnX6KAlszdZ ZWiaVolkGxt81xBMH/tTy6bDkHgscHdUAEQ3Uxy+zPsxlvx6gMKffPtoYFRqMQIPMMl9 eqPj3kaEBhQdtkLntIfYRU2+rOovS8xcOuG83SPADM/nk+NRvFYaNdKddB4kT2M2M72F mmdHOs4LjCYO1gd5EshM1wceDwVtd4WMxHb0FwulTLUc3VxM2TiWI9V6qrJZoaCEhKFT hdrI7+Zrjo7VV7r0cKcP9JWxNGC6Js5crvEvns61oyK+2WRtBsSwDrqMNLx4ALjk2ogL 8Xnw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2020-01-29 header.b=fFFnJDFD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dr22si8912953ejc.303.2021.02.21.22.30.23; Sun, 21 Feb 2021 22:30:46 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2020-01-29 header.b=fFFnJDFD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229886AbhBVG36 (ORCPT + 99 others); Mon, 22 Feb 2021 01:29:58 -0500 Received: from userp2130.oracle.com ([156.151.31.86]:40230 "EHLO userp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229780AbhBVG35 (ORCPT ); Mon, 22 Feb 2021 01:29:57 -0500 Received: from pps.filterd (userp2130.oracle.com [127.0.0.1]) by userp2130.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 11M6PA0k059973; Mon, 22 Feb 2021 06:29:00 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : mime-version : content-type : in-reply-to; s=corp-2020-01-29; bh=9rROBaHXtAdoPay+/wxFmTz+pB912Bh3fUYAYRMbSiU=; b=fFFnJDFDzU3TnjOFy2tZvzYc8RsYdT+fyu46rPP4AjXkN+nrqREcfDVcNCQa9tUjFybc ztkTzJGTl83TI6QXgOT8ym+oFslSdLtXPfWVmWYrUrncGcUa+9lRr4wsFwVOCiWFv8fp A1jEdD4xDnZpH5cwnKqQtBvoP6H5aWf4bURAyFptzrXIuN+9BAMtumtw3JjVJoqGpTlb ux965tL/qTeor2w0/PYMTAm6FrvBMgTvBWnK8ccnJcoZj0jXHrdNlG53LQnzVixH78ch QK5lOZ3vzNDRRH87o7k2zhfoO1+ql/S4+8hIfgnV7nK0nJyQpUu0ArRQYnbBgijNyxyZ uw== Received: from aserp3030.oracle.com (aserp3030.oracle.com [141.146.126.71]) by userp2130.oracle.com with ESMTP id 36tsuqtgv7-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 22 Feb 2021 06:29:00 +0000 Received: from pps.filterd (aserp3030.oracle.com [127.0.0.1]) by aserp3030.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 11M6QeK3167926; Mon, 22 Feb 2021 06:28:58 GMT Received: from aserv0121.oracle.com (aserv0121.oracle.com [141.146.126.235]) by aserp3030.oracle.com with ESMTP id 36uc0kveh0-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 22 Feb 2021 06:28:58 +0000 Received: from abhmp0012.oracle.com (abhmp0012.oracle.com [141.146.116.18]) by aserv0121.oracle.com (8.14.4/8.13.8) with ESMTP id 11M6SwE9004680; Mon, 22 Feb 2021 06:28:58 GMT Received: from kadam (/102.36.221.92) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Sun, 21 Feb 2021 22:28:57 -0800 Date: Mon, 22 Feb 2021 09:28:49 +0300 From: Dan Carpenter To: chakravarthikulkarni Cc: devel@driverdev.osuosl.org, Greg Kroah-Hartman , linux-kernel@vger.kernel.org, "Gustavo A. R. Silva" Subject: Re: [PATCH] staging: ks7010: Replace macros with do while loop. Message-ID: <20210222062849.GN2087@kadam> References: <20210221201327.12714-1-chakravarthikulkarni2021@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210221201327.12714-1-chakravarthikulkarni2021@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-Proofpoint-IMR: 1 X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9902 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 phishscore=0 adultscore=0 mlxscore=0 mlxlogscore=822 bulkscore=0 malwarescore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000 definitions=main-2102220057 X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9902 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 spamscore=0 priorityscore=1501 impostorscore=0 bulkscore=0 mlxscore=0 malwarescore=0 clxscore=1011 phishscore=0 mlxlogscore=821 lowpriorityscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000 definitions=main-2102220057 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 22, 2021 at 01:43:24AM +0530, chakravarthikulkarni wrote: > This commit fix errors found in checkpath.pl. > Error message is: > > It is a good idea to keep complex macros in do while loop. > Otherwise result may have side effect. > > Signed-off-by: chakravarthikulkarni This breaks the build. Also just ignore this checkpatch warning. The original defines are fine. regards, dan carpenter