From patchwork Thu Oct 4 09:08:55 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans Verkuil X-Patchwork-Id: 10625745 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 299A71926 for ; Thu, 4 Oct 2018 09:09:11 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 18B3428C6D for ; Thu, 4 Oct 2018 09:09:11 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 0CD3B28C95; Thu, 4 Oct 2018 09:09:11 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id BBD1C28C6D for ; Thu, 4 Oct 2018 09:09:10 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 5006C6E1DE; Thu, 4 Oct 2018 09:09:08 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from lb1-smtp-cloud8.xs4all.net (lb1-smtp-cloud8.xs4all.net [194.109.24.21]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9C5586E1DE for ; Thu, 4 Oct 2018 09:09:06 +0000 (UTC) Received: from tschai.fritz.box ([212.251.195.8]) by smtp-cloud8.xs4all.net with ESMTPA id 7zcygC0KD0ZZE7zd1gjcyE; Thu, 04 Oct 2018 11:09:05 +0200 From: Hans Verkuil To: linux-media@vger.kernel.org Subject: [PATCH 0/5] cec/adv/omap: fixes and new status flags Date: Thu, 4 Oct 2018 11:08:55 +0200 Message-Id: <20181004090900.32915-1-hverkuil@xs4all.nl> X-Mailer: git-send-email 2.18.0 X-CMAE-Envelope: MS4wfOHUDIiPDYOb622JxqGz7sJ0QhlbgIFACKk/7NT6oAHxkFiQcbOnxy7Db3kaDRt9Kt/1cu0cOd6nbBDm/yBqp9TrWLPWhRMY3J5pltf9/tKhEn6XnoVY mMpL2ETAhyOzwd5D1ZGSISo4CGl+MiPWVUfwZc/qZRDUcZ+i0OCbqDcH5mjJuDY8VJNAG850d8eylA0eywmxSbU0ciYhf7mCkH3Lo0ARHP3madJx2j46recs 6/MeU9buKfTrr+SWqEbo9z3ntaBs/U9unTcUVzEMcNE= X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Tomi Valkeinen , dri-devel@lists.freedesktop.org MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP From: Hans Verkuil The first patch adds new status flags to indicate when a pending message is aborted because the CEC adapter is unconfigured, and when a transmit times out (this indicates a driver bug). The second and third patch fix a minor issue with the adv HDMI receivers: if the EDID goes away, then the physical address also becomes invalid. The fourth patch fixes a race condition in the omap4 CEC driver that causes a transmit time out. The final patch drops the code in the omap4 CEC driver that attempts to set the number of retransmits: those register bits are read-only, so the code is pointless. There are no dependencies between these patches, although the first and fourth patch relate to the same problem. With the new transmit TIMEOUT status I hope that it will be easier to catch driver bugs like that earlier since this bug remained hidden for too long. Regards, Hans Hans Verkuil (5): cec: add new tx/rx status bits to detect aborts/timeouts adv7604: when the EDID is cleared, unconfigure CEC as well adv7842: when the EDID is cleared, unconfigure CEC as well omapdrm/dss/hdmi4_cec.c: clear TX FIFO before transmit_done omapdrm/dss/hdmi4_cec.c: don't set the retransmit count .../media/uapi/cec/cec-ioc-receive.rst | 25 ++++++- drivers/gpu/drm/omapdrm/dss/hdmi4_cec.c | 38 +++++------ drivers/media/cec/cec-adap.c | 66 +++++-------------- drivers/media/i2c/adv7604.c | 4 +- drivers/media/i2c/adv7842.c | 4 +- include/uapi/linux/cec.h | 3 + 6 files changed, 67 insertions(+), 73 deletions(-)