[U-Boot] [PATCH] mmc:sdhci:fix: Change default interrupts enabled at SDHCI initialization

Lukasz Majewski l.majewski at samsung.com
Mon Mar 11 09:22:58 CET 2013


Dear All,

> Hi Tom,
> 
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> > 
> > On 02/21/2013 06:33 PM, Jaehoon Chung wrote:
> > > Hi Tom,
> > > 
> > > On 02/22/2013 02:45 AM, Tom Rini wrote: On 02/21/2013 12:21 PM,
> > > Lukasz Majewski wrote:
> > >>>> Dear All,
> > >>>> 
> > >>>> I'd like to kindly ask for any feedback on this patch.
> > >>>> 
> > >>>> It is now more than month on the u-boot mailing list...
> > > 
> > > OK, sorry, the generic name of the driver threw me for a minute.
> > > I'm fine with this change going up u-boot-samsung -> u-boot-arm ->
> > > master since it's a samsung-only driver right now.  Does this work
> > > for you?
> > >> I think that this driver didn't use samsung-only. going up
> > >> u-boot-samsung? I'm not sure that.
> > 
> > Unless the context got very wrong, it touches drivers/mmc/sdhci.c
> > which is controlled by CONFIG_SDHCI which is only turned on in what
> > look like samsung platforms to me.
> > 
> 
> There is a /asm/arch-pantheon/config.h file which sets the
> CONFIG_SDHCI flag. It was added by Lei Wen (who is CC'ed to this
> thread) and looks like ARM926EJS (Marwell Semiconductor) processor
> based system.
> 
> Other systems are indeed samsung based processors. 
> 
> I don't mind if this patch would go via u-boot-samsung tree.
> 
> Moreover, since this change "touches" Lei system, I think that he
> should have expressed his opinion. Sadly this didn't happened....

Any feedback? If not, please pull this patch to u-boot mainline.

Today it is 2 months since this patch has been posted on ML. No reply
from interested people.




-- 
Best regards,

Lukasz Majewski

Samsung R&D Poland (SRPOL) | Linux Platform Group


More information about the U-Boot mailing list