Re: [PATCH] mmc: sdhci-pci-gli: Set SDR104's clock to 205MHz and enable SSC for GL975x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 13, 2020 at 4:57 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>
> On Tue, Oct 13, 2020 at 04:33:38PM +0800, Ben Chuang wrote:
> > On Tue, Oct 13, 2020 at 4:00 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > >
> > > On Tue, Oct 13, 2020 at 03:46:00PM +0800, Ben Chuang wrote:
> > > > From: Ben Chuang <ben.chuang@xxxxxxxxxxxxxxxxxxx>
> > > >
> > > > commit 786d33c887e15061ff95942db68fe5c6ca98e5fc upstream.
> > > >
> > > > Set SDR104's clock to 205MHz and enable SSC for GL9750 and GL9755
> > > >
> > > > Signed-off-by: Ben Chuang <ben.chuang@xxxxxxxxxxxxxxxxxxx>
> > > > Link: https://lore.kernel.org/r/20200717033350.13006-1-benchuanggli@xxxxxxxxx
> > > > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
> > > > Cc: <stable@xxxxxxxxxxxxxxx> # 5.4.x
> > > > ---
> > > > Hi Greg and Sasha,
> > > >
> > > > The patch is to improve the EMI of the hardware.
> > > > So it should be also required for some hardware devices using the v5.4.
> > > > Please tell me if have other questions.
> > >
> > > This looks like a "add support for new hardware" type of patch, right?
> >
> > No, this is for a mass production hardware.
>
> That does not make sense, sorry.
>
> Is this a bug that is being fixed, did the hardware work properly before
> 5.4 and now it does not?  Or has it never worked properly and 5.9 is the
> first kernel that it now works on?

It seems there is misunderstanding regarding “hardware” means.
I originally thought that the "hardware" refers to GL975x chips.

This Genesys patch is to fix the EMI problem for GL975x controller on a system.
There is a new Linux-based system now in development stage build in
the GL975x controller
encounter the EMI problem due to the Kernel 5.4 do not support Genesys
patch for EMI.
Hence we would like to add the patch to Kernel 5.4.

>
> > There are still some customer cases using v5.4 LTS hence we need to
> > add the patch for v5.4 LTS.
>
> Your customer problems are not the upstream kernel's problems, this is
> why they pay you :)

Thanks for  your reminder :)

>
> Have you read the stable kernel rules:
>     https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
>
> What category does this patch fall into?

Hmm. I think It fixes an EMI issue of hardware (system).

>
> thanks,
>
> greg k-h

Best regards,
Ben




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux