From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 3/3] drm/bridge: Add NWL MIPI DSI host controller support Date: Sat, 27 Jul 2019 05:04:44 +0300 Message-ID: <20190727020444.GC4902@pendragon.ideasonboard.com> References: <3158f4f8c97c21f98c394e5631d74bc60d796522.1563983037.git.agx@sigxcpu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Fabio Estevam Cc: Guido =?utf-8?Q?G=C3=BCnther?= , David Airlie , Daniel Vetter , Rob Herring , Mark Rutland , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , NXP Linux Team , Andrzej Hajda , Neil Armstrong , Jonas Karlman , Jernej Skrabec , Lee Jones , DRI mailing list , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" List-Id: devicetree@vger.kernel.org Hello, On Fri, Jul 26, 2019 at 05:01:52PM -0300, Fabio Estevam wrote: > Hi Guido, > > Thanks for your work on this driver! > > On Wed, Jul 24, 2019 at 12:52 PM Guido Günther wrote: > > > --- /dev/null > > +++ b/drivers/gpu/drm/bridge/imx-nwl/Kconfig > > @@ -0,0 +1,15 @@ > > +config DRM_IMX_NWL_DSI > > + tristate "Support for Northwest Logic MIPI DSI Host controller" > > + depends on DRM && (ARCH_MXC || ARCH_MULTIPLATFORM || COMPILE_TEST) > > This IP could potentially be found on other SoCs, so no need to make > it depend on ARCH_MXC. I'd go even further and not use the prefix imx in the driver name or anywhere in the code. [snip] -- Regards, Laurent Pinchart