linux-kernel - Re: [PATCH 2/3] i2c-eg20t: Call init() when wait-event timeout occurs

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:	Thu, 19 Apr 2012 15:19:14 +0900
From:	Tomoya MORINAGA <tomoya.rohm@...il.com>
To:	Wolfram Sang <w.sang@...gutronix.de>
Cc:	Jean Delvare <khali@...ux-fr.org>, Ben Dooks <ben-linux@...ff.org>,
	Feng Tang <feng.tang@...el.com>,
	Alexander Stein <alexander.stein@...tec-electronic.com>,
	linux-i2c@...r.kernel.org, linux-kernel@...r.kernel.org,
	qi.wang@...el.com, yong.y.wang@...el.com, joel.clark@...el.com,
	kok.howg.ewe@...el.com
Subject: Re: [PATCH 2/3] i2c-eg20t: Call init() when wait-event timeout occurs

On Thu, Apr 19, 2012 at 12:05 AM, Wolfram Sang <w.sang@...gutronix.de> wrote:
> On Mon, Mar 26, 2012 at 02:55:24PM +0900, Tomoya MORINAGA wrote:
>>
>> Signed-off-by: Tomoya MORINAGA <tomoya.rohm@...il.com>
>
> Woha, that's copy&paste code all over. May I ask to refactor this?
> Something like:
>
>        rtn = pch_..._wait_for_checked_xfer();
>        if (rtn)
>                return rtn;
>        // All the things you have to do when rtn == 0
>
> in that new function (the name was only a suugestion), you can do all
> the checks which are now copy-pasted, e.g.:
>
>  458         rtn = pch_i2c_wait_for_xfer_complete(adap);
>  459         if (rtn == 0) {
>  460                 if (pch_i2c_getack(adap)) {
>  461                         pch_dbg(adap, "Receive NACK for slave address"
>  462                                 "setting\n");
>  463                         return -EIO;
>  464                 }
>  465         } else if (rtn == -EIO) { /* Arbitration Lost */
>  466                 pch_err(adap, "Lost Arbitration\n");
>  467                 pch_clrbit(adap->pch_base_address, PCH_I2CSR, I2CMAL_BIT);
>  468                 pch_clrbit(adap->pch_base_address, PCH_I2CSR, I2CMIF_BIT);
>  469                 pch_i2c_init(adap);
>  470                 return -EAGAIN;
>  471         } else { /* wait-event timeout */
>  472                 pch_i2c_stop(adap);
>                     // Your current fixes added here
>  473                 return -ETIME;
>  474         }
>             return 0;
>
> It is only pseudo-code, but I think it can be done and will help the driver.
>

OK.
I'm ready to submit new patch you are saying.

So, firstly, could you accept this patch ?
After your accept, new patch series can be applicable.

thanks.
-- 
ROHM Co., Ltd.
tomoya
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/