On Thu, Jun 08, 2023 at 10:50:36AM +0200, Michal Simek wrote: > > > On 6/7/23 21:13, Tom Rini wrote: > > On Thu, May 25, 2023 at 09:32:59AM +0530, Venkatesh Yadav Abbarapu wrote: > > > > > From: Algapally Santosh Sagar <santoshsagar.algapa...@amd.com> > > > > > > The baudrate configured in .config is taken by default by serial. If > > > change of baudrate is required then the .config needs to changed and > > > u-boot recompilation is required or the u-boot environment needs to be > > > updated. > > > > > > To avoid this, support is added to fetch the baudrate directly from the > > > device tree file and update. > > > The serial, prints the log with the configured baudrate in the dtb. > > > The commit c4df0f6f315c ("arm: mvebu: Espressobin: Set default value for > > > $fdtfile env variable") is taken as reference for changing the default > > > environment variable. > > > > > > The default environment stores the default baudrate value, When default > > > baudrate and dtb baudrate are not same glitches are seen on the serial. > > > So, the environment also needs to be updated with the dtb baudrate to > > > avoid the glitches on the serial. > > > > > > Signed-off-by: Algapally Santosh Sagar <santoshsagar.algapa...@amd.com> > > > Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbar...@amd.com> > > > --- > > > doc/README.serial_dt_baud | 41 +++++++++++++++++++++++++++++++++ > > > > Sorry I missed this until v5. This needs to be in rST and under doc/ > > somewhere (and make htmldocs happy). > > Any other issue with implementation?
Nope, I noticed this while preparing to apply it to -next. -- Tom
signature.asc
Description: PGP signature