lkml.org 
[lkml]   [2003]   [Mar]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] i2c driver changes for 2.5.64
On Thu, Mar 13, 2003 at 04:55:00PM -0800, Greg KH wrote:
> +#include <linux/i2c.h>
> +#include <asm/io.h>
> +
> +MODULE_LICENSE("GPL");

This should be at the bottom of the file with the other MODULE_* stuff

> +#ifdef I2C_FUNC_SMBUS_BLOCK_DATA_PEC
> +#define HAVE_PEC
> +#endif

Needs more explanation.

> +
> +#ifndef PCI_DEVICE_ID_INTEL_82801CA_SMBUS
> +#define PCI_DEVICE_ID_INTEL_82801CA_SMBUS 0x2483
> +#endif
> +
> +#ifndef PCI_DEVICE_ID_INTEL_82801DB_SMBUS
> +#define PCI_DEVICE_ID_INTEL_82801DB_SMBUS 0x24C3
> +#endif

Should go to pci_ids.h

> +
> +static int supported[] = {PCI_DEVICE_ID_INTEL_82801AA_3,
> + PCI_DEVICE_ID_INTEL_82801AB_3,
> + PCI_DEVICE_ID_INTEL_82801BA_2,
> + PCI_DEVICE_ID_INTEL_82801CA_SMBUS,
> + PCI_DEVICE_ID_INTEL_82801DB_SMBUS,
> + 0 };

Shouldn't be required with new-style pci probing.

> +static int force_addr = 0;
> +MODULE_PARM(force_addr, "i");
> +MODULE_PARM_DESC(force_addr,
> + "Forcibly enable the I801 at the given address. "
> + "EXTREMELY DANGEROUS!");

What about using Rusty's new module parameter stuff?

> +static unsigned short i801_smba = 0;
> +static struct pci_dev *I801_dev = NULL;
> +static int isich4 = 0;

This is in .bss..

> +/* Detect whether a I801 can be found, and initialize it, where necessary.
> + Note the differences between kernels with the old PCI BIOS interface and
> + newer kernels with the real PCI interface. In compat.h some things are
> + defined to make the transition easier. */
> +int i801_setup(void)

Merge with caller, comment is b0rked.

> +{
> + int error_return = 0;
> + int *num = supported;
> + unsigned char temp;
> +
> + /* First check whether we can access PCI at all */
> + if (pci_present() == 0) {
> + printk(KERN_WARNING "i2c-i801.o: Error: No PCI-bus found!\n");
> + error_return = -ENODEV;
> + goto END;
> + }

END is a bad name for a label.

> + /* Look for each chip */
> + /* Note: we keep on searching until we have found 'function 3' */
> + I801_dev = NULL;
> + do {
> + if((I801_dev = pci_find_device(PCI_VENDOR_ID_INTEL,
> + *num, I801_dev))) {
> + if(PCI_FUNC(I801_dev->devfn) != 3)
> + continue;
> + break;
> + }
> + num++;
> + } while (*num != 0);

You already changed it to new-style pci probing in the next patch, didn't
you?

> + if (check_region(i801_smba, (isich4 ? 16 : 8))) {

no way!

> +int i801_transaction(void)

static..

> + int temp;
> + int result = 0;
> + int timeout = 0;
> +
> +#ifdef DEBUG
> + printk
> + (KERN_DEBUG "i2c-i801.o: Transaction (pre): CNT=%02x, CMD=%02x, ADD=%02x, DAT0=%02x, "
> + "DAT1=%02x\n", inb_p(SMBHSTCNT), inb_p(SMBHSTCMD),
> + inb_p(SMBHSTADD), inb_p(SMBHSTDAT0), inb_p(SMBHSTDAT1));
> +#endif

pr_debug

> + temp = inb_p(SMBHSTSTS);
> + if (i == 1) {
> + /* Erronenous conditions before transaction:
> + * Byte_Done, Failed, Bus_Err, Dev_Err, Intr, Host_Busy */
> + errmask=0x9f;
> + } else {
> + /* Erronenous conditions during transaction:
> + * Failed, Bus_Err, Dev_Err, Intr */
> + errmask=0x1e;
> + }

Indentation look b0rked.

-
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/

\
 
 \ /
  Last update: 2005-03-22 13:33    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site