[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_OSjJad7QGOCrWQL+LcjtCHQsCefpJ=K0mYUayFe9bUAA@mail.gmail.com>
Date: Thu, 13 Jun 2019 14:52:34 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: Sam Ravnborg <sam@...nborg.org>
Cc: Hariprasad Kelam <hariprasad.kelam@...il.com>,
LKML <linux-kernel@...r.kernel.org>,
Yongqiang Sun <yongqiang.sun@....com>,
Charlene Liu <charlene.liu@....com>,
Leo Li <sunpeng.li@....com>,
Bhawanpreet Lakha <Bhawanpreet.Lakha@....com>,
Gloria Li <geling.li@....com>,
amd-gfx list <amd-gfx@...ts.freedesktop.org>,
Nicholas Kazlauskas <nicholas.kazlauskas@....com>,
David Airlie <airlied@...ux.ie>,
Dmytro Laktyushkin <Dmytro.Laktyushkin@....com>,
Tony Cheng <tony.cheng@....com>,
Maling list - DRI developers
<dri-devel@...ts.freedesktop.org>,
Alex Deucher <alexander.deucher@....com>,
Christian König <christian.koenig@....com>,
Anthony Koo <Anthony.Koo@....com>
Subject: Re: [PATCH] drm/amd/display: fix compilation error
On Thu, Jun 13, 2019 at 2:42 PM Sam Ravnborg <sam@...nborg.org> wrote:
>
> Hi Alex.
>
> On Wed, Jun 12, 2019 at 10:35:26PM -0400, Alex Deucher wrote:
> > On Wed, Jun 12, 2019 at 10:34 PM Hariprasad Kelam
> > <hariprasad.kelam@...il.com> wrote:
> > >
> > > this patch fixes below compilation error
> > >
> > > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: In
> > > function ‘dcn10_apply_ctx_for_surface’:
> > > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2378:3:
> > > error: implicit declaration of function ‘udelay’
> > > [-Werror=implicit-function-declaration]
> > > udelay(underflow_check_delay_us);
> > >
> > > Signed-off-by: Hariprasad Kelam <hariprasad.kelam@...il.com>
>
> Am I right in assuming you took this patch?
>
> I expect that new code using udelay was added to the amd tree,
> and when merged with drm-misc-next it failed, because drm-misc-next no
> longer had drmP.h included so no implicit include of delay.h
>
> The root cause was that my original patchset should have been based
> on the amd tree, and applied there :-(
No worries. I've picked it up.
Alex
Powered by blists - more mailing lists