* [PATCH] firmware: arm_scmi: remove duplicate include header @ 2023-02-10 7:20 ye.xingchen 2023-02-10 11:29 ` Cristian Marussi 2023-03-09 14:21 ` Sudeep Holla 0 siblings, 2 replies; 5+ messages in thread From: ye.xingchen @ 2023-02-10 7:20 UTC (permalink / raw) To: sudeep.holla; +Cc: cristian.marussi, linux-arm-kernel, linux-kernel From: Ye Xingchen <ye.xingchen@zte.com.cn> linux/of.h is included more than once. Signed-off-by: Ye Xingchen <ye.xingchen@zte.com.cn> --- drivers/firmware/arm_scmi/bus.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/firmware/arm_scmi/bus.c b/drivers/firmware/arm_scmi/bus.c index 73140b854b31..b2d242178fbc 100644 --- a/drivers/firmware/arm_scmi/bus.c +++ b/drivers/firmware/arm_scmi/bus.c @@ -14,7 +14,6 @@ #include <linux/kernel.h> #include <linux/slab.h> #include <linux/device.h> -#include <linux/of.h> #include "common.h" -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] firmware: arm_scmi: remove duplicate include header 2023-02-10 7:20 [PATCH] firmware: arm_scmi: remove duplicate include header ye.xingchen @ 2023-02-10 11:29 ` Cristian Marussi 2023-02-10 20:16 ` Saravana Kannan 2023-02-23 14:42 ` Sudeep Holla 2023-03-09 14:21 ` Sudeep Holla 1 sibling, 2 replies; 5+ messages in thread From: Cristian Marussi @ 2023-02-10 11:29 UTC (permalink / raw) To: ye.xingchen; +Cc: sudeep.holla, linux-arm-kernel, saravanak, arnd, linux-kernel On Fri, Feb 10, 2023 at 03:20:07PM +0800, ye.xingchen@zte.com.cn wrote: > From: Ye Xingchen <ye.xingchen@zte.com.cn> > > linux/of.h is included more than once. > > Signed-off-by: Ye Xingchen <ye.xingchen@zte.com.cn> > --- > drivers/firmware/arm_scmi/bus.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/firmware/arm_scmi/bus.c b/drivers/firmware/arm_scmi/bus.c > index 73140b854b31..b2d242178fbc 100644 > --- a/drivers/firmware/arm_scmi/bus.c > +++ b/drivers/firmware/arm_scmi/bus.c > @@ -14,7 +14,6 @@ > #include <linux/kernel.h> > #include <linux/slab.h> > #include <linux/device.h> > -#include <linux/of.h> > Hi Ye, thanks for this. Looking at next-20230210 it seem like the linux/of.h include that you removed was added by: commit 96da4a99ce507bb98e2ba77018a2f91999c8c12b Author: Saravana Kannan <saravanak@google.com> Date: Mon Feb 6 17:42:03 2023 -0800 firmware: arm_scmi: Set fwnode for the scmi_device ...AND indeed there was already a linux/of.h include added by a preceding commit by myself commit d3cd7c525fd2ecce3a6c963f314969a54783d211 Author: Cristian Marussi <cristian.marussi@arm.com> Date: Thu Dec 22 18:50:45 2022 +0000 firmware: arm_scmi: Refactor protocol device creation ...BUT they both hit next- together so the clash. My series is queued for the next merge window on soc/, not sure about the state of Saravana series, but if that series will be queued too for v6.3 we'll need to pick up your fix indeed, Ye. Probably better to wait and pick it up in v6.3-rc1 once the situation will be more clear as what is going in. Up to Sudeep anyway the choice. Thanks, Cristian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] firmware: arm_scmi: remove duplicate include header 2023-02-10 11:29 ` Cristian Marussi @ 2023-02-10 20:16 ` Saravana Kannan 2023-02-23 14:42 ` Sudeep Holla 1 sibling, 0 replies; 5+ messages in thread From: Saravana Kannan @ 2023-02-10 20:16 UTC (permalink / raw) To: Cristian Marussi Cc: ye.xingchen, sudeep.holla, linux-arm-kernel, arnd, linux-kernel On Fri, Feb 10, 2023 at 3:29 AM Cristian Marussi <cristian.marussi@arm.com> wrote: > > On Fri, Feb 10, 2023 at 03:20:07PM +0800, ye.xingchen@zte.com.cn wrote: > > From: Ye Xingchen <ye.xingchen@zte.com.cn> > > > > linux/of.h is included more than once. > > > > Signed-off-by: Ye Xingchen <ye.xingchen@zte.com.cn> > > --- > > drivers/firmware/arm_scmi/bus.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/firmware/arm_scmi/bus.c b/drivers/firmware/arm_scmi/bus.c > > index 73140b854b31..b2d242178fbc 100644 > > --- a/drivers/firmware/arm_scmi/bus.c > > +++ b/drivers/firmware/arm_scmi/bus.c > > @@ -14,7 +14,6 @@ > > #include <linux/kernel.h> > > #include <linux/slab.h> > > #include <linux/device.h> > > -#include <linux/of.h> > > > > Hi Ye, > > thanks for this. > > Looking at next-20230210 it seem like the linux/of.h include that you > removed was added by: > > commit 96da4a99ce507bb98e2ba77018a2f91999c8c12b > Author: Saravana Kannan <saravanak@google.com> > Date: Mon Feb 6 17:42:03 2023 -0800 > > firmware: arm_scmi: Set fwnode for the scmi_device > > ...AND indeed there was already a linux/of.h include added by a preceding > commit by myself > > commit d3cd7c525fd2ecce3a6c963f314969a54783d211 > Author: Cristian Marussi <cristian.marussi@arm.com> > Date: Thu Dec 22 18:50:45 2022 +0000 > > firmware: arm_scmi: Refactor protocol device creation > > ...BUT they both hit next- together so the clash. > > My series is queued for the next merge window on soc/, not sure about the > state of Saravana series, but if that series will be queued too for v6.3 > we'll need to pick up your fix indeed, Ye. AFAIU, my series is queued up for 6.3. -Saravana _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] firmware: arm_scmi: remove duplicate include header 2023-02-10 11:29 ` Cristian Marussi 2023-02-10 20:16 ` Saravana Kannan @ 2023-02-23 14:42 ` Sudeep Holla 1 sibling, 0 replies; 5+ messages in thread From: Sudeep Holla @ 2023-02-23 14:42 UTC (permalink / raw) To: Cristian Marussi Cc: ye.xingchen, linux-arm-kernel, Sudeep Holla, saravanak, arnd, linux-kernel On Fri, Feb 10, 2023 at 11:29:11AM +0000, Cristian Marussi wrote: > On Fri, Feb 10, 2023 at 03:20:07PM +0800, ye.xingchen@zte.com.cn wrote: > > From: Ye Xingchen <ye.xingchen@zte.com.cn> > > > > linux/of.h is included more than once. > > > > Signed-off-by: Ye Xingchen <ye.xingchen@zte.com.cn> > > --- > > drivers/firmware/arm_scmi/bus.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/firmware/arm_scmi/bus.c b/drivers/firmware/arm_scmi/bus.c > > index 73140b854b31..b2d242178fbc 100644 > > --- a/drivers/firmware/arm_scmi/bus.c > > +++ b/drivers/firmware/arm_scmi/bus.c > > @@ -14,7 +14,6 @@ > > #include <linux/kernel.h> > > #include <linux/slab.h> > > #include <linux/device.h> > > -#include <linux/of.h> > > > > Hi Ye, > > thanks for this. > > Looking at next-20230210 it seem like the linux/of.h include that you > removed was added by: > > commit 96da4a99ce507bb98e2ba77018a2f91999c8c12b > Author: Saravana Kannan <saravanak@google.com> > Date: Mon Feb 6 17:42:03 2023 -0800 > > firmware: arm_scmi: Set fwnode for the scmi_device > > ...AND indeed there was already a linux/of.h include added by a preceding > commit by myself > > commit d3cd7c525fd2ecce3a6c963f314969a54783d211 > Author: Cristian Marussi <cristian.marussi@arm.com> > Date: Thu Dec 22 18:50:45 2022 +0000 > > firmware: arm_scmi: Refactor protocol device creation > > ...BUT they both hit next- together so the clash. > > My series is queued for the next merge window on soc/, not sure about the > state of Saravana series, but if that series will be queued too for v6.3 > we'll need to pick up your fix indeed, Ye. > > Probably better to wait and pick it up in v6.3-rc1 once the situation > will be more clear as what is going in. > > Up to Sudeep anyway the choice. > Yes I will pick this up once we have the issue(i.e. both the above commit) in the main upstream branch. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] firmware: arm_scmi: remove duplicate include header 2023-02-10 7:20 [PATCH] firmware: arm_scmi: remove duplicate include header ye.xingchen 2023-02-10 11:29 ` Cristian Marussi @ 2023-03-09 14:21 ` Sudeep Holla 1 sibling, 0 replies; 5+ messages in thread From: Sudeep Holla @ 2023-03-09 14:21 UTC (permalink / raw) To: ye.xingchen Cc: Sudeep Holla, cristian.marussi, linux-arm-kernel, linux-kernel On Fri, 10 Feb 2023 15:20:07 +0800, ye.xingchen@zte.com.cn wrote: > linux/of.h is included more than once. > Applied to sudeep.holla/linux (for-next/scmi/fixes), thanks! [1/1] firmware: arm_scmi: remove duplicate include header https://git.kernel.org/sudeep.holla/c/418a406d92cc -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-03-09 14:24 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-02-10 7:20 [PATCH] firmware: arm_scmi: remove duplicate include header ye.xingchen 2023-02-10 11:29 ` Cristian Marussi 2023-02-10 20:16 ` Saravana Kannan 2023-02-23 14:42 ` Sudeep Holla 2023-03-09 14:21 ` Sudeep Holla
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).