Re: [PATCH] drm/i915/cnl: Removing missing DDI_E bits from CNL.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



merged to dinq, thanks for the review.


On Tue, Aug 8, 2017 at 4:36 AM, Imre Deak <imre.deak@xxxxxxxxx> wrote:
> On Thu, Aug 03, 2017 at 03:51:37PM -0700, Rodrigo Vivi wrote:
>> DDI_E is not supported on CNL-U and CNL-Y
>>
>> When adding the inital support we noticed DDI_E wasn't supported
>> and removed it on v4 and v5 of that patch.
>> However for some reason I missed or put back these 2 chunks.
>>
>> Time to clean it up to avoid later confusion.
>>
>> Fixes: 8bcd3dd41766 ("drm/i915/cnl: Add power wells for CNL")
>> Cc: Clint Taylor <clinton.a.taylor@xxxxxxxxx>
>> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>> Cc: Imre Deak <imre.deak@xxxxxxxxx>
>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx>
>> Cc: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx>
>> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
>
> Looks ok and also the correct thing for CNL_H/S, where the port E DDI
> power well is separate from port A and not dependent on power well #2
> (unlike SKL):
>
> Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx>
>
>> ---
>>  drivers/gpu/drm/i915/intel_runtime_pm.c | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c
>> index 347484ed6d2c..e77e84a6fd2f 100644
>> --- a/drivers/gpu/drm/i915/intel_runtime_pm.c
>> +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
>> @@ -1796,7 +1796,6 @@ void intel_display_power_put(struct drm_i915_private *dev_priv,
>>       BIT_ULL(POWER_DOMAIN_PORT_DDI_B_LANES) |                \
>>       BIT_ULL(POWER_DOMAIN_PORT_DDI_C_LANES) |                \
>>       BIT_ULL(POWER_DOMAIN_PORT_DDI_D_LANES) |                \
>> -     BIT_ULL(POWER_DOMAIN_PORT_DDI_E_LANES) |                \
>>       BIT_ULL(POWER_DOMAIN_AUX_B) |                       \
>>       BIT_ULL(POWER_DOMAIN_AUX_C) |                   \
>>       BIT_ULL(POWER_DOMAIN_AUX_D) |                   \
>> @@ -1805,7 +1804,6 @@ void intel_display_power_put(struct drm_i915_private *dev_priv,
>>       BIT_ULL(POWER_DOMAIN_INIT))
>>  #define CNL_DISPLAY_DDI_A_IO_POWER_DOMAINS (         \
>>       BIT_ULL(POWER_DOMAIN_PORT_DDI_A_IO) |           \
>> -     BIT_ULL(POWER_DOMAIN_PORT_DDI_E_IO) |           \
>>       BIT_ULL(POWER_DOMAIN_INIT))
>>  #define CNL_DISPLAY_DDI_B_IO_POWER_DOMAINS (         \
>>       BIT_ULL(POWER_DOMAIN_PORT_DDI_B_IO) |           \
>> --
>> 2.13.2
>>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux