linux-kernel - [PATCH v3 0/3] i2c: imx: Fix handling of arbitration loss

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20201006105135.28985-1-ceggers@arri.de>
Date:   Tue, 6 Oct 2020 12:51:32 +0200
From:   Christian Eggers <ceggers@...i.de>
To:     Oleksij Rempel <linux@...pel-privat.de>,
        Shawn Guo <shawnguo@...nel.org>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Fabio Estevam <festevam@...il.com>,
        Uwe Kleine-König 
        <u.kleine-koenig@...gutronix.de>
CC:     Pengutronix Kernel Team <kernel@...gutronix.de>,
        NXP Linux Team <linux-imx@....com>,
        <linux-i2c@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>
Subject: [PATCH v3 0/3] i2c: imx: Fix handling of arbitration loss

On Tuesday, 6 October 2020, 08:05:28 CEST, Uwe Kleine-König wrote:
> Could we please move clearing an irq to a dedicated function? Such that
> it looks like:
> 
> 	/* check for arbitration lost */
> 	if (temp & I2SR_IAL) {
> 		i2c_imx_clear_irq(i2c_imx, I2SR_IAL);
> 		return -EAGAIN;
> 	}
done

Changes in v2:
---------------
- Don't accidently clear additional status flags on Vybrid
  (reported by Uwe Kleine-Koenig)

Changes in v3:
---------------
- dedicated function for clearing an irq