RE: [PATCH 1/1] gpu: drm: omapdrm: dss-of: add missing of_node_put after calling of_parse_phandle

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



 
>>
>> >
>> >Just an aside: When you do the same bugfix for multiple places it's
>> >good practice to submit it as one series (and cc everyone involved).
>> >Increases the odds that someone is in a good mood and reviews them
>> >all, instead of just the one affecting their own driver.
>>
>> Thanks, I realized that, and did it for later fixes for drm.
>> But if the bug fixes are at several subsystems, I think we should
>> split patch set per subsystem.
>
>Yeah, splitting per subsystem makes sense I'd say. I merged some of your patches,
>others are merged by driver maintainers directly. Is there anything left? If so, pls
>resend those as a new series to make sure it's all in one place.
>-Daniel
>

Thanks, Daniel.

I am afraid I don't know which one is merged, do you have a tree? And, I can re-send them all.
The latest patches which merged Linux-next is at [1], I remembered I dropped
one since the fix has already been there.

git log --oneline miss-of-node-put (for drm)

e167f2a gpu: drm: vc4_hdmi: add missing of_node_put after calling of_parse_phandle
9f86a38 gpu: drm: sun4i_drv: add missing of_node_put after calling of_parse_phandle
e0475ac gpu: drm: sti_vtg: add missing of_node_put after calling of_parse_phandle
99ad059 gpu: drm: sti_hqvdp: add missing of_node_put after calling of_parse_phandle
7d90d71 gpu: drm: sti_vdo: add missing of_node_put after calling of_parse_phandle
57c9e36 gpu: drm: sti_compositor: add missing of_node_put after calling of_parse_phandle
2cb11fb gpu: drm: rockchip_drm_drv: add missing of_node_put after calling of_parse_phandle
2668b9c gpu: drm: omapdrm: dss-of: add missing of_node_put after calling of_parse_phandle
ab81c2e gpu: drm: omapdrm: connector-dvi: add missing of_node_put after calling of_parse_phandle
4fc7ab7 gpn: drm: fsl_tcon: add missing of_node_put after calling of_parse_phandle
df5f61f gpu: drm: exynos_hdmi: add missing of_node_put after calling of_parse_phandle
6d294e6 gpu: drm: arcpgu_drv: add missing of_node_put after calling of_parse_phandle


[1] https://git.kernel.org/cgit/linux/kernel/git/peter.chen/usb.git/log/?h=fix-missing-of-node-put


Peter
>>
>> Peter
>>
>> >
>> >> ---
>> >>  drivers/gpu/drm/omapdrm/dss/dss-of.c | 7 ++++---
>> >>  1 file changed, 4 insertions(+), 3 deletions(-)
>> >>
>> >> diff --git a/drivers/gpu/drm/omapdrm/dss/dss-of.c
>> >> b/drivers/gpu/drm/omapdrm/dss/dss-of.c
>> >> index bf407b6..1ee6e5e 100644
>> >> --- a/drivers/gpu/drm/omapdrm/dss/dss-of.c
>> >> +++ b/drivers/gpu/drm/omapdrm/dss/dss-of.c
>> >> @@ -126,15 +126,16 @@ u32 dss_of_port_get_port_number(struct
>> >> device_node *port)
>> >>
>> >>  static struct device_node *omapdss_of_get_remote_port(const struct
>> >> device_node *node)  {
>> >> -	struct device_node *np;
>> >> +	struct device_node *np, *np_parent;
>> >>
>> >>  	np = of_parse_phandle(node, "remote-endpoint", 0);
>> >>  	if (!np)
>> >>  		return NULL;
>> >>
>> >> -	np = of_get_next_parent(np);
>> >> +	np_parent = of_get_next_parent(np);
>> >> +	of_node_put(np);
>> >>
>> >> -	return np;
>> >> +	return np_parent;
>> >>  }
>> >>
>> >>  struct device_node *
>> >> --
>> >> 1.9.1
>> >>
>> >> _______________________________________________
>> >> dri-devel mailing list
>> >> dri-devel@xxxxxxxxxxxxxxxxxxxxx
>> >> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>> >
>> >--
>> >Daniel Vetter
>> >Software Engineer, Intel Corporation
>> >http://blog.ffwll.ch
>
>--
>Daniel Vetter
>Software Engineer, Intel Corporation
>http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux