On Wed, Aug 17, 2016 at 1:31 PM, Simon Horman
<horms+renesas@xxxxxxxxxxxx> wrote:
> Enable the exposed SD card slots in the DT of the r8a7796/salvator-x.
>
> Based on work for the r8a7795/salvator-x by Ai Kyuse.
>
> Cc: Ai Kyuse <ai.kyuse.uw@xxxxxxxxxxx>
> Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 84 ++++++++++++++++++++++
> 1 file changed, 84 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> index bb34b8d69c79..f06621f2ee4c 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts
> @@ -36,12 +86,46 @@
> groups = "scif2_data_a";
> function = "scif2";
> };
> +
> + sdhi0_pins: sd0 {
> + groups = "sdhi0_data4", "sdhi0_ctrl";
> + function = "sdhi0";
> + };
> +
> + sdhi3_pins: sd3 {
> + groups = "sdhi3_data4", "sdhi3_ctrl";
> + function = "sdhi3";
> + };
> };
>
> &extal_clk {
> clock-frequency = <16666666>;
> };
>
> +&sdhi0 {
> + pinctrl-0 = <&sdhi0_pins>;
> + pinctrl-names = "default";
> +
> + vmmc-supply = <&vcc_sdhi0>;
> + vqmmc-supply = <&vccq_sdhi0>;
> + cd-gpios = <&gpio3 12 GPIO_ACTIVE_LOW>;
> + wp-gpios = <&gpio3 13 GPIO_ACTIVE_HIGH>;
BTW, is there a specific reason CD and WP use GPIOs, and not the
dedicated sdhi0_cd and sdhi0_wp pin config?
It's done the same on r8a7795/salvator-x.
> + bus-width = <4>;
> + status = "okay";
> +};
> +
> +&sdhi3 {
> + pinctrl-0 = <&sdhi3_pins>;
> + pinctrl-names = "default";
> +
> + vmmc-supply = <&vcc_sdhi3>;
> + vqmmc-supply = <&vccq_sdhi3>;
> + cd-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>;
> + wp-gpios = <&gpio4 16 GPIO_ACTIVE_HIGH>;
Likewise.
> + bus-width = <4>;
> + status = "okay";
> +};
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
