On Mon, Nov 26, 2018 at 9:23 AM Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > > + ret = mmc_gpiod_request_cd(mmc, "cd", 0, false, 0, NULL); > > Shouldn't you set the override_active_level argument to true in order for the > raw GPIO accessors to be used, as explained in the commit message ? As with the other patch, my committ message is confusing. Since I fixed all the descriptors to indicate active high/low, they can be trusted and we should not override the active level. Thanks! Yours, Linus Walleij