Hi Ivan, On Thu, Aug 25, 2022 at 06:25:14PM +0300, Ivan Bornyakov wrote: > Hi, Johannes! > > I just came across your patches. Surprisingly, our work interferes. > > I recently posted patch-series for configuring ECP5 and was asked to make > generalized sysCONFIG driver with support for both ECP5 and MachXO2, which > I did. Sadly I don't have hardware with MachXO2, but you clearly do :) > > Please, take a look at > > https://lore.kernel.org/linux-fpga/20220825112433.14583-1-i.bornyakov@xxxxxxxxxxx/ > > and please help test MachXO2 variant. When we pull this off, you may add I2C > interface on top. You are adding a new driver for something we already have a driver for in the tree. The final goal should be that we only have a single driver for sysconfig based FPGAs in the tree. Johannes' series is a step in that direction: He cleans up the existing driver and starts abstracting out common sysconfig functions so that they can be used by both the I2C and SPI interface. He just told me that the abstraction is likely not enough to integrate ECP5 support right away, one reason being that the machxo2 has a flash whereas the ECP5 does not. Unless you can explain why the existing driver is broken beyond repair I think we should rather incrementally improve the existing driver instead of adding a new one with a conflicting compatible. So despite you were in the room earlier I think you should rather base your work on Johannes' series. Just my 2 cents, maybe one of the maintainers has a few words on it. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
