RE: [PATCH] auxdisplay: remove PARPORT dependency

From: hartleys
Date: Wed Feb 11 2009 - 11:52:34 EST


On Wednesday, February 11, 2009 3:38 AM, Miguel Ojeda wrote:
>> 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.
>
> That's true; however, I think it is intentional: cfag12864bfb
> depends on cfags12864b, and this one depends on ks0108, which
> depends on parport and, for the moment, there aren't more entries
> on "Auxiliary Display support". Thereafter, if PARPORT is not
> selected, the user is not bothered with an additional menu entry
> on "Device Drivers".
>
> I don't know which is the trend nowadays in this subject (i.e. to
> show every menu althought empty or not). I see Andrew has already
> added the patch, so I bet on his decision ;)

The arm branch Kconfig was updated awhile back to include auxdisplay.

I have been messing with a couple LCD devices similar to the cfag12864.
Every time I update my kernel source I have needed to add this patch
since my arm platform does not have PARPORT_PC.

That was my stimulus for the patch. It does cause the empty menu to
appear but some others already do that so I didn't think it was a
problem.

Regards,
Hartley
--
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/