From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: tvp5150 regression after commit 9f924169c035 Date: Wed, 3 Feb 2016 11:42:19 -0300 Message-ID: <56B211CB.7060900@osg.samsung.com> References: <56B204CB.60602@osg.samsung.com> <20160203142323.GA8620@tetsubishi> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160203142323.GA8620@tetsubishi> Sender: linux-media-owner@vger.kernel.org To: Wolfram Sang Cc: linux-i2c@vger.kernel.org, Mauro Carvalho Chehab , Linux Media Mailing List List-Id: linux-i2c@vger.kernel.org Hello Wolfram, On 02/03/2016 11:23 AM, Wolfram Sang wrote: > Hi, > > thanks for reporting the issue! > Thanks for the quick response. >> Not filling the OMAP I2C driver's runtime PM callbacks does not help either. >> >> Any hints about the proper way to fix this issue? > > Can the I2C device be en-/disabled in some way? Which board is this > happening with? Any specs for it publicly available? > Yes, the device has a PDN (Power-Down) pin that puts the device in stand by mode but it is not used by the driver. The only thing that the driver does is to toggle the PDN and RESET pins to put the device in normal operation. In fact, the behavior after commit 9f924169c035 is the same as when the power sequence does not happen. That confused me at the beginning and I was chasing a red herring by looking at the OMAP GPIO driver and GPIO core. The datasheet of the tvp5151 is public in case you need more info [0]. This is happening on an OMAP3 DM3730 IGEPv2 board [1] using an expansion board [2] that has the tvp5151 video decoder among other peripherals. I don't think the schematics for these boards are public but I've them in case you need me to look at something. > Regards, > > Wolfram > [0]: http://www.ti.com/lit/ds/symlink/tvp5151.pdf [1]: https://isee.biz/products/igep-processor-boards/igepv2-dm3730 [2]: https://isee.biz/products/igep-expansion-boards/igepv2-expansion Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America