On Wed, May 17, 2023 at 10:26:17PM +0300, Sakari Ailus wrote: > Hi Dmitry, > > On Wed, May 17, 2023 at 12:12:05PM -0700, Dmitry Torokhov wrote: > > On Wed, May 17, 2023 at 09:21:00PM +0300, Sakari Ailus wrote: > > > Hi Arnd, > > > > > > On Mon, Jan 30, 2023 at 02:09:47PM +0100, Arnd Bergmann wrote: > > > > From: Arnd Bergmann <arnd@xxxxxxxx> > > > > > > > > The gpio usage in the function is fairly straightforward, > > > > but the normal gpiod_get() interface cannot be used here > > > > since the gpio is referenced by a child node of the device. > > > > > > > > Using devm_fwnode_gpiod_get_index() is the best alternative > > > > here. > > > > > > > > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > > > > > > I've picked > > > <URL:https://patchwork.linuxtv.org/project/linux-media/patch/20230130131003.668888-1-arnd@xxxxxxxxxx/> > > > instead. I hope that's fine. Also cc Dmitry. > > > > What do you mean "instead"? This is the exact patch that started this > > thread, and it is broken (uses wrong name of the GPIO and wrong polarity). > > > > I'd much rather you picked up > > https://lore.kernel.org/all/Y92VLGLQJZ%2FUDRx1@xxxxxxxxxx/ > > > > Thanks. > > Ah, the URL in my e-mail was wrong. I have > <URL:https://patchwork.linuxtv.org/project/linux-media/patch/Y92VLGLQJZ/UDRx1@xxxxxxxxxx/>, > i.e. the same patch. Ah, I see, thank you. -- Dmitry
