[PATCH v5 2/2] serial: zynqmp: Fetch baudrate from dtb and update

[Tom Rini]

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.algapally at 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.algapally at amd.com>
> > > Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230608/2136483f/attachment.sig>