Peter Samuelson wrote: > [ChenLi Tien, from http://members.home.net/puresoft/cmedia.html] > > - * Copyright (C) 1999 ChenLi Tien ([EMAIL PROTECTED]) > > - * > > - * Based on the PCI drivers by Thomas Sailer ([EMAIL PROTECTED]) > > + * Copyright (C) 1999 ChenLi Tien ([EMAIL PROTECTED]) > > + * C-media support ([EMAIL PROTECTED]) > > This is somewhat impolite -- unless the driver is *not* actually based > on Tom's work. > I didn't notice someone add this line, but I will be happy to add it. > > > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,3,0) > > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0) > > + owner: THIS_MODULE, > > +#endif > > llseek: cm_llseek, > > ioctl: cm_ioctl_mixdev, > > open: cm_open_mixdev, > > release: cm_release_mixdev, > > +#else > > + &cm_llseek, > > + NULL, /* read */ > > + NULL, /* write */ > > + NULL, /* readdir */ > > + NULL, /* poll */ > > + &cm_ioctl_mixdev, > > + NULL, /* mmap */ > > + &cm_open_mixdev, > > + NULL, /* flush */ > > + &cm_release_mixdev, > > + NULL, /* fsync */ > > + NULL, /* fasync */ > > + NULL, /* check_media_change */ > > + NULL, /* revalidate */ > > + NULL, /* lock */ > > +#endif > > I don't think the (2,3,0) ifdef is necessary. Just use the labeled > initializers for all kernels. See also cm_audio_fops, cm_dsp_fops, > cm_midi_fops, cm_dmfm_fops. Yes, as 2.3.x series is not for end-user, I can remove them. I keep it for easy to tell what's different for kernel 2.3 and 2.4. > > > > +#ifdef MODULE > > +MODULE_PARM(mpu_io, "i"); > > +MODULE_PARM(fm_io, "i"); > > +MODULE_PARM(spdif_inverse, "i"); > > +MODULE_PARM(spdif_loop, "i"); > > +MODULE_PARM(four_sp, "i"); > > +MODULE_PARM(rear_out, "i"); > > +MODULE_PARM(modem, "i"); > > +MODULE_PARM(joystick, "i"); > > #endif > > No need for '#ifdef MODULE'. I will remove it if kernel 2.2 can work. > > > > +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0) > > +static int __init init_cmpci(void) > > +#elif LINUX_VERSION_CODE >= KERNEL_VERSION(2,3,0) > > +#ifdef MODULE > > +int __init init_module(void) > > +#else > > int __init init_cmpci(void) > > #endif > > +#else > > +#ifdef MODULE > > +__initfunc(int init_module(void)) > > +#else > > +__initfunc(int init_cmpci(void)) > > +#endif > > +#endif > > __init is fine in 2.2, no need for conditional __initfunc(). > > Peter Thanks for your suggestion, I can change it. ChenLi Tien - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/