Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp2154036pxf; Sat, 3 Apr 2021 13:11:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw4+OPXLvQOxp4PBO8IuyhGo9mlQoimJ4Q8n0uzhPVSXOqXvVDG1vSCpLejuSsyXoxVCNVX X-Received: by 2002:a92:7b05:: with SMTP id w5mr14468134ilc.232.1617480664908; Sat, 03 Apr 2021 13:11:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617480664; cv=none; d=google.com; s=arc-20160816; b=opOiLo2HZpdrMuF+ICPzoyzRIMNanb4kL4086y4eFwKjHaXJq+7DNO0ntbzdfEPdR8 cY1P71eMY+aWTxc3ktaRA1X35oHpvXht5RhMFqvYRNL6s1t8vAqjDRzin17aax/ibIg1 xTYxXy1uyWZsCOm1WZ8OYRu1yMo6dAbTNSBq955YUEBGZSm3KmaFD15KzWyZfql8lDGo 8gqlDLH75gsdV4kuMb93wrEGDPH4sBDD93Dsh99qkA3khGJi8Nyw1792+1KT34o/kLdE 47DxfaUw7kop/slggKtcPy7gnjmJlVPV2hlMb7FTyzgeYCwXq1ZAyDkEv4MIWytIWnTR bjJQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:date:cc:to:from:subject :message-id; bh=JerojFCEuJ9zcVz6qFSt8SovMoi/Y84UZsbmbKW7RTI=; b=DoIxD6+vSNV4TuBikwu3vWviup0pRcCLHDk2pIXzZOeHdt3TBAd0VArc6rUEf4ucBP qxxb//4M13gsqiNPmolOyky3eovYdFWg7VrlqdJCs8Ykb7vKFdW7ZDt1tcRfDsTv8E+g 5ZJPlaWqSOQiKhQw7xCCYegeK7FyFgpbV+rDLQA+8GSR6YC9nw9AIaGOvZiLycM88VR0 QtNIb7hrR+C5fC8fKKLfSUNFuoz/IQ+i+kCR1Jiv+fvX9zEUzSSTKTOkSmK2W/BtKgR0 zLJtXGpXg73NN/BmsPcCqudv6Dgqp6lDSleREC9srjuxgDy7Is/vFj3adWrT8Izwe4A/ bKcw== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id o24si12293906jah.41.2021.04.03.13.10.14; Sat, 03 Apr 2021 13:11:04 -0700 (PDT) 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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236625AbhDCUCL (ORCPT + 99 others); Sat, 3 Apr 2021 16:02:11 -0400 Received: from smtprelay0010.hostedemail.com ([216.40.44.10]:35426 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S231658AbhDCUCK (ORCPT ); Sat, 3 Apr 2021 16:02:10 -0400 Received: from omf13.hostedemail.com (clb03-v110.bra.tucows.net [216.40.38.60]) by smtprelay05.hostedemail.com (Postfix) with ESMTP id CD8A5180458DB; Sat, 3 Apr 2021 20:02:06 +0000 (UTC) Received: from [HIDDEN] (Authenticated sender: joe@perches.com) by omf13.hostedemail.com (Postfix) with ESMTPA id D32991124F4; Sat, 3 Apr 2021 20:02:05 +0000 (UTC) Message-ID: <844503af879d05b209d9af98e378350b4f74ff9d.camel@perches.com> Subject: Re: [PATCH v3 00/30] staging: rtl8723bs: remove RT_TRACE logs in core/* From: Joe Perches To: Fabio Aiuto Cc: gregkh@linuxfoundation.org, dan.carpenter@oracle.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Date: Sat, 03 Apr 2021 13:02:04 -0700 In-Reply-To: <20210403172858.GA2397@agape.jhs> References: <1cd79d781cdcccf621ce8e104a9cdf1e90e7f803.camel@perches.com> <20210403152143.GA1403@agape.jhs> <7120836b08b680d54c15fd89b8630ccc67a53116.camel@perches.com> <20210403172858.GA2397@agape.jhs> Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.38.1-1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=0.10 X-Rspamd-Server: rspamout02 X-Stat-Signature: gabuok9ucs7h7afg833xcucuojww8dgz X-Rspamd-Queue-Id: D32991124F4 X-Session-Marker: 6A6F6540706572636865732E636F6D X-Session-ID: U2FsdGVkX1/9nFMw3M0TUvd6otDijDvItut39GnlUgo= X-HE-Tag: 1617480125-621146 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2021-04-03 at 19:28 +0200, Fabio Aiuto wrote: > On Sat, Apr 03, 2021 at 09:17:37AM -0700, Joe Perches wrote: > > On Sat, 2021-04-03 at 17:21 +0200, Fabio Aiuto wrote: > > > On Sat, Apr 03, 2021 at 08:02:25AM -0700, Joe Perches wrote: > > > > On Sat, 2021-04-03 at 11:13 +0200, Fabio Aiuto wrote: > > > > > This patchset removes all RT_TRACE usages in core/ files. > > > > > > > > and hal and include and os_dep > > > > > > Hi, > > > > > > I was just about to send the second patchset relative to hal/ files. > > > The whole has been split up in directories in order to reduce the > > > number of patch per patchset > > > > > It's a good idea, but the patches relative to RT_TRACE removal > > > could be huge > > > > That's really not a significant issue. > > Simplicity in review is also important. > > Mechanization of patch creation can reduce review efforts. > > Maybe I wrongly associated simplicity with patch dimensions, but maybe > for patches this simple have expert reviewers some tool for > automatic review? Coccinelle is a relatively trusted tool and using it as a scripting mechanism where the script is shown as part of the commit message gives confidence that the change it produces can be applied without significant doubt. To improve confidence that any change that does not have an output object code delta, comparing the object code produced before and after the change is useful. Showing that the code has been both compiled and compared in the commit message also improves confidence that the change is useful and can be applied.