Petr Cvek <petr.cvek@xxxxxx> writes: >> So Guennadi, is it possible to add a gpio through platform data to ov9640 >> driver, does it make sense, and would you accept to have the reset handled there >> ? And if yes, would you, Petr, accept to revamp your patch to have the reset and >> power handled in ov9640 ? >> > > OK, why not, so power and reset gpio with polarity settings? Even better pass ... nothing through platform-data. I think there is now a gpio binding for devices which works this way : - in ov9640 : devm_gpiod_get(dev, "reset") - in magician: GPIO_LOOKUP() and gpiod_add_lookup_table() In this way I think we'll gather at the same time : - no new platform data - ov9640 will follow the same path for devicetree and platform data > Anyway I'm planning to send patch for OV9640 in future too (color correction > matrix is not complete and some registers too). Good. Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
