From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: i2c_adapter.class and new style drivers Date: Tue, 22 Jan 2008 15:23:47 +0100 Message-ID: <20080122152347.1ebd0f4c@hyperion.delvare> References: <9e4733910801211549v74ca13c4ya07179d36b35bff0@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <9e4733910801211549v74ca13c4ya07179d36b35bff0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org Errors-To: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org To: Jon Smirl Cc: Linux I2C List-Id: linux-i2c@vger.kernel.org Hi Jon, On Mon, 21 Jan 2008 18:49:34 -0500, Jon Smirl wrote: > Is i2c_adapter.class relevant anymore with new style drivers? No, it's not. The class is meant to limit the probing scope of legacy i2c chip drivers. New-style i2c chip drivers shouldn't care. Note that these classes aren't going away anytime soon though: some chip types just have to be probed (e.g. hardware monitoring chips or EEPROMs on PC motherboards) so even if these drivers are eventually converted to new-style i2c chip drivers, some (possibly external) probing will still be necessary, and the class will be there to limit the scope of this probing. > On 1/2/08, Jochen Friedrich wrote: > > IMHO, there should be a node attribute to override > > i2c_adapter.class. Legacy i2c drivers (in particular v4l and dvb > > drivers) use this class to decide which adapter to bind to. > > dbox2 needs I2C_CLASS_TV_DIGITAL (4). I am fine with a node attribute allowing control of this attribute, that could be handy for testing and debugging purposes. If someone submits a patch implementing this, I'll review and merge it. -- Jean Delvare _______________________________________________ i2c mailing list i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org http://lists.lm-sensors.org/mailman/listinfo/i2c