* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-04-29 23:43 [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev() Douglas Anderson
@ 2022-04-30 0:34 ` Saravana Kannan
2022-04-30 20:21 ` Sakari Ailus
2022-05-01 7:49 ` Andy Shevchenko
2 siblings, 0 replies; 7+ messages in thread
From: Saravana Kannan @ 2022-04-30 0:34 UTC (permalink / raw)
To: Douglas Anderson
Cc: Rafael J . Wysocki, swboyd, Andy Shevchenko, freedreno,
Bjorn Andersson, Sakari Ailus, linux-arm-msm, Daniel Scally,
Greg Kroah-Hartman, Heikki Krogerus, Rafael J. Wysocki,
linux-acpi, linux-kernel
On Fri, Apr 29, 2022 at 4:43 PM Douglas Anderson <dianders@chromium.org> wrote:
>
> Due to a subtle typo, instead of commit 87ffea09470d ("device
> property: Introduce fwnode_for_each_parent_node()") being a no-op
> change, it ended up causing the display on my sc7180-trogdor-lazor
> device from coming up unless I added "fw_devlink=off" to my kernel
> command line. Fix the typo.
Thank you for the fix!
Reviewed-by: Saravana Kannan <saravanak@google.com>
-Saravana
>
> Fixes: 87ffea09470d ("device property: Introduce fwnode_for_each_parent_node()")
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
>
> drivers/base/property.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/base/property.c b/drivers/base/property.c
> index 36401cfe432c..52e85dcb20b5 100644
> --- a/drivers/base/property.c
> +++ b/drivers/base/property.c
> @@ -600,7 +600,7 @@ struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode)
> struct device *dev;
>
> fwnode_for_each_parent_node(fwnode, parent) {
> - dev = get_dev_from_fwnode(fwnode);
> + dev = get_dev_from_fwnode(parent);
> if (dev) {
> fwnode_handle_put(parent);
> return dev;
> --
> 2.36.0.464.gb9c8b46e94-goog
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-04-29 23:43 [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev() Douglas Anderson
2022-04-30 0:34 ` Saravana Kannan
@ 2022-04-30 20:21 ` Sakari Ailus
2022-05-01 7:49 ` Andy Shevchenko
2 siblings, 0 replies; 7+ messages in thread
From: Sakari Ailus @ 2022-04-30 20:21 UTC (permalink / raw)
To: Douglas Anderson
Cc: Rafael J . Wysocki, swboyd, Andy Shevchenko, freedreno,
Saravana Kannan, Bjorn Andersson, linux-arm-msm, Daniel Scally,
Greg Kroah-Hartman, Heikki Krogerus, Rafael J. Wysocki,
linux-acpi, linux-kernel
On Fri, Apr 29, 2022 at 04:43:47PM -0700, Douglas Anderson wrote:
> Due to a subtle typo, instead of commit 87ffea09470d ("device
> property: Introduce fwnode_for_each_parent_node()") being a no-op
> change, it ended up causing the display on my sc7180-trogdor-lazor
> device from coming up unless I added "fw_devlink=off" to my kernel
> command line. Fix the typo.
>
> Fixes: 87ffea09470d ("device property: Introduce fwnode_for_each_parent_node()")
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
Good catch, thanks!
Reviewed-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
>
> drivers/base/property.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/base/property.c b/drivers/base/property.c
> index 36401cfe432c..52e85dcb20b5 100644
> --- a/drivers/base/property.c
> +++ b/drivers/base/property.c
> @@ -600,7 +600,7 @@ struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode)
> struct device *dev;
>
> fwnode_for_each_parent_node(fwnode, parent) {
> - dev = get_dev_from_fwnode(fwnode);
> + dev = get_dev_from_fwnode(parent);
> if (dev) {
> fwnode_handle_put(parent);
> return dev;
--
Sakari Ailus
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-04-29 23:43 [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev() Douglas Anderson
2022-04-30 0:34 ` Saravana Kannan
2022-04-30 20:21 ` Sakari Ailus
@ 2022-05-01 7:49 ` Andy Shevchenko
2022-05-05 12:20 ` Rafael J. Wysocki
2 siblings, 1 reply; 7+ messages in thread
From: Andy Shevchenko @ 2022-05-01 7:49 UTC (permalink / raw)
To: Douglas Anderson
Cc: Rafael J . Wysocki, Stephen Boyd, Andy Shevchenko, freedreno,
Saravana Kannan, Bjorn Andersson, Sakari Ailus, linux-arm-msm,
Daniel Scally, Greg Kroah-Hartman, Heikki Krogerus,
Rafael J. Wysocki, ACPI Devel Maling List,
Linux Kernel Mailing List
On Sat, Apr 30, 2022 at 3:00 PM Douglas Anderson <dianders@chromium.org> wrote:
>
> Due to a subtle typo, instead of commit 87ffea09470d ("device
> property: Introduce fwnode_for_each_parent_node()") being a no-op
> change, it ended up causing the display on my sc7180-trogdor-lazor
> device from coming up unless I added "fw_devlink=off" to my kernel
> command line. Fix the typo.
Sorry and merci pour la fix!
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Fixes: 87ffea09470d ("device property: Introduce fwnode_for_each_parent_node()")
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
>
> drivers/base/property.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/base/property.c b/drivers/base/property.c
> index 36401cfe432c..52e85dcb20b5 100644
> --- a/drivers/base/property.c
> +++ b/drivers/base/property.c
> @@ -600,7 +600,7 @@ struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode)
> struct device *dev;
>
> fwnode_for_each_parent_node(fwnode, parent) {
> - dev = get_dev_from_fwnode(fwnode);
> + dev = get_dev_from_fwnode(parent);
> if (dev) {
> fwnode_handle_put(parent);
> return dev;
> --
> 2.36.0.464.gb9c8b46e94-goog
>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-05-01 7:49 ` Andy Shevchenko
@ 2022-05-05 12:20 ` Rafael J. Wysocki
2022-05-14 3:58 ` Saravana Kannan
0 siblings, 1 reply; 7+ messages in thread
From: Rafael J. Wysocki @ 2022-05-05 12:20 UTC (permalink / raw)
To: Andy Shevchenko, Douglas Anderson
Cc: Rafael J . Wysocki, Stephen Boyd, Andy Shevchenko, freedreno,
Saravana Kannan, Bjorn Andersson, Sakari Ailus, linux-arm-msm,
Daniel Scally, Greg Kroah-Hartman, Heikki Krogerus,
Rafael J. Wysocki, ACPI Devel Maling List,
Linux Kernel Mailing List
On Sun, May 1, 2022 at 9:50 AM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Sat, Apr 30, 2022 at 3:00 PM Douglas Anderson <dianders@chromium.org> wrote:
> >
> > Due to a subtle typo, instead of commit 87ffea09470d ("device
> > property: Introduce fwnode_for_each_parent_node()") being a no-op
> > change, it ended up causing the display on my sc7180-trogdor-lazor
> > device from coming up unless I added "fw_devlink=off" to my kernel
> > command line. Fix the typo.
>
> Sorry and merci pour la fix!
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Applied, thanks!
> > Fixes: 87ffea09470d ("device property: Introduce fwnode_for_each_parent_node()")
> > Signed-off-by: Douglas Anderson <dianders@chromium.org>
> > ---
> >
> > drivers/base/property.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/base/property.c b/drivers/base/property.c
> > index 36401cfe432c..52e85dcb20b5 100644
> > --- a/drivers/base/property.c
> > +++ b/drivers/base/property.c
> > @@ -600,7 +600,7 @@ struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode)
> > struct device *dev;
> >
> > fwnode_for_each_parent_node(fwnode, parent) {
> > - dev = get_dev_from_fwnode(fwnode);
> > + dev = get_dev_from_fwnode(parent);
> > if (dev) {
> > fwnode_handle_put(parent);
> > return dev;
> > --
> > 2.36.0.464.gb9c8b46e94-goog
> >
>
>
> --
> With Best Regards,
> Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-05-05 12:20 ` Rafael J. Wysocki
@ 2022-05-14 3:58 ` Saravana Kannan
2022-05-14 5:47 ` Greg Kroah-Hartman
0 siblings, 1 reply; 7+ messages in thread
From: Saravana Kannan @ 2022-05-14 3:58 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Andy Shevchenko, Douglas Anderson, Rafael J . Wysocki,
Stephen Boyd, Andy Shevchenko, freedreno, Bjorn Andersson,
Sakari Ailus, linux-arm-msm, Daniel Scally, Greg Kroah-Hartman,
Heikki Krogerus, ACPI Devel Maling List,
Linux Kernel Mailing List
On Thu, May 5, 2022 at 5:21 AM Rafael J. Wysocki <rafael@kernel.org> wrote:
>
> On Sun, May 1, 2022 at 9:50 AM Andy Shevchenko
> <andy.shevchenko@gmail.com> wrote:
> >
> > On Sat, Apr 30, 2022 at 3:00 PM Douglas Anderson <dianders@chromium.org> wrote:
> > >
> > > Due to a subtle typo, instead of commit 87ffea09470d ("device
> > > property: Introduce fwnode_for_each_parent_node()") being a no-op
> > > change, it ended up causing the display on my sc7180-trogdor-lazor
> > > device from coming up unless I added "fw_devlink=off" to my kernel
> > > command line. Fix the typo.
> >
> > Sorry and merci pour la fix!
> > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>
> Applied, thanks!
Has this been picked up by one of the driver-core branches yet? I was
poking around that
git repo and didn't see any commit with this title. This breaks
fw_devlink in a severe manner, so I want to make sure it gets into
5.18.
-Saravana
-Saravana
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] device property: Fix recent breakage of fwnode_get_next_parent_dev()
2022-05-14 3:58 ` Saravana Kannan
@ 2022-05-14 5:47 ` Greg Kroah-Hartman
0 siblings, 0 replies; 7+ messages in thread
From: Greg Kroah-Hartman @ 2022-05-14 5:47 UTC (permalink / raw)
To: Saravana Kannan
Cc: Rafael J. Wysocki, Andy Shevchenko, Douglas Anderson,
Rafael J . Wysocki, Stephen Boyd, Andy Shevchenko, freedreno,
Bjorn Andersson, Sakari Ailus, linux-arm-msm, Daniel Scally,
Heikki Krogerus, ACPI Devel Maling List,
Linux Kernel Mailing List
On Fri, May 13, 2022 at 08:58:12PM -0700, Saravana Kannan wrote:
> On Thu, May 5, 2022 at 5:21 AM Rafael J. Wysocki <rafael@kernel.org> wrote:
> >
> > On Sun, May 1, 2022 at 9:50 AM Andy Shevchenko
> > <andy.shevchenko@gmail.com> wrote:
> > >
> > > On Sat, Apr 30, 2022 at 3:00 PM Douglas Anderson <dianders@chromium.org> wrote:
> > > >
> > > > Due to a subtle typo, instead of commit 87ffea09470d ("device
> > > > property: Introduce fwnode_for_each_parent_node()") being a no-op
> > > > change, it ended up causing the display on my sc7180-trogdor-lazor
> > > > device from coming up unless I added "fw_devlink=off" to my kernel
> > > > command line. Fix the typo.
> > >
> > > Sorry and merci pour la fix!
> > > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> >
> > Applied, thanks!
>
> Has this been picked up by one of the driver-core branches yet? I was
> poking around that
> git repo and didn't see any commit with this title. This breaks
> fw_devlink in a severe manner, so I want to make sure it gets into
> 5.18.
Look in linux-next:
117ef574074d ("device property: Fix recent breakage of fwnode_get_next_parent_dev()")
Rafael's tree is not my tree :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread