Re: [PATCH] mmc: sd: Add a variable to check a faulty device

[Greg KH <gregkh@linuxfoundation.org>]

On Thu, Feb 15, 2024 at 10:03:45AM +0900, 이승희 wrote:
> > -----Original Message-----
> > From: Ulf Hansson <ulf.hansson@linaro.org>
> > Sent: Wednesday, February 14, 2024 8:27 PM
> > To: Seunghui Lee <sh043.lee@samsung.com>
> > Cc: linux-mmc@vger.kernel.org; linux-kernel@vger.kernel.org;
> > gregkh@linuxfoundation.org; avri.altman@wdc.com; grant.jung@samsung.com;
> > jt77.jang@samsung.com; dh0421.hwang@samsung.com; junwoo80.lee@samsung.com;
> > jangsub.yi@samsung.com; cw9316.lee@samsung.com; sh8267.baek@samsung.com;
> > wkon.kim@samsung.com
> > Subject: Re: [PATCH] mmc: sd: Add a variable to check a faulty device
> > 
> > On Tue, 13 Feb 2024 at 06:13, Seunghui Lee <sh043.lee@samsung.com> wrote:
> > >
> > > In mobile devices, suspend/resume situations are frequent.
> > > In the case of a defective SD card in which initialization fails,
> > > unnecessary initialization time is consumed for each resume.
> > > A field is needed to check that SD card initialization has failed on
> > > the host. It could be used to remove unnecessary initialization.
> > 
> > It's not clear to me, under what circumstance you want to optimize for.
> > 
> > Is the SD card ever getting properly initialized during boot?
> > 
> > Kind regards
> > Uffe
> > 
> We receive a lot of reports about SD card issues in the market.
> There was no problem with the first time at the time of use, and there are many cases where people recognize that it is not recognized later on. In most cases, this is a problem with the SD card itself.
> 
> SD card users cannot determine whether or not an SD card is a problem, so they should be guided in this regard.
> It is necessary to distinguish whether the SD card is inserted but unrecognized or the SD card itself is not inserted, and if there is a field that can check for initialization failure, it will facilitate guidance, so we considered the patch.
> 
> The variable's usage is expected to be used through the sysfs node in the vendor module.

What "vendor module"?  You need to include all of the needed code here
please.

thanks,

greg k-h