mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* BUG in modutils or drivers/isdn/hisax/
@ 2001-01-23  0:31 Ingo Oeser
  2001-01-23 11:28 ` Karsten Keil
  0 siblings, 1 reply; 4+ messages in thread
From: Ingo Oeser @ 2001-01-23  0:31 UTC (permalink / raw)
  To: Keith Owens; +Cc: Karsten Keil, linux-kernel

Hi Keith,
hi Karsten,
hi linux-kernel,

the current modutils (2.4.1) cannot read the
__module_pci_device_table of a kernel/drivers/isdn/hisax/hisax.o
module of linux 2.4.0 (vanilla).

What's wrong with it?

The compiler is gcc 2.95.2 (as shipped with debian potato). All
other module device tables can be read. USB has not been
compiled.


Thanks & Regards

Ingo Oeser


Now the gory details (ask for more if needed)...

I checked:

depmod -e -n /path/to/hisax.o (showed an empty pci table)
depmod -e -n /path/to/8139too.o (showed a complete pci table)

configured in core:

CONFIG_X86=y
CONFIG_ISA=y
CONFIG_UID16=y
CONFIG_EXPERIMENTAL=y
CONFIG_MODULES=y
CONFIG_KMOD=y
CONFIG_M586TSC=y
CONFIG_X86_WP_WORKS_OK=y
CONFIG_X86_INVLPG=y
CONFIG_X86_CMPXCHG=y
CONFIG_X86_BSWAP=y
CONFIG_X86_POPAD_OK=y
CONFIG_X86_USE_STRING_486=y
CONFIG_X86_ALIGNMENT_16=y
CONFIG_X86_TSC=y
CONFIG_NOHIGHMEM=y
CONFIG_NET=y
CONFIG_PCI=y
CONFIG_PCI_GOANY=y
CONFIG_PCI_BIOS=y
CONFIG_PCI_DIRECT=y
CONFIG_SYSVIPC=y
CONFIG_SYSCTL=y
CONFIG_KCORE_ELF=y
CONFIG_BINFMT_ELF=y
CONFIG_PM=y
CONFIG_ACPI=y
CONFIG_PARPORT_PC_FIFO=y
CONFIG_PARPORT_PC_SUPERIO=y
CONFIG_PARPORT_1284=y
CONFIG_BLK_DEV_LOOP=y
CONFIG_BLK_DEV_RAM=y
CONFIG_PACKET_MMAP=y
CONFIG_NETFILTER=y
CONFIG_FILTER=y
CONFIG_UNIX=y
CONFIG_INET=y
CONFIG_IP_MULTICAST=y
CONFIG_SYN_COOKIES=y
CONFIG_IP_NF_NAT_NEEDED=y
CONFIG_IP_NF_NAT_NEEDED=y
CONFIG_IDE=y
CONFIG_BLK_DEV_IDE=y
CONFIG_BLK_DEV_IDEDISK=y
CONFIG_BLK_DEV_IDEPCI=y
CONFIG_IDEPCI_SHARE_IRQ=y
CONFIG_BLK_DEV_IDEDMA_PCI=y
CONFIG_IDEDMA_PCI_AUTO=y
CONFIG_BLK_DEV_IDEDMA=y
CONFIG_BLK_DEV_VIA82CXXX=y
CONFIG_IDEDMA_AUTO=y
CONFIG_BLK_DEV_IDE_MODES=y
CONFIG_BLK_DEV_SR_VENDOR=y
CONFIG_SCSI_MULTI_LUN=y
CONFIG_NETDEVICES=y
CONFIG_NET_ETHERNET=y
CONFIG_NET_PCI=y
CONFIG_PPP_MULTILINK=y
CONFIG_ISDN_PPP=y
CONFIG_ISDN_PPP_VJ=y
CONFIG_ISDN_MPP=y
CONFIG_HISAX_EURO=y
CONFIG_DE_AOC=y
CONFIG_HISAX_FRITZPCI=y
CONFIG_VT=y
CONFIG_VT_CONSOLE=y
CONFIG_SERIAL=y
CONFIG_SERIAL_CONSOLE=y
CONFIG_SERIAL_EXTENDED=y
CONFIG_SERIAL_DETECT_IRQ=y
CONFIG_AUTOFS4_FS=y
CONFIG_CRAMFS=y
CONFIG_RAMFS=y
CONFIG_JOLIET=y
CONFIG_PROC_FS=y
CONFIG_EXT2_FS=y
CONFIG_MSDOS_PARTITION=y
CONFIG_NLS=y
CONFIG_VGA_CONSOLE=y
CONFIG_MAGIC_SYSRQ=y

configured as modules:

CONFIG_PARPORT=m
CONFIG_PARPORT_PC=m
CONFIG_PNP=m
CONFIG_ISAPNP=m
CONFIG_PACKET=m
CONFIG_NET_IPIP=m
CONFIG_IP_NF_CONNTRACK=m
CONFIG_IP_NF_FTP=m
CONFIG_IP_NF_IPTABLES=m
CONFIG_IP_NF_MATCH_LIMIT=m
CONFIG_IP_NF_MATCH_MAC=m
CONFIG_IP_NF_MATCH_MARK=m
CONFIG_IP_NF_MATCH_MULTIPORT=m
CONFIG_IP_NF_MATCH_TOS=m
CONFIG_IP_NF_MATCH_STATE=m
CONFIG_IP_NF_MATCH_UNCLEAN=m
CONFIG_IP_NF_MATCH_OWNER=m
CONFIG_IP_NF_FILTER=m
CONFIG_IP_NF_TARGET_REJECT=m
CONFIG_IP_NF_TARGET_MIRROR=m
CONFIG_IP_NF_NAT=m
CONFIG_IP_NF_TARGET_MASQUERADE=m
CONFIG_IP_NF_TARGET_REDIRECT=m
CONFIG_IP_NF_MANGLE=m
CONFIG_IP_NF_TARGET_TOS=m
CONFIG_IP_NF_TARGET_MARK=m
CONFIG_IP_NF_TARGET_LOG=m
CONFIG_IP_NF_COMPAT_IPCHAINS=m
CONFIG_BLK_DEV_IDECD=m
CONFIG_BLK_DEV_IDETAPE=m
CONFIG_BLK_DEV_IDESCSI=m
CONFIG_SCSI=m
CONFIG_BLK_DEV_SD=m
CONFIG_CHR_DEV_OSST=m
CONFIG_BLK_DEV_SR=m
CONFIG_CHR_DEV_SG=m
CONFIG_NE2K_PCI=m
CONFIG_8139TOO=m
CONFIG_RTL8129=m
CONFIG_PPP=m
CONFIG_PPP_ASYNC=m
CONFIG_PPP_SYNC_TTY=m
CONFIG_PPP_DEFLATE=m
CONFIG_PPP_BSDCOMP=m
CONFIG_PPPOE=m
CONFIG_ISDN=m
CONFIG_ISDN_DRV_HISAX=m
CONFIG_PRINTER=m
CONFIG_RTC=m
CONFIG_ISO9660_FS=m
CONFIG_NLS_CODEPAGE_437=m
CONFIG_NLS_CODEPAGE_850=m
CONFIG_NLS_ISO8859_1=m
CONFIG_NLS_ISO8859_15=m
CONFIG_NLS_UTF8=m

That's all so far. Need more? Just ask.
-- 
10.+11.03.2001 - 3. Chemnitzer LinuxTag <http://www.tu-chemnitz.de/linux/tag>
         <<<<<<<<<<<<       come and join the fun       >>>>>>>>>>>>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: BUG in modutils or drivers/isdn/hisax/
  2001-01-23  0:31 BUG in modutils or drivers/isdn/hisax/ Ingo Oeser
@ 2001-01-23 11:28 ` Karsten Keil
  2001-01-23 14:13   ` Ingo Oeser
  0 siblings, 1 reply; 4+ messages in thread
From: Karsten Keil @ 2001-01-23 11:28 UTC (permalink / raw)
  To: Ingo Oeser; +Cc: Keith Owens, Karsten Keil, linux-kernel

Hi,

On Tue, Jan 23, 2001 at 01:31:55AM +0100, Ingo Oeser wrote:
> Hi Keith,
> hi Karsten,
> hi linux-kernel,
> 
> the current modutils (2.4.1) cannot read the
> __module_pci_device_table of a kernel/drivers/isdn/hisax/hisax.o
> module of linux 2.4.0 (vanilla).
> 
> What's wrong with it?

Nothing. Only the HFC-PCI part in hisax has such a table yet, all other
card drivers in hisax don't have one at the moment.

-- 
Karsten Keil
SuSE Labs
ISDN development
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: BUG in modutils or drivers/isdn/hisax/
  2001-01-23 11:28 ` Karsten Keil
@ 2001-01-23 14:13   ` Ingo Oeser
  2001-01-23 16:42     ` Kai Germaschewski
  0 siblings, 1 reply; 4+ messages in thread
From: Ingo Oeser @ 2001-01-23 14:13 UTC (permalink / raw)
  To: Karsten Keil; +Cc: linux-kernel

Hi Karsten,

On Tue, Jan 23, 2001 at 12:28:49PM +0100, Karsten Keil wrote:
> > the current modutils (2.4.1) cannot read the
> > __module_pci_device_table of a kernel/drivers/isdn/hisax/hisax.o
> > module of linux 2.4.0 (vanilla).
> > 
> > What's wrong with it?
> 
> Nothing. Only the HFC-PCI part in hisax has such a table yet, all other
> card drivers in hisax don't have one at the moment.

To quote drivers/isdn/hisax/config.c:1710-1713
static struct pci_device_id hisax_pci_tbl[] __initdata = {
#ifdef CONFIG_HISAX_FRTIZPCI
        {PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,       PCI_ANY_ID, PCI_ANY_ID},
#endif

To quote my .config:
CONFIG_ISDN_DRV_HISAX=m
CONFIG_HISAX_FRITZPCI=y

So the bug is indeed in the driver and is a speling mistake
Patch is:

--- linux/drivers/isdn/hisax/config.c.orig	Fri Dec 29 23:07:22 2000
+++ linux/drivers/isdn/hisax/config.c	Tue Jan 23 15:07:54 2001
@@ -1708,8 +1708,8 @@
 }
 
 static struct pci_device_id hisax_pci_tbl[] __initdata = {
-#ifdef CONFIG_HISAX_FRTIZPCI
-	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,        PCI_ANY_ID, PCI_ANY_ID},
+#ifdef CONFIG_HISAX_FRITZPCI
+	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,	PCI_ANY_ID, PCI_ANY_ID},
 #endif
 #ifdef CONFIG_HISAX_DIEHLDIVA
 	{PCI_VENDOR_ID_EICON,    PCI_DEVICE_ID_EICON_DIVA20,     PCI_ANY_ID, PCI_ANY_ID},


Please apply.

Regards

Ingo Oeser
-- 
10.+11.03.2001 - 3. Chemnitzer LinuxTag <http://www.tu-chemnitz.de/linux/tag>
         <<<<<<<<<<<<       come and join the fun       >>>>>>>>>>>>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: BUG in modutils or drivers/isdn/hisax/
  2001-01-23 14:13   ` Ingo Oeser
@ 2001-01-23 16:42     ` Kai Germaschewski
  0 siblings, 0 replies; 4+ messages in thread
From: Kai Germaschewski @ 2001-01-23 16:42 UTC (permalink / raw)
  To: Ingo Oeser; +Cc: Karsten Keil, linux-kernel, Kai Germaschewski



On Tue, 23 Jan 2001, Ingo Oeser wrote:

> To quote drivers/isdn/hisax/config.c:1710-1713
> static struct pci_device_id hisax_pci_tbl[] __initdata = {
> #ifdef CONFIG_HISAX_FRTIZPCI
>         {PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,       PCI_ANY_ID, PCI_ANY_ID},
> #endif
> 
> To quote my .config:
> CONFIG_ISDN_DRV_HISAX=m
> CONFIG_HISAX_FRITZPCI=y
> 
> So the bug is indeed in the driver and is a speling mistake
> Patch is:
> 
> --- linux/drivers/isdn/hisax/config.c.orig	Fri Dec 29 23:07:22 2000
> +++ linux/drivers/isdn/hisax/config.c	Tue Jan 23 15:07:54 2001
> @@ -1708,8 +1708,8 @@
>  }
>  
>  static struct pci_device_id hisax_pci_tbl[] __initdata = {
> -#ifdef CONFIG_HISAX_FRTIZPCI
> -	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,        PCI_ANY_ID, PCI_ANY_ID},
> +#ifdef CONFIG_HISAX_FRITZPCI
> +	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,	PCI_ANY_ID, PCI_ANY_ID},
>  #endif
>  #ifdef CONFIG_HISAX_DIEHLDIVA
>  	{PCI_VENDOR_ID_EICON,    PCI_DEVICE_ID_EICON_DIVA20,     PCI_ANY_ID, PCI_ANY_ID},
> 

Close, but not quite. (Try compiling after your patch :)

Correct patch:

--- linux-2.4.0/drivers/isdn/hisax/config.c	Fri Dec 29 23:07:22 2000
+++ linux-2.4.1-pre10.work/drivers/isdn/hisax/config.c	Tue Jan 23 11:23:54 2001
@@ -1708,8 +1708,8 @@
 }
 
 static struct pci_device_id hisax_pci_tbl[] __initdata = {
-#ifdef CONFIG_HISAX_FRTIZPCI
-	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_FRITZ,        PCI_ANY_ID, PCI_ANY_ID},
+#ifdef CONFIG_HISAX_FRITZPCI
+	{PCI_VENDOR_ID_AVM,      PCI_DEVICE_ID_AVM_A1,          PCI_ANY_ID, PCI_ANY_ID},
 #endif
 #ifdef CONFIG_HISAX_DIEHLDIVA
 	{PCI_VENDOR_ID_EICON,    PCI_DEVICE_ID_EICON_DIVA20,     PCI_ANY_ID, PCI_ANY_ID},

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2001-01-23 16:42 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-01-23  0:31 BUG in modutils or drivers/isdn/hisax/ Ingo Oeser
2001-01-23 11:28 ` Karsten Keil
2001-01-23 14:13   ` Ingo Oeser
2001-01-23 16:42     ` Kai Germaschewski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®