On Mon, Aug 06, 2018 at 10:21:46AM +1000, NeilBrown wrote:
> On Fri, Aug 03 2018, Sergio Paracuellos wrote:
> 
> > This patch series include an attempt to avoid the use of custom
> > read and writes in driver code and use PCI subsystem common ones.
> >
> > In order to do this 'map_bus' callback is implemented and also
> > data structures for driver are included. The regs base address
> > ranges and data is being readed from device tree and the driver
> > gets clean a lot of code.
> >
> > This patchet also removes all legacy PCI code using now PCI_DRIVERS_GENERIC
> > kernel api.
> >
> > Changes in v10:
> >     - PATCH 1: Do 'ioremap' directly using 'ranges.cpu_addr' from the
> >       io resource and add it directly as a resource to the resources list
> >       like mips pci-legacy code do for this. We cannot do anything else 
> > since
> >       using new kernel pci generic apis (call to 
> > 'devm_of_pci_get_host_bridge_resources')
> >       we get bad io resource getting 'OF_BAD_ADDR' because of the fact that 
> > the 
> >       io resource for this platform starts in  0x1e160000 which is higher 
> > than 
> >       IO_SPACE_LIMIT (0xFFFF). Because of this io resource is not added to 
> > the resources
> >       list and system gets into a not working state.
> >     - Other patches rebased and adapted to this changes.
> 
> And...... we have a winner !!!
> 
> Tested-by: NeilBrown <n...@brown.name>
> 
> I haven't actually looked at the code - just compiled, booted, saw that
> the SATA drive appears to work.

Yeah!!!  Sergio, nice job.

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to