lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 15 Feb 2022 21:05:02 +0300 From: Ivan Bornyakov <i.bornyakov@...rotek.ru> To: Moritz Fischer <mdf@...nel.org> Cc: Conor Dooley <mail@...chuod.ie>, hao.wu@...el.com, yilun.xu@...el.com, trix@...hat.com, linux-kernel@...r.kernel.org, linux-fpga@...r.kernel.org, system@...rotek.ru Subject: Re: [PATCH v2] fpga: microsemi-spi: add Microsemi FPGA manager Hello Conor, Moritz On Tue, Feb 15, 2022 at 09:52:30AM -0800, Moritz Fischer wrote: > Hi Conor, Ivan, > > On Tue, Feb 15, 2022 at 05:37:04PM +0000, Conor Dooley wrote: > > Hey Ivan, > > Firstly thanks for the patch(es), stumbled across them today. > > As you may know Microsemi has been acquired by Microchip, so > > s/microsemi/microchip/ please. This would make the correct vendor > > prefix for compatible strings "microchip". While you've said this is > > for the PolarFire FPGA, there is prescendent for using "mpfs" for the > > PolarFire SoC FPGA in the kernel - so if you could change the uses of > > "polarfire" to "mpf" that'd be great. > > I personally don't have a strong opinion on hte microchip vs microsemi > here. We have precedent with intel/altera. > Me neither, so I'll do what Conor asked. > > > > The current item on my own todo list is the opposite side of this, > > reprogramming the FPGA via the system controller acting as a SPI > > master for PolarFire SoC. > > I will get back to you when I have a better idea of what (if any) code > > can be made generic between both modes. In the meantime, I will get > > together a setup to test SPI slave reprogramming of the PolarFire (SoC) > > > > Thanks, > > Conor <conor.dooley@...rochip.com> > > Thanks for chiming in. Always nice to have vendors help out reviewing. > Yeah, that's great, thanks in advance Conor. > > > Add support to the FPGA manager for programming Microsemi Polarfire > > > FPGAs over slave SPI interface. > > > > > > Signed-off-by: Ivan Bornyakov <i.bornyakov@...rotek.ru> > > > --- > > > Changelog: > > > v1 -> v2: fix printk formating > > > > > > drivers/fpga/Kconfig | 9 + > > > drivers/fpga/Makefile | 1 + > > > drivers/fpga/microsemi-spi.c | 366 +++++++++++++++++++++++++++++++++++ > > > 3 files changed, 376 insertions(+) > > > create mode 100644 drivers/fpga/microsemi-spi.c > > > > > --<snip>-- > > I'll take a closer look once the bot's complaints are addressed. > > Thanks, > Moritz Thanks in advance Moritz.
Powered by blists - more mailing lists