diff mbox series

usb: musb-new: sunxi: remove unused define

Message ID 20230607231640.28162-1-CFSworks@gmail.com
State Superseded
Delegated to: Andre Przywara
Headers show
Series usb: musb-new: sunxi: remove unused define | expand

Commit Message

Sam Edwards June 7, 2023, 11:16 p.m. UTC
Signed-off-by: Sam Edwards <CFSworks@gmail.com>
---
 drivers/usb/musb-new/sunxi.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Marek Vasut June 8, 2023, 4:51 a.m. UTC | #1
On 6/8/23 01:16, Sam Edwards wrote:
> Signed-off-by: Sam Edwards <CFSworks@gmail.com>
> ---
>   drivers/usb/musb-new/sunxi.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/usb/musb-new/sunxi.c b/drivers/usb/musb-new/sunxi.c
> index 2b954601a0..85d815ad75 100644
> --- a/drivers/usb/musb-new/sunxi.c
> +++ b/drivers/usb/musb-new/sunxi.c
> @@ -81,8 +81,6 @@
>    * From usbc/usbc.c
>    ******************************************************************************/
>   
> -#define OFF_SUN6I_AHB_RESET0	0x2c0
> -
>   struct sunxi_musb_config {
>   	struct musb_hdrc_config *config;
>   	bool has_sram;

Please add missing commit message and send V2.

Otherwise

Reviewed-by: Marek Vasut <marex@denx.de>
Sam Edwards June 8, 2023, 6:45 p.m. UTC | #2
Hi Marek,

> Please add missing commit message and send V2.

Ah, apologies! I had (mistakenly) believed that the summary line would 
be a sufficient commit message for something so trivial.

Instead I did a little research into how this got here and wrote a quick 
explanation of why it's unused. I figure that would be a bit more 
helpful for someone reading the commit history than just reiterating 
that "It's, y'know, unused." :)

> Otherwise
> 
> Reviewed-by: Marek Vasut <marex@denx.de>

Thank you for the speedy review!

Cheers,
Sam
diff mbox series

Patch

diff --git a/drivers/usb/musb-new/sunxi.c b/drivers/usb/musb-new/sunxi.c
index 2b954601a0..85d815ad75 100644
--- a/drivers/usb/musb-new/sunxi.c
+++ b/drivers/usb/musb-new/sunxi.c
@@ -81,8 +81,6 @@ 
  * From usbc/usbc.c
  ******************************************************************************/
 
-#define OFF_SUN6I_AHB_RESET0	0x2c0
-
 struct sunxi_musb_config {
 	struct musb_hdrc_config *config;
 	bool has_sram;