[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2023080438-reacquire-obsessed-3e81@gregkh>
Date: Fri, 4 Aug 2023 15:47:26 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Trilok Soni <quic_tsoni@...cinc.com>
Cc: Souradeep Chowdhury <quic_schowdhu@...cinc.com>,
Andy Gross <agross@...nel.org>,
Konrad Dybcio <konrad.dybcio@...ainline.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Bjorn Andersson <andersson@...nel.org>,
Rob Herring <robh+dt@...nel.org>, Alex Elder <elder@...e.org>,
Arnd Bergmann <arnd@...db.de>,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
Sibi Sankar <quic_sibis@...cinc.com>,
Rajendra Nayak <quic_rjendra@...cinc.com>
Subject: Re: [PATCH V25 0/3] misc: Add driver support for Data Capture and
Compare unit(DCC)
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
Powered by blists - more mailing lists