From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from perceval.ideasonboard.com ([213.167.242.64]:51200 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751817AbeE3Lw3 (ORCPT ); Wed, 30 May 2018 07:52:29 -0400 From: Laurent Pinchart To: Geert Uytterhoeven Cc: Jacopo Mondi , Yoshinori Sato , Rich Felker , Hans Verkuil , Linux-sh list , Linux Media Mailing List , Linux-Renesas Subject: Re: [PATCH] media: arch: sh: migor: Fix TW9910 PDN gpio Date: Wed, 30 May 2018 14:52:31 +0300 Message-ID: <2981239.tGoCg7U0XF@avalon> In-Reply-To: References: <1527671604-18768-1-git-send-email-jacopo+renesas@jmondi.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Geert, On Wednesday, 30 May 2018 12:30:49 EEST Geert Uytterhoeven wrote: > Hi Jacopo, > > On Wed, May 30, 2018 at 11:13 AM, Jacopo Mondi wrote: > > The TW9910 PDN gpio (power down) is listed as active high in the chip > > manual. It turns out it is actually active low as when set to physical > > level 0 it actually turns the video decoder power off. > > So the picture "Typical TW9910 External Circuitry" in the datasheet, which > ties PDN to GND permanently, is wrong? The SH PTT2 line is connected directory to the TW9910 PDN signal, without any inverter on the board. The PDN signal is clearly documented as active-high in the TW9910 datasheet. Something is thus weird. Jacopo, is it possible to measure the PDN signal on the board as close as possible to the TW9910 to see if it works as expected ? -- Regards, Laurent Pinchart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Wed, 30 May 2018 11:52:31 +0000 Subject: Re: [PATCH] media: arch: sh: migor: Fix TW9910 PDN gpio Message-Id: <2981239.tGoCg7U0XF@avalon> List-Id: References: <1527671604-18768-1-git-send-email-jacopo+renesas@jmondi.org> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Jacopo Mondi , Yoshinori Sato , Rich Felker , Hans Verkuil , Linux-sh list , Linux Media Mailing List , Linux-Renesas Hi Geert, On Wednesday, 30 May 2018 12:30:49 EEST Geert Uytterhoeven wrote: > Hi Jacopo, > > On Wed, May 30, 2018 at 11:13 AM, Jacopo Mondi wrote: > > The TW9910 PDN gpio (power down) is listed as active high in the chip > > manual. It turns out it is actually active low as when set to physical > > level 0 it actually turns the video decoder power off. > > So the picture "Typical TW9910 External Circuitry" in the datasheet, which > ties PDN to GND permanently, is wrong? The SH PTT2 line is connected directory to the TW9910 PDN signal, without any inverter on the board. The PDN signal is clearly documented as active-high in the TW9910 datasheet. Something is thus weird. Jacopo, is it possible to measure the PDN signal on the board as close as possible to the TW9910 to see if it works as expected ? -- Regards, Laurent Pinchart