From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161512AbbBDUeA (ORCPT ); Wed, 4 Feb 2015 15:34:00 -0500 Received: from bh-25.webhostbox.net ([208.91.199.152]:38970 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934120AbbBDUd6 (ORCPT ); Wed, 4 Feb 2015 15:33:58 -0500 Date: Wed, 4 Feb 2015 12:33:52 -0800 From: Guenter Roeck To: Wolfram Sang Cc: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] eeprom: at24: Add support for large EEPROMs connected to SMBus adapters Message-ID: <20150204203352.GA18073@roeck-us.net> References: <1423067017-27607-1-git-send-email-linux@roeck-us.net> <20150204174723.GB30930@katana> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150204174723.GB30930@katana> User-Agent: Mutt/1.5.23 (2014-03-12) X-Authenticated_sender: guenter@roeck-us.net X-OutGoing-Spam-Status: No, score=-1.0 X-CTCH-PVer: 0000001 X-CTCH-Spam: Unknown X-CTCH-VOD: Unknown X-CTCH-Flags: 0 X-CTCH-RefID: str=0001.0A020203.54D28236.01BD,ss=1,re=0.001,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 X-CTCH-Score: 0.001 X-CTCH-ScoreCust: 0.000 X-CTCH-Rules: C_4847, X-CTCH-SenderID: linux@roeck-us.net X-CTCH-SenderID-Flags: 0 X-CTCH-SenderID-TotalMessages: 3 X-CTCH-SenderID-TotalSpam: 0 X-CTCH-SenderID-TotalSuspected: 0 X-CTCH-SenderID-TotalConfirmed: 0 X-CTCH-SenderID-TotalBulk: 0 X-CTCH-SenderID-TotalVirus: 0 X-CTCH-SenderID-TotalRecipients: 0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: mailgid no entry from get_relayhosts_entry X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 04, 2015 at 06:47:23PM +0100, Wolfram Sang wrote: > > On Wed, Feb 04, 2015 at 08:23:37AM -0800, Guenter Roeck wrote: > > Large EEPROMS (24c32 and larger) require a two-byte data address > > instead of just a single byte. Implement support for such EEPROMs > > with SMBus commands. > > > > Support has limitations (reads are not multi-master safe) and is slow, > > but it works. Practical use is for a system with 24c32 connected to > > Intel 82801I (ICH9). > > Can't you simply use i2c-dev to access the EEPROM? In multi-master > environments, things can really go wrong, so I wouldn't like to add > something dangerous by default. Maybe with a module parameter named > "allow-multimaster-unsafe-access-to-large-eeproms-with-smbus" which is > default off. But I'd really prefer the i2c-dev solution. Hooking a 16bit > EEPROM to SMBus is daring, after all. SMBus is multi-master, too. > Hi Wolfram, thinking about it some more, the problem you are concerned about really is that 24c32 and larger EEPROMs can not be reliably supported in multi-master configurations with SMBus adapters. My solution to that problem would be not to build such hardware. Your solution appears to be not to support 24c32 and larger EEPROMs on SMBUs adapters at all, which I consider highly restrictive. Please reconsider. Thanks, Guenter