[PATCH] Documentation: drm: Remove dangling pointer from drm-mm.rst

Sean Paul sean at poorly.run
Thu Nov 29 16:21:47 UTC 2018


On Thu, Nov 29, 2018 at 11:14 AM Christian König
<ckoenig.leichtzumerken at gmail.com> wrote:
>
> Am 29.11.18 um 17:11 schrieb Sean Paul:
> > On Thu, Nov 29, 2018 at 10:58 AM Koenig, Christian
> > <Christian.Koenig at amd.com> wrote:
> >> Am 29.11.18 um 16:55 schrieb Sean Paul:
> >>> From: Sean Paul <seanpaul at chromium.org>
> >>>
> >>> Fixes htmldocs build error:
> >>> Error: Cannot open file ../drivers/gpu/drm/drm_global.c
> >>>
> >>> Fixes: 2bb42410b1bd ("drm: Remove drm_global.{c,h} v2")
> >>> Cc: Thomas Zimmermann <tzimmermann at suse.de>
> >>> Cc: Christian König <christian.koenig at amd.com>
> >>> Cc: Junwei Zhang <Jerry.Zhang at amd.com>
> >>> Cc: Alex Deucher <alexander.deucher at amd.com>
> >>> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> >>> Cc: Maxime Ripard <maxime.ripard at bootlin.com>
> >>> Cc: Sean Paul <sean at poorly.run>
> >>> Cc: David Airlie <airlied at linux.ie>
> >>> Cc: Christian Koenig <christian.koenig at amd.com>
> >>> Cc: Huang Rui <ray.huang at amd.com>
> >>> Cc: dri-devel at lists.freedesktop.org
> >>> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> >> Reviewed-by: Christian König <christian.koenig at amd.com>
> >>
> > Thanks for the review. Perhaps the best course of action for this
> > patch is to have Dave pick it directly and then we can backmerge it
> > into amdgpu/drm-misc-next from there? I could also spin a topic branch
> > that we could all merge.
> >
> > Thoughts?
>
> My preference is that I pick it up and push it to Dave through Alex
> fixes push.
>
> But just because the patch which broke this came the same way as well.
>

Yeah, unfortunately it's proliferated, and I'd rather not add hops to
getting drm-misc-next fixed. Let's just do a topic branch and set a PR
to Dave from that. We can both merge the topic branch asap and let
Dave pick it up at his leisure.

Sean

> Either way works for me,
> Christian.
>
> >
> > Sean
> >
> >>> ---
> >>>    Documentation/gpu/drm-mm.rst | 3 ---
> >>>    1 file changed, 3 deletions(-)
> >>>
> >>> diff --git a/Documentation/gpu/drm-mm.rst b/Documentation/gpu/drm-mm.rst
> >>> index d0f3c6b032009..54a696d961a7c 100644
> >>> --- a/Documentation/gpu/drm-mm.rst
> >>> +++ b/Documentation/gpu/drm-mm.rst
> >>> @@ -79,9 +79,6 @@ count for the TTM, which will call your initialization function.
> >>>
> >>>    See the radeon_ttm.c file for an example of usage.
> >>>
> >>> -.. kernel-doc:: drivers/gpu/drm/drm_global.c
> >>> -   :export:
> >>> -
> >>>
> >>>    The Graphics Execution Manager (GEM)
> >>>    ====================================
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>


More information about the dri-devel mailing list