12 Feb, 2019

1 commit

  • This will allow __drm_mode_object_file to be extended to perform
    access control checks based on the file in use.

    v2: Also fix up vboxvideo driver in staging

    [airlied: merging early as this is an API change]

    Suggested-by: Daniel Vetter
    Signed-off-by: Keith Packard
    Signed-off-by: Dave Airlie
    (cherry picked from commit 418da17214aca5ef5f0b6f7588905ee7df92f98f)

    Keith Packard
     

14 Jul, 2017

1 commit

  • The function has little to do with atomic, it's just where it has so
    far been needed. So, rename it to drm_property_replace_blob, move it
    to drm_property.c and export it.

    Change the semantics to return whether the blob was replaced instead
    of using an extra argument for that.

    Signed-off-by: Peter Rosin
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20170713162538.22788-2-peda@axentia.se

    Peter Rosin
     

09 Jun, 2017

1 commit

  • There is no reason why the name field should not be const, but
    several why it should. The struct should only be used by
    drm_property_create_enum() and there the name-field from the struct
    is passed to drm_property_add_enum(), which takes a const char * as
    a parameter.

    Signed-off-by: Jyri Sarha
    Signed-off-by: Ville Syrjälä
    Link: http://patchwork.freedesktop.org/patch/msgid/5dd3b6a1e20452bd8abdcbc55d1e8d7f56262266.1496161066.git.jsarha@ti.com

    Jyri Sarha
     

28 Feb, 2017

1 commit

  • For consistency with other reference counting APIs in the kernel, add
    drm_property_blob_get() and drm_property_blob_put() to reference count
    DRM blob properties.

    Compatibility aliases are added to keep existing code working. To help
    speed up the transition, all the instances of the old functions in the
    DRM core are already replaced in this commit.

    A semantic patch is provided that can be used to convert all drivers to
    the new helpers.

    Reviewed-by: Sean Paul
    Acked-by: Christian König
    Signed-off-by: Thierry Reding
    Link: http://patchwork.freedesktop.org/patch/msgid/20170228144643.5668-7-thierry.reding@gmail.com

    Thierry Reding
     

25 Jan, 2017

1 commit

  • I just learned that &struct_name.member_name works and looks pretty
    even. It doesn't (yet) link to the member directly though, which would
    be really good for big structures or vfunc tables (where the
    per-member kerneldoc tends to be long).

    Also some minor drive-by polish where it makes sense, I read a lot
    of docs ...

    v2: Review from Eric.

    Cc: Jani Nikula
    Cc: Chris Wilson
    Reviewed-by: Eric Engestrom
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20170125062657.19270-4-daniel.vetter@ffwll.ch

    Daniel Vetter
     

19 Sep, 2016

1 commit

  • Now that there's less stuff in there I noticed that I overlooked them.
    Sprinkle some docs over them while at it.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160831160913.12991-2-daniel.vetter@ffwll.ch

    Daniel Vetter
     

29 Aug, 2016

2 commits

  • - remove kerneldoc for drm-internal functions
    - drm_property_replace_global_blob isn't actually atomic, and doesn't
    need to be. Update docs&comments to match
    - document all the types and try to link things a bit better
    - nits all over

    v2: Appease checkpatch in the moved code (Archit)

    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-9-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • This just contains the base property classes and all the code to
    handle blobs. I think for any kind of standardized/shared properties
    it's better to have separate files - this is fairly big already as-is.

    v2: resurrect misplaced hunk (Daniel Stone)

    Cc: Daniel Stone
    Reviewed-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160829082757.17913-7-daniel.vetter@ffwll.ch

    Daniel Vetter