RE: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-17 Thread Jan.Sommer
> -Original Message- > From: devel On Behalf Of Chris Johns > Sent: Tuesday, February 16, 2021 10:53 PM > To: Gedare Bloom ; Peter Dufault > Cc: devel@rtems.org > Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi > [...] > > We h

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread Chris Johns
On 17/2/21 7:34 am, Gedare Bloom wrote: > On Tue, Feb 16, 2021 at 10:44 AM wrote: >>> On Feb 16, 2021, at 11:33 , Gedare Bloom wrote: The umlaut is part of the official brand. That can be tricky, depending on the corporate policy. > I find it odd to avoid Unicode and not allow

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread Gedare Bloom
On Tue, Feb 16, 2021 at 10:44 AM wrote: > > > > > On Feb 16, 2021, at 11:33 , Gedare Bloom wrote: > > > >> The umlaut is part of the official brand. > >> That can be tricky, depending on the corporate policy. > >> > >>> I find it odd to avoid Unicode and not allow umlauts in 2021. > >>> > >> > >

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread dufault
> On Feb 16, 2021, at 11:33 , Gedare Bloom wrote: > >> The umlaut is part of the official brand. >> That can be tricky, depending on the corporate policy. >> >>> I find it odd to avoid Unicode and not allow umlauts in 2021. >>> >> > > We do allow unicode with justification. For names, I

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread Gedare Bloom
On Tue, Feb 16, 2021 at 8:47 AM wrote: > > > -Original Message- > > From: dufa...@hda.com > > Sent: Tuesday, February 16, 2021 2:05 PM > > To: Sommer, Jan > > Cc: j...@rtems.org; devel@rtems.org > > Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add

RE: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread Jan.Sommer
> -Original Message- > From: dufa...@hda.com > Sent: Tuesday, February 16, 2021 2:05 PM > To: Sommer, Jan > Cc: j...@rtems.org; devel@rtems.org > Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi > > You could use "fuer&

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread dufault
F-8. > > > On Feb 15, 2021, at 03:21 , jan.som...@dlr.de wrote: > > > > Hi Chris, > > > >> -----Original Message----- > >> From: Chris Johns > >> Sent: Sunday, February 14, 2021 10:20 PM > >> To: Sommer, Jan ; devel@rtems.org > >>

RE: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-16 Thread Jan.Sommer
ris, > >> -Original Message- >> From: Chris Johns mailto:chr...@rtems.org>> >> Sent: Sunday, February 14, 2021 10:20 PM >> To: Sommer, Jan mailto:jan.som...@dlr.de>>; >> devel@rtems.org<mailto:devel@rtems.org> >> Subject: Re: [PATC

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-15 Thread Joel Sherrill
t; From: Chris Johns > >> Sent: Sunday, February 14, 2021 10:20 PM > >> To: Sommer, Jan ; devel@rtems.org > >> Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for > cadence-spi > >> > >> Hi Jan, > >> > >> Thank you for the

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-15 Thread Peter Dufault
021, at 03:21 , jan.som...@dlr.de wrote: > > Hi Chris, > >> -Original Message- >> From: Chris Johns >> Sent: Sunday, February 14, 2021 10:20 PM >> To: Sommer, Jan ; devel@rtems.org >> Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-sp

RE: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-15 Thread Jan.Sommer
Hi Chris, > -Original Message- > From: Chris Johns > Sent: Sunday, February 14, 2021 10:20 PM > To: Sommer, Jan ; devel@rtems.org > Subject: Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi > > Hi Jan, > > Thank you for the changes. I have

Re: [PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-14 Thread Chris Johns
Hi Jan, Thank you for the changes. I have one question inlined below ... On 14/2/21 10:30 pm, Jan Sommer wrote: > --- > bsps/include/dev/spi/cadence-spi-regs.h | 84 + > bsps/include/dev/spi/cadence-spi.h | 48 +++ > bsps/shared/dev/spi/cadence-spi.c | 437

[PATCH v3 1/3] bsps/xilinx_zynq: Add SPI driver for cadence-spi

2021-02-14 Thread Jan Sommer
--- bsps/include/dev/spi/cadence-spi-regs.h | 84 + bsps/include/dev/spi/cadence-spi.h | 48 +++ bsps/shared/dev/spi/cadence-spi.c | 437 3 files changed, 569 insertions(+) create mode 100644 bsps/include/dev/spi/cadence-spi-regs.h create mode 100644