From patchwork Fri May 17 06:38:03 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jean-Francois Moine X-Patchwork-Id: 2584141 Return-Path: X-Original-To: patchwork-dri-devel@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by patchwork2.kernel.org (Postfix) with ESMTP id 2C821DF215 for ; Fri, 17 May 2013 18:31:36 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 08E28E64F7 for ; Fri, 17 May 2013 11:31:36 -0700 (PDT) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from smtp1-g21.free.fr (smtp1-g21.free.fr [212.27.42.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 02EE0E5CBD for ; Thu, 16 May 2013 23:37:02 -0700 (PDT) Received: from armhf (unknown [IPv6:2a01:e35:2f5c:9de0:212:bfff:fe1e:9ce4]) by smtp1-g21.free.fr (Postfix) with ESMTP id 29348940186; Fri, 17 May 2013 08:36:47 +0200 (CEST) Date: Fri, 17 May 2013 08:38:03 +0200 From: Jean-Francois Moine To: linux-arm-kernel@lists.infradead.org Subject: [PATCH] drm/i2c: nxp-tda998x: fix abort of some i2c exchanges Message-ID: <20130517083803.5f34816e@armhf> X-Mailer: Claws Mail 3.9.1 (GTK+ 2.24.17; arm-unknown-linux-gnueabihf) Mime-Version: 1.0 X-Mailman-Approved-At: Fri, 17 May 2013 11:31:13 -0700 Cc: Jason Cooper , dri-devel@lists.freedesktop.org, Darren Etheridge , Russell King , Sebastian Hesselbarth X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org Errors-To: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org The signals SIGALRM (most often) and SIGPOLL (sometimes) may be raised after irq or some parameter change and this aborts the i2c exchanges. The problem is solved by masking these signals. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 40 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 39 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index e68b58a..672d045 100644 --- a/drivers/gpu/drm/i2c/tda998x_drv.c +++ b/drivers/gpu/drm/i2c/tda998x_drv.c @@ -262,6 +262,28 @@ struct tda998x_priv { #define TDA19989N2 0x0202 #define TDA19988 0x0301 +/* + * the signals SIGALRM (most often) and SIGPOLL (sometimes) + * may be raised after irq or some parameter change + * and this aborts the i2c exchanges + * the problem is solved by masking these signals + */ +static void savesig(sigset_t *p_oldset) +{ + sigset_t newset; + + sigemptyset(&newset); + sigaddset(&newset, SIGALRM); + sigaddset(&newset, SIGPOLL); + + sigprocmask(SIG_BLOCK, &newset, p_oldset); + +} +static void restoresig(sigset_t *p_oldset) +{ + sigprocmask(SIG_SETMASK, p_oldset, NULL); +} + static void cec_write(struct drm_encoder *encoder, uint16_t addr, uint8_t val) { @@ -426,6 +448,7 @@ static void tda998x_encoder_dpms(struct drm_encoder *encoder, int mode) { struct tda998x_priv *priv = to_tda998x_priv(encoder); + sigset_t oldset; /* we only care about on or off: */ if (mode != DRM_MODE_DPMS_ON) @@ -434,6 +457,8 @@ tda998x_encoder_dpms(struct drm_encoder *encoder, int mode) if (mode == priv->dpms) return; + savesig(&oldset); + switch (mode) { case DRM_MODE_DPMS_ON: /* enable audio and video ports */ @@ -458,6 +483,8 @@ tda998x_encoder_dpms(struct drm_encoder *encoder, int mode) break; } + restoresig(&oldset); + priv->dpms = mode; } @@ -494,11 +521,14 @@ tda998x_encoder_mode_set(struct drm_encoder *encoder, struct drm_display_mode *adjusted_mode) { struct tda998x_priv *priv = to_tda998x_priv(encoder); + sigset_t oldset; uint16_t hs_start, hs_end, line_start, line_end; uint16_t vwin_start, vwin_end, de_start, de_end; uint16_t ref_pix, ref_line, pix_start2; uint8_t reg, div, rep; + savesig(&oldset); + hs_start = mode->hsync_start - mode->hdisplay; hs_end = mode->hsync_end - mode->hdisplay; line_start = 1; @@ -613,6 +643,8 @@ tda998x_encoder_mode_set(struct drm_encoder *encoder, /* must be last register set: */ reg_clear(encoder, REG_TBG_CNTRL_0, TBG_CNTRL_0_SYNC_ONCE); + + restoresig(&oldset); } static enum drm_connector_status @@ -728,15 +760,21 @@ static int tda998x_encoder_get_modes(struct drm_encoder *encoder, struct drm_connector *connector) { - struct edid *edid = (struct edid *)do_get_edid(encoder); + struct edid *edid; + sigset_t oldset; int n = 0; + savesig(&oldset); + + edid = (struct edid *) do_get_edid(encoder); if (edid) { drm_mode_connector_update_edid_property(connector, edid); n = drm_add_edid_modes(connector, edid); kfree(edid); } + restoresig(&oldset); + return n; }