Hi Marek On 05/12/2017 10:51 PM, Marek Vasut wrote: > On 05/12/2017 07:27 PM, patrice.chot...@st.com wrote: >> From: Patrice Chotard <patrice.chot...@st.com> > > Commit message does not explain WHY this change is needed. In fact ... > commit message is missing altogether ...
Yes, my bad i forget it, i will add it in the v3 Thanks Patrice > >> Signed-off-by: Patrice Chotard <patrice.chot...@st.com> >> --- >> >> v2: _ create this independant path for printf() replacement >> >> drivers/usb/host/ehci-generic.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/usb/host/ehci-generic.c >> b/drivers/usb/host/ehci-generic.c >> index 0000808..39b56de 100644 >> --- a/drivers/usb/host/ehci-generic.c >> +++ b/drivers/usb/host/ehci-generic.c >> @@ -74,7 +74,7 @@ static int ehci_usb_probe(struct udevice *dev) >> if (ret < 0) >> break; >> if (clk_enable(&clk)) { >> - printf("failed to enable clock %d\n", i); >> + error("failed to enable clock %d\n", i); >> clk_free(&clk); >> goto clk_err; >> } >> @@ -88,7 +88,7 @@ static int ehci_usb_probe(struct udevice *dev) >> if (ret < 0) >> break; >> if (reset_deassert(&reset)) { >> - printf("failed to deassert reset %d\n", i); >> + error("failed to deassert reset %d\n", i); >> reset_free(&reset); >> goto reset_err; >> } >> > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot