[PATCH v2 0/1] [SRU][J:linux-bluefield] sdhci_of_dwcmshc: sync up

Liming Sun limings at nvidia.com
Thu May 23 15:53:46 UTC 2024


The v2 has an additional "Signed-off-by" after the "cherry picked " line in the commit message according to the comment I got for v1.

Yes, I should have mentioned it in the cover note.

Thanks,
Liming

> -----Original Message-----
> From: Tim Gardner <tim.gardner at canonical.com>
> Sent: Thursday, May 23, 2024 11:20 AM
> To: Liming Sun <limings at nvidia.com>; kernel-team at lists.ubuntu.com
> Subject: Re: [PATCH v2 0/1] [SRU][J:linux-bluefield] sdhci_of_dwcmshc: sync
> up
> 
> On 5/21/24 12:50 PM, Liming Sun wrote:
> > BugLink:
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs.
> launchpad.net%2Fbugs%2F2064184&data=05%7C02%7Climings%40nvidia.co
> m%7Cbd49506fe84d45b407c508dc7b3bc25f%7C43083d15727340c1b7db39
> efd9ccc17a%7C0%7C0%7C638520743821668010%7CUnknown%7CTWFpbGZ
> sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6
> Mn0%3D%7C0%7C%7C%7C&sdata=Og%2BNzmI%2FN1DihBgrbSpUMui1X%2
> FzbXpVbLFEboexfRJ0%3D&reserved=0
> >
> > SRU Justification:
> >
> > [Impact]
> > It shouldn't affect normal functionalities. With this fix, it'll remove a
> > warning when unloading and re-loading the sdhci_of_dwcmshc kernel
> driver.
> >
> > [Fix]
> > The fix unload the runtime PM when unloading the sdhci_of_dwcmshc
> driver.
> >
> > [Test Case]
> > Normal regression;
> > Unload and load sdhci_of_dwcmshc.ko again.
> >
> > [Regression Potential]
> > Same behavior from user perspective.
> >
> > Liming Sun (1):
> >    sdhci-of-dwcmshc: disable PM runtime in dwcmshc_remove()
> >
> >   drivers/mmc/host/sdhci-of-dwcmshc.c | 28 +++++++++++++++++-----------
> >   1 file changed, 17 insertions(+), 11 deletions(-)
> >
> 
> Some days I look at dozens of patches, so I don't remember what the
> problem was with v1. Why is this a v2 ? That explanation should be in
> the cover note.
> 
> --
> -----------
> Tim Gardner
> Canonical, Inc




More information about the kernel-team mailing list