On Thu, Aug 24, 2023 at 8:39 AM <Balamanikandan.Gunasundar@xxxxxxxxxxxxx> wrote: > >> +#if IS_ENABLED(CONFIG_MMC_ATMELMCI) > >> + if (of_device_is_compatible(np->parent, "atmel,hsmci") && > > The only difference above is "np->parent" while the existing code uses > "np". This is because the compatible string is defined in the parent > node here while the others have it in the same node. Aha. What about this right before the for-loop then: /* The Atmel MSMCI has the property in a child node of the device */ if (IS_ENABLED(CONFIG_MMC_ATMELMCI) && of_device_is_compatible(np->parent, "atmel,hsmci")) np = parent->np; Yours, Linus Walleij