On 6/17/19 8:49 AM, Lukasz Majewski wrote:
Hi Marek,
On 6/16/19 12:34 AM, Lukasz Majewski wrote:
This commit converts mxs_spi driver to support DM/DTS.
Signed-off-by: Lukasz Majewski lukma@denx.de
Is the non-DM part needed for anything ?
Do you mean the non-DM part of the mxs_gpio driver? Yes, it is used by not converted boards.
This is a SPI driver though.
I recall the SPL jumps back to BootROM when loading the U-Boot proper. So if not, drop it.
Also, please don't do partial conversion for iMX28 only, do the iMX23 part as well, it cannot be hard.
Maybe it is not hard, but I cannot test it properly as I don't have i.MX23 device. If you are offering your help with testing (i.e. you do have the access to i.MX23 device and you will test those changes) I can add support for it.
Otherwise, NO, I will not add ANY new untested code.
In general, you don't have to add any code, the iMX23/28 SPI IP is very much the same hardware, DTTO for most of the other blocks. If there are any differences between iMX23/28, they are already handled in the existing driver(s).
Half-way converted drivers in fact increase maintenance burden, because then we have to deal with two different variants of the code, instead of only one. That's why I would like to see this practice go away wherever possible, and in this case it is possible.
If you need someone to test your changes, CC the board maintainers, that's standard practice. If they don't respond, that cannot be helped.