On Wed, Apr 08, 2015 at 11:04:34AM -0700, Jonathan Richardson wrote: > - A new config for non-BCMA chips has been added. > - Common code between the BCMA and non BCMA version are shared. > - Function pointers to set read/write functions to abstract bcma > and non-bcma versions are provided. > - DT is now mandatory. Hard coded SPI devices are removed and must be > set in DT. > - Remove function was unnecessary and removed. This looks like it should be a patch series in itself - for example, the move to using function pointers as a read/write operation looks like something that could easily be pulled out, as could the removal of unused functions. Having things split out makes life a lot easier for review since it makes it much easier to check if the change is doing the things it's supposed to be doing.
Attachment:
signature.asc
Description: Digital signature
