Hi all, It's my first patch contribution, do I need to do anything more ?
Best regards, Vincent 2017-10-26 13:52 GMT+02:00 Stefan Agner <stefan.ag...@toradex.com>: > > > On 26.10.2017 13:25, Lukasz Majewski wrote: > > Hi Vincent, > > > >> This prevents board resets when calling sdp command on boards which > >> have a watchdog. > >> > >> Signed-off-by: Vincent Prince <vincent.prince...@gmail.com> > >> --- > >> drivers/usb/gadget/f_sdp.c | 4 ++++ > >> 1 file changed, 4 insertions(+) > >> > >> diff --git a/drivers/usb/gadget/f_sdp.c b/drivers/usb/gadget/f_sdp.c > >> index 0fae66b..c3eba6d 100644 > >> --- a/drivers/usb/gadget/f_sdp.c > >> +++ b/drivers/usb/gadget/f_sdp.c > >> @@ -32,6 +32,7 @@ > >> #include <spl.h> > >> #include <image.h> > >> #include <imximage.h> > >> +#include <watchdog.h> > >> > >> #define HID_REPORT_ID_MASK 0x000000ff > >> > >> @@ -602,6 +603,8 @@ int sdp_init(int controller_index) > >> puts("\rCTRL+C - Operation aborted.\n"); > >> return 1; > >> } > >> + > >> + WATCHDOG_RESET(); > >> usb_gadget_handle_interrupts(controller_index); > >> } > >> > >> @@ -712,6 +715,7 @@ void sdp_handle(int controller_index) > >> return; > >> } > >> > >> + WATCHDOG_RESET(); > >> usb_gadget_handle_interrupts(controller_index); > >> > >> sdp_handle_in_ep(); > > Reviewed-by: Lukasz Majewski <lu...@denx.de> > > Thanks for the patch! Looks good to me too. > Reviewed-by: Stefan Agner <stefan.ag...@toradex.com> > > Best regards, > Stefan > > > > > > > Best regards, > > > > Lukasz Majewski > > > > -- > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot