From patchwork Thu Nov 4 11:09:22 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 12602877 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 066C4C433FE for ; Thu, 4 Nov 2021 11:09:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E183A61213 for ; Thu, 4 Nov 2021 11:09:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231371AbhKDLMZ (ORCPT ); Thu, 4 Nov 2021 07:12:25 -0400 Received: from mslow1.mail.gandi.net ([217.70.178.240]:36913 "EHLO mslow1.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231373AbhKDLMW (ORCPT ); Thu, 4 Nov 2021 07:12:22 -0400 Received: from relay9-d.mail.gandi.net (unknown [217.70.183.199]) by mslow1.mail.gandi.net (Postfix) with ESMTP id 238ABD0692 for ; Thu, 4 Nov 2021 11:09:01 +0000 (UTC) Received: (Authenticated sender: jacopo@jmondi.org) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id C9FC5FF802; Thu, 4 Nov 2021 11:08:37 +0000 (UTC) From: Jacopo Mondi To: Kieran Bingham , Laurent Pinchart , =?utf-8?q?Nik?= =?utf-8?q?las_S=C3=B6derlund?= Cc: Jacopo Mondi , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org Subject: [PATCH v2 0/2] max9271: Fix pclk_detect silent failure Date: Thu, 4 Nov 2021 12:09:22 +0100 Message-Id: <20211104110924.248444-1-jacopo+renesas@jmondi.org> X-Mailer: git-send-email 2.33.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org Repeatedly reading register 0x15 to validate the incoming pixel clock causes sporadic read errors which went silently ignored, causing the camera module to fail to start streaming. Fix that by ignoring the read error and while at it rework the error message handling in all functions. v1->v2: - Drop v1 [1/2] - [2/2] new patch. Handle all bus access errors in the call sites Jacopo Mondi (2): media: max9271: Ignore busy loop read errors media: max9271: Fail loud on bus errors in call sites drivers/media/i2c/max9271.c | 116 ++++++++++++++++++++++-------------- 1 file changed, 71 insertions(+), 45 deletions(-) --- 2.33.1