<div dir="auto">Christian submitted a series of patches to eliminate unicode. I'd prefer if we avoided it </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 15, 2021, 8:20 AM Peter Dufault <<a href="mailto:dufault@hda.com">dufault@hda.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">An international project should allow UTF in source code.  You might need to hunt what setting you need (for gnome-terminal "export LANG=en_US.UTF-8" works for me in the US).  I'm surprised if anyone is working on a system that can't at least display UTF-8.<br>
<br>
> On Feb 15, 2021, at 03:21 , <a href="mailto:Jan.Sommer@dlr.de" target="_blank" rel="noreferrer">Jan.Sommer@dlr.de</a> wrote:<br>
> <br>
> Hi Chris,<br>
> <br>
>> -----Original Message-----<br>
>> From: Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">chrisj@rtems.org</a>><br>
>> Sent: Sunday, February 14, 2021 10:20 PM<br>
>> To: Sommer, Jan <<a href="mailto:Jan.Sommer@dlr.de" target="_blank" rel="noreferrer">Jan.Sommer@dlr.de</a>>; <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
>> Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi<br>
>> <br>
>> Hi Jan,<br>
>> <br>
>> Thank you for the changes. I have one question inlined below ...<br>
>> <br>
>> On 14/2/21 10:30 pm, Jan Sommer wrote:<br>
>>> ---<br>
>>> bsps/include/dev/spi/cadence-spi-regs.h |  84 +++++<br>
>>> bsps/include/dev/spi/cadence-spi.h      |  48 +++<br>
>>> bsps/shared/dev/spi/cadence-spi.c       | 437<br>
>> ++++++++++++++++++++++++<br>
>>> 3 files changed, 569 insertions(+)<br>
>>> create mode 100644 bsps/include/dev/spi/cadence-spi-regs.h<br>
>>> create mode 100644 bsps/include/dev/spi/cadence-spi.h<br>
>>> create mode 100644 bsps/shared/dev/spi/cadence-spi.c<br>
>>> <br>
>>> diff --git a/bsps/include/dev/spi/cadence-spi-regs.h<br>
>>> b/bsps/include/dev/spi/cadence-spi-regs.h<br>
>>> new file mode 100644<br>
>>> index 0000000000..2851c88df1<br>
>>> --- /dev/null<br>
>>> +++ b/bsps/include/dev/spi/cadence-spi-regs.h<br>
>>> @@ -0,0 +1,84 @@<br>
>>> +/* SPDX-License-Identifier: BSD-2-Clause */<br>
>>> +<br>
>>> +/*<br>
>>> + * Copyright (C) 2020 Jan Sommer, Deutsches Zentrum für Luft- und<br>
>> Raumfahrt e. V.<br>
>> <br>
>> Is there a unicode character in this line? I cannot see anything in our coding<br>
>> standard about unicode characters in source and I am not sure what the<br>
>> actual policy is. I seem to remember some have been removed in the past. I<br>
>> thought I would ask on the list now before we push these changes.<br>
>> <br>
> <br>
> There is an u-Umlaut in it. That is probably it. As it is part of the name, it's hard to get rid of.<br>
> I could probably use the English name (German Aerospace Center) which has no special characters.<br>
> Would that be preferred?<br>
> <br>
> Best regards,<br>
> <br>
>    Jan<br>
> <br>
>> Chris<br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
<br>
Peter<br>
-----------------<br>
Peter Dufault<br>
HD Associates, Inc.      Software and System Engineering<br>
<br>
This email is delivered through the public internet using protocols subject to interception and tampering.<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>