Re: [PATCH 1/3] drm/radeon: Make CIK support in Radeon conditional (v2)

["Michel Dänzer" <michel-otUistvHUpPR7s880joybQ@public.gmane.org>]

On 08/04/17 05:15 AM, Felix Kuehling wrote:
> Advertise CIK PCI IDs only when they are not supported by amdgpu.
> Use the CONFIG_DRM_AMDGPU_CIK to check so that a single option in
> the kernel config keeps both drivers in sync.
> 
> This is the simplest possible change. A more complete solution
> may want to conditionally disable more CIK-specific code in the
> Radeon driver.
> 
> v2: Add CONFIG_DRM_RADEON_FORCE_CIK option
> 
> Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>

I don't think this is good enough a solution.

I want to be able to choose the driver to use at runtime, for each GPU
individually. To make that possible, we probably need some kind of
kernel command line option which is respected by both radeon and amdgpu
drivers, plus possibly additional per-driver module parameters which can
override that kernel command line option.

CONFIG_DRM_AMDGPU_CIK/SI would then simply define which driver gets to
drive CIK/SI GPUs by default.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx