* LVDS (LDB) driver for i.MX53 IPU
@ 2013-03-04 17:51 Martin Fuzzey
2013-03-10 22:32 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: Martin Fuzzey @ 2013-03-04 17:51 UTC (permalink / raw)
To: linux-arm-kernel
Hi Sascha,
I need to add LVDS support to your staging imx/drm driver.
First of all are there any patches floating around to do that? (a search
of the mailing list didn't turn up any).
From a hardware point of view it looks quite simple (just one register
to configure) but unfortunately that register is in the IOMUXC region.
That means that if I write a new platform driver similar to
drivers/staging/imx-drm/parallel-display.c (with a new DT node pointing
to the reg) the request_resource() will fail since it will already be
claimed by the iomuxc driver.
The only solutions I can think of are:
1) Add exported function to the iomuxc driver to get the register
or
2) Directly ioremap() the address
Both of these sound horrible.
Do you have any suggestions?
I guess the generic question I'm asking is how to handle the case where
unrelated drivers require access to the same memory regions?
Regards,
Martin
^ permalink raw reply [flat|nested] 4+ messages in thread
* LVDS (LDB) driver for i.MX53 IPU
2013-03-04 17:51 LVDS (LDB) driver for i.MX53 IPU Martin Fuzzey
@ 2013-03-10 22:32 ` Sascha Hauer
2013-05-24 12:35 ` Guennadi Liakhovetski
0 siblings, 1 reply; 4+ messages in thread
From: Sascha Hauer @ 2013-03-10 22:32 UTC (permalink / raw)
To: linux-arm-kernel
Hi Martin,
On Mon, Mar 04, 2013 at 06:51:10PM +0100, Martin Fuzzey wrote:
> Hi Sascha,
>
> I need to add LVDS support to your staging imx/drm driver.
>
> First of all are there any patches floating around to do that? (a
> search of the mailing list didn't turn up any).
>
> From a hardware point of view it looks quite simple (just one
> register to configure) but unfortunately that register is in the
> IOMUXC region.
>
> That means that if I write a new platform driver similar to
> drivers/staging/imx-drm/parallel-display.c (with a new DT node
> pointing to the reg) the request_resource() will fail since it will
> already be claimed by the iomuxc driver.
>
> The only solutions I can think of are:
> 1) Add exported function to the iomuxc driver to get the register
> or
> 2) Directly ioremap() the address
>
> Both of these sound horrible.
>
> Do you have any suggestions?
>
> I guess the generic question I'm asking is how to handle the case
> where unrelated drivers require access to the same memory regions?
>
We have a driver for the ldb. Indeed the driver itself is trivial, but
the integration is tricky. For accessing the register the syscon binding
can be used. The clock is also a problem, also the mux to the IPU(s)
I'll talk with Philipp tomorrow and I think we can send you the current
state then.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 4+ messages in thread
* LVDS (LDB) driver for i.MX53 IPU
2013-03-10 22:32 ` Sascha Hauer
@ 2013-05-24 12:35 ` Guennadi Liakhovetski
2013-05-24 13:12 ` Martin Fuzzey
0 siblings, 1 reply; 4+ messages in thread
From: Guennadi Liakhovetski @ 2013-05-24 12:35 UTC (permalink / raw)
To: linux-arm-kernel
Hi Sascha
On Sun, 10 Mar 2013, Sascha Hauer wrote:
> Hi Martin,
>
> On Mon, Mar 04, 2013 at 06:51:10PM +0100, Martin Fuzzey wrote:
> > Hi Sascha,
> >
> > I need to add LVDS support to your staging imx/drm driver.
> >
> > First of all are there any patches floating around to do that? (a
> > search of the mailing list didn't turn up any).
> >
> > From a hardware point of view it looks quite simple (just one
> > register to configure) but unfortunately that register is in the
> > IOMUXC region.
> >
> > That means that if I write a new platform driver similar to
> > drivers/staging/imx-drm/parallel-display.c (with a new DT node
> > pointing to the reg) the request_resource() will fail since it will
> > already be claimed by the iomuxc driver.
> >
> > The only solutions I can think of are:
> > 1) Add exported function to the iomuxc driver to get the register
> > or
> > 2) Directly ioremap() the address
> >
> > Both of these sound horrible.
> >
> > Do you have any suggestions?
> >
> > I guess the generic question I'm asking is how to handle the case
> > where unrelated drivers require access to the same memory regions?
>
> We have a driver for the ldb. Indeed the driver itself is trivial, but
> the integration is tricky. For accessing the register the syscon binding
> can be used. The clock is also a problem, also the mux to the IPU(s)
>
> I'll talk with Philipp tomorrow and I think we can send you the current
> state then.
How's it looking in the meantime? Any plans to mainline it? Or would you
be prepared to publish what you've got to try and see whether others can
help you with this?
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 4+ messages in thread
* LVDS (LDB) driver for i.MX53 IPU
2013-05-24 12:35 ` Guennadi Liakhovetski
@ 2013-05-24 13:12 ` Martin Fuzzey
0 siblings, 0 replies; 4+ messages in thread
From: Martin Fuzzey @ 2013-05-24 13:12 UTC (permalink / raw)
To: linux-arm-kernel
Hi Guennadi,
On 24/05/13 14:35, Guennadi Liakhovetski wrote:
> Hi Sascha
>
> On Sun, 10 Mar 2013, Sascha Hauer wrote:
>
>>
>> We have a driver for the ldb. Indeed the driver itself is trivial, but
>> the integration is tricky. For accessing the register the syscon binding
>> can be used. The clock is also a problem, also the mux to the IPU(s)
>>
>> I'll talk with Philipp tomorrow and I think we can send you the current
>> state then.
> How's it looking in the meantime? Any plans to mainline it? Or would you
> be prepared to publish what you've got to try and see whether others can
> help you with this?
Philipp submitted them on March 28 "[PATCH v3 0/11] LVDS Display Bridge
support for i.MX"
The drivers/staging parts don't seem to have been applied (not in
linux/next at any rate)
I've added them to my local tree and they work fine (on i.MX53)
Martin
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-05-24 13:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-04 17:51 LVDS (LDB) driver for i.MX53 IPU Martin Fuzzey
2013-03-10 22:32 ` Sascha Hauer
2013-05-24 12:35 ` Guennadi Liakhovetski
2013-05-24 13:12 ` Martin Fuzzey
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.