linux-kernel - RE: [PATCH v8 1/3] i2c: tegra: use readx_poll_timeout after config

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]

Date:   Fri, 26 Aug 2016 09:19:40 +0000
From:   Shardar Mohammed <smohammed@...dia.com>
To:     Wolfram Sang <wsa@...-dreams.de>
CC:     Laxman Dewangan <ldewangan@...dia.com>,
        "swarren@...dotorg.org" <swarren@...dotorg.org>,
        "thierry.reding@...il.com" <thierry.reding@...il.com>,
        "gnurou@...il.com" <gnurou@...il.com>,
        "linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>,
        "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v8 1/3] i2c: tegra: use readx_poll_timeout after
 config_load reg programmed

> -----Original Message-----
> From: Wolfram Sang [mailto:wsa@...-dreams.de]
> Sent: Friday, August 26, 2016 1:10 AM
> To: Shardar Mohammed
> Cc: Laxman Dewangan; swarren@...dotorg.org;
> thierry.reding@...il.com; gnurou@...il.com; linux-i2c@...r.kernel.org;
> linux-tegra@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH v8 1/3] i2c: tegra: use readx_poll_timeout after
> config_load reg programmed
> 
> * PGP Signed by an unknown key
> 
> 
> > > > +		err = readx_poll_timeout(readl, i2c_dev->base + reg_offset,
> > > > +					 val, val == 0, 1000,
> > > > +					 I2C_CONFIG_LOAD_TIMEOUT);
> > >
> > > Any reason we can't use readl_poll_timeout() here?
> > [Shardar] Can be used, will change and push the new patchset.
> 
> Still interested? Should be an easy change, but I am reluctant to fix it here
> with no testing.
[Shardar] Will push the change by tomorrow.
> 
> 
> * Unknown Key
> * 0x14A029B6