[PATCH v3] zynq-uart: Fix set_attributes implementation

Gedare Bloom gedare at rtems.org
Thu Dec 3 16:42:53 UTC 2020


On Thu, Dec 3, 2020 at 8:32 AM Sebastian Huber <
sebastian.huber at embedded-brains.de> wrote:

> On 03/12/2020 16:25, Kinsey Moore wrote:
>
> > The zynq-uart set_attributes implementation was configured to always
> > return false which causes spconsole01 to fail. This restores the
> > disabled implementation which sets the baud rate registers
> > appropriately and allows spconsole01 to pass. This also expands the
> > set_attributes functionality to allow setting of the stop bits,
> > character width, and parity.
>
> The patch is fine from my point of view.
>
> The coding style is not in line with the RTEMS Project guidelines.
> Should we even review patches with respect to the coding style?
>
>
I don't keep it strict in the bsps/ area.

Hopefully, we can make tools help us ;)


> --
> embedded brains GmbH
> Herr Sebastian HUBER
> Dornierstr. 4
> 82178 Puchheim
> Germany
> email: sebastian.huber at embedded-brains.de
> phone: +49-89-18 94 741 - 16
> fax:   +49-89-18 94 741 - 08
>
> Registergericht: Amtsgericht München
> Registernummer: HRB 157899
> Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
> Unsere Datenschutzerklärung finden Sie hier:
> https://embedded-brains.de/datenschutzerklaerung/
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20201203/ca1d76bf/attachment-0001.html>


More information about the devel mailing list