dri-devel June 2017

dri-devel@lists.freedesktop.org

228 participants
1479 discussions

[tegra-drm:drm/tegra/for-next 2/20] htmldocs: drivers/gpu/host1x/bus.c:50: warning: Excess function parameter 'driver' description in 'host1x_subdev_add'
by kbuild test robot 15 Jun '17

15 Jun '17

tree: git://anongit.freedesktop.org/tegra/linux.git drm/tegra/for-next head: 43240bbd871e2c8f89584d369278a3d18680d9ea commit: fa6d095eb23a8b1aae78d221879032497f6e457f [2/20] drm/tegra: Add driver documentation reproduce: make htmldocs All warnings (new ones prefixed by >>): WARNING: convert(1) not found, for SVG to PDF conversion install ImageMagick (https://www.imagemagick.org) arch/x86/include/asm/uaccess_32.h:1: warning: no structured comments found include/linux/init.h:1: warning: no structured comments found include/linux/mod_devicetable.h:686: warning: Excess struct/union/enum/typedef member 'ver_major' description in 'fsl_mc_device_id' include/linux/mod_devicetable.h:686: warning: Excess struct/union/enum/typedef member 'ver_minor' description in 'fsl_mc_device_id' kernel/sched/core.c:2088: warning: No description found for parameter 'rf' kernel/sched/core.c:2088: warning: Excess function parameter 'cookie' description in 'try_to_wake_up_local' include/linux/kthread.h:26: warning: Excess function parameter '...' description in 'kthread_create' kernel/sys.c:1: warning: no structured comments found include/linux/device.h:969: warning: No description found for parameter 'dma_ops' drivers/dma-buf/seqno-fence.c:1: warning: no structured comments found include/linux/iio/iio.h:597: warning: No description found for parameter 'trig_readonly' include/linux/iio/trigger.h:151: warning: No description found for parameter 'indio_dev' include/linux/iio/trigger.h:151: warning: No description found for parameter 'trig' include/linux/device.h:970: warning: No description found for parameter 'dma_ops' include/linux/usb/gadget.h:230: warning: No description found for parameter 'claimed' include/linux/usb/gadget.h:230: warning: No description found for parameter 'enabled' include/linux/usb/gadget.h:408: warning: No description found for parameter 'quirk_altset_not_supp' include/linux/usb/gadget.h:408: warning: No description found for parameter 'quirk_stall_not_supp' include/linux/usb/gadget.h:408: warning: No description found for parameter 'quirk_zlp_not_supp' include/drm/drm_drv.h:524: warning: No description found for parameter 'set_busid' include/drm/drm_drv.h:524: warning: No description found for parameter 'irq_handler' include/drm/drm_drv.h:524: warning: No description found for parameter 'irq_preinstall' include/drm/drm_drv.h:524: warning: No description found for parameter 'irq_postinstall' include/drm/drm_drv.h:524: warning: No description found for parameter 'irq_uninstall' include/drm/drm_drv.h:524: warning: No description found for parameter 'debugfs_init' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_open_object' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_close_object' include/drm/drm_drv.h:524: warning: No description found for parameter 'prime_handle_to_fd' include/drm/drm_drv.h:524: warning: No description found for parameter 'prime_fd_to_handle' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_export' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_import' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_pin' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_unpin' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_res_obj' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_get_sg_table' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_import_sg_table' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_vmap' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_vunmap' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_prime_mmap' include/drm/drm_drv.h:524: warning: No description found for parameter 'gem_vm_ops' include/drm/drm_drv.h:524: warning: No description found for parameter 'major' include/drm/drm_drv.h:524: warning: No description found for parameter 'minor' include/drm/drm_drv.h:524: warning: No description found for parameter 'patchlevel' include/drm/drm_drv.h:524: warning: No description found for parameter 'name' include/drm/drm_drv.h:524: warning: No description found for parameter 'desc' include/drm/drm_drv.h:524: warning: No description found for parameter 'date' include/drm/drm_drv.h:524: warning: No description found for parameter 'driver_features' include/drm/drm_drv.h:524: warning: No description found for parameter 'ioctls' include/drm/drm_drv.h:524: warning: No description found for parameter 'num_ioctls' include/drm/drm_drv.h:524: warning: No description found for parameter 'fops' include/drm/drm_color_mgmt.h:1: warning: no structured comments found drivers/gpu/drm/drm_plane_helper.c:403: warning: No description found for parameter 'ctx' drivers/gpu/drm/drm_plane_helper.c:404: warning: No description found for parameter 'ctx' drivers/gpu/drm/i915/intel_lpe_audio.c:350: warning: No description found for parameter 'dp_output' drivers/gpu/drm/i915/intel_lpe_audio.c:350: warning: No description found for parameter 'link_rate' drivers/gpu/drm/i915/intel_lpe_audio.c:351: warning: No description found for parameter 'dp_output' drivers/gpu/drm/i915/intel_lpe_audio.c:351: warning: No description found for parameter 'link_rate' >> drivers/gpu/host1x/bus.c:50: warning: Excess function parameter 'driver' description in 'host1x_subdev_add' Documentation/core-api/assoc_array.rst:13: WARNING: Enumerated list ends without a blank line; unexpected unindent. Documentation/doc-guide/sphinx.rst:126: ERROR: Unknown target name: "sphinx c domain". kernel/sched/fair.c:7650: WARNING: Inline emphasis start-string without end-string. kernel/time/timer.c:1200: ERROR: Unexpected indentation. kernel/time/timer.c:1202: ERROR: Unexpected indentation. kernel/time/timer.c:1203: WARNING: Block quote ends without a blank line; unexpected unindent. include/linux/wait.h:122: WARNING: Block quote ends without a blank line; unexpected unindent. include/linux/wait.h:125: ERROR: Unexpected indentation. include/linux/wait.h:127: WARNING: Block quote ends without a blank line; unexpected unindent. kernel/time/hrtimer.c:990: WARNING: Block quote ends without a blank line; unexpected unindent. kernel/signal.c:322: WARNING: Inline literal start-string without end-string. include/linux/iio/iio.h:219: ERROR: Unexpected indentation. include/linux/iio/iio.h:220: WARNING: Block quote ends without a blank line; unexpected unindent. include/linux/iio/iio.h:226: WARNING: Definition list ends without a blank line; unexpected unindent. drivers/iio/industrialio-core.c:638: ERROR: Unknown target name: "iio_val". drivers/iio/industrialio-core.c:645: ERROR: Unknown target name: "iio_val". drivers/message/fusion/mptbase.c:5051: WARNING: Definition list ends without a blank line; unexpected unindent. drivers/tty/serial/serial_core.c:1898: WARNING: Definition list ends without a blank line; unexpected unindent. drivers/pci/pci.c:3456: ERROR: Unexpected indentation. include/linux/regulator/driver.h:271: ERROR: Unknown target name: "regulator_regmap_x_voltage". include/linux/spi/spi.h:370: ERROR: Unexpected indentation. drivers/gpu/drm/drm_scdc_helper.c:203: ERROR: Unexpected indentation. drivers/gpu/drm/drm_scdc_helper.c:204: WARNING: Block quote ends without a blank line; unexpected unindent. drivers/gpu/drm/drm_ioctl.c:690: WARNING: Definition list ends without a blank line; unexpected unindent. Documentation/gpu/todo.rst:111: ERROR: Unknown target name: "drm_fb". sound/soc/soc-core.c:2670: ERROR: Unknown target name: "snd_soc_daifmt". sound/core/jack.c:312: ERROR: Unknown target name: "snd_jack_btn". Documentation/usb/typec.rst:116: ERROR: Error in "kernel-doc" directive: invalid option block. vim +50 drivers/gpu/host1x/bus.c 776dc384 Thierry Reding 2013-10-14 34 776dc384 Thierry Reding 2013-10-14 35 struct host1x_subdev { 776dc384 Thierry Reding 2013-10-14 36 struct host1x_client *client; 776dc384 Thierry Reding 2013-10-14 37 struct device_node *np; 776dc384 Thierry Reding 2013-10-14 38 struct list_head list; 776dc384 Thierry Reding 2013-10-14 39 }; 776dc384 Thierry Reding 2013-10-14 40 776dc384 Thierry Reding 2013-10-14 41 /** 776dc384 Thierry Reding 2013-10-14 42 * host1x_subdev_add() - add a new subdevice with an associated device node 466749f1 Thierry Reding 2017-04-10 43 * @device: host1x device to add the subdevice to 466749f1 Thierry Reding 2017-04-10 44 * @driver: host1x driver 466749f1 Thierry Reding 2017-04-10 45 * @np: device node 776dc384 Thierry Reding 2013-10-14 46 */ 776dc384 Thierry Reding 2013-10-14 47 static int host1x_subdev_add(struct host1x_device *device, 776dc384 Thierry Reding 2013-10-14 48 struct device_node *np) 776dc384 Thierry Reding 2013-10-14 49 { 776dc384 Thierry Reding 2013-10-14 @50 struct host1x_subdev *subdev; 776dc384 Thierry Reding 2013-10-14 51 776dc384 Thierry Reding 2013-10-14 52 subdev = kzalloc(sizeof(*subdev), GFP_KERNEL); 776dc384 Thierry Reding 2013-10-14 53 if (!subdev) 776dc384 Thierry Reding 2013-10-14 54 return -ENOMEM; 776dc384 Thierry Reding 2013-10-14 55 776dc384 Thierry Reding 2013-10-14 56 INIT_LIST_HEAD(&subdev->list); 776dc384 Thierry Reding 2013-10-14 57 subdev->np = of_node_get(np); 776dc384 Thierry Reding 2013-10-14 58 :::::: The code at line 50 was first introduced by commit :::::: 776dc38403676f499a73d32e2e7c61eb5b42f736 drm/tegra: Move subdevice infrastructure to host1x :::::: TO: Thierry Reding <treding(a)nvidia.com> :::::: CC: Thierry Reding <treding(a)nvidia.com> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0

[PATCH] drm: Remove duplicate forward declaration
by Dawid Kurek 15 Jun '17

15 Jun '17

Forward declarations in C are great but I'm pretty sure one is enough. Signed-off-by: Dawid Kurek <dawikur(a)gmail.com> --- include/drm/drm_connector.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h index 4eeda12..930222c 100644 --- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -29,8 +29,6 @@ #include <uapi/drm/drm_mode.h> -struct drm_device; - struct drm_connector_helper_funcs; struct drm_modeset_acquire_ctx; struct drm_device; -- 2.10.0 --- This email has been checked for viruses by Avast antivirus software. https://www.avast.com/antivirus

2 1

[PATCH] gpu: drm: mgag200: remove unnecessary variable in mga_crtc_mode_set
by Gustavo A. R. Silva 15 Jun '17

15 Jun '17

Remove unnecessary variable _misc_ and refactor the code. A value is assigned to variable _misc_ at lines 1002 and 1004 and then overwritten at line 1127, just before it can be used. Besides, this variable is only being used as an argument when calling macro WREG8() and, it is not implied in any other operation. Addresses-Coverity-ID: 1397681 Signed-off-by: Gustavo A. R. Silva <garsilva(a)embeddedor.com> --- drivers/gpu/drm/mgag200/mgag200_mode.c | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c index adb411a..960e43b 100644 --- a/drivers/gpu/drm/mgag200/mgag200_mode.c +++ b/drivers/gpu/drm/mgag200/mgag200_mode.c @@ -923,7 +923,6 @@ static int mga_crtc_mode_set(struct drm_crtc *crtc, int pitch; int option = 0, option2 = 0; int i; - unsigned char misc = 0; unsigned char ext_vga[6]; u8 bppshift; @@ -998,11 +997,6 @@ static int mga_crtc_mode_set(struct drm_crtc *crtc, break; } - if (mode->flags & DRM_MODE_FLAG_NHSYNC) - misc |= 0x40; - if (mode->flags & DRM_MODE_FLAG_NVSYNC) - misc |= 0x80; - for (i = 0; i < sizeof(dacvalue); i++) { if ((i <= 0x17) || @@ -1124,8 +1118,7 @@ static int mga_crtc_mode_set(struct drm_crtc *crtc, ext_vga[1] |= 0x88; /* Set pixel clocks */ - misc = 0x2d; - WREG8(MGA_MISC_OUT, misc); + WREG8(MGA_MISC_OUT, 0x2d); mga_crtc_set_plls(mdev, mode->clock); @@ -1145,9 +1138,7 @@ static int mga_crtc_mode_set(struct drm_crtc *crtc, WREG_ECRT(0, ext_vga[0]); /* Enable mga pixel clock */ - misc = 0x2d; - - WREG8(MGA_MISC_OUT, misc); + WREG8(MGA_MISC_OUT, 0x2d); if (adjusted_mode) memcpy(&mdev->mode, mode, sizeof(struct drm_display_mode)); -- 2.5.0

1 0

[drm-intel:for-linux-next 2/3] drivers/gpu/drm/i915/i915_gem_execbuffer.c:300:2-7: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values.
by kbuild test robot 15 Jun '17

15 Jun '17

tree: git://anongit.freedesktop.org/drm-intel for-linux-next head: 8c45cec48e5871f93e56650f7e476d4ea7174a0e commit: d55495b4dcce2efb4656edfe211eb0bfb27c3387 [2/3] drm/i915: Use vma->exec_entry as our double-entry placeholder coccinelle warnings: (new ones prefixed by >>) >> drivers/gpu/drm/i915/i915_gem_execbuffer.c:300:2-7: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values. Please review and possibly fold the followup patch. --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

2 2

[pull] amdgpu drm-next-4.13
by Alex Deucher 15 Jun '17

15 Jun '17

Hi Dave, A few more patches for 4.13. Mostly bug fixes and code cleanup. This is on top of my pull request from last week. The following changes since commit b58c11314a1706bf094c489ef5cb28f76478c704: drm/amdgpu: drop deprecated drm_get_pci_dev and drm_put_dev (2017-06-08 10:54:39 -0400) are available in the git repository at: git://people.freedesktop.org/~agd5f/linux drm-next-4.13 for you to fetch changes up to a1924005a2e9bfcc4e217b4acd0a4f2421969040: drm/amdgpu: Fix compiler warnings (2017-06-15 11:50:36 -0400) ---------------------------------------------------------------- Alex Deucher (3): drm/amdgpu/gfx: fix MEC interrupt enablement for pipes != 0 drm/amdgpu/gfx9: fix compute ring doorbell index drm/amdgpu: add virtual display support for raven Alex Xie (3): drm/amdgpu: remove duplicate function prototypes drm/amdgpu: fix a typo in comment drm/amdgpu: move comment to the right place Eric Huang (3): drm/amd/powerplay: update vega10_ppsmc.h drm/amd/powerplay: add GPU power display for vega10 drm/amd/powerplay: add avfs control for Vega10 Harish Kasiviswanathan (6): drm/amdgpu: Add vm context module param drm/amdgpu: Add amdgpu_sync_wait drm/amdgpu: Support page directory update via CPU drm/amdgpu: Support page table update via CPU drm/amdgpu: vm_update_ptes remove code duplication drm/amdgpu: Fix compiler warnings Hawking Zhang (2): drm/amdgpu: add new member in gpu_info fw drm/amdgpu: avoid to reset wave_front_size to 0 Huang Rui (2): drm/amdgpu: export test ib debugfs interface drm/amdgpu: fix missed gpu info firmware when cache firmware during S3 Rex Zhu (1): drm/amd/powerplay: fix copy error in powerplay. Tom St Denis (12): drm/amd/amdgpu: gfx9 tidy ups (v2) drm/amd/amdgpu: Rename KIQ ring to avoid spaces drm/amd/amdgpu: Add offset variant to SOC15 macros drm/amd/amdgpu: Port GFXHUB over to new SOC15 macros drm/amd/amdgpu: Cleanup gfxhub read-modify-write patterns drm/amd/amdgpu: Port MMHUB over to new SOC15 macros drm/amd/amdgpu: Port UVD 7.0 over to new SOC15 macros drm/amd/amdgpu: Port NBIO v6.1 driver over to new SOC15 macros drm/amd/amdgpu: Port NBIO v7.0 driver over to new SOC15 macros drm/amd/amdgpu: Port PSP v3.1 over to new SOC15 macros drm/amd/amdgpu: Port PSP v10.0 over to new SOC15 macros drm/amd/amdgpu: Port VCN over to new SOC15 macros horchen (1): drm/amdgpu: add contiguous flag in ucode bo create drivers/gpu/drm/amd/amdgpu/amdgpu.h | 8 +- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 81 ++++++- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 4 + drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 3 +- drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c | 19 ++ drivers/gpu/drm/amd/amdgpu/amdgpu_sync.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 3 +- drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | 3 + drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 250 +++++++++++++++++---- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 20 +- drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 57 +++-- drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 57 +++-- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 89 ++++---- drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 37 ++- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 25 +-- drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c | 54 ++--- drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c | 42 ++-- drivers/gpu/drm/amd/amdgpu/psp_v10_0.c | 14 +- drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 32 +-- drivers/gpu/drm/amd/amdgpu/soc15.c | 2 + drivers/gpu/drm/amd/amdgpu/soc15_common.h | 14 ++ drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c | 174 +++++++------- drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 164 +++++++------- drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 27 ++- drivers/gpu/drm/amd/powerplay/inc/vega10_ppsmc.h | 3 +- 27 files changed, 765 insertions(+), 422 deletions(-)

1 0

[PATCH] dim: allow setup to work with different usernames
by Lionel Landwerlin 15 Jun '17

15 Jun '17

If your username on fd.o differs from your local username, you'll run into issues while setting up dim. Let's use regexp to filter remotes so it doesn't fail. Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin(a)intel.com> --- dim | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/dim b/dim index 89378033f472..a812320581c2 100755 --- a/dim +++ b/dim @@ -228,21 +228,22 @@ function url_to_remote # url { local url remote - url="$1" + url=$1 + login_url=$(echo $url | sed -e 's,ssh://,ssh://[a-zA-Z0-9]+@?,') if [[ -z "$url" ]]; then echoerr "$0 without url" return 1 fi - remote=$(git remote -v | grep -m 1 "$url" | cut -f 1) + remote=$(git remote -v | grep -m 1 -E "$login_url" | cut -f 1) if [[ -z "$remote" ]]; then - git_url=$(echo $url | sed -e 's/git\./anongit./' -e 's/ssh:/git:/') - remote=$(git remote -v | grep -m 1 "$git_url" | cut -f 1) + git_url=$(echo $url | sed -e 's/git\./anongit./' -e 's,ssh://[a-zA-Z0-9]+@?,git://,') + remote=$(git remote -v | grep -m 1 -E "$git_url" | cut -f 1) if [[ -z "$remote" ]]; then - echoerr "No git remote for url $url or $git_url found in $(pwd)" + echoerr "No git remote for url $url, $login_url or $git_url found in $(pwd)" echoerr "Please set it up using:" echoerr " $ git remote add <name> $url" echoerr "with a name of your choice." -- 2.11.0

1 0

[PULL] drm-intel-fixes
by Jani Nikula 15 Jun '17

15 Jun '17

Hi Dave, a GVT fix and a couple of rotation related fixes. BR, Jani. The following changes since commit ef6c4d75e35345f8f362d6754bcd9a28292a897c: drm/i915: fix warning for unused variable (2017-06-08 17:09:44 +0300) are available in the git repository at: git://anongit.freedesktop.org/git/drm-intel tags/drm-intel-fixes-2017-06-15 for you to fetch changes up to c380f681245d7ae57f17d9ebbbe8f8f1557ee1fb: drm/i915: Fix GVT-g PVINFO version compatibility check (2017-06-13 11:19:22 +0300) ---------------------------------------------------------------- drm/i915 fixes for v4.12-rc6 ---------------------------------------------------------------- Ville Syrjälä (2): drm/i915: Fix scaling check for 90/270 degree plane rotation drm/i915: Fix SKL+ watermarks for 90/270 rotation Zhenyu Wang (1): drm/i915: Fix GVT-g PVINFO version compatibility check drivers/gpu/drm/i915/i915_pvinfo.h | 8 ++------ drivers/gpu/drm/i915/i915_vgpu.c | 10 ++++------ drivers/gpu/drm/i915/intel_display.c | 14 ++++++++------ drivers/gpu/drm/i915/intel_pm.c | 36 ++++++++++++++++++++++++------------ 4 files changed, 38 insertions(+), 30 deletions(-) -- Jani Nikula, Intel Open Source Technology Center

1 0

[PATCH] drm/msm/hdmi: Use bitwise operators when building register values
by Liviu Dudau 15 Jun '17

15 Jun '17

Commit c0c0d9eeeb8d ("drm/msm: hdmi audio support") uses logical OR operators to build up a value to be written in the REG_HDMI_AUDIO_INFO0 and REG_HDMI_AUDIO_INFO1 registers when it should have used bitwise operators. Signed-off-by: Liviu Dudau <liviu.dudau(a)arm.com> Fixes: c0c0d9eeeb8d ("drm/msm: hdmi audio support") --- drivers/gpu/drm/msm/hdmi/hdmi_audio.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c index 8177e8511afd..9c34b91ae329 100644 --- a/drivers/gpu/drm/msm/hdmi/hdmi_audio.c +++ b/drivers/gpu/drm/msm/hdmi/hdmi_audio.c @@ -175,10 +175,10 @@ int msm_hdmi_audio_update(struct hdmi *hdmi) /* configure infoframe: */ hdmi_audio_infoframe_pack(info, buf, sizeof(buf)); hdmi_write(hdmi, REG_HDMI_AUDIO_INFO0, - (buf[3] << 0) || (buf[4] << 8) || - (buf[5] << 16) || (buf[6] << 24)); + (buf[3] << 0) | (buf[4] << 8) | + (buf[5] << 16) | (buf[6] << 24)); hdmi_write(hdmi, REG_HDMI_AUDIO_INFO1, - (buf[7] << 0) || (buf[8] << 8)); + (buf[7] << 0) | (buf[8] << 8)); hdmi_write(hdmi, REG_HDMI_GC, 0); -- 2.13.1

1 0

[PATCH v3 00/20] Tegra DRM fixes
by Dmitry Osipenko 15 Jun '17

15 Jun '17

I've dropped the two "GART restoring" patches from the series, postponing them till a full solution of GART utilization would be ready. The "Forbid relocation address shifting in the firewall" patch has been reverted to v1, Mikko and me agreed on that it is a reasonable variant. All the patches got a r-b, series should be good to go. Dmitry Osipenko (19): drm/tegra: Fix lockup on a use of staging API drm/tegra: Correct idr_alloc() minimum id drm/tegra: Check for malformed offsets and sizes in the 'submit' IOCTL drm/tegra: Correct copying of waitchecks and disable them in the 'submit' IOCTL drm/tegra: Check syncpoint ID in the 'submit' IOCTL drm/tegra: dc: Avoid reset asserts on Tegra20 drm/tegra: dc: Apply clipping to the plane drm/tegra: dc: Disable plane if it is invisible gpu: host1x: Initialize firewall class to the jobs one gpu: host1x: Correct host1x_job_pin() error handling gpu: host1x: Do not leak BO's phys address to userspace gpu: host1x: Forbid relocation address shifting in the firewall gpu: host1x: Forbid RESTART opcode in the firewall gpu: host1x: Forbid unrelated SETCLASS opcode in the firewall gpu: host1x: Correct swapped arguments in the is_addr_reg() definition gpu: host1x: Check waits in the firewall gpu: host1x: Remove unused 'struct host1x_cmdbuf' gpu: host1x: Remove unused host1x_cdma_stop() definition gpu: host1x: At first try a non-blocking allocation for the gather copy Mikko Perttunen (1): gpu: host1x: Refactor channel allocation code drivers/gpu/drm/tegra/dc.c | 92 +++++++++++++++-------- drivers/gpu/drm/tegra/drm.c | 141 +++++++++++++++++++++++++++++------ drivers/gpu/drm/tegra/drm.h | 1 + drivers/gpu/drm/tegra/gem.c | 5 -- drivers/gpu/drm/tegra/gem.h | 5 ++ drivers/gpu/drm/tegra/gr2d.c | 11 ++- drivers/gpu/drm/tegra/gr3d.c | 4 +- drivers/gpu/drm/tegra/vic.c | 4 +- drivers/gpu/host1x/cdma.h | 1 - drivers/gpu/host1x/channel.c | 147 +++++++++++++++++++++++-------------- drivers/gpu/host1x/channel.h | 21 ++++-- drivers/gpu/host1x/debug.c | 47 +++++------- drivers/gpu/host1x/dev.c | 7 +- drivers/gpu/host1x/dev.h | 6 +- drivers/gpu/host1x/hw/channel_hw.c | 4 - drivers/gpu/host1x/job.c | 124 ++++++++++++++++++++++++------- drivers/gpu/host1x/job.h | 14 ---- include/linux/host1x.h | 13 +++- 18 files changed, 440 insertions(+), 207 deletions(-) -- 2.13.0

2 23

[PATCH][drm-next] drm/i915/cnl: make function cnl_ddi_dp_set_dpll_hw_state static
by Colin King 15 Jun '17

15 Jun '17

From: Colin Ian King <colin.king(a)canonical.com> The function cnl_ddi_dp_set_dpll_hw_state does not need to be in global scope, so make it static. Cleans up sparse warning: "symbol 'cnl_ddi_dp_set_dpll_hw_state' was not declared. Should it be static?" Signed-off-by: Colin Ian King <colin.king(a)canonical.com> --- drivers/gpu/drm/i915/intel_dpll_mgr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dpll_mgr.c b/drivers/gpu/drm/i915/intel_dpll_mgr.c index 8e669b6254ae..2f7b0e64f628 100644 --- a/drivers/gpu/drm/i915/intel_dpll_mgr.c +++ b/drivers/gpu/drm/i915/intel_dpll_mgr.c @@ -2292,8 +2292,9 @@ static bool cnl_ddi_hdmi_pll_dividers(struct intel_crtc *crtc, return true; } -bool cnl_ddi_dp_set_dpll_hw_state(int clock, - struct intel_dpll_hw_state *dpll_hw_state) +static bool +cnl_ddi_dp_set_dpll_hw_state(int clock, + struct intel_dpll_hw_state *dpll_hw_state) { uint32_t cfgcr0; -- 2.11.0

3 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

dri-devel June 2017