From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH V2] drivers: i2c: tegra: fix checkpatch defects Date: Fri, 31 May 2019 15:45:06 +0300 Message-ID: <2d83ad21-b012-1b3e-fc9a-741972ee3ba9@gmail.com> References: <1559196850-7007-1-git-send-email-bbiswas@nvidia.com> <911e52ed-8f3c-583a-7610-e38723219eca@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Bitan Biswas , Laxman Dewangan , Thierry Reding , Jonathan Hunter , linux-i2c@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Shardar Mohammed , Sowjanya Komatineni , Mantravadi Karthik List-Id: linux-tegra@vger.kernel.org 31.05.2019 12:04, Bitan Biswas пишет: > > > On 5/30/19 4:43 AM, Dmitry Osipenko wrote: >> 30.05.2019 14:36, Dmitry Osipenko пишет: >>> 30.05.2019 9:14, Bitan Biswas пишет: >>>> Fix checkpatch.pl warning(s)/error(s)/check(s) in i2c-tegra.c >>>> except for BUG/BUG_ON checks >>> >>> Please turn the BUG_ON's into WARN_ON's. The machine won't go on fire, >>> hence there is absolutely no good reason in making system unusable on a >>> software bug. BUG_ON may be more useful for development, but not for a >>> casual daily usage. > I shall update the patch to remove the BUG/BUG_ON calls. Please replace them with WARN_ON_ONCE.