On Thu, Sep 5, 2013 at 4:03 PM, Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote:
> On Wed, Sep 04, 2013 at 08:54:17PM +0800, Dong Aisheng wrote:
>> This is needed for supporting ultra high speed cards like SD3.0 cards.
>>
>> Signed-off-by: Dong Aisheng <b29396@xxxxxxxxxxxxx>
>> ---
>> arch/arm/boot/dts/imx6dl.dtsi | 33 ++++++++++++++++++++++++++++++
>> arch/arm/boot/dts/imx6q.dtsi | 33 ++++++++++++++++++++++++++++++
>> arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 4 ++-
>> 3 files changed, 69 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi
>> index 2b3ecd6..e983b81 100644
>> --- a/arch/arm/boot/dts/imx6dl.dtsi
>> +++ b/arch/arm/boot/dts/imx6dl.dtsi
>> @@ -203,6 +203,39 @@
>> MX6DL_PAD_SD3_DAT3__SD3_DATA3 0x17059
>> >;
>> };
>> +
>> + pinctrl_usdhc3_3: usdhc3grp-3 { /* 100Mhz */
>> + fsl,pins = <
>> + MX6DL_PAD_SD3_CMD__SD3_CMD 0x170B9
>> + MX6DL_PAD_SD3_CLK__SD3_CLK 0x100B9
>> + MX6DL_PAD_SD3_DAT0__SD3_DATA0 0x170B9
>> + MX6DL_PAD_SD3_DAT1__SD3_DATA1 0x170B9
>> + MX6DL_PAD_SD3_DAT2__SD3_DATA2 0x170B9
>> + MX6DL_PAD_SD3_DAT3__SD3_DATA3 0x170B9
>> + MX6DL_PAD_SD3_DAT4__SD3_DATA4 0x170B9
>> + MX6DL_PAD_SD3_DAT5__SD3_DATA5 0x170B9
>> + MX6DL_PAD_SD3_DAT6__SD3_DATA6 0x170B9
>> + MX6DL_PAD_SD3_DAT7__SD3_DATA7 0x170B9
>> + MX6DL_PAD_GPIO_18__SD3_VSELECT 0x17059
>> + >;
>> + };
>
> No please.
>
> in pinctrl_usdhc3_x 'x' is the mux option. Lets do not degrade this to
> an arbitrary number. We should use prefixes like '4bit', '100mhz' or
> combinations thereof for further options.
>
The original design does not have this assumption.
The 'x' includes different mux or config.
It may be hard to name for all different configs since it's board related.
However i don't think it's bad idea for this case.
How about pinctrl_usdhc3_3_100mhz and pinctrl_usdhc3_3_200mhz?
e.g.
pinctrl_usdhc3_3: usdhc3grp-3 { /* default */
fsl,pins = <...>;
}
pinctrl_usdhc3_3_100mhz: usdhc3grp-3-100mhz {
fsl,pins = <...>;
}
pinctrl_usdhc3_3_200mhz: usdhc3grp-3-200mhz {
fsl,pins = <...>;
}
Regards
Dong Aisheng
> Sascha
>
> --
> Pengutronix e.K. | |
> Industrial Linux Solutions | http://www.pengutronix.de/ |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
> _______________________________________________
> 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 linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
