Re: [PATCH v2] misc: rtsx: separate aspm mode into MODE_REG and MODE_CFG

From: Chris Chiu
Date: Tue Jun 08 2021 - 23:02:10 EST


On Tue, Jun 8, 2021 at 9:17 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>
> On Tue, Jun 08, 2021 at 07:16:14PM +0800, Chris Chiu wrote:
> > On Tue, Jun 8, 2021 at 4:59 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > >
> > > On Tue, Jun 08, 2021 at 11:43:03AM +0800, Chris Chiu wrote:
> > > > On Mon, Jun 7, 2021 at 6:16 PM <ricky_wu@xxxxxxxxxxx> wrote:
> > > > >
> > > > > From: Ricky Wu <ricky_wu@xxxxxxxxxxx>
> > > > >
> > > > > aspm (Active State Power Management)
> > > > > rtsx_comm_set_aspm: this function is for driver to make sure
> > > > > not enter power saving when processing of init and card_detcct
> > > > > ASPM_MODE_CFG: 8411 5209 5227 5229 5249 5250
> > > > > Change back to use original way to control aspm
> > > > > ASPM_MODE_REG: 5227A 524A 5250A 5260 5261 5228
> > > > > Keep the new way to control aspm
> > > > >
> > > > > Signed-off-by: Ricky Wu <ricky_wu@xxxxxxxxxxx>
> > > > > ---
> > > > Reported-by: Chris Chiu <chris.chiu@xxxxxxxxxxxxx>

Tested-by: Gordon Lack <gordon.lack@xxxxxxxxxxxxx>
Fixes: 121e9c6b5c4c ("misc: rtsx: modify and fix init_hw function")

> > >
> > > Can you test this to verify it works?
> > >
> > > thanks,
> > >
> > > greg k-h
> >
> > It's verified and confirmed by the user at comment #17 of
> > https://bugs.launchpad.net/bugs/1929444.
>
> That's not useful to put in a changelog text. Please respond with a
> real tag that I can use.
>
> thanks,
>
> greg k-h