Re: [PATCH 2/2] mmc: dw_mmc: add dw_mmc-k3 for k3 platform

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

 



On Wed, 2013-12-11 at 04:45 +0100, Arnd Bergmann wrote:
> On Wednesday 11 December 2013, Zhangfei Gao wrote:
> > Thanks for the good suggestion, it can be abstracted to clock.
> > 
> > The issue for this version's ip is there are some registers need to be
> > set, including fixed factor and phase, which may be required to be
> > tuned, however it can be hide in clock.
> > 
> > Double checked next version's ip, it is more intelligent and only
> > fixed factor is required, more like a clock, and all these register
> > accessing are NOT required.
> > 
> > Will update new version to abstract these chip depended registers to clock.
> 
> Ok, please stay in contact with Dinh Nguyen over this, since his side
> is work-in-progress and we are still evaluating how to best encode
> the phase setting in DT in a generic form.

Will the bindings "samsung,dw-mshc-sdr-timing" and
"samsung,dw-mshc-ddr-timing" also work for the k3 platform as well? I
think they should work, but just wanted to verify.

Heiko, on the rockchip platform, can you check to see if you have these
cclk_in_drv and cclk_sample settings for the phase shift of the clocks?
The SD/MMC IP version(2.40a) on Rockchip is the same version that is on
SOCFPGA, so I think these settings are there. They can be located in
some other "system" register block like SOCFPGA. The possible reason
that you have not need to set them in the Rockchip code could be that
the bootloader/firmware has already set them if you are booting from the
SD/MMC source. You may need to set the clock phase settings if you boot
from a non-SD source.

I think the reading of the "samsung,dw-mshc-ddr-timing" and
"samsung,dw-msh-sdr-timing" bindings can also be moved into the generic
dw_mmc driver if the above 2 conditions are correct.

Thanks,
Din
> 
> 	Arnd
> 



--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux