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

2020-10-30 Thread Alex Deucher
On Fri, Oct 30, 2020 at 6:11 AM 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 horror show

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

2020-10-26 Thread Thomas Zimmermann
Hi Am 23.10.20 um 14:28 schrieb Daniel Vetter: > 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 horror show :-) >

Re: [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

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

2020-10-24 Thread Maxime Ripard
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