Re: [Intel-gfx] [PATCH] drm/: Constify struct drm_driver

2020-10-25 Thread Daniel Vetter
On Sun, Oct 25, 2020 at 11:23 PM Sam Ravnborg wrote: > > Hi Daniel. > > On Fri, Oct 23, 2020 at 06:04:44PM +0200, Daniel Vetter wrote: > > Only the following drivers aren't converted: > > - amdgpu, because of the driver_feature mangling due to virt support > > - nouveau, because DRIVER_ATOMIC uapi

Re: [Intel-gfx] [PATCH] drm/: Constify struct drm_driver

2020-10-25 Thread Sam Ravnborg
Hi Daniel. On Fri, Oct 23, 2020 at 06:04:44PM +0200, Daniel Vetter wrote: > Only the following drivers aren't converted: > - amdgpu, because of the driver_feature mangling due to virt support > - nouveau, because DRIVER_ATOMIC uapi is still not the default on the > platforms where it's supported

Re: [Intel-gfx] [PATCH 2/5] drm: Compile out legacy chunks from struct drm_device

2020-10-25 Thread Sam Ravnborg
On Fri, Oct 23, 2020 at 02:28:08PM +0200, Daniel Vetter wrote: > This means some very few #ifdef in code, but it allows us to > enlist the compiler to make sure this stuff isn't used anymore. > > More important, only legacy drivers change drm_device (for the > legacy_dev_list shadow attach managem

Re: [Intel-gfx] [PATCH 4/5] drm: Allow const struct drm_driver

2020-10-25 Thread Sam Ravnborg
On Fri, Oct 23, 2020 at 02:28:10PM +0200, Daniel Vetter wrote: > It's nice if a big function/ioctl table like this is const. Only > downside here is that we need a few more #ifdef to paper over the > differences when CONFIG_DRM_LEGACY is enabled. Maybe provides more > motivation to sunset that horr