[PATCH v3] zynq-uart: Fix set_attributes implementation

[Kinsey Moore]

From: Gedare Bloom <gedare at rtems.org>
Sent: Thursday, December 3, 2020 10:43
To: Sebastian Huber <sebastian.huber at embedded-brains.de>
Cc: Kinsey Moore <kinsey.moore at oarcorp.com>; devel at rtems.org
Subject: Re: [PATCH v3] zynq-uart: Fix set_attributes implementation



On Thu, Dec 3, 2020 at 8:32 AM Sebastian Huber <sebastian.huber at embedded-brains.de<mailto: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 ;)

I was mostly trying to match similar code in the same area of the codebase, but I’ll do some cleanup as per Gedare’s other email and submit a hopefully final version. I look forward to having a pre-commit hook that yells at me for improper code formatting (or just fixes it blindly).

Kinsey
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20201203/e061479b/attachment-0001.html>