Hi Michal,
On 2021/06/22 21:44, Michal Simek wrote:
Hi,
On 6/22/21 6:24 AM, Kunihiko Hayashi wrote:
This adds serial parameters that include stop bit mode, parity mode, and character length. Mark parity and space parity modes are not supported.
Signed-off-by: Kunihiko Hayashi hayashi.kunihiko@socionext.com
drivers/serial/serial_zynq.c | 64 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 64 insertions(+)
[snip]
I am just curious how you have tested it because only hook is in test/dm/serial.c and I can't see no way how to change this setting via u-boot command line.
I was misunderstanding.
Surely there is no way to execute .setconfig function, and neither command line nor devicetree actually affects the serial mode. The mode just inherits that of the previous firmware.
That being said I see that this change adds 184 bytes which is quite a lot especially for SPL on zynqmp. That's why would like to know how this feature should be used. If make sense for example to limit it to only full U-Boot.
I see. I didn't think enough about the size limit of SPL. Anyway, I withdraw this patch.
Thank you,
--- Best Regards Kunihiko Hayashi