On Wed, Jul 13, 2016 at 01:22:06AM +0000, Peter Chen wrote:
>
> >
> >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
>
> 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
