Hi,
> * Girish. S. G. <[EMAIL PROTECTED]> [080822 15:42]:
>> Tony,
>>
>> > Hi,
>> >
arch/arm/configs/omap_3430sdp_defconfig | 12
>> >> arch/arm/mach-omap2/serial.c | 166 ++---
>> >> drivers/serial/Kconfig | 23
>> >> drivers/serial/Makefile | 1
>> >> drivers/serial/omap-serial.c | 887
>> >> ++++++++++++++++++++++++++++++++
>> >> include/linux/serial_core.h | 3
>> -static struct platform_device serial_device = {
>> >> - .name = "serial8250",
>> >> - .id = PLAT8250_DEV_PLATFORM,
>> >> - .dev = {
>> >> - .platform_data = serial_platform_data,
>> >> - },
>> >> -};
>> >
>> > What about powering UART on/off? I suggest you provide set_power()
>> > function in platform_data. That way the UART power function can be
>> > generic on later omaps, or external like the FPGA on omap1510.
>>
>> There isn't any specific power on/off for UART as such(or any on omap1510?).
>> But yes we can have all the clk and pm related functions to be grouped under
>> uart_ops->pm().
>
> Yeah I believe it's an external power for early omaps. So some board
> specific function pointer should be available for that.
Ok, this can be done.
>
>>
>> AFAIK using ttyS name will break hot-plug 8250 ports, such as CF ports.
>> > How about using ttyO or something similar? I guess the minor number also
>> > needs to be different then.
>> >
>> > In general, will this driver also work on DaVinci? Maybe use name like
>> > serial_ti or similar?
>>
>> I don't think the name would conflict as only our uart instance would be up
>> and
>> working. Right now this driver is supported and tested only on OMAP2/OMAP3.
>> But
>> yes, once it's in it can be easily ported on other TI platforms.
>
> The thing is this driver won't be accepted upstream if it conflicts
> with hot-pluggable 8250 ports. AFAIK, you need to get a new id for it,
> and call it something like ttyO instead.
In that case there should be a new id. I'll check on this and modify it
accordingly.
Regards,
Girish
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
