On 27 December 2014 at 05:10, Bin Meng <bmeng...@gmail.com> wrote:
> Use ePAPR defined properties for x86-uart: clock-frequency and
> current-speed. Assign the value of clock-frequency in device tree
> to plat->clock of x86-uart instead of using hardcoded number.
>
> Signed-off-by: Bin Meng <bmeng...@gmail.com>
>
> ---
>
> Changes in v2:
> - New patch to use ePAPR defined properties for x86-uart
>
> arch/x86/dts/serial.dtsi | 5 ++---
> drivers/serial/serial_x86.c | 8 +++++++-
> 2 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/arch/x86/dts/serial.dtsi b/arch/x86/dts/serial.dtsi
> index ebdda76..9b097f4 100644
> --- a/arch/x86/dts/serial.dtsi
> +++ b/arch/x86/dts/serial.dtsi
> @@ -3,8 +3,7 @@
> compatible = "x86-uart";
> reg = <0x3f8 8>;
> reg-shift = <0>;
> - io-mapped = <1>;
> - multiplier = <1>;
> - baudrate = <115200>;
> + clock-frequency = <1843200>;
> + current-speed = <115200>;
Where is current speed used? If it is needed, please update the
binding at doc/device-tree-bindings/serial/ns16550.txt
> };
> };
> diff --git a/drivers/serial/serial_x86.c b/drivers/serial/serial_x86.c
> index e81e035..4bf6062 100644
> --- a/drivers/serial/serial_x86.c
> +++ b/drivers/serial/serial_x86.c
> @@ -6,9 +6,12 @@
>
> #include <common.h>
> #include <dm.h>
> +#include <fdtdec.h>
> #include <ns16550.h>
> #include <serial.h>
>
> +DECLARE_GLOBAL_DATA_PTR;
> +
> static const struct udevice_id x86_serial_ids[] = {
> { .compatible = "x86-uart" },
> { }
> @@ -22,10 +25,13 @@ static int x86_serial_ofdata_to_platdata(struct udevice
> *dev)
> ret = ns16550_serial_ofdata_to_platdata(dev);
> if (ret)
> return ret;
> - plat->clock = 1843200;
> +
> + plat->clock = fdtdec_get_int(gd->fdt_blob, dev->of_offset,
> + "clock-frequency", 1843200);
>
> return 0;
> }
> +
> U_BOOT_DRIVER(serial_ns16550) = {
> .name = "serial_x86",
> .id = UCLASS_SERIAL,
> --
> 1.8.2.1
>
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
