Re: [PATCH] assign a default class to MPC adapter

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 16 Apr 2009 15:36:12 +0200, Michael Lawnick wrote:
> Wolfram Sang said the following:
> >> Could you please elaborate?
> > 
> > Instead of probing you specify the devices on the i2c-bus in the dts. Example
> > for the pcm032 looks like this (from the kernel-tree):
> > 
> > i2c@3d40 {
> > 	#address-cells = <1>;
> > 	#size-cells = <0>;
> > 	compatible = "fsl,mpc5200b-i2c","fsl,mpc5200-i2c","fsl-i2c";
> > 	reg = <0x3d40 0x40>;
> > 	interrupts = <2 16 0>;
> > 	fsl5200-clocking;
> > 
> > 	rtc@51 {
> > 		compatible = "nxp,pcf8563";
> > 		reg = <0x51>;
> > 	};
> > 	eeprom@52 {
> > 		compatible = "at24,24c32";
> > 		reg = <0x52>;
> > 	};
> > };
> > 
> > Boards which relied on probing need to have such child nodes added.
> 
> Ok, that doesn't help :-(
> I hoped you show me a way how to set the class of adapter. My clients
> are loaded with insmod, so dts has no effect (or is this wrong?)

No, you don't load "clients" with insmod. You load i2c device drivers
with insmod. This is in no way incompatible with what Wolfram is
suggesting. "Clients" (or i2c devices) can (and in general should)
exist independent of their driver being loaded or not.

> The current situation is:
> I can't see a method beside force to instantiate a client at runtime.

That's right, but why do you insist on instantiating them at runtime?

> But force bypasses clients detect method and is deprecated anyway if I
> got Jean right.

I confirm that the force module parameters are planed for removal
(although I can really call them deprecated in the absence of a
replacement at this time - I know there's a patch from you pending my
review.)

> Currently I'm lost. Jean NACKs all attempts to do detected client
> instantiations at runtime, you do a NACK on making adapter working
> properly by indicating to patches that have no more relevance.
> 
> Can't anybody help me?

The question is: why every other user of i2c-mpc is apparently able to
declare i2c devices through dts and you aren't?

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux