linux-kernel - Re: [PATCH -mm] char: drivers use/need PCI

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <456DB524.8000008@gmail.com>
Date:	Wed, 29 Nov 2006 17:28:20 +0100
From:	Jiri Slaby <jirislaby@...il.com>
To:	Randy Dunlap <randy.dunlap@...cle.com>
CC:	lkml <linux-kernel@...r.kernel.org>, akpm <akpm@...l.org>
Subject: Re: [PATCH -mm] char: drivers use/need PCI

Randy Dunlap wrote:
> Jiri Slaby wrote:
>> Randy Dunlap wrote:
>>> From: Randy Dunlap <randy.dunlap@...cle.com>
>>>
>>> With CONFIG_PCI=n:
>>> drivers/char/mxser_new.c: In function 'mxser_release_res':
>>> drivers/char/mxser_new.c:2383: warning: implicit declaration of
>>> function 'pci_release_region'
>>> drivers/char/mxser_new.c: In function 'mxser_probe':
>>> drivers/char/mxser_new.c:2578: warning: implicit declaration of
>>> function 'pci_request_region'
>>> drivers/built-in.o: In function `sx_remove_card':
>>> sx.c:(.text.sx_remove_card+0x65): undefined reference to
>>> `pci_release_region'
>>> drivers/char/isicom.c: In function 'isicom_probe':
>>> drivers/char/isicom.c:1793: warning: implicit declaration of function
>>> 'pci_request_region'
>>> drivers/char/isicom.c:1827: warning: implicit declaration of function
>>> 'pci_release_region'
>>>
>>> Signed-off-by: Randy Dunlap <randy.dunlap@...cle.com>
>>> ---
>>>  drivers/char/Kconfig |    6 +++---
>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> --- linux-2.6.19-rc6-mm2.orig/drivers/char/Kconfig
>>> +++ linux-2.6.19-rc6-mm2/drivers/char/Kconfig
>>> @@ -203,7 +203,7 @@ config MOXA_SMARTIO
>>>  
>>>  config MOXA_SMARTIO_NEW
>>>      tristate "Moxa SmartIO support v. 2.0 (EXPERIMENTAL)"
>>> -    depends on SERIAL_NONSTANDARD
>>> +    depends on SERIAL_NONSTANDARD && PCI
>>>      help
>>>        Say Y here if you have a Moxa SmartIO multiport serial card
>>> and/or
>>>        want to help develop a new version of this driver.
>>> @@ -218,7 +218,7 @@ config MOXA_SMARTIO_NEW
>>>  
>>>  config ISI
>>>      tristate "Multi-Tech multiport card support (EXPERIMENTAL)"
>>> -    depends on SERIAL_NONSTANDARD
>>> +    depends on SERIAL_NONSTANDARD && PCI
>>>      select FW_LOADER
>>>      help
>>>        This is a driver for the Multi-Tech cards which provide several
>>> @@ -312,7 +312,7 @@ config SPECIALIX_RTSCTS
>>>  
>>>  config SX
>>>      tristate "Specialix SX (and SI) card support"
>>> -    depends on SERIAL_NONSTANDARD
>>> +    depends on SERIAL_NONSTANDARD && PCI
>>>      help
>>>        This is a driver for the SX and SI multiport serial cards.
>>>        Please read the file <file:Documentation/sx.txt> for details.
>>
>> Nack. I have to correct the mxser and sx code. Thanks,
> 
> Sure, either way is OK.  Thanks.

But those drivers support ISA devices too. Ok then, let "&& PCI" be as a correct
temporary way and I'll add "|| ISA" after the proposed code fix :).

regards,
-- 
http://www.fi.muni.cz/~xslaby/            Jiri Slaby
faculty of informatics, masaryk university, brno, cz
e-mail: jirislaby gmail com, gpg pubkey fingerprint:
B674 9967 0407 CE62 ACC8  22A0 32CC 55C3 39D4 7A7E
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/