All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>
Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
	Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
	linux-media@vger.kernel.org, kyungmin.park@samsung.com,
	m.szyprowski@samsung.com, riverful.kim@samsung.com,
	sw0312.kim@samsung.com, devicetree-discuss@lists.ozlabs.org,
	linux-samsung-soc@vger.kernel.org, b.zolnierkie@samsung.com
Subject: Re: [RFC/PATCH 02/13] media: s5p-csis: Add device tree support
Date: Tue, 31 Jul 2012 23:37:20 +0200	[thread overview]
Message-ID: <1964196.hivxX7VrnL@avalon> (raw)
In-Reply-To: <5017D1CB.80709@samsung.com>

Hi Sylwester,

On Tuesday 31 July 2012 14:38:35 Sylwester Nawrocki wrote:
> On 07/31/2012 01:05 PM, Laurent Pinchart wrote:
> >>>>> What about CSI receivers that can reroute the lanes internally ? We
> >>>>> would need to specify lane indices for each lane then, maybe with
> >>>>> something like
> >>>>> 
> >>>>> clock-lane =<0>;
> >>>>> data-lanes =<2 3 1>;
> >>>> 
> >>>> Sounds good to me. And the clock-lane could be made optional, as not
> >>>> all devices would need it.
> >>>> 
> >>>> However, as far as I can see, there is currently no generic API for
> >>>> handling this kind of data structure. E.g. number of cells for the
> >>>> "interrupts" property is specified with an additional
> >>>> "#interrupt-cells" property.
> >>>> 
> >>>> It would have been much easier to handle something like:
> >>>> 
> >>>> data-lanes = <2>, <3>, <1>;
> >>>> 
> >>>> i.e. an array of the lane indexes.
> >>> 
> >>> I'm fine with that.
> >> 
> >> ...on a second thought: shouldn't this be handled by pinctrl? Or is it
> >> some CSI-2 module internal lane switching, not the global SoC pin
> >> function configuration?
> > 
> > On the hardware I came across, it's handled by the CSI2 receiver, not the
> > SoC pinmux feature.
> 
> Same here. Is there any hardware known to mux those MIPI-CSI
> D-PHY high speed differential signals with general purpose IO pins ?
> 
> Or are there mostly dedicated pins used ?

The OMAP3 multiplexes the CSI pins with other functions.

> However, if there are cases the lane configuration is done solely at CSI2
> receiver level, there seems little point in involving the pinctrl API.

The OMAP3 handles lane routing in the CSI2 receiver.

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2012-07-31 21:37 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-25 19:47 [RFC/PATCH 0/13] Add device tree support for s5p-fimc SoC camera host interface driver Sylwester Nawrocki
2012-05-25 19:52 ` [RFC/PATCH 01/13] ARM: Samsung: Extend MIPI PHY callback with an index argument Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 02/13] media: s5p-csis: Add device tree support Sylwester Nawrocki
2012-07-16  8:55     ` Guennadi Liakhovetski
2012-07-17 18:16       ` Sylwester Nawrocki
2012-07-26 14:38         ` Laurent Pinchart
2012-07-26 19:51           ` Sylwester Nawrocki
2012-07-26 22:35             ` Laurent Pinchart
2012-07-31 10:58               ` Guennadi Liakhovetski
     [not found]                 ` <Pine.LNX.4.64.1207311257020.27888-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
2012-07-31 11:05                   ` Laurent Pinchart
2012-07-31 11:05                     ` Laurent Pinchart
2012-07-31 12:38                     ` Sylwester Nawrocki
2012-07-31 21:37                       ` Laurent Pinchart [this message]
2012-07-31  9:34           ` Guennadi Liakhovetski
2012-05-25 19:52   ` [RFC/PATCH 03/13] ARM: Samsung: Remove unused fields from FIMC and CSIS platform data Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 04/13] devicetree: Add common video devices bindings documentation Sylwester Nawrocki
2012-07-16  9:09     ` Guennadi Liakhovetski
2012-07-18 16:58       ` Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 05/13] media: s5p-fimc: Add device tree support for FIMC devices Sylwester Nawrocki
2012-07-16  9:13     ` Guennadi Liakhovetski
2012-07-17 20:15       ` Sylwester Nawrocki
2012-07-18  8:17         ` Guennadi Liakhovetski
2012-07-18 19:53           ` Sylwester Nawrocki
2012-07-26 14:54             ` Laurent Pinchart
2012-07-30 21:35               ` Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 06/13] media: s5p-fimc: Add device tree support for FIMC-LITE Sylwester Nawrocki
2012-07-16  9:15     ` Guennadi Liakhovetski
     [not found]       ` <Pine.LNX.4.64.1207161114130.12302-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
2012-07-17 18:55         ` Sylwester Nawrocki
2012-07-17 18:55           ` Sylwester Nawrocki
2012-07-18  7:57           ` Guennadi Liakhovetski
2012-07-18 17:46             ` Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 07/13] media: s5p-fimc: Enable device tree based media device instantiation Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 08/13] ARM: dts: Add FIMC and MIPI-CSIS devices to Exynos4210 DT source Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 09/13] media: s5k6aa: Add support for device tree based instantiation Sylwester Nawrocki
2012-07-16  9:42     ` Guennadi Liakhovetski
2012-07-18  9:18       ` Sylwester Nawrocki
2012-07-26 15:09         ` Laurent Pinchart
2012-07-31  9:56           ` Guennadi Liakhovetski
2012-07-31 10:57             ` Laurent Pinchart
2012-07-31 11:14               ` Guennadi Liakhovetski
2012-07-31 11:22                 ` Laurent Pinchart
2012-07-31 11:29                   ` Guennadi Liakhovetski
2012-07-31 11:48                     ` Laurent Pinchart
2012-07-31 12:26                       ` Guennadi Liakhovetski
2012-07-31 12:46                         ` Sylwester Nawrocki
2012-07-31 12:59                           ` Guennadi Liakhovetski
2012-07-31 13:28                             ` Sylwester Nawrocki
2012-07-31 21:46                               ` Laurent Pinchart
2012-07-26 15:21     ` Laurent Pinchart
2012-07-26 20:39       ` Sylwester Nawrocki
2012-07-26 22:50         ` Laurent Pinchart
2012-08-19 10:02           ` Sakari Ailus
2012-05-25 19:52   ` [RFC/PATCH 10/13] ARM: dts: Add camera devices to exynos4210-nuri.dts Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 11/13] media: s5p-fimc: Keep local copy of sensors platform data Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 12/13] media: s5p-fimc: Add device tree based sensors registration Sylwester Nawrocki
2012-07-16  9:51     ` Guennadi Liakhovetski
2012-07-18 17:28       ` Sylwester Nawrocki
2012-05-25 19:52   ` [RFC/PATCH 13/13] media: s5p-fimc: Add parallel video port pin configuration Sylwester Nawrocki
2012-05-25 19:52   ` [PATCH 14/14] s5p-fimc: Add FIMC and MIPI-CSIS devices to CAM power domain Sylwester Nawrocki
2012-07-26 14:42   ` [RFC/PATCH 01/13] ARM: Samsung: Extend MIPI PHY callback with an index argument Laurent Pinchart
2012-07-26 20:15     ` Sylwester Nawrocki

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=1964196.hivxX7VrnL@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=g.liakhovetski@gmx.de \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=riverful.kim@samsung.com \
    --cc=s.nawrocki@samsung.com \
    --cc=sw0312.kim@samsung.com \
    --cc=sylvester.nawrocki@gmail.com \
    /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.