RE: [PATCH] auxdisplay: remove PARPORT dependency

[hartleys]

On Monday, February 09, 2009 3:37 PM, Andrew Morton wrote:
> (please cc the maintainers on patches)

Sorry about missing that.  I wasn't sure how patches to Kconfig were
handled.

>> Remove PARPORT dependency for Auxiliary Display support.
>> 
>> This is not needed since the dependency for the KS0108 driver is 
>> PARPORT_PC.
>> 
>
> OK, but the patch also accidentally removes the parport dependency
> from cfag12864b.c and cfag12864bfb.c.
>
> A quick grep indicates that those drivers don't need parport, but I
> didn't look very closely.

I think the dependency will be handled automatically.  Both cfag12864b.c
and cfag12864bfb.c are enabled by CFAG12864B which has a dependency on
KS0108, which it turn depends on PARPORT_PC.

>> 
>> diff --git a/drivers/auxdisplay/Kconfig b/drivers/auxdisplay/Kconfig 
>> index 14b9d5f..c07e725 100644
>> --- a/drivers/auxdisplay/Kconfig
>> +++ b/drivers/auxdisplay/Kconfig
>> @@ -6,7 +6,6 @@
>>  #
>>  
>>  menuconfig AUXDISPLAY
>> -	depends on PARPORT
>>  	bool "Auxiliary Display support"
>>  	---help---
>>  	  Say Y here to get to see options for auxiliary display
drivers.
>> @@ -14,7 +13,7 @@ menuconfig AUXDISPLAY
>>  
>>  	  If you say N, all options in this submenu will be skipped and 
>> disabled.
>>  
>> -if AUXDISPLAY && PARPORT
>> +if AUXDISPLAY
>>  
>>  config KS0108
>>  	tristate "KS0108 LCD Controller" 
>
> Your email client wordwraps patches.  Please fix that up for next
time.

Sorry about that also.  Do you have any idea how to correctly configure
Outlook to not mess with wordwraps?  I know, it's a stupid client, but's
that's what I'm stuck with here at work.  I've been beating my head with
it trying to get it to work correctly.

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