Re: [PATCH v1 2/3] arm: snapdragon: Remove QCS615 MMC node disable fixup

[Sumit Garg <sumit.garg@kernel.org>]

On Mon, Jan 26, 2026 at 09:29:37PM +0530, Balaji Selvanathan wrote:
> Remove the QCS615-specific device tree fixup that was disabling the
> MMC node at /soc@0/mmc@7c4000. This fixup is no longer needed as the
> MMC/SDCC functionality should now be properly supported through the
> clock infrastructure and device tree configuration.
> 
> This patch is based on the changes in the upstream submission:
> https://lore.kernel.org/u-boot/20260113115404.1448759-4-aswin.murugan@oss.qualcomm.com/

Rather drop this patch entirely since fixing eMMC as you did as part of
patch #1 is the right thing to do for upstream. The series you
referenced can be adjusted accordingly.

-Sumit

> 
> Signed-off-by: Balaji Selvanathan <balaji.selvanathan@oss.qualcomm.com>
> ---
>  arch/arm/mach-snapdragon/of_fixup.c | 14 --------------
>  1 file changed, 14 deletions(-)
> 
> diff --git a/arch/arm/mach-snapdragon/of_fixup.c b/arch/arm/mach-snapdragon/of_fixup.c
> index 9b6e1847489..216cc816d95 100644
> --- a/arch/arm/mach-snapdragon/of_fixup.c
> +++ b/arch/arm/mach-snapdragon/of_fixup.c
> @@ -194,20 +194,6 @@ EVENT_SPY_FULL(EVT_OF_LIVE_BUILT, qcom_of_fixup_nodes);
>  static void soc_specific_fixups(struct fdt_header *fdt)
>  {
>  	int ret;
> -
> -	/* QCS615-specific fixup: Disable MMC node */
> -	if (fdt_node_check_compatible(fdt, 0, "qcom,qcs615") == 0) {
> -		int path_offset;
> -		char prop_val[] = "disabled";
> -
> -		path_offset = fdt_path_offset(fdt, "/soc@0/mmc@7c4000");
> -		if (path_offset >= 0) {
> -			ret = fixup_dt_node(fdt, path_offset, "status",
> -					    (void *)prop_val, SET_PROP_STRING);
> -			if (ret)
> -				log_err("Failed to disable MMC node for QCS615: %d\n", ret);
> -		}
> -	}
>  }
>  
>  int ft_board_setup(void __maybe_unused *blob, struct bd_info __maybe_unused *bd)
> -- 
> 2.34.1
>