Re: [PATCH] New documentation

Riley Williams (rhw@MemAlpha.CX)
Wed, 2 Jun 1999 13:24:43 +0100 (GMT)


Hi Philip.

>> 4. The options relating to the PARPORT subsystem all have names
>> ending with a dot,

> They do not. The options relating to PARIDE look like they
> might consist of two parts separated by a dot ("pg.drive0=...")
> but the options for parport don't feature dots anywhere that I'm
> aware of.

Oooooopppsss...sorry, s/PARPORT/PARIDE/ 8-Z

Typo on my part when entering the email. However, the following is the
relevant section of init/main.c relating to these options...

===8<=== CUT ===>8===
#ifdef CONFIG_PARIDE_PD
{ "pd.", pd_setup },
#endif
#ifdef CONFIG_PARIDE_PCD
{ "pcd.", pcd_setup },
#endif
#ifdef CONFIG_PARIDE_PF
{ "pf.", pf_setup },
#endif
#ifdef CONFIG_PARIDE_PT
{ "pt.", pt_setup },
#endif
#ifdef CONFIG_PARIDE_PG
{ "pg.", pg_setup },
#endif
===8<=== CUT ===>8===

As you can see, all of the above options are listed as being a series
of letters terminated by a dot, nothing else being shown as following
the dot in any of those cases.

>>+ LPT Printer support is enabled.
>>+ parport= [HW,LP]
>>+ plip= [LP,NET] Parallel port network link.
>>+ lp= [LPT] Parallel Printer.

> If `LP' and `LPT' are supposed to be the same, this is bogus.

They were all meant to be LPT rather than LP. Also, I'm not quite sure
what you mean by "this is bogus", so if you could explain, it would be
appreciated.

Here's the patch to correct those errors you identified that I
understood.

--- kernparm.diff~ Wed Jun 2 12:05:54 1999
+++ kernparm.diff Wed Jun 2 13:19:59 1999
@@ -247,7 +247,7 @@
+
+ panic=
+
-+ parport= [HW,LP]
++ parport= [HW,LPT]
+
+ pas16= [HW,SCSI]
+
@@ -265,7 +265,7 @@
+
+ pirq= [SMP,APIC]
+
-+ plip= [LP,NET] Parallel port network link.
++ plip= [LPT,NET] Parallel port network link.
+
+ profile=
+

Best wishes from Riley.

+----------------------------------------------------------------------+
| There is something frustrating about the quality and speed of Linux |
| development, ie., the quality is too high and the speed is too high, |
| in other words, I can implement this XXXX feature, but I bet someone |
| else has already done so and is just about to release their patch. |
+----------------------------------------------------------------------+
* ftp://ftp.MemAlpha.cx/pub/rhw/Linux
* http://www.MemAlpha.cx/kernel.versions.html

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