
Hi Simon,
On Sun, Dec 28, 2014 at 9:55 AM, Simon Glass sjg@chromium.org wrote:
On 27 December 2014 at 05:10, Bin Meng bmeng.cn@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.cn@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
The current-speed is the baud rate of the serial port. Currently it is put there for ePAPR completeness and not used by U-Boot as U-Boot is using gd->baudrate for the serial console.
[snip]
Regards, Bin