<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 27, 2019 at 1:10 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 26/03/2019 15:17, Lou Woods wrote:<br>
> From: Lou Woods <Lou.Woods@OARCorp.com><br>
><br>
> ---<br>
>   bsps/arm/xilinx-zynq/console/zynq-uart.c | 16 +++++++++++-----<br>
>   1 file changed, 11 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/bsps/arm/xilinx-zynq/console/zynq-uart.c b/bsps/arm/xilinx-zynq/console/zynq-uart.c<br>
> index fa91f3f..9c21f6f 100644<br>
> --- a/bsps/arm/xilinx-zynq/console/zynq-uart.c<br>
> +++ b/bsps/arm/xilinx-zynq/console/zynq-uart.c<br>
> @@ -262,12 +262,16 @@ static void zynq_uart_write_support(<br>
>   #endif<br>
>   }<br>
>   <br>
> +/*<br>
> + * Disable this because the initialization is done by code generated<br>
> + * by the Xilinx code generator.<br>
> + */<br>
<br>
The main purpose of the set attributes function is to apply the settings <br>
specified by the user via the Termios interface. Returning false just <br>
indicates that this function is not implemented. If you want to hide <br>
this fact from the user, then you can simply return true ...<br></blockquote><div><br></div><div>We discussed this alternative with Chris and this was the end result.</div><div><br></div><div>git blame shows that Chris added the if 0 and then you added the return false</div><div>which broke the callers. From that point forward, no interactive test ran successfully. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +#if 0<br>
>   static bool zynq_uart_set_attributes(<br>
>     rtems_termios_device_context *context,<br>
>     const struct termios *term<br>
>   )<br>
>   {<br>
> -#if 0<br>
>     volatile zynq_uart *regs = zynq_uart_get_regs(minor);<br>
>     uint32_t brgr = 0;<br>
>     uint32_t bauddiv = 0;<br>
> @@ -283,14 +287,16 @@ static bool zynq_uart_set_attributes(<br>
>     regs->control |= ZYNQ_UART_CONTROL_RXEN | ZYNQ_UART_CONTROL_TXEN;<br>
>   <br>
>     return true;<br>
> -#else<br>
> -  return false;<br>
<br>
... here.<br>
<br>
> -#endif<br>
>   }<br>
> +#endif<br>
>   <br>
>   const rtems_termios_device_handler zynq_uart_handler = {<br>
>     .first_open = zynq_uart_first_open,<br>
> -  .set_attributes = zynq_uart_set_attributes,<br>
> +#if 0<br>
> +  .set_attributes = zynq_uart_set_attributes;<br>
> +#else<br>
> +  .set_attributes = NULL,<br>
> +#endif<br>
>     .write = zynq_uart_write_support,<br>
>   #ifdef ZYNQ_CONSOLE_USE_INTERRUPTS<br>
>     .last_close = zynq_uart_last_close,<br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div>