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.