Hi, This patch misses dsi bus un-registration. However I have reviewed two patches of Takashi which solves the same issue more completely [1]. These patches were not accepted neither.
[1]: https://lists.freedesktop.org/archives/dri-devel/2016-August/114870.html Regards Andrzej On 11.10.2016 10:40, Sun, Jing A wrote: > Dear Maintainers, > > It's needed that DRM Driver module could be removed and reloaded after kernel > booting on the projects that I have been working on, and I hope such module > type change could be accepted. Looks like Iwai has similar change request as > well. Would you please review it and let us know if any concerns? > > Regards, > Sun, Jing > > > -----Original Message----- > From: Takashi Iwai [mailto:tiwai at suse.de] > Sent: Monday, October 10, 2016 5:57 PM > To: Jani Nikula > Cc: Sun, Jing A; airlied at linux.ie; Vetter, Daniel; linux-kernel at > vger.kernel.org; dri-devel at lists.freedesktop.org; Takashi Iwai; a.hajda at > samsung.com; Thierry Reding > Subject: Re: [PATCH]"drm: change DRM_MIPI_DSI module type from "bool" to > "tristate". > > On Mon, 10 Oct 2016 10:28:31 +0200, > Jani Nikula wrote: >> On Mon, 10 Oct 2016, "Sun, Jing A" <jing.a.sun at intel.com> wrote: >>> Dear Maintainers, >>> >>> Please kindly review my patch as below. It's based on the mainline branch. >>> >>> From b401009f79883ac5e9d41525c9d54b800ece2e22 Mon Sep 17 00:00:00 >>> 2001 >>> From: Jing SUN <jing.a.sun at intel.com> >>> Date: Mon, 10 Oct 2016 14:06:54 +0800 >>> Subject: [PATCH 1/1] drm: change DRM_MIPI_DSI module type from >>> "bool" to "tristate". >>> >>> A lot of drm driver modules, which are designed to be loadable, >>> select DRM_MIPI_DSI, while that being "bool" prevents those from >>> getting reloaded. >> You're missing some lists and people from distribution, added now. >> >> See the discussion starting at [1]. I don't know if anything has >> happened since then. Takashi? > Unfortunately, nothing seems to have happened since my last patch due to > little interest. I'd be glad if the patch is revived. > > (BTW, I'm traveling in these two weeks, so the further reply will be > delayed.) > > > thanks, > > Takashi > > >> BR, >> Jani. >> >> >> [1] http://lkml.kernel.org/r/s5hh9bhvj7j.wl-tiwai at suse.de >> >> >>> Signed-off-by: Jing SUN <jing.a.sun at intel.com> >>> --- >>> drivers/gpu/drm/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig index >>> fc35731..67668a0 100644 >>> --- a/drivers/gpu/drm/Kconfig >>> +++ b/drivers/gpu/drm/Kconfig >>> @@ -22,7 +22,7 @@ menuconfig DRM >>> (/dev/agpgart) support if it is available for your platform. >>> >>> config DRM_MIPI_DSI >>> - bool >>> + tristate >>> depends on DRM >>> >>> config DRM_DP_AUX_CHARDEV >> -- >> Jani Nikula, Intel Open Source Technology Center >> > >
