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.amba...@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
