[PATCH v2 2/5] configs: npcm: support more uart baud rate

[Jim Liu]

Hi Tom

Thanks for your review.
Google and other customers need to change baud rate so I added this table.

I will modify the name to CFG_SYS_BAUDRATE_TABLE.

Best regards,
Jim


On Wed, Oct 11, 2023 at 7:20 AM Tom Rini <trini at konsulko.com> wrote:
>
> On Wed, Oct 11, 2023 at 04:45:30PM +0800, Jim Liu wrote:
> > add uart baud rate table to arbel(npcm8xx) and poleg(npcm7xx)
> >
> > Signed-off-by: Jim Liu <JJLIU0 at nuvoton.com>
> > ---
> >  include/configs/arbel.h | 5 ++++-
> >  include/configs/poleg.h | 2 ++
> >  2 files changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/include/configs/arbel.h b/include/configs/arbel.h
> > index 891257bc93..c45b2ac137 100644
> > --- a/include/configs/arbel.h
> > +++ b/include/configs/arbel.h
> > @@ -7,10 +7,13 @@
> >  #define __CONFIG_ARBEL_H
> >
> >  #define CFG_SYS_SDRAM_BASE           0x0
> > -#define CFG_SYS_BOOTMAPSZ            (30 << 20)
> > +#define CFG_SYS_BOOTMAPSZ            (128 << 20)
> >  #define CFG_SYS_BOOTM_LEN            (20 << 20)
> >  #define CFG_SYS_INIT_RAM_ADDR        CFG_SYS_SDRAM_BASE
> >  #define CFG_SYS_INIT_RAM_SIZE        0x8000
> > +#define CONFIG_SYS_BAUDRATE_TABLE    { 9600, 14400, 19200, 38400, 57600, 115200, 230400, \
> > +                                       380400, 460800, 921600 }
> > +
> >
> >  /* Default environemnt variables */
> >  #define CFG_EXTRA_ENV_SETTINGS   "uimage_flash_addr=80400000\0"   \
> > diff --git a/include/configs/poleg.h b/include/configs/poleg.h
> > index 1e96e838be..fd0e9a7362 100644
> > --- a/include/configs/poleg.h
> > +++ b/include/configs/poleg.h
> > @@ -13,6 +13,8 @@
> >  #define CFG_SYS_BOOTMAPSZ            (0x30 << 20)
> >  #define CFG_SYS_SDRAM_BASE           0x0
> >
> > +#define CONFIG_SYS_BAUDRATE_TABLE    { 57600, 115200, 230400, 460800 }
> > +
> >  /* Default environemnt variables */
> >  #define CFG_EXTRA_ENV_SETTINGS   "uimage_flash_addr=80200000\0"   \
> >               "stdin=serial\0"   \
>
> This is now CFG_SYS_BAUDRATE_TABLE, do you really need something other
> than the default as well?
>
> --
> Tom