Hi Doug, On Tue, Jan 8, 2013 at 12:34 AM, Doug Anderson <diand...@chromium.org> wrote: > > Alim, > > On Mon, Jan 7, 2013 at 3:19 AM, Alim Akhtar <alim.akh...@samsung.com> wrote: > > As per the current exynos-dw-mshc bindings, dw-mshc-sdr-timing and > > dw-mshc-ddr-timing properties are having only two cells, these properties > > are wrongly set for exynos5250 based cros5250 and smdk5250 platfroms. This > > patch > > corrects above timing propreties for above platfroms > > Thanks for catching this. Given that this just removes a number from > the dts file that isn't read by the code, I don't expect any > functional change and I don't see any. > > I also double-checked usage and compared to the current Chrome OS > tree. The third number (that you removed) is already correctly > specified in the dts as "samsung,dw-mshc-ciu-div". :)
Yes, no functional changes as the removed number is passed using dw-mshc-ciu-div property now as mentioned by you. > > > > > Signed-off-by: Alim Akhtar <alim.akh...@samsung.com> > > Tested-by: Doug Anderson <diand...@chromium.org> > Acked-by: Doug Anderson <diand...@chromium.org> Thanks!!! > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" > in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html