12 Oct, 2016

1 commit

  • Pull drm updates from Dave Airlie:
    "Core:
    - Fence destaging work
    - DRIVER_LEGACY to split off legacy drm drivers
    - drm_mm refactoring
    - Splitting drm_crtc.c into chunks and documenting better
    - Display info fixes
    - rbtree support for prime buffer lookup
    - Simple VGA DAC driver

    Panel:
    - Add Nexus 7 panel
    - More simple panels

    i915:
    - Refactoring GEM naming
    - Refactored vma/active tracking
    - Lockless request lookups
    - Better stolen memory support
    - FBC fixes
    - SKL watermark fixes
    - VGPU improvements
    - dma-buf fencing support
    - Better DP dongle support

    amdgpu:
    - Powerplay for Iceland asics
    - Improved GPU reset support
    - UVD/VEC powergating support for CZ/ST
    - Preinitialised VRAM buffer support
    - Virtual display support
    - Initial SI support
    - GTT rework
    - PCI shutdown callback support
    - HPD IRQ storm fixes

    amdkfd:
    - bugfixes

    tilcdc:
    - Atomic modesetting support

    mediatek:
    - AAL + GAMMA engine support
    - Hook up gamma LUT
    - Temporal dithering support

    imx:
    - Pixel clock from devicetree
    - drm bridge support for LVDS bridges
    - active plane reconfiguration
    - VDIC deinterlacer support
    - Frame synchronisation unit support
    - Color space conversion support

    analogix:
    - PSR support
    - Better panel on/off support

    rockchip:
    - rk3399 vop/crtc support
    - PSR support

    vc4:
    - Interlaced vblank timing
    - 3D rendering CPU overhead reduction
    - HDMI output fixes

    tda998x:
    - HDMI audio ASoC support

    sunxi:
    - Allwinner A33 support
    - better TCON support

    msm:
    - DT binding cleanups
    - Explicit fence-fd support

    sti:
    - remove sti415/416 support

    etnaviv:
    - MMUv2 refactoring
    - GC3000 support

    exynos:
    - Refactoring HDMI DCC/PHY
    - G2D pm regression fix
    - Page fault issues with wait for vblank

    There is no nouveau work in this tree, as Ben didn't get a pull
    request in, and he was fighting moving to atomic and adding mst
    support, so maybe best it waits for a cycle"

    * tag 'drm-for-v4.9' of git://people.freedesktop.org/~airlied/linux: (1412 commits)
    drm/crtc: constify drm_crtc_index parameter
    drm/i915: Fix conflict resolution from backmerge of v4.8-rc8 to drm-next
    drm/i915/guc: Unwind GuC workqueue reservation if request construction fails
    drm/i915: Reset the breadcrumbs IRQ more carefully
    drm/i915: Force relocations via cpu if we run out of idle aperture
    drm/i915: Distinguish last emitted request from last submitted request
    drm/i915: Allow DP to work w/o EDID
    drm/i915: Move long hpd handling into the hotplug work
    drm/i915/execlists: Reinitialise context image after GPU hang
    drm/i915: Use correct index for backtracking HUNG semaphores
    drm/i915: Unalias obj->phys_handle and obj->userptr
    drm/i915: Just clear the mmiodebug before a register access
    drm/i915/gen9: only add the planes actually affected by ddb changes
    drm/i915: Allow PCH DPLL sharing regardless of DPLL_SDVO_HIGH_SPEED
    drm/i915/bxt: Fix HDMI DPLL configuration
    drm/i915/gen9: fix the watermark res_blocks value
    drm/i915/gen9: fix plane_blocks_per_line on watermarks calculations
    drm/i915/gen9: minimum scanlines for Y tile is not always 4
    drm/i915/gen9: fix the WaWmMemoryReadLatency implementation
    drm/i915/kbl: KBL also needs to run the SAGV code
    ...

    Linus Torvalds
     

22 Sep, 2016

5 commits

  • We removed it in

    commit 6ab10b76ff6252bd9be0849c40f5865e39a29961
    Author: Daniel Vetter
    Date: Fri Aug 12 22:48:45 2016 +0200

    drm/kms: Nuke dirty_info property

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Signed-off-by: Sean Paul
    Link: http://patchwork.freedesktop.org/patch/msgid/1474448370-32227-7-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Again move it from the unmaintainable csv into DOC free-form overview
    sections.

    v2: Types Lionel&Sean spotted.

    Cc: Lionel Landwerlin
    Reviewed-by: Sean Paul
    Reviewed-by: Lionel Landwerlin
    Signed-off-by: Daniel Vetter
    Signed-off-by: Sean Paul
    Link: http://patchwork.freedesktop.org/patch/msgid/1474448370-32227-6-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Try to spec a bit more precisely how they all fit together, now that
    at least the code is for all the additional properties is in one
    place.

    Also remove the entries for the standardized properties from the
    table, because that thing is supremely unmaintaineable.

    v2: Fix typos Sean spotted.

    Cc: Ville Syrjälä
    Cc: Sean Paul
    Cc: Benjamin Gaignard
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Signed-off-by: Sean Paul
    Link: http://patchwork.freedesktop.org/patch/msgid/1474448370-32227-4-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Big thing is untangling and carefully documenting the different uapi
    types of planes. I also sprinkled a few more cross references around
    to make this easier to discover.

    As usual, remove the kerneldoc for internal functions which are not
    exported. Aside: We should probably go OCD on all the ioctl handlers
    and consistenly give them an _ioctl postfix.

    Acked-by: Archit Taneja
    Signed-off-by: Daniel Vetter
    Signed-off-by: Sean Paul
    Link: http://patchwork.freedesktop.org/patch/msgid/1474448370-32227-2-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Just pure code movement, cleanup and polish will happen in later
    patches.

    v2: Don't forget all the ioctl! To extract those cleanly I decided to
    put check_src_coords into drm_framebuffer.c (and give it a
    drm_framebuffer_ prefix), since that just checks framebuffer
    constraints.

    v3: rebase over PAGE_FLIP_TARGET.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter

    [seanpaul]
    This patch as posted on the list was rebased on:

    commit 6f00975c619064a18c23fd3aced325ae165a73b9
    Author: Daniel Vetter
    Date: Sat Aug 20 12:22:11 2016 +0200

    drm: Reject page_flip for !DRIVER_MODESET

    so as a result of moving the page_flip ioctl, this fix has
    been rolled into this patch.

    Signed-off-by: Sean Paul

    Daniel Vetter
     

19 Sep, 2016

2 commits

  • Some were still left in drm_crtc.h. Also include drm_edid.h in the
    rst files.

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

    Daniel Vetter
     
  • We don't want to burry the bridge structures kerneldoc in drm_crtc.h.

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

    Daniel Vetter
     

01 Sep, 2016

1 commit

  • Also provide some pointers for building IGT as some kernel hackers might
    not be that familiar with building stuff on Linux distros.

    Signed-off-by: Tomeu Vizoso
    Cc: Daniel Vetter
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1472715695-19812-1-git-send-email-tomeu.vizoso@collabora.com

    Tomeu Vizoso
     

29 Aug, 2016

5 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
     
  • Just for the struct drm_mode_object base class. The header file was
    already partially extracted to help untangle the include loops.

    v2:
    - Also move the generic get/set property ioctls. At first this seemed
    like a bad idea since it requires making drm_mode_crtc_set_obj_prop
    non-static. But eventually that will get split away too (like
    the connector version already is) for both crtc and planes. Hence I
    reconsidered.

    - drm_mode_object.[hc] instead of drm_modeset.[hc], which requires
    renaming the drm_modeset.h header I already started building up.
    This is more consistent (matches the name of the main structure),
    and I want to be able to use drm_modeset.[hc] for the basic modeset
    init/cleanup functionality like drm_mode_config_init.

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

    Daniel Vetter
     
  • - Move missing bits into struct drm_encoder docs.
    - Explain that encoders are 95% internal and only 5% uapi, and that in
    general the uapi part is broken.
    - Remove verbose comments for functions not exposed to drivers.

    v2: Review from Archit:
    - Appease checkpatch in the moved code.
    - Make it clearer that bridges are not exposed to userspace.

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

    Daniel Vetter
     
  • Same treatment as before. Only hiccup is drm_crtc_mask, which
    unfortunately can't be resolved until drm_crtc.h is less of a monster.
    Untangle the header loop with a forward declaration for that static
    inline.

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

    Daniel Vetter
     

24 Aug, 2016

2 commits

  • Reported-by: Chris Wilson
    Cc: Chris Wilson
    Fixes: b754b35b089d ("vgaarbiter: rst-ifiy and polish kerneldoc")
    Reviewed-by: Chris Wilson
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/20160824101916.30125-1-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Everyone knows them, except all the new folks joining from the ARM
    side haven't lived through all the pain of the past years and are
    entirely surprised when I raise this. Definitely time to document
    this.

    Last time this was a big discussion was about 6 years ago, when qcom
    tried to land a kernel driver without userspace. Dave Airlie made the
    rules really clear:

    http://airlied.livejournal.com/73115.html

    This write-up here is essentially what I've put into a presentation a
    while ago, which was also reviewed by Dave:

    http://blog.ffwll.ch/2015/05/gfx-kernel-upstreaming-requirements.html

    v2: Fix typos Eric&Rob spotted.

    v3: Nitpick from Jani.

    Cc: Dave Airlie
    Cc: Oded Gabbay
    Cc: Russell King
    Cc: Tomi Valkeinen
    Cc: Eric Anholt
    Cc: Thomas Hellstrom
    Cc: Sinclair Yeh
    Cc: Lucas Stach
    Cc: Benjamin Gaignard
    Cc: Mark Yao
    Cc: Laurent Pinchart
    Cc: Ben Skeggs
    Cc: Rob Clark
    Cc: CK Hu
    Cc: Xinliang Liu
    Cc: Philipp Zabel
    Cc: Stefan Agner
    Cc: Inki Dae
    Cc: Maxime Ripard
    Cc: Boris Brezillon
    Cc: Jani Nikula
    Cc: Daniel Vetter
    Cc: Thierry Reding
    Cc: Christian König
    Cc: Alex Deucher
    Cc: Gerd Hoffmann
    Cc: Brian Starkey
    Cc: Liviu Dudau
    Cc: Alexey Brodkin
    Acked-by: Dave Airlie
    Reviewed-by: Rob Clark
    Reviewed-by: Christian König
    Reviewed-by: Eric Anholt
    Reviewed-by: Jani Nikula
    Signed-off-by: Daniel Vetter
    Signed-off-by: Dave Airlie

    Daniel Vetter
     

19 Aug, 2016

1 commit


17 Aug, 2016

4 commits

  • Move the documentation into Documentation/gpu, link it up and pull in
    the kernel doc.

    No actual text changes except that I did polish the kerneldoc a bit,
    especially for vga_client_register().

    v2: Remove some rst from vga-switcheroo.rst that I don't understand,
    but which seems to be the reason why the new vgaarbiter.rst sometimes
    drops out of the sidebar index.

    v3: Drop one level of headings and clarify the vgaarb one a bit.

    v4: Fix some typos (Sean).

    Cc: Jonathan Corbet
    Cc: linux-doc@vger.kernel.org
    Cc: Sean Paul
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-20-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • - Shuffle docs from drm-kms.rst into the structure docs where it makes
    sense.
    - Put the remaining bits into a new overview section.

    One thing I've changed is around probing: Old docs says that you
    _must_ use the probe helpers, which isn't correct. Helpers are always
    optional.

    v2: Review from Sean.

    Cc: Sean Paul
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-17-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • There's not much point in kerneldoc if it's not included:
    - It won't show up in the pretty html pages.
    - The comments itself won't get parsed, which means 0day won't pick up
    changes, resulting in stale docs fast.

    Also, uapi really should be core, not helpers, so move drm_blend.c to
    that. That also means that the zpos normilize function loses it's
    helper status (and we might as well call it always). For that,
    EXPORT_SYMBOL. Just spotted while integrating docs and noticing that
    one was missing.

    With sphinx there's really no excuse any more to not build the docs
    and make sure it's all nice!

    $ make DOCBOOKS="" htmldocs

    Cc: Marek Szyprowski
    Cc: Benjamin Gaignard
    Cc: Laurent Pinchart
    Cc: Ville Syrjälä
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-15-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Pulls in quite a lot of connector related structures (cmdline mode,
    force/status enums, display info), but I think that all makes perfect
    sense.

    Also had to move a few more core kms object stuff into drm_modeset.h.

    And as a first cleanup remove the kerneldoc for the 2 connector IOCTL
    - DRM core docs are aimed at drivers, no point documenting internal in
    excruciating detail.

    v2: And also pull in all the connector property code.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-14-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     

16 Aug, 2016

7 commits

  • - Move the intro section into a DOC comment, and update it slightly.
    - kernel-doc for struct drm_framebuffer!

    v2:
    - Copypaste fail (Sean).
    - Explain the linear @offsets clearer (Ville).

    Cc: Sean Paul
    Cc: Ville Syrjälä
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-12-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • Also start with drm_modeset.h with the core bits, since we need
    to untangle this mess somehow. That allows us to move the drm_modes.h
    include to the right spot, except for the temporary connector status
    enum. That will get fixed as soon as drm_connector.h exists.

    v2: Rebase.

    v3: Move drm_crtc_force_disable_all back again, that wasn't meant to
    be moved (Sean).

    v4: Rebase.

    Cc: Sean Paul
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter

    Daniel Vetter
     
  • Accidentally the wrong file. Oops.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-10-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • - Again adjust headings a bit, and don't mix up the initialization
    sections with other stuff.
    - Remove the doc for output polling, that vfunc is now properly
    documented in the vfunc reference sections.
    - Move the grab-bag with all the core stuff (i.e. drm_crtc.[hc]) to
    the front for a more prominent place.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-5-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • - Readjust headings - we lost one level through the extraction into a
    separate .rst file.
    - Merge helper reference sections with the helper documentation - that
    split was just an artifact of the docbook toolchain sucking at too
    deep nesting levels. No such problems with sphinx.
    - Move the cma helpers in with the gem documentation, since they're
    helpers to implement gem using CMA/dma memory as a backend.

    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-4-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • While reviewing docs I spotted that we have a few functions that
    really just don't fit into their containing helper library section.
    Extract them and shovel them all into a new library for random one-off
    aux stuff.

    v2: Remove wrongly added files for real.

    Cc: Sean Paul
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-3-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • - Move the common vtable stuff to the top
    - Move "Tile Group" to a more appropriate heading level
    - Throw away the old intro for the crtc helpers (it's entirely stale,
    e.g. helpers have become modular years ago), and replace it with a
    general intro about the motivation behind helpers.
    - Reorder helpers to group them together a bit better, and explain
    that grouping in the intro.
    - Make sure the introductory DOC section is always first.

    v2:
    - Remove bogus files accidentally added (Sean).
    - Spelling fixes (Sean).

    Cc: Sean Paul
    Reviewed-by: Sean Paul
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1471034937-651-2-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     

15 Aug, 2016

2 commits

  • - refactor ddi buffer programming a bit (Ville)
    - large-scale renaming to untangle naming in the gem code (Chris)
    - rework vma/active tracking for accurately reaping idle mappings of shared
    objects (Chris)
    - misc dp sst/mst probing corner case fixes (Ville)
    - tons of cleanup&tunings all around in gem
    - lockless (rcu-protected) request lookup, plus use it everywhere for
    non(b)locking waits (Chris)
    - pipe crc debugfs fixes (Rodrigo)
    - random fixes all over

    * tag 'drm-intel-next-2016-08-08' of git://anongit.freedesktop.org/drm-intel: (222 commits)
    drm/i915: Update DRIVER_DATE to 20160808
    drm/i915: fix aliasing_ppgtt leak
    drm/i915: Update comment before i915_spin_request
    drm/i915: Use drm official vblank_no_hw_counter callback.
    drm/i915: Fix copy_to_user usage for pipe_crc
    Revert "drm/i915: Track active streams also for DP SST"
    drm/i915: fix WaInsertDummyPushConstPs
    drm/i915: Assert that the request hasn't been retired
    drm/i915: Repack fence tiling mode and stride into a single integer
    drm/i915: Document and reject invalid tiling modes
    drm/i915: Remove locking for get_tiling
    drm/i915: Remove pinned check from madvise ioctl
    drm/i915: Reduce locking inside swfinish ioctl
    drm/i915: Remove (struct_mutex) locking for busy-ioctl
    drm/i915: Remove (struct_mutex) locking for wait-ioctl
    drm/i915: Do a nonblocking wait first in pread/pwrite
    drm/i915: Remove unused no-shrinker-steal
    drm/i915: Tidy generation of the GTT mmap offset
    drm/i915/shrinker: Wait before acquiring struct_mutex under oom
    drm/i915: Simplify do_idling() (Ironlake vt-d w/a)
    ...

    Dave Airlie
     
  • With the gpu/conf.py, the gpu folder can be build and distributed
    stand-alone. To compile only the html of 'gpu' folder use::

    make SPHINXDIRS="gpu" htmldocs

    Signed-off-by: Markus Heiser
    Signed-off-by: Jonathan Corbet

    Markus Heiser
     

08 Aug, 2016

1 commit

  • It's super confusing that new drivers need to be marked with
    DRIVER_MODESET when really it means DRIVER_MODERN. Much better to
    invert the meaning and rename it to something that's suitably
    off-putting.

    Since there's over 100 places using DRIVER_MODESET we need to roll out
    this change without a flag day.

    v2: Update docs.

    Reviewed-by: Frank Binns
    Reviewed-by: David Herrmann
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1470251470-30830-1-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     

05 Aug, 2016

2 commits


29 Jul, 2016

1 commit

  • version 8:
    - move drm_blend.o from drm-y to drm_kms_helper-y to avoid
    EXPORT_SYMBOL(drm_atomic_helper_normalize_zpos)
    - remove dead function declarations in drm_crtc.h

    version 7:
    - remove useless EXPORT_SYMBOL()
    - better z-order wording in Documentation

    version 6:
    - add zpos in gpu documentation file
    - merge Ville patch about zpos initial value and API improvement.
    I have split Ville patch between zpos core and drivers

    version 5:
    - remove zpos range check and comeback to 0 to N-1
    normalization algorithm

    version 4:
    - make sure that normalized zpos value is stay
    in the defined property range and warn user if not

    This patch adds support for generic plane's zpos property property with
    well-defined semantics:
    - added zpos properties to plane and plane state structures
    - added helpers for normalizing zpos properties of given set of planes
    - well defined semantics: planes are sorted by zpos values and then plane
    id value if zpos equals

    Normalized zpos values are calculated automatically when generic
    muttable zpos property has been initialized. Drivers can simply use
    plane_state->normalized_zpos in their atomic_check and/or plane_update
    callbacks without any additional calls to DRM core.

    Signed-off-by: Marek Szyprowski

    Compare to Marek's original patch zpos property is now specific to each
    plane and no more to the core.
    Normalize function take care of the range of per plane defined range
    before set normalized_zpos.

    Signed-off-by: Benjamin Gaignard
    Reviewed-by: Ville Syrjälä
    Acked-by: Laurent Pinchart
    Tested-by: Laurent Pinchart

    Cc: Inki Dae
    Cc: Daniel Vetter
    Cc: Ville Syrjala
    Cc: Joonyoung Shim
    Cc: Seung-Woo Kim
    Cc: Andrzej Hajda
    Cc: Krzysztof Kozlowski
    Cc: Bartlomiej Zolnierkiewicz
    Cc: Tobias Jakobi
    Cc: Gustavo Padovan
    Cc: vincent.abriou@st.com
    Cc: fabien.dessenne@st.com
    Cc: Laurent Pinchart

    Marek Szyprowski
     

19 Jul, 2016

3 commits

  • This is unfortunately not all the sphinx noise when building the gpu
    documentations. But the remaining warnings have completely bogus line
    and probably also file sources, and I just can't find them.

    Probably some serious debuggging of the line annotations in the
    sphinx/kernel-doc toolchain needed first.

    Acked-by: Chris Wilson
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1468612088-9721-11-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     
  • The drm_irq docs want one function from drmP.h, but that one is a
    serious mess. Extract it, and while at it improve the docs a bit.
    There's a bit a header loop issue since core data structures like
    drm_device and drm_driver aren't in their own headers yet, which means
    the drm_irq.h include in drmP.h needs to be in just the right spot :(

    Also noticed that drm_vblank_crtc->last_wait is entirely unused,
    remove it.

    v2: git add drm_irq.h ...

    Acked-by: Chris Wilson
    Signed-off-by: Daniel Vetter

    Daniel Vetter
     
  • Right now there's nothing, and kernel-doc produces a warning because
    of that. Remove it until we need it for a clean build.

    Cc: Laurent Pinchart
    Acked-by: Chris Wilson
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/1468612088-9721-1-git-send-email-daniel.vetter@ffwll.ch

    Daniel Vetter
     

12 Jul, 2016

1 commit

  • Fix up formatting glitches remaining after the automatic rst conversion.

    Cc: Jonathan Corbet
    Cc: Jani Nikula
    Signed-off-by: Lukas Wunner
    Signed-off-by: Daniel Vetter
    Link: http://patchwork.freedesktop.org/patch/msgid/3a369602ae35fdbf5e4a12f7e172088c89fa27bb.1468225027.git.lukas@wunner.de

    Lukas Wunner
     

22 Jun, 2016

2 commits

  • Also extract drm_auth.h for nicer grouping.

    v2: Nuke the other comments since they don't really explain a lot, and
    within the drm core we generally only document functions exported to
    drivers: The main audience for these docs are driver writers.

    v3: Limit the exposure of drm_master internals by only including
    drm_auth.h where it is neede (Chris).

    v4: Spelling polish (Emil).

    Cc: Chris Wilson
    Reviewed-by: Chris Wilson
    Reviewed-by: Emil Velikov
    Signed-off-by: Daniel Vetter

    Daniel Vetter
     
  • We already have a fallback in place to fill out the unique from
    dev->unique, which is set to something reasonable in drm_dev_alloc.

    Which means we only need to have a special set_busid for pci devices,
    to be able to care the backwards compat code for drm 1.1 around, which
    libdrm still needs.

    While developing and testing this patch things blew up in really
    interesting ways, and the code is rather confusing in naming things
    between the kernel code, ioctl #defines and libdrm. For the next brave
    dragon slayer, document all this madness properly in the userspace
    interface section of gpu.tmpl.

    v2: Make drm_dev_set_unique static and update kerneldoc.

    v3: Entire rewrite, plus document what's going on for posterity in the
    gpu docbook uapi section.

    v4: Drop accidental amdgpu hunk (Emil).

    v5: Drop accidental omapdrm vblank counter change (Emil).

    v6: Rebase on top of the sphinx conversion.

    Cc: Gustavo Padovan
    Cc: Emil Velikov
    Tested-by: Gustavo Padovan (virt_gpu)
    Reviewed-by: Emil Velikov
    Signed-off-by: Daniel Vetter

    Daniel Vetter