Re: [PATCH v3 0/7] Introduce features and debugfs/sysfs entries for MHI

From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Bhaumik Bhatt <bbhatt@codeaurora.org>
Cc: linux-arm-msm@vger.kernel.org, hemantk@codeaurora.org,
	jhugo@codeaurora.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/7] Introduce features and debugfs/sysfs entries for MHI
Date: Thu, 21 May 2020 21:02:08 +0530	[thread overview]
Message-ID: <20200521153208.GB16101@Mani-XPS-13-9360> (raw)
In-Reply-To: <20200521132318.GA3797@Mani-XPS-13-9360>

On Thu, May 21, 2020 at 06:53:18PM +0530, Manivannan Sadhasivam wrote:
> Hi,
> 
> On Mon, May 18, 2020 at 01:03:54PM -0700, Bhaumik Bhatt wrote:
> > Introduce independent bus and device voting mechanism for clients and save
> > hardware information from BHI.
> > Allow reading and modifying some MHI variables for debug, test, and
> > informational purposes using debugfs.
> > Read values for device specific hardware information to be used by OEMs in
> > factory testing such as serial number and PK hash using sysfs.
> > 
> 
> I think this series is not yet ready. So will not merge any patches in this
> series for 5.8.
> 
> Or let me know if there are any independent patches which should get merged.
> I'm planning to send the final 5.8 series to Greg by Friday.
> 

Sorry. I just realised that we are running out of time as we are ending rc6. So
I've sent the 5.8 series to Greg.

Thanks,
Mani

> Thanks,
> Mani
> 
> > This set of patches was tested on arm64 and x86.
> > 
> > v3:
> > -Add patch to check for pending packets in suspend as a dependency for the
> > independent voting mechanism introduction
> > -Include register dump entry for debugfs to dump MHI, BHI, and BHIe registers
> > -Update commit message for the debugfs patch
> > -Updated Documentation/ABI with the required info for sysfs
> > -Updated debugfs patch to include a new KConfig entry and dependencies
> > -Updated reviewed-by for some patches
> > 
> > v2:
> > -Added a new debugfs.c file for specific debugfs entries and code
> > -Updated commit text and addressed some comments for voting change
> > -Made sure sysfs is only used for serial number and OEM PK hash usage
> > 
> > Bhaumik Bhatt (7):
> >   bus: mhi: core: Abort suspends due to outgoing pending packets
> >   bus: mhi: core: Introduce independent voting mechanism
> >   bus: mhi: core: Use generic name field for an MHI device
> >   bus: mhi: core: Introduce helper function to check device state
> >   bus: mhi: core: Introduce debugfs entries and counters for MHI
> >   bus: mhi: core: Read and save device hardware information from BHI
> >   bus: mhi: core: Introduce sysfs entries for MHI
> > 
> >  Documentation/ABI/stable/sysfs-bus-mhi |  25 ++
> >  MAINTAINERS                            |   1 +
> >  drivers/bus/mhi/Kconfig                |   8 +
> >  drivers/bus/mhi/core/Makefile          |   5 +-
> >  drivers/bus/mhi/core/boot.c            |  17 +-
> >  drivers/bus/mhi/core/debugfs.c         | 501 +++++++++++++++++++++++++++++++++
> >  drivers/bus/mhi/core/init.c            |  80 +++++-
> >  drivers/bus/mhi/core/internal.h        |  29 ++
> >  drivers/bus/mhi/core/main.c            |   6 +-
> >  drivers/bus/mhi/core/pm.c              |  79 ++++--
> >  include/linux/mhi.h                    |  39 ++-
> >  11 files changed, 745 insertions(+), 45 deletions(-)
> >  create mode 100644 Documentation/ABI/stable/sysfs-bus-mhi
> >  create mode 100644 drivers/bus/mhi/core/debugfs.c
> > 
> > -- 
> > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> > a Linux Foundation Collaborative Project