On Wed, Jun 17, 2015 at 11:39:27PM +0800, [email protected] wrote:
> From: Frank Li <[email protected]>
> 
> +&uart1 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_uart1>;
> +     status = "okay";
> +};
> +
> +&uart2 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_uart2>;
> +     fsl,uart-has-rtscts;
> +     /* for DTE mode, add below change */
> +     /* fsl,dte-mode; */
> +     /* pinctrl-0 = <&pinctrl_uart2dte>; */
> +     status = "okay";
> +};
> +
> +&usdhc1 {
> +     pinctrl-names = "default", "state_100mhz", "state_200mhz";
> +     pinctrl-0 = <&pinctrl_usdhc1>;
> +     pinctrl-1 = <&pinctrl_usdhc1_100mhz>;
> +     pinctrl-2 = <&pinctrl_usdhc1_200mhz>;
> +     cd-gpios = <&gpio1 19 0>;

Use the correct polarity here. This is probably GPIO_ACTIVE_LOW.

> +     keep-power-in-suspend;
> +     enable-sdio-wakeup;
> +     vmmc-supply = <&reg_sd1_vmmc>;
> +     status = "okay";
> +};
> +
> +&usdhc2 {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_usdhc2>;
> +     no-1-8-v;
> +     keep-power-in-suspend;
> +     enable-sdio-wakeup;
> +     status = "okay";
> +};
> +
> +&iomuxc {
> +     pinctrl-names = "default";
> +     pinctrl-0 = <&pinctrl_hog_1>;
> +     imx6ul-evk {

You can put the pinctrl nodes directly under the iomux node since:
5fcdf6a pinctrl: imx: Allow parsing DT without function nodes

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 |
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to