[U-Boot] [PATCH 1/9] ioport.h: Remove struct resource & co
Paul Burton
paul.burton at imgtec.com
Fri Jan 29 16:58:40 CET 2016
On Fri, Jan 29, 2016 at 03:06:33PM +0100, Marek Vasut wrote:
> On Friday, January 29, 2016 at 02:54:47 PM, Paul Burton wrote:
> > We only use struct resource in a single place (drivers/usb/dwc3/core.h)
> > for a field (xhci_resources) which is never used. Only ARM currently
> > defines resource_size_t which means linux/ioport.h only compiles there.
> > In preparation for making use of the IORESOURCE_ flags, remove struct
> > resource & the various declarations of functions which we don't
> > implement.
> >
> > Signed-off-by: Paul Burton <paul.burton at imgtec.com>
> > ---
> >
> > drivers/usb/dwc3/core.h | 1 -
> > include/linux/ioport.h | 104
> > ------------------------------------------------ 2 files changed, 105
> > deletions(-)
>
> I believe the driver is imported from Linux kernel, so it'd be much better
> to sync the driver with mainline Linux instead of starting to diverge.
>
> Best regards,
> Marek Vasut
Hi Marek,
The problem is that the driver can't use struct resource because U-Boot
has none of the infrastructure around it. The driver model doesn't use
struct resource, there's basically nothing in U-Boot to fill out the
struct. So unless that changes this dwc3 driver will always have to
handle resources differently to on Linux.
I therefore don't see any good reason to keep around an unused struct
which will only currently compile for one architecture, for a driver
which can't use it in U-Boot anyway.
The alternative to this patch would be to define resource_size_t for
other architectures, but then we're just left with dead code.
Thanks,
Paul
More information about the U-Boot
mailing list