Hi Alim and Tomasz,
Thanks for the review.
On Wed, Oct 22, 2014 at 3:05 AM, Alim Akhtar <alim.akhtar@xxxxxxxxx> wrote:
> Hi Tomasz,
>
> On Tue, Oct 21, 2014 at 7:30 PM, Tomasz Figa <tomasz.figa@xxxxxxxxx> wrote:
>> On 21.10.2014 15:52, Alim Akhtar wrote:
>>>> +&mmc_0 {
>>>> + status = "okay";
>>>> + num-slots = <1>;
>>>> + broken-cd;
>>>> + caps2-mmc-hs200-1_8v;
>>> Please use mmc_hs200-1_8v instead.
>>
>> I guess you mean "mmc-hs200-1_8v" (with a hyphen between mmc and hs200).
>>
> Yes, right, that was a typo. Thanks.
Will modify it to "mmc-hs200-1_8v".
>>>> + supports-highspeed;
>>> As per synopsys-dw-mshc DT binding documentation, supports-highspeed
>>> property is deprecated, so please use common DT binding for this,
>>> which is "cap-mmc-highspeed".
>>>> + non-removable;
>>>> + card-detect-delay = <200>;
>>>> + clock-frequency = <800000000>;
>>>> + samsung,dw-mshc-ciu-div = <3>;
>>>> + samsung,dw-mshc-sdr-timing = <0 4>;
>>>> + samsung,dw-mshc-ddr-timing = <0 2>;
>>>> + pinctrl-names = "default";
>>>> + pinctrl-0 = <&sd0_clk &sd0_cmd &sd0_qrdy &sd0_bus1 &sd0_bus4 &sd0_bus8>;
>>>> + bus-width = <8>;
>>>> +};
>>>> +
>>>> +&mmc_2 {
>>>> + status = "okay";
>>>> + num-slots = <1>;
>>>> + supports-highspeed;
>>> Here also common DT binding please "cap-sd-highspeed"
>>
>> Above you suggest "cap-mmc-highspeed" to replace the same deprecated
>> property, but here "cap-sd-highspeed". What is the rationale behind
>> using only one particular new property and not both for both controllers?
>>
> Well, on this particular board dw_mmc channels are dedicated to use
> only one kind of card. e.g. mmc_0 is for eMMC and which is
> non-removable and mmc_2 is for SD cards. Although controller supports
> both the properties on all channel on this SoC.
Will use "cap-sd/mmc-highspeed" instead of "supports-highspeed".
Regards,
Abhilash
>
>> Best regards,
>> Tomasz
>
>
>
> --
> Regards,
> Alim
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
