2022-07-27 03:56:40

by Stanley Chu

[permalink] [raw]
Subject: [PATCH v2 1/5] scsi: ufs: ufs-mediatek: Remove redundant header files

Remove redundant header files like
<linux/sched/clock.h>

Signed-off-by: Stanley Chu <[email protected]>
Change-Id: I7796a23a5fdc767a4d00475b399844ceeaded0a8
---
drivers/ufs/host/ufs-mediatek.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-mediatek.c
index c958279bdd8f..ff6fd8f52ebc 100644
--- a/drivers/ufs/host/ufs-mediatek.c
+++ b/drivers/ufs/host/ufs-mediatek.c
@@ -19,7 +19,6 @@
#include <linux/pm_qos.h>
#include <linux/regulator/consumer.h>
#include <linux/reset.h>
-#include <linux/sched/clock.h>
#include <linux/soc/mediatek/mtk_sip_svc.h>

#include <ufs/ufshcd.h>
--
2.18.0


2022-07-27 19:23:00

by Bart Van Assche

[permalink] [raw]
Subject: Re: [PATCH v2 1/5] scsi: ufs: ufs-mediatek: Remove redundant header files

On 7/26/22 20:44, Stanley Chu wrote:
> Remove redundant header files like
> <linux/sched/clock.h>
>
> Signed-off-by: Stanley Chu <[email protected]>
> Change-Id: I7796a23a5fdc767a4d00475b399844ceeaded0a8

Please leave out Change-Id tags from patches that are sent upstream.

Otherwise this patch looks good to me.

Thanks,

Bart.

2022-07-28 01:59:13

by Stanley Jhu

[permalink] [raw]
Subject: Re: [PATCH v2 1/5] scsi: ufs: ufs-mediatek: Remove redundant header files

Hi Bart,

On Thu, Jul 28, 2022 at 3:03 AM Bart Van Assche <[email protected]> wrote:
>
> On 7/26/22 20:44, Stanley Chu wrote:
> > Remove redundant header files like
> > <linux/sched/clock.h>
> >
> > Signed-off-by: Stanley Chu <[email protected]>
> > Change-Id: I7796a23a5fdc767a4d00475b399844ceeaded0a8
>
> Please leave out Change-Id tags from patches that are sent upstream.

Ooops, I forgot to clean them up in v2. Will fix it in the next version.

Thanks, Stanley