Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-25 Thread Pali Rohár
On Friday 24 September 2021 18:07:36 Tom Rini wrote: > On Fri, Sep 24, 2021 at 09:08:15PM +0200, Pali Rohár wrote: > > Hello! > > > > On Monday 13 September 2021 18:11:38 Tom Rini wrote: > > > On Mon, Sep 13, 2021 at 05:03:13PM -0500, Alex G. wrote: > > > > > > > > > > > > On 9/13/21 4:24 PM, To

Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-24 Thread Tom Rini
On Fri, Sep 24, 2021 at 09:08:15PM +0200, Pali Rohár wrote: > Hello! > > On Monday 13 September 2021 18:11:38 Tom Rini wrote: > > On Mon, Sep 13, 2021 at 05:03:13PM -0500, Alex G. wrote: > > > > > > > > > On 9/13/21 4:24 PM, Tom Rini wrote: > > > > In order to move CONFIG_SYS_BAUDRATE_TABLE to K

Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-24 Thread Pali Rohár
Hello! On Monday 13 September 2021 18:11:38 Tom Rini wrote: > On Mon, Sep 13, 2021 at 05:03:13PM -0500, Alex G. wrote: > > > > > > On 9/13/21 4:24 PM, Tom Rini wrote: > > > In order to move CONFIG_SYS_BAUDRATE_TABLE to Kconfig, we need to rework > > > the logic a bit. Rename the users of CONFIG

Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-13 Thread Tom Rini
On Mon, Sep 13, 2021 at 05:03:13PM -0500, Alex G. wrote: [snip] > Suggestion II: Define the Kconfig SYS_BAUDRATE_TABLE table to a C array, > like 'default "{ 300, 420, 690}" ' and forego the #ifdefs in serial.h Oh! And FWIW, I don't _think_ this works. I'd be quite happy to be proven wrong, but

Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-13 Thread Tom Rini
On Mon, Sep 13, 2021 at 05:03:13PM -0500, Alex G. wrote: > > > On 9/13/21 4:24 PM, Tom Rini wrote: > > In order to move CONFIG_SYS_BAUDRATE_TABLE to Kconfig, we need to rework > > the logic a bit. Rename the users of CONFIG_SYS_BAUDRATE_TABLE to > > SYS_BAUDRATE_TABLE. Introduce a series of CON

Re: [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-13 Thread Alex G.
On 9/13/21 4:24 PM, Tom Rini wrote: In order to move CONFIG_SYS_BAUDRATE_TABLE to Kconfig, we need to rework the logic a bit. Rename the users of CONFIG_SYS_BAUDRATE_TABLE to SYS_BAUDRATE_TABLE. Introduce a series of CONFIG_BAUDRATE_TABLE_... that include some number of baud rates. These ma

[PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE

2021-09-13 Thread Tom Rini
In order to move CONFIG_SYS_BAUDRATE_TABLE to Kconfig, we need to rework the logic a bit. Rename the users of CONFIG_SYS_BAUDRATE_TABLE to SYS_BAUDRATE_TABLE. Introduce a series of CONFIG_BAUDRATE_TABLE_... that include some number of baud rates. These match all existing users. The help for each