Re: [PATCH v1 0/7] MHI Emergency download and flash programmer support

[Bhaumik Bhatt]

Hi Mani,
On 2021-03-31 04:16 AM, Manivannan Sadhasivam wrote:
> On Mon, Mar 29, 2021 at 06:28:17PM -0700, Bhaumik Bhatt wrote:
> > Allow handling EDL mode after SYS_ERROR occurs by reading the 
> > execution
> > environment post handling and move to power on reset state to 
> > accommodate the
> > scenario.
> > Handle EDL mode properly and wait for ready instead of just exiting 
> > from the
> > firmware load.
> > Allow use of the Flash Programmer execution environment as a mission 
> > mode
> > use case for a blank NAND power up scenario.
> > Always attempt a wait for MHI ready state as device could be waiting 
> > for the
> > host to do so after pass through execution environment is seen.
> > Introduce patch to improve state awareness and aid in debugging.
> >
> > This patch series was tested on x86_64 architecture.
>
> These patches were part of a different series before. It is good that
> you splitted the patches but you should mention that here.
>
> Thanks,
> Mani
I will keep that in mind when submitting next time.

> > Bhaumik Bhatt (6):
> >   bus: mhi: core: Rely on accurate method to determine EDL mode
> >   bus: mhi: core: Wait for ready after an EDL firmware download
> >   bus: mhi: core: Handle EDL mode entry appropriately
> >   bus: mhi: core: Identify Flash Programmer as a mission mode use case
> >   bus: mhi: core: Wait for MHI READY state in most scenarios
> >   bus: mhi: core: Improve state strings for debug messages
> >
> > Carl Yin (1):
> >   bus: mhi: core: Add support for Flash Programmer execution 
> > environment
> >
> >  drivers/bus/mhi/core/boot.c     | 13 +++++++------
> >  drivers/bus/mhi/core/init.c     | 34 
> > ++++++++++++++++++----------------
> >  drivers/bus/mhi/core/internal.h |  4 +++-
> >  drivers/bus/mhi/core/main.c     |  3 +++
> >  drivers/bus/mhi/core/pm.c       | 28 +++++++++++++++++++++++++---
> >  include/linux/mhi.h             |  4 +++-
> >  6 files changed, 59 insertions(+), 27 deletions(-)
> >
> > --
> > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
> > Forum,
> > a Linux Foundation Collaborative Project

Thanks,
Bhaumik
---
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
Forum,
a Linux Foundation Collaborative Project