[U-Boot] [PATCH v3 8/8] usb: common: Use a global macro for 'min3'
Marek Vasut
marex at denx.de
Sun Apr 14 19:11:44 CEST 2013
Dear Vivek Gautam,
> We can use a common global macro for calculating minimum of
> 3 numbers. Put the same in 'common header' and let 'ehci'
> use it.
>
> Signed-off-by: Vivek Gautam <gautam.vivek at samsung.com>
> ---
>
> This patch added in V3(current-version) of this patch-series.
>
> drivers/usb/host/ehci-hcd.c | 10 ----------
> include/common.h | 2 ++
> 2 files changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
> index c816878..bcecae3 100644
> --- a/drivers/usb/host/ehci-hcd.c
> +++ b/drivers/usb/host/ehci-hcd.c
> @@ -587,16 +587,6 @@ fail:
> return -1;
> }
>
> -static inline int min3(int a, int b, int c)
> -{
> -
> - if (b < a)
> - a = b;
> - if (c < a)
> - a = c;
> - return a;
> -}
> -
> int
> ehci_submit_root(struct usb_device *dev, unsigned long pipe, void *buffer,
> int length, struct devrequest *req)
> diff --git a/include/common.h b/include/common.h
> index d41aeb4..37269c7 100644
> --- a/include/common.h
> +++ b/include/common.h
> @@ -211,6 +211,8 @@ typedef void (interrupt_handler_t)(void *);
> #define MIN(x, y) min(x, y)
> #define MAX(x, y) max(x, y)
>
> +#define min3(a, b, c) min(min(a, b), c)
> +
You might want to keep it as an inline-function to allow GCC do the type-
checking?
> /*
> * Return the absolute value of a number.
> *
Best regards,
Marek Vasut
More information about the U-Boot
mailing list