Re: [PATCH 2.6.19-rc1 full] drivers: add LCD support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/23/06, Franck Bui-Huu <[email protected]> wrote:
Franck Bui-Huu wrote:
> Hi
>
>
> sorry for coming lately, I just noticed your patch in -mm tree.
>
> Did you took a look at drivers/video/arcfb.c driver ?
> It seems to be a fb driver for ks108 lcd controller. A lot of code
> is related to the platform though and the controller is driven
> through GPIO but have you tried to split the code for sharing
> controller specific code. Note, I don't say it's a good idea, it's
> just a question that comes in mind.
>
> Also you create driver/auxdisplay directory whereas drivers such
> the one I mentioned previously is located in drivers/video. Why
> not putting your driver in driver/video ?

Is this driver is already no more maintained ?


The driver is waiting in the -mm tree (-mm2 right now) for being
included in the mainline kernel sometime in the future. If it is
included, I will maintain it as I coded it as it apears in the
MAINTAINERS file. Why are you so worried about it if I can ask? Do you
want some more features or something like that?

I missed the other two questions you wrote few days ago. About the
second one, that was discussed a lot in the past and the people
decided that (it wasn't my idea). About the first one, well, my ks0108
code is the one for the wiring of an auxiliary LCD, so if you read the
discussion you will find the people wanted to split video things and
other auxiliary displays, so I think it is better to split it.
(Anyway, I'm answering quickly, I haven't checked the code you talk
about, but I will anyway).

Thanks,
    Miguel Ojeda
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux