[U-Boot] [PATCH] serial: serial_x86: add ns16550a compatible ids

Add ns16550a compatible ids, so that other arch may use this standard driver.
Signed-off-by: Thomas Chou thomas@wytron.com.tw --- drivers/serial/serial_x86.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/serial/serial_x86.c b/drivers/serial/serial_x86.c index 4bf6062..f378eee 100644 --- a/drivers/serial/serial_x86.c +++ b/drivers/serial/serial_x86.c @@ -14,6 +14,8 @@ DECLARE_GLOBAL_DATA_PTR;
static const struct udevice_id x86_serial_ids[] = { { .compatible = "x86-uart" }, + { .compatible = "ns16550a" }, + { .compatible = "ns16550" }, { } };

Hi Simon,
On 2015年11月09日 16:53, Thomas Chou wrote:
Add ns16550a compatible ids, so that other arch may use this standard driver.
Signed-off-by: Thomas Chou thomas@wytron.com.tw
drivers/serial/serial_x86.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/serial/serial_x86.c b/drivers/serial/serial_x86.c index 4bf6062..f378eee 100644 --- a/drivers/serial/serial_x86.c +++ b/drivers/serial/serial_x86.c @@ -14,6 +14,8 @@ DECLARE_GLOBAL_DATA_PTR;
static const struct udevice_id x86_serial_ids[] = { { .compatible = "x86-uart" },
- { .compatible = "ns16550a" },
- { .compatible = "ns16550" }, { } };
I want to use serial_x86.c for nios2. I think there is no need to duplicate another one for altera/nios2. Is there a proper way to change Kconfig/Makefile?
Best regards, Thomas

Hi Thomas,
On Mon, Nov 9, 2015 at 5:17 PM, Thomas Chou thomas@wytron.com.tw wrote:
Hi Simon,
On 2015年11月09日 16:53, Thomas Chou wrote:
Add ns16550a compatible ids, so that other arch may use this standard driver.
Signed-off-by: Thomas Chou thomas@wytron.com.tw
drivers/serial/serial_x86.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/serial/serial_x86.c b/drivers/serial/serial_x86.c index 4bf6062..f378eee 100644 --- a/drivers/serial/serial_x86.c +++ b/drivers/serial/serial_x86.c @@ -14,6 +14,8 @@ DECLARE_GLOBAL_DATA_PTR;
static const struct udevice_id x86_serial_ids[] = { { .compatible = "x86-uart" },
{ .compatible = "ns16550a" },
};{ .compatible = "ns16550" }, { }
I want to use serial_x86.c for nios2. I think there is no need to duplicate another one for altera/nios2. Is there a proper way to change Kconfig/Makefile?
Please check my previous proposal on serial driver clean up.
http://lists.denx.de/pipermail/u-boot/2015-August/223758.html
Regards, Bin

Hi Bin,
On 2015年11月09日 20:03, Bin Meng wrote:
I want to use serial_x86.c for nios2. I think there is no need to duplicate another one for altera/nios2. Is there a proper way to change Kconfig/Makefile?
Please check my previous proposal on serial driver clean up.
http://lists.denx.de/pipermail/u-boot/2015-August/223758.html
Thanks a lot for the pointer.
I think your suggestion is more practical. We can add clock-frequency to those have fixed clock. Then merge them to one driver.
For those need dynamic clock like tegra, we may ether leave them along. Or we can use the .data of compatible ids to fetch the clock.
We will need this soon.
Best regards, Thomas

+Tom, Stephen
Hi,
On 9 November 2015 at 05:19, Thomas Chou thomas@wytron.com.tw wrote:
Hi Bin,
On 2015年11月09日 20:03, Bin Meng wrote:
I want to use serial_x86.c for nios2. I think there is no need to duplicate another one for altera/nios2. Is there a proper way to change Kconfig/Makefile?
Please check my previous proposal on serial driver clean up.
http://lists.denx.de/pipermail/u-boot/2015-August/223758.html
Thanks a lot for the pointer.
I think your suggestion is more practical. We can add clock-frequency to those have fixed clock. Then merge them to one driver.
Sounds good.
For those need dynamic clock like tegra, we may ether leave them along. Or we can use the .data of compatible ids to fetch the clock.
Yes I think we should just add the property. It's silly to expect SPL to parse the entire clock logic in the DT. We can leave Tegra on its own for now.
We will need this soon.
Regards. Simon
participants (3)
-
Bin Meng
-
Simon Glass
-
Thomas Chou