[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5028AB81.8040801@suse.cz>
Date: Mon, 13 Aug 2012 09:23:45 +0200
From: Jiri Slaby <jslaby@...e.cz>
To: Rusty Russell <rusty@...abs.org>
CC: gregkh@...uxfoundation.org, alan@...ux.intel.com,
linux-kernel@...r.kernel.org, jirislaby@...il.com
Subject: Re: [PATCH 12/41] mxser: allow overlapping vector
On 08/13/2012 03:50 AM, Rusty Russell wrote:
> On Tue, 7 Aug 2012 21:47:37 +0200, Jiri Slaby <jslaby@...e.cz> wrote:
>> For many cards, this saves some IO space because interrupt status port
>> has precedence over the rest of ports on the card. Hence it can be
>> mapped to a hole in I/O ports.
>>
>> Here we add a kernel parameter which allows that if a user wants to.
>> But they need to explicitly enable it by a module parameter.
>>
>> Signed-off-by: Jiri Slaby <jslaby@...e.cz>
>> ---
>> drivers/tty/mxser.c | 35 +++++++++++++++++++++++++++++++----
>> 1 file changed, 31 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/tty/mxser.c b/drivers/tty/mxser.c
>> index c162ee9..e64fe40 100644
>> --- a/drivers/tty/mxser.c
>> +++ b/drivers/tty/mxser.c
>> @@ -2337,11 +2337,36 @@ static struct tty_port_operations mxser_port_ops = {
>> * The MOXA Smartio/Industio serial driver boot-time initialization code!
>> */
>>
>> +static bool allow_overlapping_vector;
>> +module_param(allow_overlapping_vector, bool, 444);
>
> You mean 0444, right?
Of course I did. I wonder why patch posters like numbers instead of
names. I will change this to S_IRUGO.
thanks,
--
js
suse labs
--
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/
Powered by blists - more mailing lists