[PATCH 1/2] tty/serial: Update of_serial's XSCALE port type to be generic
Grant Likely
grant.likely at secretlab.ca
Fri May 20 05:11:11 EST 2011
On Thu, May 12, 2011 at 10:41:13AM -0600, Stephen Warren wrote:
> Instead of adding a specific nvidia,tegra250-uart entry, add a generic
> xscale entry that can be used by any XSCALE-compatible device. This avoids
> us having to add a new entry per vendor to of_serial.c, just like all the
> other already extant entries.
>
> I assume this patch will be squashed into or replace
> b9e7db240617f12dd371b0a363042fb007e97ed7
> tty/serial: Add devicetree support for nVidia Tegra serial ports
>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> ---
> drivers/tty/serial/of_serial.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/tty/serial/of_serial.c b/drivers/tty/serial/of_serial.c
> index 0d3f138..bfc1c2e 100644
> --- a/drivers/tty/serial/of_serial.c
> +++ b/drivers/tty/serial/of_serial.c
> @@ -168,7 +168,7 @@ static struct of_device_id __devinitdata of_platform_serial_table[] = {
> { .compatible = "ns16550", .data = (void *)PORT_16550, },
> { .compatible = "ns16750", .data = (void *)PORT_16750, },
> { .compatible = "ns16850", .data = (void *)PORT_16850, },
> - { .compatible = "nvidia,tegra250-uart", .data = (void *)PORT_XSCALE, },
> + { .compatible = "xscale", .data = (void *)PORT_XSCALE, },
As previously discussed (and repeating here just for the historical
record), this isn't really the right thing since the port isn't
actually an Xscale port and has a subtly different behaviour. The
solution discussed in our other thread about the tegra serial port
should be pursued.
g.
More information about the devicetree-discuss
mailing list