diff options
| author | Simon Glass <sjg@chromium.org> | 2020-05-10 20:39:55 +0300 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-05-18 21:54:24 +0300 |
| commit | c3dc39a2f85b16bf590789f7e283cd72275cd168 (patch) | |
| tree | fd07b448aa4192e1d4fede43e412d5fdfa584b82 /drivers/usb/eth | |
| parent | b79fdc76978515fba2c4f3b5b4f2e679c5ff1463 (diff) | |
| download | u-boot-c3dc39a2f85b16bf590789f7e283cd72275cd168.tar.xz | |
arm: Don't include common.h in header files
It is bad practice to include common.h in other header files since it can
bring in any number of superfluous definitions. It implies that some C
files don't include it and thus may be missing CONFIG options that are set
up by that file. The C files should include these themselves.
Update some header files in arch/arm to drop this.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'drivers/usb/eth')
| -rw-r--r-- | drivers/usb/eth/lan7x.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/usb/eth/lan7x.h b/drivers/usb/eth/lan7x.h index 7af610be37..35965e9645 100644 --- a/drivers/usb/eth/lan7x.h +++ b/drivers/usb/eth/lan7x.h @@ -4,7 +4,10 @@ */ #include <console.h> +#include <time.h> #include <watchdog.h> +#include <linux/delay.h> +#include <linux/errno.h> /* USB Vendor Requests */ #define USB_VENDOR_REQUEST_WRITE_REGISTER 0xA0 @@ -122,6 +125,10 @@ int lan7x_write_reg(struct usb_device *udev, u32 index, u32 data); int lan7x_read_reg(struct usb_device *udev, u32 index, u32 *data); +/* + * FIXME: Code should not be in header files. Nive this to a file common to + * the two drivers. + */ static inline int lan7x_wait_for_bit(struct usb_device *udev, const char *prefix, const u32 reg, const u32 mask, const bool set, |