| Message ID | 20200827062154.1847-1-thunder.leizhen@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [1/1] watchdog: remove unneeded inclusion of <uapi/linux/sched/types.h> | expand |
On 8/26/20 11:21 PM, Zhen Lei wrote: > There has been no reference to "struct sched_param" since > commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so > there's no need to include <uapi/linux/sched/types.h> any more, delete > it. > > Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> > --- > drivers/watchdog/watchdog_dev.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c > index 6798addabd5a067..0f18fa2433310b0 100644 > --- a/drivers/watchdog/watchdog_dev.c > +++ b/drivers/watchdog/watchdog_dev.c > @@ -43,8 +43,6 @@ > #include <linux/watchdog.h> /* For watchdog specific items */ > #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ > > -#include <uapi/linux/sched/types.h> /* For struct sched_param */ > - > #include "watchdog_core.h" > #include "watchdog_pretimeout.h" > >
Hi, Wim Van Sebroeck, Guenter Roeck: What's your opinion? Guenter Roeck given "Reviewed-by" two weeks ago. On 2020/8/27 21:40, Guenter Roeck wrote: > On 8/26/20 11:21 PM, Zhen Lei wrote: >> There has been no reference to "struct sched_param" since >> commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so >> there's no need to include <uapi/linux/sched/types.h> any more, delete >> it. >> >> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> > > Reviewed-by: Guenter Roeck <linux@roeck-us.net> > >> --- >> drivers/watchdog/watchdog_dev.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c >> index 6798addabd5a067..0f18fa2433310b0 100644 >> --- a/drivers/watchdog/watchdog_dev.c >> +++ b/drivers/watchdog/watchdog_dev.c >> @@ -43,8 +43,6 @@ >> #include <linux/watchdog.h> /* For watchdog specific items */ >> #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ >> >> -#include <uapi/linux/sched/types.h> /* For struct sched_param */ >> - >> #include "watchdog_core.h" >> #include "watchdog_pretimeout.h" >> >> > > >
On 9/7/20 12:50 AM, Leizhen (ThunderTown) wrote: > Hi, Wim Van Sebroeck, Guenter Roeck: > What's your opinion? Guenter Roeck given "Reviewed-by" two weeks ago. > The patch is in my watchdog-next branch, and Wim usually picks it up from there. Guenter > > On 2020/8/27 21:40, Guenter Roeck wrote: >> On 8/26/20 11:21 PM, Zhen Lei wrote: >>> There has been no reference to "struct sched_param" since >>> commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so >>> there's no need to include <uapi/linux/sched/types.h> any more, delete >>> it. >>> >>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> >> >> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >> >>> --- >>> drivers/watchdog/watchdog_dev.c | 2 -- >>> 1 file changed, 2 deletions(-) >>> >>> diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c >>> index 6798addabd5a067..0f18fa2433310b0 100644 >>> --- a/drivers/watchdog/watchdog_dev.c >>> +++ b/drivers/watchdog/watchdog_dev.c >>> @@ -43,8 +43,6 @@ >>> #include <linux/watchdog.h> /* For watchdog specific items */ >>> #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ >>> >>> -#include <uapi/linux/sched/types.h> /* For struct sched_param */ >>> - >>> #include "watchdog_core.h" >>> #include "watchdog_pretimeout.h" >>> >>> >> >> >> >
On 2020/9/8 10:40, Guenter Roeck wrote: > On 9/7/20 12:50 AM, Leizhen (ThunderTown) wrote: >> Hi, Wim Van Sebroeck, Guenter Roeck: >> What's your opinion? Guenter Roeck given "Reviewed-by" two weeks ago. >> > > The patch is in my watchdog-next branch, and Wim usually picks it up > from there. Oh, thanks. > > Guenter > >> >> On 2020/8/27 21:40, Guenter Roeck wrote: >>> On 8/26/20 11:21 PM, Zhen Lei wrote: >>>> There has been no reference to "struct sched_param" since >>>> commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so >>>> there's no need to include <uapi/linux/sched/types.h> any more, delete >>>> it. >>>> >>>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> >>> >>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>> >>>> --- >>>> drivers/watchdog/watchdog_dev.c | 2 -- >>>> 1 file changed, 2 deletions(-) >>>> >>>> diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c >>>> index 6798addabd5a067..0f18fa2433310b0 100644 >>>> --- a/drivers/watchdog/watchdog_dev.c >>>> +++ b/drivers/watchdog/watchdog_dev.c >>>> @@ -43,8 +43,6 @@ >>>> #include <linux/watchdog.h> /* For watchdog specific items */ >>>> #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ >>>> >>>> -#include <uapi/linux/sched/types.h> /* For struct sched_param */ >>>> - >>>> #include "watchdog_core.h" >>>> #include "watchdog_pretimeout.h" >>>> >>>> >>> >>> >>> >> > > >
On 2020/9/8 11:34, Leizhen (ThunderTown) wrote: > > > On 2020/9/8 10:40, Guenter Roeck wrote: >> On 9/7/20 12:50 AM, Leizhen (ThunderTown) wrote: >>> Hi, Wim Van Sebroeck, Guenter Roeck: >>> What's your opinion? Guenter Roeck given "Reviewed-by" two weeks ago. >>> >> >> The patch is in my watchdog-next branch, and Wim usually picks it up >> from there. > Hi, Guenter: Have you sent [GIT PULL] updates for 5.10, I don't see this patch in linux-next. > Oh, thanks. > >> >> Guenter >> >>> >>> On 2020/8/27 21:40, Guenter Roeck wrote: >>>> On 8/26/20 11:21 PM, Zhen Lei wrote: >>>>> There has been no reference to "struct sched_param" since >>>>> commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so >>>>> there's no need to include <uapi/linux/sched/types.h> any more, delete >>>>> it. >>>>> >>>>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> >>>> >>>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>>> >>>>> --- >>>>> drivers/watchdog/watchdog_dev.c | 2 -- >>>>> 1 file changed, 2 deletions(-) >>>>> >>>>> diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c >>>>> index 6798addabd5a067..0f18fa2433310b0 100644 >>>>> --- a/drivers/watchdog/watchdog_dev.c >>>>> +++ b/drivers/watchdog/watchdog_dev.c >>>>> @@ -43,8 +43,6 @@ >>>>> #include <linux/watchdog.h> /* For watchdog specific items */ >>>>> #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ >>>>> >>>>> -#include <uapi/linux/sched/types.h> /* For struct sched_param */ >>>>> - >>>>> #include "watchdog_core.h" >>>>> #include "watchdog_pretimeout.h" >>>>> >>>>> >>>> >>>> >>>> >>> >> >> >>
On 10/13/20 7:14 PM, Leizhen (ThunderTown) wrote: > > > On 2020/9/8 11:34, Leizhen (ThunderTown) wrote: >> >> >> On 2020/9/8 10:40, Guenter Roeck wrote: >>> On 9/7/20 12:50 AM, Leizhen (ThunderTown) wrote: >>>> Hi, Wim Van Sebroeck, Guenter Roeck: >>>> What's your opinion? Guenter Roeck given "Reviewed-by" two weeks ago. >>>> >>> >>> The patch is in my watchdog-next branch, and Wim usually picks it up >>> from there. >> > > Hi, Guenter: > Have you sent [GIT PULL] updates for 5.10, I don't see this patch in linux-next. > Wim does that. Unfortunately, he often does not pick up patches until shortly before he sends a pull request to Linus. Guenter >> Oh, thanks. >> >>> >>> Guenter >>> >>>> >>>> On 2020/8/27 21:40, Guenter Roeck wrote: >>>>> On 8/26/20 11:21 PM, Zhen Lei wrote: >>>>>> There has been no reference to "struct sched_param" since >>>>>> commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so >>>>>> there's no need to include <uapi/linux/sched/types.h> any more, delete >>>>>> it. >>>>>> >>>>>> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> >>>>> >>>>> Reviewed-by: Guenter Roeck <linux@roeck-us.net> >>>>> >>>>>> --- >>>>>> drivers/watchdog/watchdog_dev.c | 2 -- >>>>>> 1 file changed, 2 deletions(-) >>>>>> >>>>>> diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c >>>>>> index 6798addabd5a067..0f18fa2433310b0 100644 >>>>>> --- a/drivers/watchdog/watchdog_dev.c >>>>>> +++ b/drivers/watchdog/watchdog_dev.c >>>>>> @@ -43,8 +43,6 @@ >>>>>> #include <linux/watchdog.h> /* For watchdog specific items */ >>>>>> #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ >>>>>> >>>>>> -#include <uapi/linux/sched/types.h> /* For struct sched_param */ >>>>>> - >>>>>> #include "watchdog_core.h" >>>>>> #include "watchdog_pretimeout.h" >>>>>> >>>>>> >>>>> >>>>> >>>>> >>>> >>> >>> >>> >
diff --git a/drivers/watchdog/watchdog_dev.c b/drivers/watchdog/watchdog_dev.c index 6798addabd5a067..0f18fa2433310b0 100644 --- a/drivers/watchdog/watchdog_dev.c +++ b/drivers/watchdog/watchdog_dev.c @@ -43,8 +43,6 @@ #include <linux/watchdog.h> /* For watchdog specific items */ #include <linux/uaccess.h> /* For copy_to_user/put_user/... */ -#include <uapi/linux/sched/types.h> /* For struct sched_param */ - #include "watchdog_core.h" #include "watchdog_pretimeout.h"
There has been no reference to "struct sched_param" since commit 94beddacb53c ("sched,watchdog: Convert to sched_set_fifo()"), so there's no need to include <uapi/linux/sched/types.h> any more, delete it. Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> --- drivers/watchdog/watchdog_dev.c | 2 -- 1 file changed, 2 deletions(-)