lkml.org 
[lkml]   [2003]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Q: Maintainer for drivers/isdn/hisax in kernel-2.6
Kai Germaschewski wrote:

>I'm still there to take questions and review patches.
>
Kai, sorry for bothering you, but right now i'm stuck in the process of
building an kernel-2.6-module for an Teles PCMCIA-Card. I guess my
questions are not that ISDN-specific but more basically in building
kernel-modules (especialy in drivers/isdn/hisax). There are some
comments from my side about the current state of the code.

OK, but let's start:

According to David Hinds - the Maintainer of pcmcia-cs - in kernel 2.6
it's no more possible to include any module in the pcmcia-cs package.
There where such a module for pcmcia-cs in the past, which worked well
in kernel 2.0, 2.2 and 2.4, Actually it was'nt included in pcmcia-cs but
an patch against (with little extensions from my side to run in kernel
2.2 and later in 2.4). Based on this module I've created an module for
kernel-2.6, and there the problems begun. I've also checked some other
modules in the kernel-tree but could'nt get enlighted. In
drivers/isdn/hisax i added an Kernel-Option (CONFIG_) HISAX_TELES_CS to
Kconfig, to the Makefile, to drivers/isdn/hisax/config.c and added a new
file teles_cs.c. The Kernel-Option seems to be well interpreted in the
Makefile, but not in config.c.
First Question: Why is that?

Second Question: What is config.c for? I defined some functions as
extern there (which actually defined in teles_cs.c), which are not known
by hisax.o (teles_cs.o also linked against):

scripts/modpost vmlinux drivers/isdn/hisax/hisax.o
drivers/ide/legacy/ide-cs.o drivers/isdn/i4l/isdn.o
drivers/isdn/i4l/isdn_bsdcomp.o drivers/isdn/hisax/teles_cs.o
*** Warning: "init_teles_cs" [drivers/isdn/hisax/hisax.ko] undefined!

Third Question: Why does my teles_cs.o knows nothing about
hisax_init_pcmcia which is also defined in config.c (not defined by me)
and defined external in teles_cs.c (which compiles fine):

Oct 6 22:27:06 Toral kernel: cs: IO port probe 0x0c00-0x0cff: clean.
Oct 6 22:27:06 Toral kernel: cs: IO port probe 0x0800-0x08ff: excluding
0x800-0x817 0x828-0x837 0x840-0x84f 0x860-0x877 0x880-0x88f 0x898-0x89f
0x8a8-0x8cf 0x8e0-0x8ff
Oct 6 22:27:06 Toral kernel: cs: IO port probe 0x0100-0x04ff: excluding
0x270-0x277 0x3f8-0x3ff 0x4d0-0x4d7
Oct 6 22:27:06 Toral kernel: cs: IO port probe 0x0a00-0x0aff: clean.
Oct 6 22:27:14 Toral kernel: cs: memory probe 0xa0000000-0xa0ffffff: clean.
Oct 6 22:27:16 Toral kernel: ISDN subsystem initialized
Oct 6 22:27:16 Toral kernel: hisax: Unknown symbol init_teles_cs
Oct 6 22:27:16 Toral kernel: hisax: Unknown symbol init_teles_cs
Oct 6 22:27:16 Toral kernel: teles_cs: Unknown symbol hisax_init_pcmcia

Back to 2nd Question: What is config.c for and where is it used?
Actually i've some ideas for what it is for, but the results are not as
expected by me (see above)...

Comments: IMHO some of the files in drivers/isdn/hisax will not be used.
Example: the Option CONFIG_HISAX_ELSA_CS will not be used outside
Kconfig and the Makefile so you will get an "HiSax: Support for %s Card
not selected\n" from config.c. Also some functions from elsa_cs.c are
outdated (IMHO) - i've tried to apply this file to my teles_cs.c which
resulted in some compile-errors...

Thx for any help,
Jan

-
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:49    [W:0.057 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site