From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH] i2c_intel_mid: Fix slash in sysfs name Date: Wed, 15 Dec 2010 10:08:15 +0100 Message-ID: <20101215100815.44fb8b83@endymion.delvare> References: <20101214152849.7265.93628.stgit@bob.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20101214152849.7265.93628.stgit-Z/y2cZnRghHXmaaqVzeoHQ@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alan Cox Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org List-Id: linux-i2c@vger.kernel.org Hi Alan, On Tue, 14 Dec 2010 15:29:08 +0000, Alan Cox wrote: > From: Alan Cox > > This gets caught by the new sanity check code. New sanity check of what? Where's the code? > Instead of the slash use a > different symbol. This was originally found by Major Lee who proposed a > rather more complex patch which changed the name according to the chip > type. > > On the basis that we are in a late -rc and making Linus grumpy isn't always > a good idea (however fun) this is a simple alternative. > > Signed-off-by: Alan Cox > --- > > drivers/i2c/busses/i2c-intel-mid.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > > diff --git a/drivers/i2c/busses/i2c-intel-mid.c b/drivers/i2c/busses/i2c-intel-mid.c > index 657f314..da4bbdd 100644 > --- a/drivers/i2c/busses/i2c-intel-mid.c > +++ b/drivers/i2c/busses/i2c-intel-mid.c > @@ -1035,7 +1035,7 @@ static int __devinit intel_mid_i2c_probe(struct pci_dev *dev, > > /* Initialize struct members */ > snprintf(mrst->adap.name, sizeof(mrst->adap.name), > - "MRST/Medfield I2C at %lx", start); > + "Intel MID I2C at %lx", start); > mrst->adap.owner = THIS_MODULE; > mrst->adap.algo = &intel_mid_i2c_algorithm; > mrst->adap.dev.parent = &dev->dev; Having a slash in an I2C adapter name shouldn't be a problem. The real problem is below: err = request_irq(dev->irq, intel_mid_i2c_isr, IRQF_SHARED, mrst->adap.name, mrst); Using the adapter name to request the IRQ is silly. The very same issue was fixed in i2c-pca-platform recently: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=323584436db0cb05286425d4dfd9516fce88487f You want to use a short name for the interrupt. Either the driver name, or if you really want something device-specific, the bus device name. -- Jean Delvare