linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] video: imx: Update the manufacturer's name
@ 2011-01-10 11:47 Fabio Estevam
  2011-01-10 19:10 ` Paul Mundt
  0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2011-01-10 11:47 UTC (permalink / raw)
  To: linux-fbdev

i.MX processors are currently manufactured by Freescale, not Motorola.

Make the manufacturer's name consistent.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
Changes since v1:
- Improved commit message
- Replaced Motorola reference in imxfb.c

 drivers/video/Kconfig |    2 +-
 drivers/video/imxfb.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index 55dc6fb..0ac30ef 100644
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -414,7 +414,7 @@ config FB_SA1100
 	  Y here.
 
 config FB_IMX
-	tristate "Motorola i.MX LCD support"
+	tristate "Freescale i.MX LCD support"
 	depends on FB && (HAVE_FB_IMX || ARCH_MX1 || ARCH_MX2)
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
diff --git a/drivers/video/imxfb.c b/drivers/video/imxfb.c
index 1ab2c25..69bd4a5 100644
--- a/drivers/video/imxfb.c
+++ b/drivers/video/imxfb.c
@@ -974,6 +974,6 @@ static void __exit imxfb_cleanup(void)
 module_init(imxfb_init);
 module_exit(imxfb_cleanup);
 
-MODULE_DESCRIPTION("Motorola i.MX framebuffer driver");
+MODULE_DESCRIPTION("Freescale i.MX framebuffer driver");
 MODULE_AUTHOR("Sascha Hauer, Pengutronix");
 MODULE_LICENSE("GPL");
-- 
1.6.0.4



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2] video: imx: Update the manufacturer's name
  2011-01-10 11:47 [PATCH v2] video: imx: Update the manufacturer's name Fabio Estevam
@ 2011-01-10 19:10 ` Paul Mundt
  0 siblings, 0 replies; 2+ messages in thread
From: Paul Mundt @ 2011-01-10 19:10 UTC (permalink / raw)
  To: linux-fbdev

On Mon, Jan 10, 2011 at 09:47:41AM -0200, Fabio Estevam wrote:
> i.MX processors are currently manufactured by Freescale, not Motorola.
> 
> Make the manufacturer's name consistent.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

This is of course fine with me, but it's also a good reminder of why we
traditionally avoid having this sort of information in the option text.

Most of these sorts of drivers already have their own platform or
architecture dependencies that effectively restrict the driver to a
specific SoC or architecture where folks are likely already going to have
a pretty good idea of who the manufacturer of a given driver is and
whether they are interested in this particular driver or not.

While it's probably not an issue in your case, it can also be confusing
when legacy parts from one company are attributed to a newer incarnation
of the same company, specifically when those parts where never
manufactured under the new corporate entity. While an IP transfer does
invariably take place, parts that are already on life support from the
old corporate entity are rarely factored in to the new corporate entity's
roadmap, meaning that it's also possible to attribute part/company
relationships that don't actually exist.

While it may seem obvious to you that folks with older Motorola-branded
SoCs enable a freescale-branded driver, one could also easily infer that
the driver only applies to newer freescale-braneded parts. (Admittedly
this is not much of an issue for the SoC space, but it's certainly
something to watch out for with PCI drivers that are less rigidly locked
down in their dependency chain).

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-01-10 19:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-10 11:47 [PATCH v2] video: imx: Update the manufacturer's name Fabio Estevam
2011-01-10 19:10 ` Paul Mundt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).