Re: [PATCH v3 3/3] ARM: dts: Add support for Samsung Chagall WiFi

[Henrik Grimler]

Hi Krzysztof,

> > +/* External sdcard */
> > +&mmc_2 {
> > +	status = "okay";
> > +	bus-width = <4>;
> > +	cap-sd-highspeed;
> > +	card-detect-delay = <200>;
> > +	pinctrl-0 = <&sd2_clk &sd2_cmd &mmc2_cd &sd2_bus1 &sd2_bus4>;
> > +	pinctrl-names = "default";
> > +	samsung,dw-mshc-ciu-div = <3>;
> > +	samsung,dw-mshc-ddr-timing = <0 2>;
> > +	samsung,dw-mshc-sdr-timing = <0 4>;
> > +	sd-uhs-sdr50;
> > +	vmmc-supply = <&ldo19_reg>;
> > +	vqmmc-supply = <&ldo13_reg>;
> > +};
> > +
> > +&pinctrl_0 {
> > +	mmc2_cd: sd2-cd-pins {
> > +		samsung,pins = "gpx2-4";
> 
> Interesting... I looked at vendor sources to board-chagall and standard
> pin gpc2-2 is mentioned as PULL down and not-connected comment.
> 
> gpx2-4 seems not mentioned at all, unless other board files are actually
> used.

Gpio seems to be spread out. GPIO_T_FLASH_DETECT is defined as gpx2-4
in board-universal5420-mmc.c, and then used for card detection.
(Looking at it now again I see that write protection through sd2_wp
should also be supported for mmc_2, can add that in next patch set.)

> Anyway, if it works for you, it's good.
> 
> Rest looks ok, I'll apply after the merge window.

Thanks!

> Best regards,
> Krzysztof

Best regards,
Henrik Grimler