Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751531AbbDBUJ5 (ORCPT ); Thu, 2 Apr 2015 16:09:57 -0400 Received: from mail-lb0-f182.google.com ([209.85.217.182]:34096 "EHLO mail-lb0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750887AbbDBUJy (ORCPT ); Thu, 2 Apr 2015 16:09:54 -0400 MIME-Version: 1.0 In-Reply-To: <20150327153713.GC27238@katana> References: <20150327153713.GC27238@katana> Date: Fri, 3 Apr 2015 01:39:52 +0530 Message-ID: Subject: Re: [PATCH] i2c: cadence: Handling Slave monitor mode From: Harini Katakam To: Wolfram Sang Cc: Nava kishore Manne , Michal Simek , =?UTF-8?Q?S=C3=B6ren_Brinkmann?= , "linux-arm-kernel@lists.infradead.org" , "linux-i2c@vger.kernel.org" , "linux-kernel@vger.kernel.org" , anirudh@xilinx.com, svemula@xilinx.com, Nava kishore Manne Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1662 Lines: 44 Hi Wolfram, On Fri, Mar 27, 2015 at 9:07 PM, Wolfram Sang wrote: > On Tue, Mar 17, 2015 at 09:48:15PM +0530, Nava kishore Manne wrote: >> In slave monitor mode, the I2C interface is set up as a master and >> continues to attempt a transfer to a particular slave until the >> slave device responds with an ACK. >> >> Added this feature for zero length transfers enable the controller >> for slave monitor interrupt and get the status. Disable the slave >> monitor mode feature upon successful handling. >> >> Signed-off-by: Nava kishore Manne >> Acked-by: Harini Katakam > > I am not sure this is going to work. > > How often is this access tried when there is no device? It should be > tried only once. Once slave monitor is enabled in the host controller, it retries till there's an ACK or slave monitor is disabled. The SW wont be involved and need not be interrupted in this gap. The way this feature works in the controller though, I'm afraid there's no way to distinguish between no slave/ busy slave. The out is the software timeout in that case. Please suggest how to use this feature better. > > Those 0byte messages can be read or write, so transferring one bit, one > could say. I assume this one can only read? (which is not a show-stopper > but the cases should be handled) > Yes, only read is used. Thanks for the review. Regards, Harini -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/