[PATCH 1/3] drm/amdgpu: Really leave SI support disabled by default

deathsimple@xxxxxxxxxxx (Christian König) · Tue, 30 May 2017 09:57:38 +0200

Am 30.05.2017 um 05:26 schrieb Michel DÃ¤nzer:
> On 30/05/17 02:18 AM, Christian KÃ¶nig wrote:
>> Am 29.05.2017 um 11:20 schrieb Michel DÃ¤nzer:
>>> From: Michel DÃ¤nzer <michel.daenzer at amd.com>
>>>
>>> The default option value didn't match the help text and intention.
>>>
>>> Signed-off-by: Michel DÃ¤nzer <michel.daenzer at amd.com>
>> I'm still unsure about the last one. The feature parity is a good
>> argument but on the other hand we want people to use amdgpu for CIK
>> these days, don't we?
> We want to make it easy for people to test amdgpu on CIK, which is what
> the options added by Felix are for. IMO we should not flip the default
> (upstream) before there is feature parity.
>
>
>> Anyway Reviewed-by: Christian KÃ¶nig <christian.koenig at amd.com>.
> Thanks, I assume that applies to the whole series?

With the SI typo and the HDMI/DP audio thing pointed out by Kai fixed, 
than yes that rb applies to the whole series.

Christian.