Hi, On 1/8/2018 4:08 AM, Felipe Balbi wrote: > Thinh Nguyen <Thinh.Nguyen@xxxxxxxxxxxx> writes: >> Dump LSP and BMU debug info. >> >> Signed-off-by: Thinh Nguyen <thinhn@xxxxxxxxxxxx> >> --- >> drivers/usb/dwc3/core.h | 5 +++++ >> drivers/usb/dwc3/debugfs.c | 5 +++++ >> 2 files changed, 10 insertions(+) >> >> diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h >> index 83a74d10fa82..21a6dcd379f1 100644 >> --- a/drivers/usb/dwc3/core.h >> +++ b/drivers/usb/dwc3/core.h >> @@ -100,6 +100,11 @@ >> #define DWC3_GHWPARAMS7 0xc15c >> #define DWC3_GDBGFIFOSPACE 0xc160 >> #define DWC3_GDBGLTSSM 0xc164 >> +#define DWC3_GDBGBMU 0xc16c >> +#define DWC3_GDBGLSPMUX 0xc170 >> +#define DWC3_GDBGLSP 0xc174 >> +#define DWC3_GDBGEPINFO0 0xc178 >> +#define DWC3_GDBGEPINFO1 0xc17c > > wheren't these registers accessed with a write followed by a read? IIRC, > just writing them doesn't give us much information, no? > These registers are for internal use only. They hold the internal states that are useful for the HW engineers to debug. The databook doesn't specify that we need a write followed by a read to access these registers. BR, Thinh -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
