From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,HK_RANDOM_FROM,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AFA95C433ED for ; Sun, 4 Apr 2021 06:48:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 62E08610E8 for ; Sun, 4 Apr 2021 06:48:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229628AbhDDGsT (ORCPT ); Sun, 4 Apr 2021 02:48:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55378 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229550AbhDDGsS (ORCPT ); Sun, 4 Apr 2021 02:48:18 -0400 Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 17921C061756 for ; Sat, 3 Apr 2021 23:48:14 -0700 (PDT) Received: by mail-wr1-x42d.google.com with SMTP id f12so2221361wro.0 for ; Sat, 03 Apr 2021 23:48:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=tgrRgfC/AS75qx+sFS4KzZOps5eBumxQzPquj/kuv/s=; b=Yac76LSYnblNMeet6KFw2iCUTCT82eHZ2VzON5r5iOdiIFNBfOS7Jxf65f/kCUsm9X liZyHvWgA0l0K3RGko6Ntywzbu8CuB0Z2s/Km9/MMJnFVJGQv9ATdyxXlR/0mZa3CemX FxQlNl9qZqngZJwIAkmsk49EaHaZhyvHppXZ/tUS5FN3HG4CmiDQ5nSXb1SXhStbMiQZ ekYCjAr9rWVrg5ZjY2hGsu8Qq+OqvNj2D+ulJxpSaOals4AqJqvK7/I3GZ5ecqjb52t9 bV2SCgnZoXHOnIppogaJGfr0vehBfHWRo0nKe2tT09yLuLaE0E2gTKHlS4klRAiQyLNP 2ZxA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=tgrRgfC/AS75qx+sFS4KzZOps5eBumxQzPquj/kuv/s=; b=NPhzd4TOdaaLj4COguCMqK8LidYCslcfIVg6JTXMmY1FpGl/y523G14yywn+pYlnuA kvZWH+eocJ/zHWU71IiLeCkbDhGiLIupPoL0EIZPVtfU+2EycSdJC1VgUUWizQ2DbsKA z+qoA1N220v80AUjqus4jRa+h2QiEV8rf1EQUR2H+WM/Ym+PdISpLFr1fK5cWr0rW8EZ MIj/+ZlAFlRYE6t6yZ6JyAbAZq9DiNPh0uT0FS6uqYGxSkajbThXC/zgT1mFAw7DXhgo 2kYTVvOtGbqWMZQZl4FHlJmct5fmjRz6GRBJNy4wDd33zqYrioPTV6g6f45KTaecFN2b N7UQ== X-Gm-Message-State: AOAM533lneN71gtDy4UJeE+KC/SnQbdgc0Ckee8njpAKSS6Ecwj6XMRo iHnszoWSW9Z/6VB8XSNh8KU= X-Google-Smtp-Source: ABdhPJxGxKv+uMmzCG1cAz9sUBEHELavzR5LYu99H4bHKKBWaKeWxdpiqZ4sqz8OuFOPsTCIfh3Hfw== X-Received: by 2002:adf:a15e:: with SMTP id r30mr4808440wrr.101.1617518892649; Sat, 03 Apr 2021 23:48:12 -0700 (PDT) Received: from agape.jhs ([5.171.80.10]) by smtp.gmail.com with ESMTPSA id i17sm22888100wrp.77.2021.04.03.23.48.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 03 Apr 2021 23:48:12 -0700 (PDT) Date: Sun, 4 Apr 2021 08:48:09 +0200 From: Fabio Aiuto To: Joe Perches Cc: gregkh@linuxfoundation.org, dan.carpenter@oracle.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 00/30] staging: rtl8723bs: remove RT_TRACE logs in core/* Message-ID: <20210404064808.GA1401@agape.jhs> References: <1cd79d781cdcccf621ce8e104a9cdf1e90e7f803.camel@perches.com> <20210403152143.GA1403@agape.jhs> <7120836b08b680d54c15fd89b8630ccc67a53116.camel@perches.com> <20210403172858.GA2397@agape.jhs> <844503af879d05b209d9af98e378350b4f74ff9d.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <844503af879d05b209d9af98e378350b4f74ff9d.camel@perches.com> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Apr 03, 2021 at 01:02:04PM -0700, Joe Perches wrote: > 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. > > which tool provides an object code delta output suitable for a commit message? I dare a proposal apologizing in advance if this question should sound off topic, if it is please tell me how to ask it, is anyone of you all available being my mentor? I realized that what you told me in last weeks helps me a lot, and maybe a mentor can address me working towards more useful code bases. I choosed rtl8723bs quite by random, just to make experience in cleaning up code, and get more confident with required dev tools, but if there is some other task with higher priority that I could carry on, I will be glad to follow directives of a mentor. I really would like to become an expert kernel developer, and the way community works gets me more and more involved. Thank you again to all people helping me, fabio