Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for i2c: refactor core and break out blocks

2017-05-27 Thread Wolfram Sang
> For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_4827/ As far as I can read the above page, all non-green blocks are caused by non-I2C issues. So, I'd assume my patch set doesn't break things. Good! signature.asc Description: PGP signature

[Intel-gfx] ✗ Fi.CI.BAT: warning for i2c: refactor core and break out blocks

2017-05-27 Thread Patchwork
== Series Details == Series: i2c: refactor core and break out blocks URL : https://patchwork.freedesktop.org/series/24985/ State : warning == Summary == Series 24985v1 i2c: refactor core and break out blocks https://patchwork.freedesktop.org/api/1.0/series/24985/revisions/1/mbox/ Test

[Intel-gfx] [RFT 1/8] i2c: rename core source file to allow refactorization

2017-05-27 Thread Wolfram Sang
The I2C core became quite huge and its monolithic structure makes maintenance hard. So, prepare to break out some functionality into seperate files by renaming the source file. Note that we keep the resulting object name constant to avoid regressions. Signed-off-by: Wolfram Sang

[Intel-gfx] [RFT 6/8] docs: i2c: dev-interface: adapt to new filenames of the i2c core

2017-05-27 Thread Wolfram Sang
The I2C core files were renamed, adapt the textfile to it. Signed-off-by: Wolfram Sang --- Documentation/i2c/dev-interface | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/Documentation/i2c/dev-interface b/Documentation/i2c/dev-interface

[Intel-gfx] [RFT 5/8] i2c: break out ACPI support into seperate file

2017-05-27 Thread Wolfram Sang
Removes some ifdeffery. Also add the new file to the relevant MAINTAINERS section. Signed-off-by: Wolfram Sang --- MAINTAINERS | 1 + drivers/i2c/Makefile| 1 + drivers/i2c/i2c-core-acpi.c | 653

[Intel-gfx] [RFT 3/8] i2c: break out smbus support into seperate file

2017-05-27 Thread Wolfram Sang
Break out the exported SMBus functions and the emulation layer into a seperate file. This also involved splitting up the tracing header into an I2C and an SMBus part. Signed-off-by: Wolfram Sang --- Documentation/driver-api/i2c.rst | 3 + drivers/i2c/Makefile |

[Intel-gfx] [RFT 7/8] i2c: remove unneeded includes from core

2017-05-27 Thread Wolfram Sang
They seem like cruft to me. I couldn't find any evidence that something included from there is actually used. Signed-off-by: Wolfram Sang --- drivers/i2c/i2c-core-base.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/i2c/i2c-core-base.c

[Intel-gfx] [RFT 4/8] i2c: break out OF support into seperate file

2017-05-27 Thread Wolfram Sang
Also removes some ifdeffery. Signed-off-by: Wolfram Sang --- drivers/i2c/Makefile| 1 + drivers/i2c/i2c-core-base.c | 265 +- drivers/i2c/i2c-core-of.c | 276

[Intel-gfx] [RFT 2/8] i2c: break out slave support into seperate file

2017-05-27 Thread Wolfram Sang
Also removes some ifdeffery. Signed-off-by: Wolfram Sang --- drivers/i2c/Makefile | 1 + drivers/i2c/i2c-core-base.c | 102 +- drivers/i2c/i2c-core-slave.c | 115 +++

[Intel-gfx] [RFT 8/8] i2c: reformat core-base file header

2017-05-27 Thread Wolfram Sang
Finally, apply modern comment rules to the file header. The old style looked very non-Linuxish and challenged my eyes for some time now. I took the liberty to a) drop the two credits for the multiplexing support here in this file because there is no significant code in the base file and the two

[Intel-gfx] [RFT 0/8] i2c: refactor core and break out blocks

2017-05-27 Thread Wolfram Sang
This series was originally posted as [1] to the i2c list. There, Jani suggested to resend the series here, so it could benefit from the CI tests seeing if I2C still works after this huge restructuring (it did for my local tests). Thank you very much, this is very helpful. So, I resend here as RFT.

Re: [Intel-gfx] [PATCH 0/8] i2c: refactor core and break out blocks

2017-05-27 Thread Wolfram Sang
> If you don't mind sending the whole series to the intel-gfx list (Cc'd), > our CI will run a bunch of tests on it, exercising our use of the I2C > adapter interfaces for display data channel and I2C over Display Port > native aux. Cool, that sounds very helpful! Thanks for the offer, I'll

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/3] drm/i915/gvt: Add gvt options sanitize function

2017-05-27 Thread Patchwork
== Series Details == Series: series starting with [v2,1/3] drm/i915/gvt: Add gvt options sanitize function URL : https://patchwork.freedesktop.org/series/24982/ State : success == Summary == Series 24982v1 Series without cover letter

[Intel-gfx] [PATCH v2 2/3] drm/i915/gvt: Return -EIO if host enable_execlists not enabled when loading GVT-g

2017-05-27 Thread Chuanxiao Dong
GVT-g relies on the enable_execlists parameter in i915. If this option is not enabled for GVT-g, should return -EIO to make i915 driver loading failed. v2: - Use DMR_ERROR as it is a fatal message. (Chris) Cc: Chris Wilson Suggested-by: Joonas Lahtinen

[Intel-gfx] [PATCH v2 1/3] drm/i915/gvt: Add gvt options sanitize function

2017-05-27 Thread Chuanxiao Dong
The intel_gvt_sanitize_options will sanitize the GVT related options before doing initialize the GVT. Suggested-by: Joonas Lahtinen Cc: Joonas Lahtinen Reviewed-by: Chris Wilson Cc: Chris Wilson

[Intel-gfx] [PATCH v2 3/3] drm/i915/gvt: Return -EIO if host GuC submission is enabled when loading GVT-g

2017-05-27 Thread Chuanxiao Dong
Currently GVT-g cannot work properly when host GuC submission is enabled, so make the driver loading failed in this case. v2: - use DRM_ERROR as it is a fatal message. (Chris) Suggested-by: Joonas Lahtinen Suggested-by: Chris Wilson

[Intel-gfx] [PATCH v2 0/3] drm/i915/gvt: GVT-g options sanitize series

2017-05-27 Thread Chuanxiao Dong
In this series, options sanitize API is added for GVT-g, and GVT-g init will fail with -EIO if detected incompatible i915 parameters. Chuanxiao Dong (3): drm/i915/gvt: Add gvt options sanitize function drm/i915/gvt: Return -EIO if host enable_execlists not enabled when loading GVT-g

Re: [Intel-gfx] [PATCH 1/3] drm/i915/gvt: Add gvt options sanitize function

2017-05-27 Thread Dong, Chuanxiao
> -Original Message- > From: intel-gvt-dev [mailto:intel-gvt-dev-boun...@lists.freedesktop.org] On > Behalf Of Chris Wilson > Sent: Friday, May 26, 2017 7:09 PM > To: Dong, Chuanxiao > Cc: intel-gfx@lists.freedesktop.org; intel-gvt-...@lists.freedesktop.org >

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gvt: dma-buf support for GVT-g (rev6)

2017-05-27 Thread Patchwork
== Series Details == Series: drm/i915/gvt: dma-buf support for GVT-g (rev6) URL : https://patchwork.freedesktop.org/series/23686/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h

[Intel-gfx] [PATCH v6 2/6] drm/i915/gvt: OpRegion support for GVT-g

2017-05-27 Thread Xiaoguang Chen
OpRegion is needed to support display related operation for intel vgpu. A vfio device region is added to intel vgpu to deliver the host OpRegion information to user space so user space can construct the OpRegion for vgpu. Signed-off-by: Bing Niu Signed-off-by: Xiaoguang Chen

[Intel-gfx] [PATCH v6 5/6] drm/i915/gvt: Dmabuf support for GVT-g

2017-05-27 Thread Xiaoguang Chen
dmabuf for GVT-g can be exported to users who can use the dmabuf to show the desktop of vm which use intel vgpu. Currently we provide query and create new dmabuf operations. Users of dmabuf can cache some created dmabufs and related information such as the framebuffer's address, size, tiling

[Intel-gfx] [PATCH v6 6/6] drm/i915/gvt: Adding interface so user space can get the dma-buf

2017-05-27 Thread Xiaoguang Chen
User space should create the management fd for the dma-buf operation first. Then user can query the plane information and create dma-buf if necessary using the management fd. Signed-off-by: Xiaoguang Chen --- drivers/gpu/drm/i915/gvt/dmabuf.c | 12

[Intel-gfx] [PATCH v6 3/6] drm/i915/gvt: Frame buffer decoder support for GVT-g

2017-05-27 Thread Xiaoguang Chen
decode frambuffer attributes of primary, cursor and sprite plane Signed-off-by: Xiaoguang Chen --- drivers/gpu/drm/i915/gvt/Makefile | 3 +- drivers/gpu/drm/i915/gvt/display.c| 2 +- drivers/gpu/drm/i915/gvt/display.h| 2 +

[Intel-gfx] [PATCH v6 0/6] drm/i915/gvt: Dma-buf support for GVT-g

2017-05-27 Thread Xiaoguang Chen
v5->v6: 1) align the dma-buf life cycle with the vfio device. 2) add the dma-buf releated operations in a separate patch. 3) i915 releated changes. v4->v5: 1) fix bug while checking whether the gem obj is gvt's dma-buf when user change caching mode or domains. Add a helper function to do it. 2)

[Intel-gfx] [PATCH v6 1/6] drm/i915/gvt: Extend the GVT-g architecture to support vfio device region

2017-05-27 Thread Xiaoguang Chen
Signed-off-by: Xiaoguang Chen --- drivers/gpu/drm/i915/gvt/kvmgt.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/gvt/kvmgt.c b/drivers/gpu/drm/i915/gvt/kvmgt.c index 1ae0b40..3c6a02b 100644 ---

[Intel-gfx] [PATCH v6 4/6] vfio: Define vfio based vgpu's dma-buf operations

2017-05-27 Thread Xiaoguang Chen
Here we defined a new ioctl to create a fd for a vfio device based on the input type. Now only one type is supported that is a dma-buf management fd. Two ioctls are defined for the dma-buf management fd: query the vfio vgpu's plane information and create a dma-buf for a plane. Signed-off-by: