Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 4 Aug 2023 15:47:26 +0200
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     Trilok Soni <quic_tsoni@quicinc.com>
Cc:     Souradeep Chowdhury <quic_schowdhu@quicinc.com>,
        Andy Gross <agross@kernel.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Rob Herring <robh+dt@kernel.org>, Alex Elder <elder@ieee.org>,
        Arnd Bergmann <arnd@arndb.de>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        Sibi Sankar <quic_sibis@quicinc.com>,
        Rajendra Nayak <quic_rjendra@quicinc.com>
Subject: Re: [PATCH V25 0/3] misc: Add driver support for Data Capture and
 Compare unit(DCC)
Message-ID: <2023080438-reacquire-obsessed-3e81@gregkh>
References: <cover.1687945879.git.quic_schowdhu@quicinc.com>
 <f25f8c43-2996-23ff-e6af-9e39b7fced86@quicinc.com>
 <774b688f-0324-9097-6504-58d15a3e6afb@quicinc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <774b688f-0324-9097-6504-58d15a3e6afb@quicinc.com>
Precedence: bulk

On Thu, Aug 03, 2023 at 07:35:18AM -0700, Trilok Soni wrote:
> On 8/3/2023 12:06 AM, Souradeep Chowdhury wrote:
> > 
> > 
> > On 6/28/2023 3:53 PM, Souradeep Chowdhury wrote:
> 
> ...
> 
> > > 
> > > https://git.codelinaro.org/clo/le/platform/vendor/qcom-opensource/tools/-/tree/opensource-tools.lnx.1.0.r176-rel/dcc_parser
> > > 
> > > Changes in v25
> > > 
> > > * Updated the documentation of the structure dcc_config_entry as per
> > > the comments in V23
> > > * Updated the documentation of the dcc Kconfig definition as per
> > > comment in V24
> > > * Used u64 where applicable
> > > * Removed the mutex locks where it is not needed
> > > * Removed the use of unlikely keyword
> > > * Renamed "nr_link_list" to "max_link_list"
> > > 
> > > Souradeep Chowdhury (3):
> > >    dt-bindings: misc: qcom,dcc: Add the dtschema
> > >    misc: dcc: Add driver support for Data Capture and Compare unit(DCC)
> > >    MAINTAINERS: Add the entry for DCC(Data Capture and Compare) driver
> > >      support
> > > 
> > >   Documentation/ABI/testing/debugfs-driver-dcc  |   10 +-
> > >   .../devicetree/bindings/misc/qcom,dcc.yaml    |   44 +
> > >   MAINTAINERS                                   |    8 +
> > >   drivers/misc/Kconfig                          |    8 +
> > >   drivers/misc/Makefile                         |    1 +
> > >   drivers/misc/qcom-dcc.c                       | 1312 +++++++++++++++++
> > >   6 files changed, 1378 insertions(+), 5 deletions(-)
> > >   create mode 100644 Documentation/devicetree/bindings/misc/qcom,dcc.yaml
> > >   create mode 100644 drivers/misc/qcom-dcc.c
> > 
> > Gentle Ping
> 
> Thank you for the reminder Souradeep. Greg and others, please see if we need
> any changes here or it can be picked up?

It would help if the code would actually build:

drivers/misc/qcom-dcc.c: In function ‘ready_read’:
drivers/misc/qcom-dcc.c:853:13: error: unused variable ‘ret’ [-Werror=unused-variable]
  853 |         int ret = 0;
      |             ^~~

{sigh}

How in the world was this ever tested?

greg k-h