Re: [PATCH 4/5] drm/mgag200: Init and finalize devices in mgag200_device_{init,fini}()

2020-05-06 Thread Thomas Zimmermann
Hi Sam Am 05.05.20 um 18:49 schrieb Sam Ravnborg: > Hi Thomas. > > On Tue, May 05, 2020 at 11:56:48AM +0200, Thomas Zimmermann wrote: >> Device initialization is now done in mgag200_device_init(). Specifically, >> the function allocates the DRM device and sets up the respective fields >> in

Re: [PATCH 4/5] drm/mgag200: Init and finalize devices in mgag200_device_{init,fini}()

2020-05-06 Thread Thomas Zimmermann
Hi Am 05.05.20 um 16:14 schrieb Daniel Vetter: > On Tue, May 05, 2020 at 11:56:48AM +0200, Thomas Zimmermann wrote: >> Device initialization is now done in mgag200_device_init(). Specifically, >> the function allocates the DRM device and sets up the respective fields >> in struct mga_device. >>

Re: [PATCH 4/5] drm/mgag200: Init and finalize devices in mgag200_device_{init,fini}()

2020-05-05 Thread Sam Ravnborg
Hi Thomas. On Tue, May 05, 2020 at 11:56:48AM +0200, Thomas Zimmermann wrote: > Device initialization is now done in mgag200_device_init(). Specifically, > the function allocates the DRM device and sets up the respective fields > in struct mga_device. > > A call to mgag200_device_fini()

Re: [PATCH 4/5] drm/mgag200: Init and finalize devices in mgag200_device_{init,fini}()

2020-05-05 Thread Daniel Vetter
On Tue, May 05, 2020 at 11:56:48AM +0200, Thomas Zimmermann wrote: > Device initialization is now done in mgag200_device_init(). Specifically, > the function allocates the DRM device and sets up the respective fields > in struct mga_device. > > A call to mgag200_device_fini() finalizes struct

[PATCH 4/5] drm/mgag200: Init and finalize devices in mgag200_device_{init, fini}()

2020-05-05 Thread Thomas Zimmermann
Device initialization is now done in mgag200_device_init(). Specifically, the function allocates the DRM device and sets up the respective fields in struct mga_device. A call to mgag200_device_fini() finalizes struct mga_device. The old function mgag200_driver_load() and mgag200_driver_unload()