[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 18/37] mips: baudbase is 115200 by default
From: |
Aleksandar Markovic |
Subject: |
Re: [PATCH v4 18/37] mips: baudbase is 115200 by default |
Date: |
Mon, 25 Nov 2019 13:22:07 +0100 |
On Mon, Nov 25, 2019 at 12:26 PM Philippe Mathieu-Daudé
<address@hidden> wrote:
>
> On 11/25/19 11:12 AM, Marc-André Lureau wrote:
> > Hi
> >
> > On Mon, Nov 25, 2019 at 2:07 PM Aleksandar Markovic
> > <address@hidden> wrote:
> >>
> >>
> >>
> >> On Wednesday, November 20, 2019, Marc-André Lureau <address@hidden> wrote:
> >>>
> >>> Signed-off-by: Marc-André Lureau <address@hidden>
> >>> ---
> >>> hw/mips/mips_mipssim.c | 1 -
> >>> 1 file changed, 1 deletion(-)
> >>>
> >>> diff --git a/hw/mips/mips_mipssim.c b/hw/mips/mips_mipssim.c
> >>> index bfafa4d7e9..3cd0e6eb33 100644
> >>> --- a/hw/mips/mips_mipssim.c
> >>> +++ b/hw/mips/mips_mipssim.c
> >>> @@ -223,7 +223,6 @@ mips_mipssim_init(MachineState *machine)
> >>> if (serial_hd(0)) {
> >>> DeviceState *dev = qdev_create(NULL, TYPE_SERIAL_IO);
> >>>
> >>> - qdev_prop_set_uint32(DEVICE(dev), "baudbase", 115200);
> >>> qdev_prop_set_chr(dev, "chardev", serial_hd(0));
> >>> qdev_set_legacy_instance_id(dev, 0x3f8, 2);
> >>> qdev_init_nofail(dev);
> >>> --
> >>
> >>
> >> Please mention in your commit message where the default baudbase is set.
> >
> > ok
> >
> >> Also, is there a guarantie that default value 115200 will never change in
> >> future?
> >
> > The level of stability on properties in general is unclear to me.
> >
> > Given that 115200 is standard for serial, it is unlikely to change
> > though.. We can have an assert there instead?
> >
> > Peter, what do you think? thanks
>
> This property confused me by the past. It is _not_ the baudrate.
The name is "boudbase" (whatever that means), so not "boudrate".
Can we perhaps track the "inventor" of the property?
Google search for the word "boudbase" gives me an address of a person
in Maryland, whose last name is Boudbase, and also another address of
apparently the same person in Fresno, CA. No serial-device-related
results occurred (within the first page of Google results, at least).
Sincerely,
Aleksandar
> It is the input frequency clocking the UART ('XIN' pin, Xtal INput).
>
> Each board has its own frequency, and it can even be variable (the clock
> domain tree can reconfigure it at a different rate).
>
> I'm not sure it makes sense to have a default, and I don't know what is
> the frequency modeled by the SPIM simulator.
>
- Re: [PATCH v4 17/37] mips: inline serial_init(), (continued)
Re: [PATCH v4 17/37] mips: inline serial_init(), Aleksandar Markovic, 2019/11/27
[PATCH v4 18/37] mips: baudbase is 115200 by default, Marc-André Lureau, 2019/11/20
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Peter Maydell, 2019/11/21
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Aleksandar Markovic, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Marc-André Lureau, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default,
Aleksandar Markovic <=
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Philippe Mathieu-Daudé, 2019/11/25
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Marc-André Lureau, 2019/11/27
- Re: [PATCH v4 18/37] mips: baudbase is 115200 by default, Aleksandar Markovic, 2019/11/27
[PATCH v4 19/37] mips: use sysbus_add_io(), Marc-André Lureau, 2019/11/20
[PATCH v4 20/37] mips: use sysbus_mmio_get_region() instead of internal fields, Marc-André Lureau, 2019/11/20