Hi Jules, On Wed, Dec 09, 2020 at 03:55:45PM +0100, Jules Maselbas wrote: > Theses functions were only used in dwc2/core.c and where defined as > static. > > Signed-off-by: Jules Maselbas <jmaselbas@xxxxxxxxx> > --- > drivers/usb/dwc2/core.c | 5 +++++ > drivers/usb/dwc2/dwc2.h | 6 +----- > 2 files changed, 6 insertions(+), 5 deletions(-) > > diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c > index c4a3cc789..a61acf553 100644 > --- a/drivers/usb/dwc2/core.c > +++ b/drivers/usb/dwc2/core.c > @@ -1,6 +1,11 @@ > // SPDX-License-Identifier: GPL-2.0+ > #include "dwc2.h" > > +static void dwc2_set_param_otg_cap(struct dwc2 *dwc2); > +static void dwc2_set_param_phy_type(struct dwc2 *dwc2); > +static void dwc2_set_param_speed(struct dwc2 *dwc2); > +static void dwc2_set_param_phy_utmi_width(struct dwc2 *dwc2); > + There should be a forward declaration necessary for static functions. You are changing the declarations to static here, but not the implementations further down in this file, so this shouldn't compile. > /* Returns the controller's GHWCFG2.OTG_MODE. */ > static unsigned int dwc2_op_mode(struct dwc2 *dwc2) > { > diff --git a/drivers/usb/dwc2/dwc2.h b/drivers/usb/dwc2/dwc2.h > index 196f4a07f..70a747302 100644 > --- a/drivers/usb/dwc2/dwc2.h > +++ b/drivers/usb/dwc2/dwc2.h > @@ -8,13 +8,9 @@ > #include "core.h" > > /* Core functions */ > -void dwc2_set_param_otg_cap(struct dwc2 *dwc2); > -void dwc2_set_param_phy_type(struct dwc2 *dwc2); > -void dwc2_set_param_speed(struct dwc2 *dwc2); > -void dwc2_set_param_phy_utmi_width(struct dwc2 *dwc2); > -void dwc2_set_default_params(struct dwc2 *dwc2); > int dwc2_core_snpsid(struct dwc2 *dwc2); > void dwc2_get_hwparams(struct dwc2 *dwc2); > +void dwc2_set_default_params(struct dwc2 *dwc2); No need to move this declaration. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox