[PATCH v5 0/2] Add support to fetch baudrate from dtb

[Michal Simek]

Hi Simon,

On 5/25/23 06:02, Venkatesh Yadav Abbarapu wrote:
> In this patch series
> - Add support in Kconfig and convert for armada boards
> - Fetch baudrate from the dtb and update
> 
> Changes in v5:
> - Adding DEFAULT_ENV_IS_RW Kconfig in missing files
> - Updating DEFAULT_ENV_IS_RW to CONFIG_DEFAULT_ENV_IS_RW
> 
> Changes in v4:
> - Moved SERIAL_DT_BAUD to another patch
> - Added doc file for fetching serial baudrate from DT.
> - Changed Kconfig SERIAL_DT_BAUD to OF_SERIAL_DT_BAUD
> - Added function docs wherever required.
> - Moved changes from fdtdec api to ofnode
> - Changed serial_get_valid_baudrate to check_valid_baudrate
> - Added function fetch_baud_from_dtb to fetch baud from DT
> - Used dectoul() for parsing baudrate
> 
> Changes in v3:
> - Add SERIAL_DT_BAUD to Kconfig
> - Moved DEFAULT_ENV_IS_RW to Kconfig also updated armada files
> - Moved filler changes from zynqmp.h to generic file env_default.h
> - Removed ENV_RW_FILLER and added padding in the generic file env_default.h.
> - Print baudrate parameter properly when SERIAL_DT is enabled.
> 
> Changes in v2:
> - Changed to #ifdef from #if CONFIG_IS_ENABLED to enable patching in
> spl.
> - Added SPL_ENV_SUPPORT dependency in SERIAL_DT_BAUD to allow SPL
> compilation.
> - Moved DEFAULT_ENV_IS_RW to Kconfig also updated armada files
> - Moved ENV_RW_FILLER to generic file env_default.h.
> - Increased the ENV_RW_FILLER padding to support 8000000 baud.
> 
> Algapally Santosh Sagar (2):
>    configs: Add support in Kconfig and convert for armada boards
>    serial: zynqmp: Fetch baudrate from dtb and update
> 
>   configs/eDPU_defconfig                      |  1 +
>   configs/mvebu_db-88f3720_defconfig          |  1 +
>   configs/mvebu_espressobin-88f3720_defconfig |  1 +
>   configs/uDPU_defconfig                      |  1 +
>   doc/README.serial_dt_baud                   | 41 ++++++++++++++++++++
>   drivers/core/ofnode.c                       | 20 ++++++++++
>   drivers/serial/Kconfig                      | 15 ++++++++
>   drivers/serial/serial-uclass.c              | 42 +++++++++++++++++++++
>   include/configs/mvebu_armada-37xx.h         |  1 -
>   include/dm/ofnode.h                         | 14 ++++++-
>   include/env_default.h                       |  8 +++-
>   include/env_internal.h                      |  2 +-
>   include/serial.h                            | 15 ++++++++
>   13 files changed, 156 insertions(+), 6 deletions(-)
>   create mode 100644 doc/README.serial_dt_baud
> 

as you commented previous series. Can you please comment these patches?

Thanks,
Michal