drm/i915: the great header refactoring, part one

Submitted by Jani Nikula on April 4, 2019, 9:14 p.m.

Details

Reviewer None
Submitted April 4, 2019, 9:14 p.m.
Last Updated April 6, 2019, 8:08 a.m.
Revision 2

Cover Letter(s)

Revision 1
      intel_drv.h has grown out of proportions, and turned into a dumping
ground. Way back when it was useful to have only a handful of headers,
but we're long past that.

Start splitting off per-module headers. The basic principles:

* Make the new headers self-contained (i.e. can be compiled without
  including other headers first), and test this using the new infra for
  that.

* Use minimal includes for making the headers self-contained. Use
  forward declarations for structs where applicable, and e.g. include
  <linux/types.h> instead of <linux/kernel.h>.

* Only split off the headers, and mostly refrain from doing other
  refactoring while at it. (There are a few minor things.)

* Mostly only split off function declarations. Splitting off types is
  left for follow-up work.

* Include the new headers only where needed. This leads to a lot of
  includes here and there, but on the other hand increases the clarity
  of the relationships between the modules. (And already raises a bunch
  of questions about the split and cross-calls between some
  modules. It'll be easier to analyze this.)

* Wherever adding new includes, group the includes by <linux/...> first,
  then <drm/...>, then "...", and sort the groups alphabetically.

* Choice of what to extract first here is purely arbitrary.

* Follow-up work should consider renaming functions according to the
  module, i.e. functions in intel_foo.c should be prefixed
  intel_foo_. Better naming will be helpful in further organizing the
  driver, as well as grasping the structure to begin with.

BR,
Jani.


Jani Nikula (25):
  drm/i915: make intel_frontbuffer.h self-contained
  drm/i915: extract intel_audio.h from intel_drv.h
  drm/i915: extract intel_crt.h from intel_drv.h
  drm/i915: extract intel_ddi.h from intel_drv.h
  drm/i915: extract intel_connector.h from intel_drv.h
  drm/i915: extract intel_csr.h from intel_drv.h
  drm/i915: extract intel_fbc.h from intel_drv.h
  drm/i915: extract intel_psr.h from intel_drv.h
  drm/i915: extract intel_color.h from intel_drv.h
  drm/i915: extract intel_lspcon.h from intel_drv.h
  drm/i915: extract intel_sdvo.h from intel_drv.h
  drm/i915: extract intel_hdcp.h from intel_drv.h
  drm/i915: extract intel_panel.h from intel_drv.h
  drm/i915: extract intel_pm.h from intel_drv.h
  drm/i915: extract intel_fbdev.h from intel_drv.h
  drm/i915: extract intel_dp.h from intel_drv.h
  drm/i915: extract intel_hdmi.h from intel_drv.h
  drm/i915: extract intel_atomic_plane.h from intel_drv.h
  drm/i915: extract intel_pipe_crc.h from intel_drv.h
  drm/i915: extract intel_tv.h from intel_drv.h
  drm/i915: extract intel_lvds.h from intel_drv.h
  drm/i915: extract intel_dvo.h from intel_drv.h
  drm/i915: extract intel_sprite.h from intel_drv.h
  drm/i915: extract intel_cdclk.h from intel_drv.h
  drm/i915/cdclk: have only one init/uninit function

 drivers/gpu/drm/i915/Makefile.header-test     |  24 +
 drivers/gpu/drm/i915/i915_debugfs.c           |  13 +-
 drivers/gpu/drm/i915/i915_drv.c               |  11 +-
 drivers/gpu/drm/i915/i915_drv.h               |  11 +-
 drivers/gpu/drm/i915/i915_gem.c               |   1 +
 drivers/gpu/drm/i915/i915_irq.c               |  11 +-
 drivers/gpu/drm/i915/i915_pci.c               |   1 +
 drivers/gpu/drm/i915/i915_request.c           |   3 +-
 drivers/gpu/drm/i915/i915_reset.h             |   1 +
 drivers/gpu/drm/i915/i915_suspend.c           |   5 +-
 drivers/gpu/drm/i915/icl_dsi.c                |   6 +-
 drivers/gpu/drm/i915/intel_atomic.c           |   2 +
 drivers/gpu/drm/i915/intel_atomic_plane.c     |  36 +-
 drivers/gpu/drm/i915/intel_atomic_plane.h     |  40 ++
 drivers/gpu/drm/i915/intel_audio.c            |  12 +-
 drivers/gpu/drm/i915/intel_audio.h            |  24 +
 drivers/gpu/drm/i915/intel_cdclk.c            | 121 ++--
 drivers/gpu/drm/i915/intel_cdclk.h            |  46 ++
 drivers/gpu/drm/i915/intel_color.h            |  17 +
 drivers/gpu/drm/i915/intel_connector.c        |   8 +-
 drivers/gpu/drm/i915/intel_connector.h        |  35 ++
 drivers/gpu/drm/i915/intel_crt.c              |   7 +-
 drivers/gpu/drm/i915/intel_crt.h              |  21 +
 drivers/gpu/drm/i915/intel_csr.h              |  17 +
 drivers/gpu/drm/i915/intel_ddi.c              |   9 +
 drivers/gpu/drm/i915/intel_ddi.h              |  53 ++
 drivers/gpu/drm/i915/intel_display.c          |  28 +-
 drivers/gpu/drm/i915/intel_dp.c               |  20 +-
 drivers/gpu/drm/i915/intel_dp.h               | 121 ++++
 drivers/gpu/drm/i915/intel_dp_link_training.c |   1 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |   9 +-
 drivers/gpu/drm/i915/intel_dpio_phy.c         |   1 +
 drivers/gpu/drm/i915/intel_drv.h              | 580 +-----------------
 drivers/gpu/drm/i915/intel_dvo.c              |   9 +-
 drivers/gpu/drm/i915/intel_dvo.h              |  13 +
 drivers/gpu/drm/i915/intel_fbc.c              |   3 +-
 drivers/gpu/drm/i915/intel_fbc.h              |  42 ++
 drivers/gpu/drm/i915/intel_fbdev.c            |  17 +-
 drivers/gpu/drm/i915/intel_fbdev.h            |  53 ++
 drivers/gpu/drm/i915/intel_fifo_underrun.c    |   1 +
 drivers/gpu/drm/i915/intel_frontbuffer.c      |   5 +-
 drivers/gpu/drm/i915/intel_frontbuffer.h      |  10 +
 drivers/gpu/drm/i915/intel_hdcp.h             |  33 +
 drivers/gpu/drm/i915/intel_hdmi.c             |  16 +-
 drivers/gpu/drm/i915/intel_hdmi.h             |  51 ++
 drivers/gpu/drm/i915/intel_lspcon.c           |   5 +-
 drivers/gpu/drm/i915/intel_lspcon.h           |  38 ++
 drivers/gpu/drm/i915/intel_lvds.c             |   8 +-
 drivers/gpu/drm/i915/intel_lvds.h             |  22 +
 drivers/gpu/drm/i915/intel_opregion.c         |   3 +-
 drivers/gpu/drm/i915/intel_panel.c            |   2 +
 drivers/gpu/drm/i915/intel_panel.h            |  65 ++
 drivers/gpu/drm/i915/intel_pipe_crc.h         |  35 ++
 drivers/gpu/drm/i915/intel_pm.c               |   5 +-
 drivers/gpu/drm/i915/intel_pm.h               |  72 +++
 drivers/gpu/drm/i915/intel_psr.c              |  13 +-
 drivers/gpu/drm/i915/intel_psr.h              |  40 ++
 drivers/gpu/drm/i915/intel_runtime_pm.c       |  20 +-
 drivers/gpu/drm/i915/intel_sdvo.c             |  12 +-
 drivers/gpu/drm/i915/intel_sdvo.h             |  23 +
 drivers/gpu/drm/i915/intel_sprite.c           |  16 +-
 drivers/gpu/drm/i915/intel_sprite.h           |  55 ++
 drivers/gpu/drm/i915/intel_tv.c               |   4 +-
 drivers/gpu/drm/i915/intel_tv.h               |  13 +
 drivers/gpu/drm/i915/intel_uncore.c           |   9 +-
 drivers/gpu/drm/i915/vlv_dsi.c                |  10 +-
 66 files changed, 1267 insertions(+), 751 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/intel_atomic_plane.h
 create mode 100644 drivers/gpu/drm/i915/intel_audio.h
 create mode 100644 drivers/gpu/drm/i915/intel_cdclk.h
 create mode 100644 drivers/gpu/drm/i915/intel_color.h
 create mode 100644 drivers/gpu/drm/i915/intel_connector.h
 create mode 100644 drivers/gpu/drm/i915/intel_crt.h
 create mode 100644 drivers/gpu/drm/i915/intel_csr.h
 create mode 100644 drivers/gpu/drm/i915/intel_ddi.h
 create mode 100644 drivers/gpu/drm/i915/intel_dp.h
 create mode 100644 drivers/gpu/drm/i915/intel_dvo.h
 create mode 100644 drivers/gpu/drm/i915/intel_fbc.h
 create mode 100644 drivers/gpu/drm/i915/intel_fbdev.h
 create mode 100644 drivers/gpu/drm/i915/intel_hdcp.h
 create mode 100644 drivers/gpu/drm/i915/intel_hdmi.h
 create mode 100644 drivers/gpu/drm/i915/intel_lspcon.h
 create mode 100644 drivers/gpu/drm/i915/intel_lvds.h
 create mode 100644 drivers/gpu/drm/i915/intel_panel.h
 create mode 100644 drivers/gpu/drm/i915/intel_pipe_crc.h
 create mode 100644 drivers/gpu/drm/i915/intel_pm.h
 create mode 100644 drivers/gpu/drm/i915/intel_psr.h
 create mode 100644 drivers/gpu/drm/i915/intel_sdvo.h
 create mode 100644 drivers/gpu/drm/i915/intel_sprite.h
 create mode 100644 drivers/gpu/drm/i915/intel_tv.h
    
Revision 2
      v2 of [1] with:

- each foo.c now includes the corresponding new foo.h to avoid sparse warnings

- minor sparse/checkpatch/whitespace fixes

- commit message tab screwup fixed

I didn't add changelog to each patch for the first two.

Took the liberty of adding Chris' r-b despite the changes.

BR,
Jani.


[1] https://patchwork.freedesktop.org/series/59022/


Jani Nikula (25):
  drm/i915: make intel_frontbuffer.h self-contained
  drm/i915: extract intel_audio.h from intel_drv.h
  drm/i915: extract intel_crt.h from intel_drv.h
  drm/i915: extract intel_ddi.h from intel_drv.h
  drm/i915: extract intel_connector.h from intel_drv.h
  drm/i915: extract intel_csr.h from intel_drv.h
  drm/i915: extract intel_fbc.h from intel_drv.h
  drm/i915: extract intel_psr.h from intel_drv.h
  drm/i915: extract intel_color.h from intel_drv.h
  drm/i915: extract intel_lspcon.h from intel_drv.h
  drm/i915: extract intel_sdvo.h from intel_drv.h
  drm/i915: extract intel_hdcp.h from intel_drv.h
  drm/i915: extract intel_panel.h from intel_drv.h
  drm/i915: extract intel_pm.h from intel_drv.h
  drm/i915: extract intel_fbdev.h from intel_drv.h
  drm/i915: extract intel_dp.h from intel_drv.h
  drm/i915: extract intel_hdmi.h from intel_drv.h
  drm/i915: extract intel_atomic_plane.h from intel_drv.h
  drm/i915: extract intel_pipe_crc.h from intel_drv.h
  drm/i915: extract intel_tv.h from intel_drv.h
  drm/i915: extract intel_lvds.h from intel_drv.h
  drm/i915: extract intel_dvo.h from intel_drv.h
  drm/i915: extract intel_sprite.h from intel_drv.h
  drm/i915: extract intel_cdclk.h from intel_drv.h
  drm/i915/cdclk: have only one init/uninit function

 drivers/gpu/drm/i915/Makefile.header-test     |  24 +
 drivers/gpu/drm/i915/i915_debugfs.c           |  13 +-
 drivers/gpu/drm/i915/i915_drv.c               |  11 +-
 drivers/gpu/drm/i915/i915_drv.h               |  11 +-
 drivers/gpu/drm/i915/i915_gem.c               |   1 +
 drivers/gpu/drm/i915/i915_irq.c               |  11 +-
 drivers/gpu/drm/i915/i915_pci.c               |   1 +
 drivers/gpu/drm/i915/i915_request.c           |   3 +-
 drivers/gpu/drm/i915/i915_reset.h             |   1 +
 drivers/gpu/drm/i915/i915_suspend.c           |   4 +-
 drivers/gpu/drm/i915/icl_dsi.c                |   6 +-
 drivers/gpu/drm/i915/intel_atomic.c           |   2 +
 drivers/gpu/drm/i915/intel_atomic_plane.c     |   3 +
 drivers/gpu/drm/i915/intel_atomic_plane.h     |  40 ++
 drivers/gpu/drm/i915/intel_audio.c            |  12 +-
 drivers/gpu/drm/i915/intel_audio.h            |  24 +
 drivers/gpu/drm/i915/intel_cdclk.c            | 121 ++--
 drivers/gpu/drm/i915/intel_cdclk.h            |  46 ++
 drivers/gpu/drm/i915/intel_color.c            |   1 +
 drivers/gpu/drm/i915/intel_color.h            |  17 +
 drivers/gpu/drm/i915/intel_connector.c        |   9 +-
 drivers/gpu/drm/i915/intel_connector.h        |  35 ++
 drivers/gpu/drm/i915/intel_crt.c              |   7 +-
 drivers/gpu/drm/i915/intel_crt.h              |  21 +
 drivers/gpu/drm/i915/intel_csr.c              |   3 +
 drivers/gpu/drm/i915/intel_csr.h              |  17 +
 drivers/gpu/drm/i915/intel_ddi.c              |  10 +
 drivers/gpu/drm/i915/intel_ddi.h              |  53 ++
 drivers/gpu/drm/i915/intel_display.c          |  28 +-
 drivers/gpu/drm/i915/intel_dp.c               |  20 +-
 drivers/gpu/drm/i915/intel_dp.h               | 121 ++++
 drivers/gpu/drm/i915/intel_dp_link_training.c |   1 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |   9 +-
 drivers/gpu/drm/i915/intel_dpio_phy.c         |   1 +
 drivers/gpu/drm/i915/intel_drv.h              | 581 +-----------------
 drivers/gpu/drm/i915/intel_dvo.c              |  10 +-
 drivers/gpu/drm/i915/intel_dvo.h              |  13 +
 drivers/gpu/drm/i915/intel_fbc.c              |   4 +-
 drivers/gpu/drm/i915/intel_fbc.h              |  42 ++
 drivers/gpu/drm/i915/intel_fbdev.c            |  17 +-
 drivers/gpu/drm/i915/intel_fbdev.h            |  53 ++
 drivers/gpu/drm/i915/intel_fifo_underrun.c    |   1 +
 drivers/gpu/drm/i915/intel_frontbuffer.c      |   5 +-
 drivers/gpu/drm/i915/intel_frontbuffer.h      |  10 +
 drivers/gpu/drm/i915/intel_hdcp.c             |  10 +-
 drivers/gpu/drm/i915/intel_hdcp.h             |  33 +
 drivers/gpu/drm/i915/intel_hdmi.c             |  17 +-
 drivers/gpu/drm/i915/intel_hdmi.h             |  51 ++
 drivers/gpu/drm/i915/intel_lspcon.c           |   6 +-
 drivers/gpu/drm/i915/intel_lspcon.h           |  38 ++
 drivers/gpu/drm/i915/intel_lvds.c             |   9 +-
 drivers/gpu/drm/i915/intel_lvds.h             |  22 +
 drivers/gpu/drm/i915/intel_opregion.c         |   3 +-
 drivers/gpu/drm/i915/intel_panel.c            |   3 +
 drivers/gpu/drm/i915/intel_panel.h            |  65 ++
 drivers/gpu/drm/i915/intel_pipe_crc.c         |   4 +-
 drivers/gpu/drm/i915/intel_pipe_crc.h         |  35 ++
 drivers/gpu/drm/i915/intel_pm.c               |   5 +-
 drivers/gpu/drm/i915/intel_pm.h               |  71 +++
 drivers/gpu/drm/i915/intel_psr.c              |  13 +-
 drivers/gpu/drm/i915/intel_psr.h              |  40 ++
 drivers/gpu/drm/i915/intel_runtime_pm.c       |  20 +-
 drivers/gpu/drm/i915/intel_sdvo.c             |  13 +-
 drivers/gpu/drm/i915/intel_sdvo.h             |  23 +
 drivers/gpu/drm/i915/intel_sprite.c           |  16 +-
 drivers/gpu/drm/i915/intel_sprite.h           |  55 ++
 drivers/gpu/drm/i915/intel_tv.c               |   5 +-
 drivers/gpu/drm/i915/intel_tv.h               |  13 +
 drivers/gpu/drm/i915/intel_uncore.c           |   9 +-
 drivers/gpu/drm/i915/vlv_dsi.c                |  10 +-
 70 files changed, 1273 insertions(+), 739 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/intel_atomic_plane.h
 create mode 100644 drivers/gpu/drm/i915/intel_audio.h
 create mode 100644 drivers/gpu/drm/i915/intel_cdclk.h
 create mode 100644 drivers/gpu/drm/i915/intel_color.h
 create mode 100644 drivers/gpu/drm/i915/intel_connector.h
 create mode 100644 drivers/gpu/drm/i915/intel_crt.h
 create mode 100644 drivers/gpu/drm/i915/intel_csr.h
 create mode 100644 drivers/gpu/drm/i915/intel_ddi.h
 create mode 100644 drivers/gpu/drm/i915/intel_dp.h
 create mode 100644 drivers/gpu/drm/i915/intel_dvo.h
 create mode 100644 drivers/gpu/drm/i915/intel_fbc.h
 create mode 100644 drivers/gpu/drm/i915/intel_fbdev.h
 create mode 100644 drivers/gpu/drm/i915/intel_hdcp.h
 create mode 100644 drivers/gpu/drm/i915/intel_hdmi.h
 create mode 100644 drivers/gpu/drm/i915/intel_lspcon.h
 create mode 100644 drivers/gpu/drm/i915/intel_lvds.h
 create mode 100644 drivers/gpu/drm/i915/intel_panel.h
 create mode 100644 drivers/gpu/drm/i915/intel_pipe_crc.h
 create mode 100644 drivers/gpu/drm/i915/intel_pm.h
 create mode 100644 drivers/gpu/drm/i915/intel_psr.h
 create mode 100644 drivers/gpu/drm/i915/intel_sdvo.h
 create mode 100644 drivers/gpu/drm/i915/intel_sprite.h
 create mode 100644 drivers/gpu/drm/i915/intel_tv.h
    

Revisions

Patches download mbox

# Name Submitter State A F R T
[01/25] drm/i915: make intel_frontbuffer.h self-contained Jani Nikula Accepted
[02/25] drm/i915: extract intel_audio.h from intel_drv.h Jani Nikula Accepted
[03/25] drm/i915: extract intel_crt.h from intel_drv.h Jani Nikula Accepted
[04/25] drm/i915: extract intel_ddi.h from intel_drv.h Jani Nikula New
[05/25] drm/i915: extract intel_connector.h from intel_drv.h Jani Nikula New
[06/25] drm/i915: extract intel_csr.h from intel_drv.h Jani Nikula New
[07/25] drm/i915: extract intel_fbc.h from intel_drv.h Jani Nikula New
[08/25] drm/i915: extract intel_psr.h from intel_drv.h Jani Nikula New
[09/25] drm/i915: extract intel_color.h from intel_drv.h Jani Nikula New
[10/25] drm/i915: extract intel_lspcon.h from intel_drv.h Jani Nikula New
[11/25] drm/i915: extract intel_sdvo.h from intel_drv.h Jani Nikula New
[12/25] drm/i915: extract intel_hdcp.h from intel_drv.h Jani Nikula New
[13/25] drm/i915: extract intel_panel.h from intel_drv.h Jani Nikula New
[14/25] drm/i915: extract intel_pm.h from intel_drv.h Jani Nikula New
[15/25] drm/i915: extract intel_fbdev.h from intel_drv.h Jani Nikula Accepted
[16/25] drm/i915: extract intel_dp.h from intel_drv.h Jani Nikula New
[17/25] drm/i915: extract intel_hdmi.h from intel_drv.h Jani Nikula New
[18/25] drm/i915: extract intel_atomic_plane.h from intel_drv.h Jani Nikula New
[19/25] drm/i915: extract intel_pipe_crc.h from intel_drv.h Jani Nikula New
[20/25] drm/i915: extract intel_tv.h from intel_drv.h Jani Nikula New
[21/25] drm/i915: extract intel_lvds.h from intel_drv.h Jani Nikula New
[22/25] drm/i915: extract intel_dvo.h from intel_drv.h Jani Nikula New
[23/25] drm/i915: extract intel_sprite.h from intel_drv.h Jani Nikula New
[24/25] drm/i915: extract intel_cdclk.h from intel_drv.h Jani Nikula Accepted
[25/25] drm/i915/cdclk: have only one init/uninit function Jani Nikula New

Tests

CI Bug Log - changes from CI_DRM_5875 -> Patchwork_12687
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/59022/revisions/1/mbox/

Known issues
------------

  Here are the changes found in Patchwork_12687 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_basic@basic-bsd2:
    - fi-kbl-7500u:       NOTRUN -> SKIP [fdo#109271] +9

  * igt@gem_exec_basic@gtt-bsd2:
    - fi-byt-clapper:     NOTRUN -> SKIP [fdo#109271] +12

  * igt@gem_exec_suspend@basic-s3:
    - fi-byt-clapper:     NOTRUN -> INCOMPLETE [fdo#102657]

  * igt@i915_selftest@live_contexts:
    - fi-bdw-gvtdvm:      PASS -> DMESG-FAIL [fdo#110235 ]

  * igt@i915_selftest@live_execlists:
    - fi-apl-guc:         PASS -> INCOMPLETE [fdo#103927] / [fdo#109720]

  * igt@kms_busy@basic-flip-c:
    - fi-blb-e6850:       NOTRUN -> SKIP [fdo#109271] / [fdo#109278]

  * igt@kms_chamelium@dp-crc-fast:
    - fi-kbl-7500u:       NOTRUN -> DMESG-WARN [fdo#103841]

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-c:
    - fi-blb-e6850:       NOTRUN -> SKIP [fdo#109271] +48

  * igt@runner@aborted:
    - fi-kbl-7500u:       NOTRUN -> FAIL [fdo#103841]
    - fi-apl-guc:         NOTRUN -> FAIL [fdo#108622] / [fdo#109720]

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s3:
    - fi-blb-e6850:       INCOMPLETE [fdo#107718] -> PASS

  * igt@i915_selftest@live_hangcheck:
    - fi-skl-iommu:       INCOMPLETE [fdo#108602] / [fdo#108744] -> PASS

  * igt@kms_flip@basic-flip-vs-wf_vblank:
    - fi-bsw-n3050:       FAIL [fdo#100368] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
    - {fi-icl-u3}:        FAIL [fdo#103167] -> PASS
    - fi-icl-u2:          FAIL [fdo#103167] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
  [fdo#102657]: https://bugs.freedesktop.org/show_bug.cgi?id=102657
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103841]: https://bugs.freedesktop.org/show_bug.cgi?id=103841
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
  [fdo#108602]: https://bugs.freedesktop.org/show_bug.cgi?id=108602
  [fdo#108622]: https://bugs.freedesktop.org/show_bug.cgi?id=108622
  [fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#109720]: https://bugs.freedesktop.org/show_bug.cgi?id=109720
  [fdo#110235 ]: https://bugs.freedesktop.org/show_bug.cgi?id=110235 


Participating hosts (48 -> 45)
------------------------------

  Additional (2): fi-byt-clapper fi-kbl-7500u 
  Missing    (5): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan 


Build changes
-------------

    * Linux: CI_DRM_5875 -> Patchwork_12687

  CI_DRM_5875: 5cc7c47c44aaef5bfe07e7307d06caa98e401fad @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4928: 014a6fa238322b497116b359cb92df1ce7fa8847 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_12687: 65c3d8e5d782f22612abb99cf76f5782a9641d0f @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

65c3d8e5d782 drm/i915/cdclk: have only one init/uninit function
eef88940ccfa drm/i915: extract intel_cdclk.h from intel_drv.h
502027133de0 drm/i915: extract intel_sprite.h from intel_drv.h
afe139e73b37 drm/i915: extract intel_dvo.h from intel_drv.h
1baaf662b096 drm/i915: extract intel_lvds.h from intel_drv.h
8552e096cde2 drm/i915: extract intel_tv.h from intel_drv.h
c2bed9041cd5 drm/i915: extract intel_pipe_crc.h from intel_drv.h
4fcb1b6c3c6d drm/i915: extract intel_atomic_plane.h from intel_drv.h
de1853ccd427 drm/i915: extract intel_hdmi.h from intel_drv.h
c9dd8b09cf61 drm/i915: extract intel_dp.h from intel_drv.h
9b50c8a6f31f drm/i915: extract intel_fbdev.h from intel_drv.h
9169ad03c189 drm/i915: extract intel_pm.h from intel_drv.h
7764ec95da8a drm/i915: extract intel_panel.h from intel_drv.h
444cceac5b34 drm/i915: extract intel_hdcp.h from intel_drv.h
a9b1119a8b8a drm/i915: extract intel_sdvo.h from intel_drv.h
4aef051c5084 drm/i915: extract intel_lspcon.h from intel_drv.h
1e02d4859ef6 drm/i915: extract intel_color.h from intel_drv.h
1d43269827e3 drm/i915: extract intel_psr.h from intel_drv.h
3b0c7907b1c7 drm/i915: extract intel_fbc.h from intel_drv.h
d3dbcf35782d drm/i915: extract intel_csr.h from intel_drv.h
29b3cb9772fe drm/i915: extract intel_connector.h from intel_drv.h
f7505cd98351 drm/i915: extract intel_ddi.h from intel_drv.h
17593127dc04 drm/i915: extract intel_crt.h from intel_drv.h
9173569bc454 drm/i915: extract intel_audio.h from intel_drv.h
e3273176ce64 drm/i915: make intel_frontbuffer.h self-contained
$ dim checkpatch origin/drm-tip
e3273176ce64 drm/i915: make intel_frontbuffer.h self-contained
9173569bc454 drm/i915: extract intel_audio.h from intel_drv.h
-:94: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#94: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 178 lines checked
17593127dc04 drm/i915: extract intel_crt.h from intel_drv.h
-:55: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#55: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 86 lines checked
f7505cd98351 drm/i915: extract intel_ddi.h from intel_drv.h
-:61: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#61: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 156 lines checked
29b3cb9772fe drm/i915: extract intel_connector.h from intel_drv.h
-:45: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#45: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 214 lines checked
d3dbcf35782d drm/i915: extract intel_csr.h from intel_drv.h
-:45: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#45: 
new file mode 100644

-:60: WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct drm_i915_private *' should also have an identifier name
#60: FILE: drivers/gpu/drm/i915/intel_csr.h:11:
+void intel_csr_ucode_init(struct drm_i915_private *);

-:61: WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct drm_i915_private *' should also have an identifier name
#61: FILE: drivers/gpu/drm/i915/intel_csr.h:12:
+void intel_csr_load_program(struct drm_i915_private *);

-:62: WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct drm_i915_private *' should also have an identifier name
#62: FILE: drivers/gpu/drm/i915/intel_csr.h:13:
+void intel_csr_ucode_fini(struct drm_i915_private *);

-:63: WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct drm_i915_private *' should also have an identifier name
#63: FILE: drivers/gpu/drm/i915/intel_csr.h:14:
+void intel_csr_ucode_suspend(struct drm_i915_private *);

-:64: WARNING:FUNCTION_ARGUMENTS: function definition argument 'struct drm_i915_private *' should also have an identifier name
#64: FILE: drivers/gpu/drm/i915/intel_csr.h:15:
+void intel_csr_ucode_resume(struct drm_i915_private *);

total: 0 errors, 6 warnings, 0 checks, 51 lines checked
3b0c7907b1c7 drm/i915: extract intel_fbc.h from intel_drv.h
-:76: CHECK:LINE_SPACING: Please don't use multiple blank lines
#76: FILE: drivers/gpu/drm/i915/i915_suspend.c:27:
 
+

-:134: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#134: 
new file mode 100644

total: 0 errors, 1 warnings, 1 checks, 150 lines checked
1d43269827e3 drm/i915: extract intel_psr.h from intel_drv.h
-:202: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#202: 
new file mode 100644

-:221: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects?
#221: FILE: drivers/gpu/drm/i915/intel_psr.h:15:
+#define CAN_PSR(dev_priv) (HAS_PSR(dev_priv) && dev_priv->psr.sink_support)

-:226: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#226: FILE: drivers/gpu/drm/i915/intel_psr.h:20:
+void intel_psr_disable(struct intel_dp *intel_dp,
+		      const struct intel_crtc_state *old_crtc_state);

-:231: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#231: FILE: drivers/gpu/drm/i915/intel_psr.h:25:
+			  unsigned frontbuffer_bits,

-:234: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#234: FILE: drivers/gpu/drm/i915/intel_psr.h:28:
+		     unsigned frontbuffer_bits,

total: 0 errors, 3 warnings, 2 checks, 195 lines checked
1e02d4859ef6 drm/i915: extract intel_color.h from intel_drv.h
-:33: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#33: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 43 lines checked
4aef051c5084 drm/i915: extract intel_lspcon.h from intel_drv.h
-:101: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#101: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 93 lines checked
a9b1119a8b8a drm/i915: extract intel_sdvo.h from intel_drv.h
-:75: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#75: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 57 lines checked
444cceac5b34 drm/i915: extract intel_hdcp.h from intel_drv.h
-:141: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#141: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 120 lines checked
7764ec95da8a drm/i915: extract intel_panel.h from intel_drv.h
-:189: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#189: 
new file mode 100644

-:253: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#253: FILE: drivers/gpu/drm/i915/intel_panel.h:60:
+}
+static inline void intel_backlight_device_unregister(struct intel_connector *connector)

total: 0 errors, 1 warnings, 1 checks, 197 lines checked
9169ad03c189 drm/i915: extract intel_pm.h from intel_drv.h
-:193: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#193: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 222 lines checked
9b50c8a6f31f drm/i915: extract intel_fbdev.h from intel_drv.h
-:160: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#160: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 162 lines checked
c9dd8b09cf61 drm/i915: extract intel_dp.h from intel_drv.h
-:93: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#93: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 342 lines checked
de1853ccd427 drm/i915: extract intel_hdmi.h from intel_drv.h
-:148: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#148: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 154 lines checked
4fcb1b6c3c6d drm/i915: extract intel_atomic_plane.h from intel_drv.h
-:85: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#85: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 137 lines checked
c2bed9041cd5 drm/i915: extract intel_pipe_crc.h from intel_drv.h
-:75: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#75: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 74 lines checked
8552e096cde2 drm/i915: extract intel_tv.h from intel_drv.h
-:59: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#59: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 36 lines checked
1baaf662b096 drm/i915: extract intel_lvds.h from intel_drv.h
-:75: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#75: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 56 lines checked
afe139e73b37 drm/i915: extract intel_dvo.h from intel_drv.h
-:58: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#58: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 35 lines checked
502027133de0 drm/i915: extract intel_sprite.h from intel_drv.h
-:163: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#163: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 152 lines checked
eef88940ccfa drm/i915: extract intel_cdclk.h from intel_drv.h
-:57: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 128 lines checked
65c3d8e5d782 drm/i915/cdclk: have only one init/uninit function
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: make intel_frontbuffer.h self-contained
-drivers/gpu/drm/i915/selftests/../i915_drv.h:3623:16: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/selftests/../i915_drv.h:3616:16: warning: expression using sizeof(void)
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_audio.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_crt.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_ddi.h from intel_drv.h
+drivers/gpu/drm/i915/intel_ddi.c:1046:6: warning: symbol 'hsw_fdi_link_train' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1300:5: warning: symbol 'cnl_calc_wrpll_link' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1663:6: warning: symbol 'intel_ddi_set_pipe_settings' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1708:6: warning: symbol 'intel_ddi_set_vc_payload_alloc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1724:6: warning: symbol 'intel_ddi_enable_transcoder_func' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1810:6: warning: symbol 'intel_ddi_disable_transcoder_func' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1830:5: warning: symbol 'intel_ddi_toggle_hdcp_signalling' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:1861:6: warning: symbol 'intel_ddi_connector_get_hw_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2013:6: warning: symbol 'intel_ddi_get_hw_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2083:6: warning: symbol 'intel_ddi_enable_pipe_clock' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2096:6: warning: symbol 'intel_ddi_disable_pipe_clock' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2191:4: warning: symbol 'intel_ddi_dp_voltage_max' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2235:4: warning: symbol 'intel_ddi_dp_pre_emphasis_max' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2664:5: warning: symbol 'bxt_signal_levels' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2682:5: warning: symbol 'ddi_signal_levels' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:2751:6: warning: symbol 'icl_sanitize_encoder_pll_mapping' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:3326:6: warning: symbol 'intel_ddi_fdi_post_disable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:3624:6: warning: symbol 'intel_ddi_prepare_link_retrain' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:3683:6: warning: symbol 'intel_ddi_compute_min_voltage_level' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:3692:6: warning: symbol 'intel_ddi_get_config' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_ddi.c:4136:6: warning: symbol 'intel_ddi_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_connector.h from intel_drv.h
+drivers/gpu/drm/i915/intel_connector.c:106:5: warning: symbol 'intel_connector_register' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:128:6: warning: symbol 'intel_connector_unregister' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:135:6: warning: symbol 'intel_connector_attach_encoder' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:147:6: warning: symbol 'intel_connector_get_hw_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:155:11: warning: symbol 'intel_connector_get_pipe' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:172:5: warning: symbol 'intel_connector_update_modes' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:190:5: warning: symbol 'intel_ddc_get_modes' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:214:1: warning: symbol 'intel_attach_force_audio_property' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:241:1: warning: symbol 'intel_attach_broadcast_rgb_property' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:263:1: warning: symbol 'intel_attach_aspect_ratio_property' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:272:1: warning: symbol 'intel_attach_colorspace_property' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:33:5: warning: symbol 'intel_connector_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:53:24: warning: symbol 'intel_connector_alloc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:76:6: warning: symbol 'intel_connector_free' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:85:6: warning: symbol 'intel_connector_destroy' was not declared. Should it be static?

Commit: drm/i915: extract intel_csr.h from intel_drv.h
+drivers/gpu/drm/i915/intel_csr.c:257:6: warning: symbol 'intel_csr_load_program' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_csr.c:467:6: warning: symbol 'intel_csr_ucode_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_csr.c:546:6: warning: symbol 'intel_csr_ucode_suspend' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_csr.c:565:6: warning: symbol 'intel_csr_ucode_resume' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_csr.c:585:6: warning: symbol 'intel_csr_ucode_fini' was not declared. Should it be static?

Commit: drm/i915: extract intel_fbc.h from intel_drv.h
+drivers/gpu/drm/i915/intel_fbc.c:1020:6: warning: symbol 'intel_fbc_choose_crtc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1078:6: warning: symbol 'intel_fbc_enable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1126:6: warning: symbol 'intel_fbc_disable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1146:6: warning: symbol 'intel_fbc_global_disable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1188:5: warning: symbol 'intel_fbc_reset_underrun' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1223:6: warning: symbol 'intel_fbc_handle_fifo_underrun_irq' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1250:6: warning: symbol 'intel_fbc_init_pipe_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:1305:6: warning: symbol 'intel_fbc_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:399:6: warning: symbol 'intel_fbc_is_active' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:563:6: warning: symbol 'intel_fbc_cleanup_cfb' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:852:6: warning: symbol 'intel_fbc_pre_update' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:938:6: warning: symbol 'intel_fbc_post_update' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:959:6: warning: symbol 'intel_fbc_invalidate' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_fbc.c:981:6: warning: symbol 'intel_fbc_flush' was not declared. Should it be static?
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_psr.h from intel_drv.h
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_color.h from intel_drv.h
+drivers/gpu/drm/i915/intel_color.c:1223:6: warning: symbol 'intel_color_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_color.c:838:6: warning: symbol 'intel_color_load_luts' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_color.c:845:6: warning: symbol 'intel_color_commit' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_color.c:852:5: warning: symbol 'intel_color_check' was not declared. Should it be static?

Commit: drm/i915: extract intel_lspcon.h from intel_drv.h
+drivers/gpu/drm/i915/intel_lspcon.c:183:6: warning: symbol 'lspcon_ycbcr420_config' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:427:6: warning: symbol 'lspcon_write_infoframe' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:455:6: warning: symbol 'lspcon_read_infoframe' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:463:6: warning: symbol 'lspcon_set_infoframes' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:517:5: warning: symbol 'lspcon_infoframes_enabled' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:524:6: warning: symbol 'lspcon_resume' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:544:6: warning: symbol 'lspcon_wait_pcon_mode' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lspcon.c:549:6: warning: symbol 'lspcon_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_sdvo.h from intel_drv.h
+drivers/gpu/drm/i915/intel_sdvo.c:1520:6: warning: symbol 'intel_sdvo_port_enabled' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_sdvo.c:3159:6: warning: symbol 'intel_sdvo_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_hdcp.h from intel_drv.h
-O:drivers/gpu/drm/i915/intel_connector.c:33:5: warning: symbol 'intel_connector_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:36:5: warning: symbol 'intel_connector_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1752:6: warning: symbol 'intel_hdcp_component_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1789:5: warning: symbol 'intel_hdcp_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1815:5: warning: symbol 'intel_hdcp_enable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1852:5: warning: symbol 'intel_hdcp_disable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1875:6: warning: symbol 'intel_hdcp_component_fini' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1889:6: warning: symbol 'intel_hdcp_cleanup' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1899:6: warning: symbol 'intel_hdcp_atomic_check' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:1933:6: warning: symbol 'intel_hdcp_handle_cp_irq' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:58:6: warning: symbol 'intel_hdcp_capable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdcp.c:871:6: warning: symbol 'is_hdcp_supported' was not declared. Should it be static?

Commit: drm/i915: extract intel_panel.h from intel_drv.h
-O:drivers/gpu/drm/i915/intel_connector.c:36:5: warning: symbol 'intel_connector_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_connector.c:37:5: warning: symbol 'intel_connector_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:103:25: warning: symbol 'intel_panel_edid_fixed_mode' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:1183:6: warning: symbol 'intel_panel_enable_backlight' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:1308:5: warning: symbol 'intel_backlight_device_register' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:1361:6: warning: symbol 'intel_backlight_device_unregister' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:145:25: warning: symbol 'intel_panel_vbt_fixed_mode' was not declared. Should it be static?
-drivers/gpu/drm/i915/intel_panel.c:1717:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1717:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1717:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1717:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_panel.c:1750:34: warning: too many warnings
+drivers/gpu/drm/i915/intel_panel.c:1717:34: warning: too many warnings
+drivers/gpu/drm/i915/intel_panel.c:174:1: warning: symbol 'intel_pch_panel_fitting' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:374:6: warning: symbol 'intel_gmch_panel_fitting' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:43:1: warning: symbol 'intel_fixed_panel_mode' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:61:25: warning: symbol 'intel_panel_edid_downclock_mode' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:690:6: warning: symbol 'intel_panel_set_backlight_acpi' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_panel.c:841:6: warning: symbol 'intel_panel_disable_backlight' was not declared. Should it be static?
-drivers/gpu/drm/i915/vlv_dsi.c:116:33: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/vlv_dsi.c:116:33: warning: expression using sizeof(void)

Commit: drm/i915: extract intel_pm.h from intel_drv.h
+drivers/gpu/drm/i915/i915_irq.c:1194:6: warning: symbol 'gen6_rps_reset_ei' was not declared. Should it be static?

Commit: drm/i915: extract intel_fbdev.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_dp.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_hdmi.h from intel_drv.h
+drivers/gpu/drm/i915/intel_hdmi.c:1190:6: warning: symbol 'intel_dp_dual_mode_set_tmds_output' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:2259:5: warning: symbol 'intel_hdmi_compute_config' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:2755:6: warning: symbol 'intel_hdmi_handle_sink_scrambling' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:2931:6: warning: symbol 'intel_infoframe_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:2971:6: warning: symbol 'intel_hdmi_init_connector' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:3038:6: warning: symbol 'intel_hdmi_init' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:570:5: warning: symbol 'intel_hdmi_infoframe_enable' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:582:5: warning: symbol 'intel_hdmi_infoframes_enabled' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:654:6: warning: symbol 'intel_read_infoframe' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:80:19: warning: symbol 'enc_to_intel_hdmi' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_hdmi.c:940:6: warning: symbol 'intel_hdmi_read_gcp_infoframe' was not declared. Should it be static?

Commit: drm/i915: extract intel_atomic_plane.h from intel_drv.h
+drivers/gpu/drm/i915/intel_atomic_plane.c:115:5: warning: symbol 'intel_plane_atomic_check_with_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:223:6: warning: symbol 'intel_update_plane' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:233:6: warning: symbol 'intel_update_slave' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:243:6: warning: symbol 'intel_disable_plane' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:252:6: warning: symbol 'skl_update_planes_on_crtc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:300:6: warning: symbol 'i9xx_update_planes_on_crtc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:322:37: warning: symbol 'intel_plane_helper_funcs' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:41:20: warning: symbol 'intel_plane_alloc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:71:1: warning: symbol 'intel_plane_destroy_state' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:79:6: warning: symbol 'intel_plane_free' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:94:24: warning: symbol 'intel_plane_duplicate_state' was not declared. Should it be static?

Commit: drm/i915: extract intel_pipe_crc.h from intel_drv.h
+drivers/gpu/drm/i915/intel_pipe_crc.c:558:12: warning: symbol 'intel_crtc_get_crc_sources' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_pipe_crc.c:565:5: warning: symbol 'intel_crtc_verify_crc_source' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_pipe_crc.c:585:5: warning: symbol 'intel_crtc_set_crc_source' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_pipe_crc.c:636:6: warning: symbol 'intel_crtc_enable_pipe_crc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_pipe_crc.c:656:6: warning: symbol 'intel_crtc_disable_pipe_crc' was not declared. Should it be static?

Commit: drm/i915: extract intel_tv.h from intel_drv.h
+drivers/gpu/drm/i915/intel_tv.c:1858:1: warning: symbol 'intel_tv_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_lvds.h from intel_drv.h
+drivers/gpu/drm/i915/intel_lvds.c:751:22: warning: symbol 'intel_get_lvds_encoder' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lvds.c:763:6: warning: symbol 'intel_is_dual_link_lvds' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lvds.c:80:6: warning: symbol 'intel_lvds_port_enabled' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_lvds.c:811:6: warning: symbol 'intel_lvds_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_dvo.h from intel_drv.h
+drivers/gpu/drm/i915/intel_dvo.c:402:6: warning: symbol 'intel_dvo_init' was not declared. Should it be static?

Commit: drm/i915: extract intel_sprite.h from intel_drv.h
-O:drivers/gpu/drm/i915/intel_atomic_plane.c:41:20: warning: symbol 'intel_plane_alloc' was not declared. Should it be static?
+drivers/gpu/drm/i915/intel_atomic_plane.c:42:20: warning: symbol 'intel_plane_alloc' was not declared. Should it be static?
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_cdclk.h from intel_drv.h
Okay!

Commit: drm/i915/cdclk: have only one init/uninit function
Okay!

Patches download mbox

# Name Submitter State A F R T
[v2,01/25] drm/i915: make intel_frontbuffer.h self-contained Jani Nikula New 1 1
[v2,02/25] drm/i915: extract intel_audio.h from intel_drv.h Jani Nikula New 1 1
[v2,03/25] drm/i915: extract intel_crt.h from intel_drv.h Jani Nikula New 1 1
[v2,04/25] drm/i915: extract intel_ddi.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,05/25] drm/i915: extract intel_connector.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,06/25] drm/i915: extract intel_csr.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,07/25] drm/i915: extract intel_fbc.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,08/25] drm/i915: extract intel_psr.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,09/25] drm/i915: extract intel_color.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,10/25] drm/i915: extract intel_lspcon.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,11/25] drm/i915: extract intel_sdvo.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,12/25] drm/i915: extract intel_hdcp.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,13/25] drm/i915: extract intel_panel.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,14/25] drm/i915: extract intel_pm.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,15/25] drm/i915: extract intel_fbdev.h from intel_drv.h Jani Nikula New 1 1
[v2,16/25] drm/i915: extract intel_dp.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,17/25] drm/i915: extract intel_hdmi.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,18/25] drm/i915: extract intel_atomic_plane.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,19/25] drm/i915: extract intel_pipe_crc.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,20/25] drm/i915: extract intel_tv.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,21/25] drm/i915: extract intel_lvds.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,22/25] drm/i915: extract intel_dvo.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,23/25] drm/i915: extract intel_sprite.h from intel_drv.h Jani Nikula Accepted 1 1
[v2,24/25] drm/i915: extract intel_cdclk.h from intel_drv.h Jani Nikula New 1 1
[v2,25/25] drm/i915/cdclk: have only one init/uninit function Jani Nikula New 1 1

Tests

CI Bug Log - changes from CI_DRM_5878 -> Patchwork_12694
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/59022/revisions/2/mbox/

Known issues
------------

  Here are the changes found in Patchwork_12694 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_cpu_reloc@basic:
    - fi-icl-y:           PASS -> INCOMPLETE [fdo#110246]

  * igt@gem_ctx_create@basic-files:
    - fi-gdg-551:         NOTRUN -> SKIP [fdo#109271] +106

  * igt@kms_busy@basic-flip-a:
    - fi-bsw-n3050:       NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +1

  * igt@kms_busy@basic-flip-c:
    - fi-skl-6770hq:      PASS -> SKIP [fdo#109271] / [fdo#109278] +2
    - fi-gdg-551:         NOTRUN -> SKIP [fdo#109271] / [fdo#109278]

  * igt@kms_chamelium@hdmi-crc-fast:
    - fi-bsw-n3050:       NOTRUN -> SKIP [fdo#109271] +62

  * igt@kms_chamelium@vga-edid-read:
    - fi-hsw-4770r:       NOTRUN -> SKIP [fdo#109271] +45

  * igt@kms_flip@basic-flip-vs-dpms:
    - fi-skl-6770hq:      PASS -> SKIP [fdo#109271] +33

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362] +1

  
#### Possible fixes ####

  * igt@gem_mmap@basic-small-bo:
    - {fi-icl-u3}:        DMESG-WARN [fdo#107724] -> PASS

  * igt@i915_selftest@live_uncore:
    - fi-ivb-3770:        DMESG-FAIL [fdo#110210] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
    - fi-byt-clapper:     FAIL [fdo#103167] -> PASS

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#110210]: https://bugs.freedesktop.org/show_bug.cgi?id=110210
  [fdo#110246]: https://bugs.freedesktop.org/show_bug.cgi?id=110246


Participating hosts (46 -> 44)
------------------------------

  Additional (3): fi-hsw-4770r fi-gdg-551 fi-bsw-n3050 
  Missing    (5): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-skl-6600u 


Build changes
-------------

    * Linux: CI_DRM_5878 -> Patchwork_12694

  CI_DRM_5878: ae3367460fdf5e6f5a47038c1281b502817184e1 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4931: 019f892e5d1a0a9643cb726c47ce2d99c14b444f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_12694: 8a2a62bb3557a16ffc60167fe4b49e21e63f9405 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

8a2a62bb3557 drm/i915/cdclk: have only one init/uninit function
727651faff74 drm/i915: extract intel_cdclk.h from intel_drv.h
b4673c1802b2 drm/i915: extract intel_sprite.h from intel_drv.h
1e168fb624e2 drm/i915: extract intel_dvo.h from intel_drv.h
8d7fa0132482 drm/i915: extract intel_lvds.h from intel_drv.h
1ed1091c7f13 drm/i915: extract intel_tv.h from intel_drv.h
343a406bddad drm/i915: extract intel_pipe_crc.h from intel_drv.h
cb48fc298613 drm/i915: extract intel_atomic_plane.h from intel_drv.h
db9eb9ce6912 drm/i915: extract intel_hdmi.h from intel_drv.h
7c68e0c09f32 drm/i915: extract intel_dp.h from intel_drv.h
53962f585c19 drm/i915: extract intel_fbdev.h from intel_drv.h
5281fe57b82b drm/i915: extract intel_pm.h from intel_drv.h
dd94498ded80 drm/i915: extract intel_panel.h from intel_drv.h
c10004c97f07 drm/i915: extract intel_hdcp.h from intel_drv.h
3072c769395b drm/i915: extract intel_sdvo.h from intel_drv.h
23f5cad48977 drm/i915: extract intel_lspcon.h from intel_drv.h
f695ba7c7197 drm/i915: extract intel_color.h from intel_drv.h
155ee6b0ddd6 drm/i915: extract intel_psr.h from intel_drv.h
72b59056d9db drm/i915: extract intel_fbc.h from intel_drv.h
fa52f7f70c10 drm/i915: extract intel_csr.h from intel_drv.h
b71989d51260 drm/i915: extract intel_connector.h from intel_drv.h
d3d1b197eae6 drm/i915: extract intel_ddi.h from intel_drv.h
38232a6560e4 drm/i915: extract intel_crt.h from intel_drv.h
1c4845705dd7 drm/i915: extract intel_audio.h from intel_drv.h
7dd28bb15c13 drm/i915: make intel_frontbuffer.h self-contained
$ dim checkpatch origin/drm-tip
7dd28bb15c13 drm/i915: make intel_frontbuffer.h self-contained
1c4845705dd7 drm/i915: extract intel_audio.h from intel_drv.h
-:96: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#96: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 178 lines checked
38232a6560e4 drm/i915: extract intel_crt.h from intel_drv.h
-:57: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 86 lines checked
d3d1b197eae6 drm/i915: extract intel_ddi.h from intel_drv.h
-:75: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#75: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 163 lines checked
b71989d51260 drm/i915: extract intel_connector.h from intel_drv.h
-:69: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#69: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 231 lines checked
fa52f7f70c10 drm/i915: extract intel_csr.h from intel_drv.h
-:66: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#66: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 63 lines checked
72b59056d9db drm/i915: extract intel_fbc.h from intel_drv.h
-:151: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#151: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 158 lines checked
155ee6b0ddd6 drm/i915: extract intel_psr.h from intel_drv.h
-:203: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#203: 
new file mode 100644

-:222: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects?
#222: FILE: drivers/gpu/drm/i915/intel_psr.h:15:
+#define CAN_PSR(dev_priv) (HAS_PSR(dev_priv) && dev_priv->psr.sink_support)

-:232: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#232: FILE: drivers/gpu/drm/i915/intel_psr.h:25:
+			  unsigned frontbuffer_bits,

-:235: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#235: FILE: drivers/gpu/drm/i915/intel_psr.h:28:
+		     unsigned frontbuffer_bits,

total: 0 errors, 3 warnings, 1 checks, 192 lines checked
f695ba7c7197 drm/i915: extract intel_color.h from intel_drv.h
-:47: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#47: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 50 lines checked
23f5cad48977 drm/i915: extract intel_lspcon.h from intel_drv.h
-:122: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#122: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 107 lines checked
3072c769395b drm/i915: extract intel_sdvo.h from intel_drv.h
-:89: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#89: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 64 lines checked
c10004c97f07 drm/i915: extract intel_hdcp.h from intel_drv.h
-:158: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#158: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 130 lines checked
dd94498ded80 drm/i915: extract intel_panel.h from intel_drv.h
-:203: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#203: 
new file mode 100644

-:267: CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations
#267: FILE: drivers/gpu/drm/i915/intel_panel.h:60:
+}
+static inline void intel_backlight_device_unregister(struct intel_connector *connector)

total: 0 errors, 1 warnings, 1 checks, 204 lines checked
5281fe57b82b drm/i915: extract intel_pm.h from intel_drv.h
-:205: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#205: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 228 lines checked
53962f585c19 drm/i915: extract intel_fbdev.h from intel_drv.h
-:162: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#162: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 162 lines checked
7c68e0c09f32 drm/i915: extract intel_dp.h from intel_drv.h
-:95: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#95: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 336 lines checked
db9eb9ce6912 drm/i915: extract intel_hdmi.h from intel_drv.h
-:162: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#162: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 161 lines checked
cb48fc298613 drm/i915: extract intel_atomic_plane.h from intel_drv.h
-:49: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#49: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 98 lines checked
343a406bddad drm/i915: extract intel_pipe_crc.h from intel_drv.h
-:96: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#96: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 88 lines checked
1ed1091c7f13 drm/i915: extract intel_tv.h from intel_drv.h
-:73: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#73: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 43 lines checked
8d7fa0132482 drm/i915: extract intel_lvds.h from intel_drv.h
-:89: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#89: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 63 lines checked
1e168fb624e2 drm/i915: extract intel_dvo.h from intel_drv.h
-:72: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#72: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 42 lines checked
b4673c1802b2 drm/i915: extract intel_sprite.h from intel_drv.h
-:165: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#165: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 152 lines checked
727651faff74 drm/i915: extract intel_cdclk.h from intel_drv.h
-:59: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#59: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 128 lines checked
8a2a62bb3557 drm/i915/cdclk: have only one init/uninit function
CI Bug Log - changes from CI_DRM_5878_full -> Patchwork_12694_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Known issues
------------

  Here are the changes found in Patchwork_12694_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_nop@basic-series:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#109100]

  * igt@gem_exec_schedule@pi-ringfull-bsd2:
    - shard-iclb:         NOTRUN -> SKIP [fdo#109276] +2

  * igt@gem_exec_suspend@basic-s3-devices:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#109638]

  * igt@gem_mmap_gtt@hang:
    - shard-iclb:         PASS -> FAIL [fdo#109677]

  * igt@gem_ppgtt@blt-vs-render-ctxn:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#109801]

  * igt@gem_pwrite@huge-cpu-fbr:
    - shard-iclb:         NOTRUN -> SKIP [fdo#109290]

  * igt@gem_tiled_swapping@non-threaded:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#108686]

  * igt@i915_pm_rps@min-max-config-loaded:
    - shard-iclb:         NOTRUN -> FAIL [fdo#108059]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-a:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#110222]

  * igt@kms_chamelium@dp-hpd-storm-disable:
    - shard-iclb:         NOTRUN -> SKIP [fdo#109284]

  * igt@kms_cursor_crc@cursor-64x64-suspend:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108]

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
    - shard-glk:          PASS -> FAIL [fdo#104873]

  * igt@kms_flip@2x-flip-vs-panning-vs-hang-interruptible:
    - shard-iclb:         NOTRUN -> SKIP [fdo#109274] +1

  * igt@kms_flip@busy-flip-interruptible:
    - shard-skl:          PASS -> FAIL [fdo#103257]

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-cpu:
    - shard-apl:          NOTRUN -> SKIP [fdo#109271] +22

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-shrfb-msflip-blt:
    - shard-snb:          NOTRUN -> SKIP [fdo#109271] +40

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-blt:
    - shard-iclb:         NOTRUN -> SKIP [fdo#109280] +2

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-shrfb-draw-pwrite:
    - shard-iclb:         PASS -> FAIL [fdo#103167] +5

  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-shrfb-draw-render:
    - shard-kbl:          NOTRUN -> SKIP [fdo#109271] +19

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-cpu:
    - shard-iclb:         PASS -> FAIL [fdo#109247] +20

  * igt@kms_frontbuffer_tracking@psr-rgb565-draw-blt:
    - shard-skl:          PASS -> FAIL [fdo#103167]

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-d:
    - shard-kbl:          NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +1

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence:
    - shard-skl:          PASS -> FAIL [fdo#103191] / [fdo#107362]

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d:
    - shard-skl:          NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +8

  * igt@kms_plane@pixel-format-pipe-b-planes-source-clamping:
    - shard-glk:          PASS -> SKIP [fdo#109271]

  * igt@kms_plane_alpha_blend@pipe-a-alpha-basic:
    - shard-skl:          NOTRUN -> FAIL [fdo#107815] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min:
    - shard-skl:          PASS -> FAIL [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-c-alpha-transparant-fb:
    - shard-skl:          NOTRUN -> FAIL [fdo#108145] +1

  * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
    - shard-snb:          NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +2

  * igt@kms_plane_scaling@pipe-c-scaler-with-clipping-clamping:
    - shard-apl:          NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +2

  * igt@kms_psr@psr2_cursor_plane_onoff:
    - shard-iclb:         PASS -> SKIP [fdo#109441] +3

  * igt@kms_psr@sprite_mmap_cpu:
    - shard-iclb:         PASS -> FAIL [fdo#107383] / [fdo#110215] +1

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-kbl:          PASS -> FAIL [fdo#109016]

  * igt@kms_vblank@pipe-c-ts-continuation-dpms-suspend:
    - shard-apl:          PASS -> FAIL [fdo#104894] +1

  * igt@kms_vblank@pipe-c-ts-continuation-suspend:
    - shard-iclb:         PASS -> FAIL [fdo#104894]

  * igt@perf_pmu@busy-accuracy-50-vcs1:
    - shard-skl:          NOTRUN -> SKIP [fdo#109271] +121

  * igt@prime_busy@hang-bsd:
    - shard-glk:          PASS -> INCOMPLETE [fdo#103359] / [k.org#198133]

  
#### Possible fixes ####

  * igt@gem_linear_blits@normal:
    - shard-iclb:         TIMEOUT [fdo#109673] -> PASS

  * igt@i915_pm_rpm@debugfs-forcewake-user:
    - shard-skl:          INCOMPLETE [fdo#107807] -> PASS

  * igt@i915_pm_rpm@i2c:
    - shard-iclb:         DMESG-WARN [fdo#109982] -> PASS

  * igt@i915_selftest@live_workarounds:
    - shard-iclb:         DMESG-FAIL [fdo#108954] -> PASS

  * igt@kms_atomic_transition@1x-modeset-transitions-nonblocking-fencing:
    - shard-apl:          FAIL [fdo#109660] -> PASS

  * igt@kms_atomic_transition@plane-toggle-modeset-transition:
    - shard-apl:          INCOMPLETE [fdo#103927] -> PASS

  * igt@kms_cursor_crc@cursor-256x256-suspend:
    - shard-skl:          INCOMPLETE [fdo#104108] -> PASS

  * igt@kms_flip@modeset-vs-vblank-race:
    - shard-kbl:          FAIL [fdo#103060] -> PASS

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-blt:
    - shard-iclb:         FAIL [fdo#109247] -> PASS +18

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
    - shard-iclb:         FAIL [fdo#103167] -> PASS +2

  * igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-pwrite:
    - shard-iclb:         FAIL [fdo#105682] / [fdo#109247] -> PASS +1

  * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
    - shard-skl:          FAIL [fdo#107815] / [fdo#108145] -> PASS

  * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
    - shard-skl:          FAIL [fdo#107815] -> PASS

  * igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping:
    - shard-glk:          SKIP [fdo#109271] / [fdo#109278] -> PASS

  * igt@kms_psr@psr2_primary_mmap_cpu:
    - shard-iclb:         SKIP [fdo#109441] -> PASS

  * igt@kms_psr@sprite_mmap_gtt:
    - shard-iclb:         FAIL [fdo#107383] / [fdo#110215] -> PASS +2

  * igt@kms_rotation_crc@multiplane-rotation:
    - shard-kbl:          INCOMPLETE [fdo#103665] -> PASS

  * igt@kms_vblank@pipe-a-ts-continuation-suspend:
    - shard-iclb:         FAIL [fdo#103375] -> PASS

  * igt@kms_vblank@pipe-b-ts-continuation-suspend:
    - shard-apl:          FAIL [fdo#104894] -> PASS

  * igt@perf@polling:
    - shard-iclb:         FAIL [fdo#108587] -> PASS

  
  [fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103257]: https://bugs.freedesktop.org/show_bug.cgi?id=103257
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
  [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#104873]: https://bugs.freedesktop.org/show_bug.cgi?id=104873
  [fdo#104894]: https://bugs.freedesktop.org/show_bug.cgi?id=104894
  [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107383]: https://bugs.freedesktop.org/show_bug.cgi?id=107383
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
  [fdo#108059]: https://bugs.freedesktop.org/show_bug.cgi?id=108059
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108587]: https://bugs.freedesktop.org/show_bug.cgi?id=108587
  [fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
  [fdo#108954]: https://bugs.freedesktop.org/show_bug.cgi?id=108954
  [fdo#109016]: https://bugs.freedesktop.org/show_bug.cgi?id=109016
  [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
  [fdo#109247]: https://bugs.freedesktop.org/show_bug.cgi?id=109247
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
  [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
  [fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
  [fdo#109290]: https://bugs.freedesktop.org/show_bug.cgi?id=109290
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#109638]: https://bugs.freedesktop.org/show_bug.cgi?id=109638
  [fdo#109660]: https://bugs.freedesktop.org/show_bug.cgi?id=109660
  [fdo#109673]: https://bugs.freedesktop.org/show_bug.cgi?id=109673
  [fdo#109677]: https://bugs.freedesktop.org/show_bug.cgi?id=109677
  [fdo#109801]: https://bugs.freedesktop.org/show_bug.cgi?id=109801
  [fdo#109982]: https://bugs.freedesktop.org/show_bug.cgi?id=109982
  [fdo#110215]: https://bugs.freedesktop.org/show_bug.cgi?id=110215
  [fdo#110222]: https://bugs.freedesktop.org/show_bug.cgi?id=110222
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (10 -> 9)
------------------------------

  Missing    (1): shard-hsw 


Build changes
-------------

    * Linux: CI_DRM_5878 -> Patchwork_12694

  CI_DRM_5878: ae3367460fdf5e6f5a47038c1281b502817184e1 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4931: 019f892e5d1a0a9643cb726c47ce2d99c14b444f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_12694: 8a2a62bb3557a16ffc60167fe4b49e21e63f9405 @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: make intel_frontbuffer.h self-contained
-drivers/gpu/drm/i915/selftests/../i915_drv.h:3623:16: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/selftests/../i915_drv.h:3616:16: warning: expression using sizeof(void)
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_audio.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_crt.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_ddi.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_connector.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_csr.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_fbc.h from intel_drv.h
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_psr.h from intel_drv.h
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_color.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_lspcon.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_sdvo.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_hdcp.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_panel.h from intel_drv.h
-drivers/gpu/drm/i915/vlv_dsi.c:116:33: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/vlv_dsi.c:116:33: warning: expression using sizeof(void)

Commit: drm/i915: extract intel_pm.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_fbdev.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_dp.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_hdmi.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_atomic_plane.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_pipe_crc.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_tv.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_lvds.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_dvo.h from intel_drv.h
Okay!

Commit: drm/i915: extract intel_sprite.h from intel_drv.h
+./include/uapi/linux/perf_event.h:147:56: warning: cast truncates bits from constant value (8000000000000000 becomes 0)

Commit: drm/i915: extract intel_cdclk.h from intel_drv.h
Okay!

Commit: drm/i915/cdclk: have only one init/uninit function
Okay!