Re: [PATCH] devicetree: Add video bus switch

2017-12-22 Thread Pavel Machek
Hi! > > > I don't really object using g_endpoint_config() as a temporary solution; > > > I'd like to have Laurent's opinion on that though. Another option is to > > > wait, but we've already waited a looong time (as in total). > > > > Laurent, do you have some input here? We have simple "2 camera

Re: [PATCH] devicetree: Add video bus switch

2017-12-21 Thread Ivaylo Dimitrov
Hi, On 20.12.2017 19:54, Laurent Pinchart wrote: Hi Pavel, On Saturday, 4 February 2017 23:56:10 EET Pavel Machek wrote: Hi! +Required properties +=== + +compatible : must contain "video-bus-switch" How generic is this? Should we have e.g. nokia,video-bus-switch? And if

Re: [PATCH] devicetree: Add video bus switch

2017-12-21 Thread Sakari Ailus
On Wed, Dec 20, 2017 at 07:54:12PM +0200, Laurent Pinchart wrote: > Hi Pavel, > > On Saturday, 4 February 2017 23:56:10 EET Pavel Machek wrote: > > Hi! > > > > +Required properties > > +=== > > + > > +compatible: must contain "video-bus-switch" > > >>> > >

Re: [PATCH] devicetree: Add video bus switch

2017-12-20 Thread Laurent Pinchart
Hi Pavel, On Saturday, 4 February 2017 23:56:10 EET Pavel Machek wrote: > Hi! > > +Required properties > +=== > + > +compatible : must contain "video-bus-switch" > >>> > >>> How generic is this? Should we have e.g. nokia,video-bus-switch? And > >>> if so,

Re: [PATCH] devicetree: Add video bus switch

2017-03-02 Thread Pavel Machek
On Mon 2017-02-13 12:20:35, Sakari Ailus wrote: > Hi Pavel, > > On Mon, Feb 13, 2017 at 10:54:20AM +0100, Pavel Machek wrote: > > Hi! > > > > > > Take a look at the wikipedia. If you do "one at a time" at 100Hz, you > > > > can claim it is time-domain multiplex. But we are plain switching the > >

Re: [PATCH] devicetree: Add video bus switch

2017-02-13 Thread Sakari Ailus
Hi Pavel, On Mon, Feb 13, 2017 at 10:54:20AM +0100, Pavel Machek wrote: > Hi! > > > > Take a look at the wikipedia. If you do "one at a time" at 100Hz, you > > > can claim it is time-domain multiplex. But we are plain switching the > > > cameras. It takes second (or so) to setup the pipeline. > >

Re: [PATCH] devicetree: Add video bus switch

2017-02-13 Thread Pavel Machek
Hi! > > Take a look at the wikipedia. If you do "one at a time" at 100Hz, you > > can claim it is time-domain multiplex. But we are plain switching the > > cameras. It takes second (or so) to setup the pipeline. > > > > This is not multiplex. > > The functionality is still the same, isn't it? Do

Re: [PATCH] devicetree: Add video bus switch

2017-02-10 Thread Sakari Ailus
Hi Pavel, On Fri, Feb 10, 2017 at 08:54:35PM +0100, Pavel Machek wrote: > > > >>> > diff --git > > >>> > a/Documentation/devicetree/bindings/media/video-bus-switch.txt > > >>> > b/Documentation/devicetree/bindings/media/video-bus-switch.txt > > >>> > new file mode 100644 > > >>> > index 000

Re: [PATCH] devicetree: Add video bus switch

2017-02-10 Thread Pavel Machek
On Thu 2017-02-09 16:58:29, Rob Herring wrote: > On Wed, Feb 8, 2017 at 4:34 PM, Pavel Machek wrote: > >> > + > >> > +This is a binding for a gpio controlled switch for camera interfaces. > >> > Such a > >> > +device is used on some embedded devices to connect two cameras to the > >> > same > >>

Re: [PATCH] devicetree: Add video bus switch

2017-02-10 Thread Pavel Machek
> >>> > diff --git > >>> > a/Documentation/devicetree/bindings/media/video-bus-switch.txt > >>> > b/Documentation/devicetree/bindings/media/video-bus-switch.txt > >>> > new file mode 100644 > >>> > index 000..1b9f8e0 > >>> > --- /dev/null > >>> > +++ b/Documentation/devicetree/bindings/media

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Wed, Feb 8, 2017 at 4:34 PM, Pavel Machek wrote: >> > + >> > +This is a binding for a gpio controlled switch for camera interfaces. >> > Such a >> > +device is used on some embedded devices to connect two cameras to the same >> > +interface of a image signal processor. >> > + >> > +Required pr

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Wed, Feb 8, 2017 at 4:30 PM, Pavel Machek wrote: > On Wed 2017-02-08 15:36:09, Rob Herring wrote: >> On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: >> > >> > N900 contains front and back camera, with a switch between the >> > two. This adds support for the switch component, and i

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Thu, Feb 9, 2017 at 5:02 PM, Rob Herring wrote: > On Wed, Feb 8, 2017 at 4:30 PM, Pavel Machek wrote: >> On Wed 2017-02-08 15:36:09, Rob Herring wrote: >>> On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: >>> > >>> > N900 contains front and back camera, with a switch between the >

Re: [PATCH] devicetree: Add video bus switch

2017-02-08 Thread Pavel Machek
> > + > > +This is a binding for a gpio controlled switch for camera interfaces. Such > > a > > +device is used on some embedded devices to connect two cameras to the same > > +interface of a image signal processor. > > + > > +Required properties > > +=== > > + > > +compatible : mu

Re: [PATCH] devicetree: Add video bus switch

2017-02-08 Thread Pavel Machek
On Wed 2017-02-08 15:36:09, Rob Herring wrote: > On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: > > > > N900 contains front and back camera, with a switch between the > > two. This adds support for the switch component, and it is now > > possible to select between front and back cam

Re: [PATCH] devicetree: Add video bus switch

2017-02-08 Thread Rob Herring
On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: > > N900 contains front and back camera, with a switch between the > two. This adds support for the switch component, and it is now > possible to select between front and back cameras during runtime. > > This adds documentation for the

Re: [PATCH] devicetree: Add video bus switch

2017-02-05 Thread Sebastian Reichel
Hi, On Sun, Feb 05, 2017 at 10:12:20PM +0100, Pavel Machek wrote: > > > 9) Highly reconfigurable hardware - Julien Beraud > > > > > > - 44 sub-devices connected with an interconnect. > > > - As long as formats match, any sub-device could be connected to any > > > - other sub-device through a link

Re: [PATCH] devicetree: Add video bus switch

2017-02-05 Thread Pavel Machek
Hi! > > 9) Highly reconfigurable hardware - Julien Beraud > > > > - 44 sub-devices connected with an interconnect. > > - As long as formats match, any sub-device could be connected to any > > - other sub-device through a link. > > - The result is 44 * 44 links at worst. > > - A switch sub-device

Re: [PATCH] devicetree: Add video bus switch

2017-02-04 Thread Sakari Ailus
Hi Pavel, On Sat, Feb 04, 2017 at 10:56:10PM +0100, Pavel Machek wrote: > Hi! > > > > > > +Required properties > > > > > +=== > > > > > + > > > > > +compatible : must contain "video-bus-switch" > > > > > > > > How generic is this? Should we have e.g. nokia,video-bus-switch? And

Re: [PATCH] devicetree: Add video bus switch

2017-02-04 Thread Pavel Machek
Hi! > > > > +Required properties > > > > +=== > > > > + > > > > +compatible : must contain "video-bus-switch" > > > > > > How generic is this? Should we have e.g. nokia,video-bus-switch? And if > > > so, > > > change the file name accordingly. > > > > Generic for "single GPI

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Sebastian Reichel
Hi, On Fri, Feb 03, 2017 at 10:07:28PM +0100, Pavel Machek wrote: > On Fri 2017-02-03 14:32:19, Pali Rohár wrote: > > On Friday 03 February 2017 13:35:08 Pavel Machek wrote: > > > N900 contains front and back camera, with a switch between the > > > two. This adds support for the switch component,

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Sakari Ailus
Hi Pavel, On Fri, Feb 03, 2017 at 10:06:10PM +0100, Pavel Machek wrote: > Hi! > > > My apologies for the delays in reviewing. Feel free to ping me in the future > > if this happens. :-) > > No problem :-). If you could review the C-code, too... that would be > nice. It should be in your inbox so

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Pavel Machek
On Fri 2017-02-03 14:32:19, Pali Rohár wrote: > On Friday 03 February 2017 13:35:08 Pavel Machek wrote: > > N900 contains front and back camera, with a switch between the > > two. This adds support for the switch component, and it is now > > possible to select between front and back cameras during

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Pavel Machek
Hi! > My apologies for the delays in reviewing. Feel free to ping me in the future > if this happens. :-) No problem :-). If you could review the C-code, too... that would be nice. It should be in your inbox somewhere (and I attached it below, without the dts part). > > +Required properties > >

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Pali Rohár
On Friday 03 February 2017 13:35:08 Pavel Machek wrote: > N900 contains front and back camera, with a switch between the > two. This adds support for the switch component, and it is now > possible to select between front and back cameras during runtime. IIRC for controlling cameras on N900 there a

Re: [PATCH] devicetree: Add video bus switch

2017-02-03 Thread Sakari Ailus
Hi Pavel, My apologies for the delays in reviewing. Feel free to ping me in the future if this happens. :-) On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: > > N900 contains front and back camera, with a switch between the > two. This adds support for the switch component, and it i

[PATCH] devicetree: Add video bus switch

2017-02-03 Thread Pavel Machek
N900 contains front and back camera, with a switch between the two. This adds support for the switch component, and it is now possible to select between front and back cameras during runtime. This adds documentation for the devicetree binding. Signed-off-by: Sebastian Reichel Signed-off-by: Iva