Re: [PATCH] drm: Don't export internal module variables

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

 



Hi

On Mon, Jun 2, 2014 at 10:01 AM, Daniel Vetter <daniel@xxxxxxxx> wrote:
> On Sun, Jun 01, 2014 at 02:04:45PM +0200, David Herrmann wrote:
>> Hi
>>
>> On Thu, May 29, 2014 at 7:25 PM, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote:
>> > Drivers really have no business touching these. Noticed because
>> > exynose _did_ touch the vblank off delay, which could potentially
>> > affect other drivers.
>> >
>> > drm_debug is an exception since it's used in macros and inline
>> > functions.
>> >
>> > v2: Drop bonghits changes. Note to self: Don't submit patches
>> > before first coffee.
>> >
>> > Cc: Inki Dae <inki.dae@xxxxxxxxxxx>
>> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
>> > ---
>> >  drivers/gpu/drm/drm_stub.c              | 4 ----
>> >  drivers/gpu/drm/exynos/exynos_drm_drv.c | 4 ----
>> >  drivers/gpu/drm/exynos/exynos_drm_drv.h | 2 --
>> >  3 files changed, 10 deletions(-)
>> >
>> > diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c
>> > index 3727ac8bc310..998782a88520 100644
>> > --- a/drivers/gpu/drm/drm_stub.c
>> > +++ b/drivers/gpu/drm/drm_stub.c
>> > @@ -43,17 +43,13 @@ unsigned int drm_debug = 0; /* 1 to enable debug output */
>> >  EXPORT_SYMBOL(drm_debug);
>> >
>> >  unsigned int drm_rnodes = 0;   /* 1 to enable experimental render nodes API */
>> > -EXPORT_SYMBOL(drm_rnodes);
>> >
>> >  /* 1 to allow user space to request universal planes (experimental) */
>> >  unsigned int drm_universal_planes = 0;
>> > -EXPORT_SYMBOL(drm_universal_planes);
>> >
>> >  unsigned int drm_vblank_offdelay = 5000;    /* Default to 5000 msecs. */
>> > -EXPORT_SYMBOL(drm_vblank_offdelay);
>> >
>> >  unsigned int drm_timestamp_precision = 20;  /* Default to 20 usecs. */
>> > -EXPORT_SYMBOL(drm_timestamp_precision);
>> >
>> >  /*
>> >   * Default to use monotonic timestamps for wait-for-vblank and page-flip
>> > diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c b/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> > index 2d27ba23a6a8..d7c50b0da510 100644
>> > --- a/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> > +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.c
>> > @@ -38,8 +38,6 @@
>> >  #define DRIVER_MAJOR   1
>> >  #define DRIVER_MINOR   0
>> >
>> > -#define VBLANK_OFF_DELAY       50000
>> > -
>> >  /* platform device pointer for eynos drm device. */
>> >  static struct platform_device *exynos_drm_pdev;
>> >
>> > @@ -109,8 +107,6 @@ static int exynos_drm_load(struct drm_device *dev, unsigned long flags)
>> >         /* setup possible_clones. */
>> >         exynos_drm_encoder_setup(dev);
>> >
>> > -       drm_vblank_offdelay = VBLANK_OFF_DELAY;
>>
>> You know that this is 5,000 by default and VBLANK_OFF_DELAY is 50,000?
>> Not sure whether that was a typo or was explicitly set this way. If
>> that change is intentional:
>
> Oops, didn't spot that, nice catch. git log says this was added to paper
> over a race in get/put_vblank in exynos. Wtf, imo even more reason to nuke
> this ... I've pimped the commit message a bit - still r-b?

You mean:

commit 52c68814cd7f3592292a02b890b5b8625e9069a7
Author: Inki Dae <inki.dae@xxxxxxxxxxx>
Date:   Fri Dec 16 21:31:12 2011 +0900

    drm/exynos: extend vblank off delay time.

Yeah, that one doesn't make any sense. So this is still:

Reviewed-by: David Herrmann <dh.herrmann@xxxxxxxxx>

Thanks
David
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://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