On Wed 11 Nov 09:37 CST 2020, Douglas Anderson wrote:
> After many years of struggle, commit fa2d0aa96941 ("mmc: core: Allow
> setting slot index via device tree alias") finally allows the use of
> aliases to number SD/MMC slots. Let's do that for sc7180 SoCs so that
> if eMMC and SD are both used they have consistent numbers across boots
> and kernel changes.
>
> Picking numbers can be tricky. Do we call these "1" and "2" to match
> the name in documentation or "0" and "1" with the assertion that we
> should always start at 0 and count up?
>
> While the "start counting at 0" makes sense if there are not already
> well-defined numbers for all sd/mmc controllers, in the case of sc7180
> there _are_ well defined numbers. IMO it is less confusing to use
> those and match the docs.
>
I'm happy with this motivation, patch applied.
Regards,
Bjorn
> Signed-off-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
> ---
>
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index 4e7e58c63285..625e922c273d 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -31,6 +31,8 @@ / {
> chosen { };
>
> aliases {
> + mmc1 = &sdhc_1;
> + mmc2 = &sdhc_2;
> i2c0 = &i2c0;
> i2c1 = &i2c1;
> i2c2 = &i2c2;
> --
> 2.29.2.222.g5d2a92d10f8-goog
>
