<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 9, 2021 at 5:08 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</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 9/2/21 11:42 pm, Jan Sommer wrote:<br>
> From: Kinsey Moore <<a href="mailto:kinsey.moore@oarcorp.com" target="_blank">kinsey.moore@oarcorp.com</a>><br>
> <br>
> The zynq-uart set_attributes implementation was configured to always<br>
> return false which causes spconsole01 to fail. This restores the<br>
> disabled implementation which sets the baud rate registers<br>
> appropriately and allows spconsole01 to pass. This also expands the<br>
> set_attributes functionality to allow setting of the stop bits,<br>
> character width, and parity.<br>
<br>
Hmmm I missed this change.<br>
<br>
Sebastian changed the Zynq driver to return an error. Returning an error is<br>
correct because the attributes are not being set however we have a couple of<br>
design approaches clashing.<br>
<br>
The issue pulls right back to the Xilinx SystemZ design dialogue box. The<br>
hardware designer has the ability to set the serial port defaults. These<br>
settings form part of the ps7init data the FSBL paints into the hardware very<br>
early in the boot process. Also in this data are parallel port pin settings, AXI<br>
clocks, memory and more so lots of important stuff. The Xilinx bootloader and<br>
ones I have created (I do not know about uboot) do not touch the UART hardware<br>
on purpose so the system designer has control. Then in 2017 Sebastian pushed a<br>
change to force the baudrate to a BSP specific default.<br>
<br>
This is normally all OK because all Zynq designs I know of use the default of<br>
115200,8,n,1 for the configuration. A problem arises when these defaults are varied.<br>
<br>
Maybe something explaining this in the user manual ...<br>
<br>
<a href="https://docs.rtems.org/branches/master/user/bsps/bsps-arm.html#xilinx-zynq" rel="noreferrer" target="_blank">https://docs.rtems.org/branches/master/user/bsps/bsps-arm.html#xilinx-zynq</a><br>
<br>
so we have provided information on what to change?<br></blockquote><div><br></div><div>I wonder if there should be a configuration parameter for the default settings.</div><div>At least "don't override the defaults"</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The other approach is not to touch the defaults in hardware and allow a get<br>
attributes to discover them? Does termios drivers have a low level get? I can<br>
see a set but no get in the zynq uart driver.<br></blockquote><div><br></div><div>No. This reflects termios from the POSIX perspective. </div><div><br></div><div>--joel </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Chris<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><br>
</blockquote></div></div>