Re: [Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-05-31 Thread kbuild test robot
Hi Rodrigo, [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on v4.12-rc3 next-20170531] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-05-31 Thread Srivatsa, Anusha
top.org>; Nikula, Jani ><jani.nik...@intel.com>; Vetter, Daniel <daniel.vet...@intel.com> >Subject: Re: [Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support > >Anusha, when going to merge I noticed that we had the wrong version here. > >The incorrect version would break

Re: [Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-05-31 Thread Rodrigo Vivi
Anusha, when going to merge I noticed that we had the wrong version here. The incorrect version would break hdmi on CFL. The right workaround is not to respect the CNL bios for the pin number. So, I hope I can keep your rv-b on this one, but I'd like you to confirm please. Thanks, Rodrigo. On

[Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-05-31 Thread Rodrigo Vivi
On CNP PCH based platforms the gmbus is on the south display that is on PCH. The existing implementation for previous platforms already covers the need for CNP expect for the pin pair configuration that follows similar definitions that we had on BXT. v2: Don't drop "_BXT" as the indicator of the

Re: [Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-04-07 Thread kbuild test robot
Hi Rodrigo, [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on v4.11-rc5] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[Intel-gfx] [PATCH] drm/i915/cnp: add CNP gmbus support

2017-04-06 Thread Rodrigo Vivi
On CNP PCH based platforms the gmbus is on the south display that is on PCH. The existing implementation for previous platforms already covers the need for CNP expect for the pin pair configuration that follows similar definitions that we had on BXT. v2: Don't drop "_BXT" as the indicator of the