RE: [PATCH v2 4/7] OMAP: DSS: Add i2c client driver for picodlp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Valkeinen, Tomi
> Sent: Wednesday, May 04, 2011 4:36 PM
> To: Janorkar, Mayuresh
> Cc: linux-omap@xxxxxxxxxxxxxxx; K, Mythri P
> Subject: RE: [PATCH v2 4/7] OMAP: DSS: Add i2c client driver for picodlp
> 
> On Wed, 2011-05-04 at 15:31 +0530, Janorkar, Mayuresh wrote:
> >
> > > -----Original Message-----
> > > From: Valkeinen, Tomi
> > > Sent: Wednesday, May 04, 2011 12:15 AM
> > > To: Janorkar, Mayuresh
> > > Cc: linux-omap@xxxxxxxxxxxxxxx; K, Mythri P
> > > Subject: Re: [PATCH v2 4/7] OMAP: DSS: Add i2c client driver for
> picodlp
> > >
> > > On Mon, 2011-05-02 at 20:22 +0530, Mayuresh Janorkar wrote:
> > > > The configurations and data transfer with picodlp panel happens
> through
> > > i2c.
> > > > An i2c client with name "picodlp_i2c_driver" is registered inside
> panel.
> > > >
> > > > dpp2600 requires 4 gpio lines for interfacing it with any processor,
> > > > phy_reset, ready_reset, park, display_select
> > >
> > > Hmm, so what is dpp2600? It's mentioned here for the first time, the
> > > documentation doesn't mention it.
> >
> > Patch 0 does mention about dpp2600. It means DLP Pico Processor
> > and I have also provided link to wiki page which talks more about
> dpp2600.
> > Earlier version of patch also had functions dpp2600_configure_flash.
> 
> Remember that patch 0 is just an intro for the patch set. It's not
> included in the kernel tree. So the basic rule is that patch 0 should
> not contain any important info that is not available from the patches
> itself.
> 
> Where does the DPP2600 name come from? The documentation doesn't mention
> it, it's only mentioned in the wiki page written by you. I didn't find
> anything with a quick googling either.

You can check it here:
https://focus.ti.com/myti/docs/extranet.tsp?sectionId=403
You might have to create an extranet account to access it.

> 
> > > If it means the picodlp, just use the same name all the time. If not,
> > > you could tell what it is first.
> >
> > DPP means DLP pico processor.
> 
> So DPP is the processor part inside the projector? I'm not sure if that
> knowledge is relevant here. From the kernel driver's point of view
> there's just the projector, where the GPIOs, i2c and video lines go.
> Does it matter that there's a DPP2600 processor inside?

My bad it means DLP Pico Projector.
There is no processor.

> 
> Also, in the patch comment above you write "picodlp panel". The PicoDLP
> is not panel, so you shouldn't speak of picodlp panel.
> 
>  Tomi
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux