linux-kernel - RE: [PATCH] auxdisplay: remove PARPORT dependency

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]

Date:	Mon, 9 Feb 2009 17:46:05 -0500
From:	"hartleys" <hartleys@...ionengravers.com>
To:	"Andrew Morton" <akpm@...ux-foundation.org>
Cc:	<linux-kernel@...r.kernel.org>, <miguel.ojeda.sandonis@...il.com>
Subject: RE: [PATCH] auxdisplay: remove PARPORT dependency

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/