Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-06-04 Thread Wolfgang Denk
Dear Andy, In message 2acbd3e40906031657jac92a86y9a0de40c6b1c9...@mail.gmail.com you wrote: Wolfgang, I have an identical, but different patch from Yauhen Kharuzhy That's a funny thing - identical, but different. really? :-) applied to my tree, ready for pulling... I can take it out, but

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-06-03 Thread Wolfgang Denk
Dear Ilya Yanok, In message 1242295389-3682-1-git-send-email-ya...@emcraft.com you wrote: Return value of mmc_send_if_cond() can be safely ignored (as it is done in Linux). This makes older cards work with MXC MCI controller. Signed-off-by: Ilya Yanok ya...@emcraft.com ---

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-06-03 Thread Andy Fleming
On Wed, Jun 3, 2009 at 5:15 PM, Wolfgang Denk w...@denx.de wrote: Dear Ilya Yanok, In message 1242295389-3682-1-git-send-email-ya...@emcraft.com you wrote: Return value of mmc_send_if_cond() can be safely ignored (as it is done in Linux). This makes older cards work with MXC MCI

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-15 Thread alfred steele
Hi Ilya, Just make sure that you have all needed clocks enabled. Thanks!One thing i forgot mentioning is that when i had adopted your patch on my baseline code, i started getting the infamous 'u64 ' (64 bit) division LINK errors while compiling with EABI rooting out of the clock based

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-15 Thread Ilya Yanok
Hi Alfred, alfred steele wrote: Just make sure that you have all needed clocks enabled. Thanks!One thing i forgot mentioning is that when i had adopted your patch on my baseline code, i started getting the infamous 'u64 ' (64 bit) division LINK errors while compiling with EABI rooting

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread Ilya Yanok
Hi Alfred, alfred steele wrote: Hi Ilya, /* Now try to get the SD card's operating condition */ err = sd_send_op_cond(mmc); I am using your set of patches on the MX31 board. I am not getting any command response for the sd_send_op_cond(mmc) called from the mmc. I

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread alfred steele
Thanks for the much waited reply. I've only tried my patch on i.MX27 board. I mentioned that it may work on MX3's too cause the kernel driver used as a source works on both MX2 and MX3. You need to do some changes for my driver to work on MX3. Check the pins configuration and if you have MCI

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread alfred steele
Looks like accidently somehow pressed the send button. Apologize! Thanks for the much awaited reply. I've only tried my patch on i.MX27 board. I mentioned that it may work on MX3's too cause the kernel driver used as a source works on both MX2 and MX3. You need to do some changes for my

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread Ilya Yanok
Hi Alfred, my English is not perfect really... but I'm actually having hard times trying to understand yours... So I give you my excuses in advance for possible misunderstandings... alfred steele wrote: Looks like accidently somehow pressed the send button. Apologize! Thanks for the much

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread alfred steele
Hi, my English is not perfect really... but I'm actually having hard times trying to understand yours... So I give you my excuses in advance for possible misunderstandings... IMO too, I know i have a horrible English . Trying to improve;) Pls dont hesitate in asking em to rephrase. I believe

Re: [U-Boot] [PATCH] mmc: it's safe to ignore mmc_send_if_cond() return value

2009-05-14 Thread Ilya Yanok
Hi Alfred, alfred steele wrote: We don't have clocks API in U-Boot. You need to enable the clock by setting appropriate bit in the clocks control register manually. Why do i have to touch the CCM? IIRC, Can't it be controlled with the STR_STP_CLOCK (bit 0). When i read the CCM using