Message-ID: <BAY103-F3438FAEB6A51CC5567D48DDF2B0@phx.gbl>
In-Reply-To: <Pine.LNX.4.61.0512311830030.7910@yvahk01.tjqt.qr>
From: "Jason Dravet" <dravet@hotmail.com>
To: jengelh@linux01.gwdg.de
Cc: linux-kernel@vger.kernel.org, linux-parport@lists.infradead.org
Subject: Re: RFC: add udev support to parport_pc
Date: Sat, 31 Dec 2005 12:07:47 -0600
Mime-Version: 1.0
Content-Type: text/plain; format=flowed
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1757
Lines: 47

Thank you for the reply.  Comments inline.

>I would prefer to actually see == 0x378 in the code, because the
>hexademical number is what you see everywhere else, such as the BIOS POST
>and /proc/ioports. This also applies to 0x278 and 0x3BC below.
>
This is what I wanted but I could not figure out how do it.  If you tell me 
how I will be happy to change it.  I tried  if (p->base == 0x378) but then 
class_device_create does not get executed.

> > +	{
> > +		class_device_create(parallel_class, NULL, MKDEV(6, 0), NULL,
> > "lp0");
> > +		class_device_create(parallel_class, NULL, MKDEV(99, 0), NULL,
> > "parport0");
> > +	}
>
>Background info before: Because I burnt my on-board LPT port (applying too
>much volts or milliamps), I bought a dual-slot PCI add-in card. This card
>provides "parport1" and "parport2" at ports at 0xC800 and 0xC00
>(/proc/ioports).
>
The last experience I have with off board cards was about 5 years ago.  The 
choices for the two parallel ports were 378, 278, or 3BC.  I was not aware 
that you had flexibility now.
>
>There are a number of problems in your code:
>
>1- testing just for 0x378/0x278/0x3BC is not enough
>
>2- parport0 could be 0xC800 (address may vary) if you do not
>    have any onboard LPT ports.
>     2=> that is why I think you should not reserver "lp0"/"parport0"
>         for 0x378.
As I said above I was not aware todays off board parallel ports had more 
choices.  I will see what I can do to fix this.  Do you have any 
suggestions?

Thanks,
Jason


-
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/