On Fri, Nov 30, 2018 at 5:25 PM Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > On Friday, 30 November 2018 17:35:50 EET Linus Walleij wrote: > > On Mon, Nov 26, 2018 at 9:39 AM Laurent Pinchart wrote: > > > > + status = mmc_gpiod_request_cd(mmc, NULL, 0, false, 1, NULL); > > > > > > As for patch 02/11, shouldn't the override argument be set to true ? > > > > (...) > > > > > > + status = mmc_gpiod_request_ro(mmc, NULL, 1, false, 0, NULL); > > > > > > Same here. > > > > Oh after looking, actually not. > > > > Since I also fix up all the descriptors in this patch to actually indicate > > the right polarity, we do not need to override it, the setting in the > > descriptor is correct. > > Could you then change the commit message to reflect that ? Apart from that the > patch looks fine to me. Of course. I already did, sorry for the confusion. Yours, Linus Walleij
