Messages in this thread | | | From | Arnd Bergmann <> | Subject | Re: [PATCH] serial: Add Cavium OCTEON UART definitions. | Date | Tue, 7 Oct 2008 11:34:50 +0200 |
| |
On Tuesday 07 October 2008, David Daney wrote: > > - up->port.type = PORT_16550A; > +#ifdef CONFIG_CPU_CAVIUM_OCTEON > + /* UPF_FIXED_PORT indicates an internal UART. */ > + if (up->port.flags & UPF_FIXED_PORT) > + up->port.type = PORT_OCTEON; > + else > +#endif > + up->port.type = PORT_16550A; > +
This looks somewhat wrong, IMHO a device driver should not assume that a CONFIG_CPU_* symbol is exclusive. You could have (maybe not now, but in the future) a kernel that supports running on an Octeon as well as some other Mips64 processor, and have UPF_FIXED_PORT uart on some other machine, which will make the kernel think it is a PORT_OCTEON.
Arnd <>< -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |