
20 Oct
2022
20 Oct
'22
7:27 a.m.
Hi Conor,
On Wed, 2022-10-19 at 16:47 +0100, Conor Dooley wrote:
On Wed, Oct 19, 2022 at 03:16:01PM +0000, Tudor.Ambarus@microchip.com wrote:
Hi!
On 10/19/22 17:53, Padmarao Begari wrote:
drivers/spi/microchip_qspi.c | 504 +++++++++++++++++++++++++++++++++++
The name is too generic, there are multiple QSPI IPs inside mchp. I would suggest to create an mchp directory and rename your c file to maybe something like mpfs-qspi?
The (linux) driver works for both the "hard" peripheral on mpfs & for the core in FPGA fabric. I would wouldn't mind copying the linux naming and using "microchip-corespi" & "microchip-coreqspi" as the underlying IPs are called CoreSPI and CoreQSPI respectively.
Ok, will use "microchip-coreqspi" name.
Regards Padmarao