"Karicheri, Muralidharan" <[email protected]> writes: > Kevin, > > I am working to submit the patches for camera sensor capture support > for DM355 & DM365. The sensor MT9T031 sensor has a i2c mux (pca9543) > in it's head board. Currently in our driver, we setup this i2c client > inside the board-dmxxx-evm.c. Last time when I had posted this patch > there was a comment to move this out of this file. Do you have any > suggestions? There is a patch being reviewed in the i2c mailing list > for adding support for I2C mux in i2c core... > > http://thread.gmane.org/gmane.linux.drivers.i2c/2998 > > But not sure when this will make to mainline. Until then we need an > interim solution that can be used. So I propose to add following file in > arch/arm/mach-davinci/ folder. > mt_pca9543.c > > This will get added in the Makefile as > > obj-$(CONFIG_MT_PCA9543) += mt_pca9543.o > > From board-dmxxx-evm.c, a function will be called to switch pc9543 on or > off based. This function will be defined in mt_pca9543.c > > Do you have any suggestions to handle this in a clean way?
The extentions to the i2c framework look like the right way to go to me. I suggest you contribute to the discussions on the above thread by testing it out and seeing if it works for you followed by a query on the i2c list to state that we need it on davinci and see what is left to get it to mainline. Only if the i2c core extentions are a dead end would I consider merging a temporary solution, but even then I'd want to see a clear path to the right solution. Kevin _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
