Hi Alex, On Fri, Oct 19, 2018 at 12:17 PM Alex Gonzalez <alex.gonzalez@xxxxxxxx> wrote: > +/* USDHC1 (Wireless) */ > +&usdhc1 { > + pinctrl-names = "default", "sleep"; > + pinctrl-0 = <&pinctrl_usdhc1 &pinctrl_wifibt_ctrl>; > + pinctrl-1 = <&pinctrl_usdhc1_sleep &pinctrl_wifibt_ctrl_sleep>; > + non-removable; > + no-1-8-v; > + bus-width = <4>; > + status = "okay"; > +}; > + > + Just a minor not: only one blank line is enough. Reviewed-by: Fabio Estevam <festevam@xxxxxxxxx>