[U-Boot] [PATCH v2 6/7] x86: Use ePAPR defined properties for x86-uart

[Bin Meng]

Hi Simon,

On Sun, Dec 28, 2014 at 9:55 AM, Simon Glass <sjg at chromium.org> wrote:
> On 27 December 2014 at 05:10, Bin Meng <bmeng.cn at 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 at 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