On 30/05/17 06:17 PM, Grazvydas Ignotas wrote: > On Tue, May 30, 2017 at 6:30 AM, Michel Dänzer <michel at daenzer.net> wrote: >> On 29/05/17 06:20 PM, Michel Dänzer wrote: >>> From: Michel Dänzer <michel.daenzer at amd.com> >>> >>> Even if CONFIG_DRM_AMDGPU_CIK is enabled. >>> >>> There is no feature parity yet for CIK, in particular amdgpu doesn't >>> support HDMI/DisplayPort without DC. >>> >>> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com> >> >> [...] >> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig b/drivers/gpu/drm/amd/amdgpu/Kconfig >>> index 8d36087fc186..e0121f8b436e 100644 >>> --- a/drivers/gpu/drm/amd/amdgpu/Kconfig >>> +++ b/drivers/gpu/drm/amd/amdgpu/Kconfig >>> @@ -17,11 +17,11 @@ config DRM_AMDGPU_CIK >>> help >>> Choose this option if you want to enable support for CIK asics. >>> >>> - CIK is already supported in radeon. If you enable this option, >>> - support for CIK will be provided by amdgpu and disabled in >>> - radeon by default. Use module options to override this: >>> + CIK is already supported in radeon. Support for SI in amdgpu >> >> Consider this "SI" typo fixed in v2. > > While you are here, what about adding the full codenames here? You > can't expect every user configuring the kernel to know what SI/CIK > means, it's not even documented in > https://www.x.org/wiki/RadeonFeature/ or wikipedia, while the long > codenames are. That's out of scope for this series, and I definitely won't get around to doing that before next week. Anybody feel free to beat me to it. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
