[PATCH] serial: zynq: Add support for serial parameters

[Kunihiko Hayashi]

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 at 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