From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Fabio Estevam <festevam@gmail.com>
Cc: "Mark Rutland" <mark.rutland@arm.com>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
"Jernej Skrabec" <jernej.skrabec@siol.net>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Neil Armstrong" <narmstrong@baylibre.com>,
"David Airlie" <airlied@linux.ie>,
"Lee Jones" <lee.jones@linaro.org>,
"Guido Günther" <agx@sigxcpu.org>,
"Jonas Karlman" <jonas@kwiboo.se>,
linux-kernel <linux-kernel@vger.kernel.org>,
"DRI mailing list" <dri-devel@lists.freedesktop.org>,
"Andrzej Hajda" <a.hajda@samsung.com>,
"Rob Herring" <robh+dt@kernel.org>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Robert Chiras" <robert.chiras@nxp.com>,
"Chris Healy" <cphealy@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 3/3] drm/bridge: Add NWL MIPI DSI host controller support
Date: Sat, 27 Jul 2019 05:04:44 +0300 [thread overview]
Message-ID: <20190727020444.GC4902@pendragon.ideasonboard.com> (raw)
In-Reply-To: <CAOMZO5BRbV_1du1b9eJqcBvvXSE2Mon3yxSPJxPpZgBqYNjBSg@mail.gmail.com>
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 <agx@sigxcpu.org> 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Fabio Estevam <festevam@gmail.com>
Cc: "Guido Günther" <agx@sigxcpu.org>,
"David Airlie" <airlied@linux.ie>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Rob Herring" <robh+dt@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Andrzej Hajda" <a.hajda@samsung.com>,
"Neil Armstrong" <narmstrong@baylibre.com>,
"Jonas Karlman" <jonas@kwiboo.se>,
"Jernej Skrabec" <jernej.skrabec@siol.net>,
"Lee Jones" <lee.jones@linaro.org>,
"DRI mailing list" <dri-devel@lists.freedesktop.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>
Subject: Re: [PATCH 3/3] drm/bridge: Add NWL MIPI DSI host controller support
Date: Sat, 27 Jul 2019 05:04:44 +0300 [thread overview]
Message-ID: <20190727020444.GC4902@pendragon.ideasonboard.com> (raw)
In-Reply-To: <CAOMZO5BRbV_1du1b9eJqcBvvXSE2Mon3yxSPJxPpZgBqYNjBSg@mail.gmail.com>
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 <agx@sigxcpu.org> 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
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Fabio Estevam <festevam@gmail.com>
Cc: "Guido Günther" <agx@sigxcpu.org>,
"David Airlie" <airlied@linux.ie>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Rob Herring" <robh+dt@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Andrzej Hajda" <a.hajda@samsung.com>,
"Neil Armstrong" <narmstrong@baylibre.com>,
"Jonas Karlman" <jonas@kwiboo.se>,
"Jernej Skrabec" <jernej.skrabec@siol.net>,
"Lee Jones" <lee.jones@linaro.org>,
"DRI mailing list" <dri-devel@lists.freedesktop.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
"Robert Chiras" <robert.chiras@nxp.com>,
"Chris Healy" <cphealy@gmail.com>
Subject: Re: [PATCH 3/3] drm/bridge: Add NWL MIPI DSI host controller support
Date: Sat, 27 Jul 2019 05:04:44 +0300 [thread overview]
Message-ID: <20190727020444.GC4902@pendragon.ideasonboard.com> (raw)
In-Reply-To: <CAOMZO5BRbV_1du1b9eJqcBvvXSE2Mon3yxSPJxPpZgBqYNjBSg@mail.gmail.com>
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 <agx@sigxcpu.org> 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
next prev parent reply other threads:[~2019-07-27 2:04 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-24 15:52 [PATCH 0/3] drm: bridge: Add NWL MIPI DSI host controller support Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` [PATCH 1/3] arm64: imx8mq: add imx8mq iomux-gpr field defines Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` [PATCH 2/3] dt-bindings: display/bridge: Add binding for IMX NWL mipi dsi host controller Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-26 9:23 ` Sam Ravnborg
2019-07-26 9:23 ` Sam Ravnborg
2019-07-26 9:23 ` Sam Ravnborg
2019-07-26 11:05 ` Guido Günther
2019-07-26 11:05 ` Guido Günther
2019-07-27 1:59 ` Laurent Pinchart
2019-07-27 1:59 ` Laurent Pinchart
2019-07-27 1:57 ` Laurent Pinchart
2019-07-27 1:57 ` Laurent Pinchart
2019-07-27 1:57 ` Laurent Pinchart
2019-07-31 14:37 ` Guido Günther
2019-07-31 14:37 ` Guido Günther
2019-07-24 15:52 ` [PATCH 3/3] drm/bridge: Add NWL MIPI DSI host controller support Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-24 15:52 ` Guido Günther
2019-07-26 10:08 ` Sam Ravnborg
2019-07-26 10:08 ` Sam Ravnborg
2019-07-26 20:01 ` Fabio Estevam
2019-07-26 20:01 ` Fabio Estevam
2019-07-26 20:01 ` Fabio Estevam
2019-07-27 2:04 ` Laurent Pinchart [this message]
2019-07-27 2:04 ` Laurent Pinchart
2019-07-27 2:04 ` Laurent Pinchart
2019-07-31 14:38 ` Guido Günther
2019-07-31 14:38 ` Guido Günther
2019-07-31 14:38 ` Guido Günther
2019-07-31 14:35 ` Guido Günther
2019-07-31 14:35 ` Guido Günther
2019-07-31 14:35 ` Guido Günther
2019-07-31 14:43 ` Fabio Estevam
2019-07-31 14:43 ` Fabio Estevam
2019-07-31 14:43 ` Fabio Estevam
2019-07-31 16:40 ` Jernej Škrabec
2019-07-31 16:40 ` Jernej Škrabec
2019-07-31 16:40 ` Jernej Škrabec
2019-07-31 17:40 ` Fabio Estevam
2019-07-31 17:40 ` Fabio Estevam
2019-07-31 17:40 ` Fabio Estevam
2019-07-31 16:54 ` Guido Günther
2019-07-31 16:54 ` Guido Günther
2019-07-31 16:54 ` Guido Günther
2019-07-27 2:47 ` Laurent Pinchart
2019-07-27 2:47 ` Laurent Pinchart
2019-07-27 2:47 ` Laurent Pinchart
2019-08-09 16:25 ` Guido Günther
2019-08-09 16:25 ` Guido Günther
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190727020444.GC4902@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=a.hajda@samsung.com \
--cc=agx@sigxcpu.org \
--cc=airlied@linux.ie \
--cc=cphealy@gmail.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=festevam@gmail.com \
--cc=jernej.skrabec@siol.net \
--cc=jonas@kwiboo.se \
--cc=kernel@pengutronix.de \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=narmstrong@baylibre.com \
--cc=robert.chiras@nxp.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.