Re: dvbv5-scan needs which channel file?

2014-12-30 Thread David Liontooth


Ah, thank you Olli -- much appreciated!

If dvbv5-scan expects the initial scan files in the new DVBV5 format, 
does that mean that these still somewhat mysterious initial scan files 
have to be supplied, as in the link to the dtv-scan-tables? How are 
these initial scan files themselves generated?


Surely there must be thousands of different dvb signal locations -- is 
linux-tv going to try to maintain these thousands of scan tables for 
download? What do users do when their particular location is not 
represented in the dtv-scan-tables.git?


Finally, I'm using gnutv to record television; I imagine it still only 
accepts the old format? What's the new alternative?


Cheers,
David

On 12/29/14, 11:55 PM, Olli Salonen wrote:

Hello David,

Coincidentally I was just yesterday working with dvbv5-scan and the
initial scan files. dvbv5-scan expects the initial scan files in the
new DVBV5 format. w_scan is not producing results in this format.

The scan tables at
http://git.linuxtv.org/cgit.cgi/dtv-scan-tables.git/ are in the new
format. Some of them are a bit outdated though (send in a patch if you
can update it for your area).

The v4l-utils package also includes tools to convert between the old
and the new format.

Cheers,
-olli


On 29 December 2014 at 22:09, David Liontooth lionte...@cogweb.net wrote:

Greetings --

How do you actually use dvbv5-scan? It seems to require some kind of input
file but there is no man page and the --help screen doesn't say anything
about it.

Could we document this? I tried

$ dvbv5-scan
Usage: dvbv5-scan [OPTION...] initial file
scan DVB services using the channel file

What is the channel file? Maybe the channels.conf file? (I created mine
using w_scan -ft -A3 -X -cUS -o7 -a /dev/dvb/adapter0/)

$ dvbv5-scan /etc/channels.conf
ERROR key/value without a channel group while parsing line 1 of
/etc/channels.conf

So it knows what it wants -- but what is it? Or is this a matter of dvb
versions, and my /etc/channels.conf is in the older format?

Very mysterious.

Cheers,
David
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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


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


Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-30 Thread Guennadi Liakhovetski
Hi Laurent,

First of all, sorry, I am currently on a holiday, so, replies are delayed, 
real work (reviewing or anything else) is impossible.

On Tue, 30 Dec 2014, Laurent Pinchart wrote:

 Hi Guennadi,
 
 On Friday 26 December 2014 11:38:11 Guennadi Liakhovetski wrote:
  On Fri, 26 Dec 2014, Laurent Pinchart wrote:
   On Friday 26 December 2014 10:14:26 Guennadi Liakhovetski wrote:
   On Fri, 26 Dec 2014, Laurent Pinchart wrote:
   On Friday 26 December 2014 14:37:14 Josh Wu wrote:
 
 [snip]
 
   Talking about mclk and xvclk is quite confusing. There's no mclk from
   an ov2640 point of view. The ov2640 driver should call
   v4l2_clk_get(xvclk).
   
   Yes, I also was thinking about this, and yes, requesting a xvclk clock
   would be more logical. But then, as you write below, if we let the
   v4l2_clk wrapper first check for a CCF xvclk clock, say, none is
   found. How do we then find the exported mclk V4L2 clock? Maybe
   v4l2_clk_get() should use two names?..
   
   Given that v4l2_clk_get() is only used by soc-camera drivers and that they
   all call it with the clock name set to mclk, I wonder whether we
   couldn't just get rid of struct v4l2_clk.id and ignore the id argument to
   v4l2_clk_get() when CCF isn't available. Maybe we've overdesigned
   v4l2_clk :-)
  
  Sure, that'd be fine with me, if everyone else agrees.
 
 Can you submit a patch ? That's the best way to find out if anyone objects.

Can do, sure, once I am back home and find time for this.

 [snip]
 
   v4l2_clk_get() should try to get the clock from CCF with a call to
   clk_get() first, and then look at the list of v4l2-specific clocks.
   
   Yes, how will it find the mclk when xvclk (or any other name) is
   requested? We did discuss this in the beginning and agreed to use a
   fixed clock name for the time being...
   
   Please see above.
   
   That's at least how I had envisioned it when v4l2_clk_get() was
   introduced. Let's remember that v4l2_clk was designed as a temporary
   workaround for platforms not implementing CCF yet. Is that still
   needed,
   or could be instead just get rid of it now ?
   
   I didn't check, but I don't think all platforms, handled by soc-camera,
   support CCF yet.
   
   After a quick check it looks like only OMAP1 and SH Mobile are missing.
   Atmel, MX2, MX3 and R-Car all support CCF. PXA27x has CCF support but
   doesn't enable it yet for an unknown (to me) reason.
   
   The CEU driver is used on both arch/sh and arch/arm/mach-shmobile. The
   former will most likely never receive CCF support, and the latter is
   getting fixed. As arch/sh isn't maintained anymore I would be fine with
   dropping CEU support for it.
   
   OMAP1 is thus the only long-term show-stopper. What should we do with it ?
  
  Indeed, what should we? :)
 
 You're listed as the soc-camera maintainer, so you should provide an answer 
 to 
 that question :-)

Thanks for ar reminder;)

 I'll propose one, let's drop the omap1-camera driver (I've 
 CC'ed the original author of the driver to this e-mail).

Let's see what they reply, but I don't tgink Josh will want to wait that 
long. And until omap1 is in the mainline we cannot drop v4l2_clk.

Thanks
Guennadi

 
 -- 
 Regards,
 
 Laurent Pinchart
 
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-30 Thread Laurent Pinchart
Hi Guennadi,

On Tuesday 30 December 2014 09:36:31 Guennadi Liakhovetski wrote:
 Hi Laurent,
 
 First of all, sorry, I am currently on a holiday, so, replies are delayed,
 real work (reviewing or anything else) is impossible.

Sure, no worries. Enjoy your holidays without thinking too much about soc-
camera :-)

 On Tue, 30 Dec 2014, Laurent Pinchart wrote:
  On Friday 26 December 2014 11:38:11 Guennadi Liakhovetski wrote:
  On Fri, 26 Dec 2014, Laurent Pinchart wrote:
  On Friday 26 December 2014 10:14:26 Guennadi Liakhovetski wrote:
  On Fri, 26 Dec 2014, Laurent Pinchart wrote:
  On Friday 26 December 2014 14:37:14 Josh Wu wrote:
 
  [snip]
  
  Talking about mclk and xvclk is quite confusing. There's no mclk
  from an ov2640 point of view. The ov2640 driver should call
  v4l2_clk_get(xvclk).
  
  Yes, I also was thinking about this, and yes, requesting a xvclk
  clock would be more logical. But then, as you write below, if we let
  the v4l2_clk wrapper first check for a CCF xvclk clock, say, none is
  found. How do we then find the exported mclk V4L2 clock? Maybe
  v4l2_clk_get() should use two names?..
  
  Given that v4l2_clk_get() is only used by soc-camera drivers and that
  they all call it with the clock name set to mclk, I wonder whether we
  couldn't just get rid of struct v4l2_clk.id and ignore the id argument
  to v4l2_clk_get() when CCF isn't available. Maybe we've overdesigned
  v4l2_clk :-)
  
  Sure, that'd be fine with me, if everyone else agrees.
  
  Can you submit a patch ? That's the best way to find out if anyone
  objects.
 
 Can do, sure, once I am back home and find time for this.
 
  [snip]
  
  v4l2_clk_get() should try to get the clock from CCF with a call to
  clk_get() first, and then look at the list of v4l2-specific clocks.
  
  Yes, how will it find the mclk when xvclk (or any other name) is
  requested? We did discuss this in the beginning and agreed to use a
  fixed clock name for the time being...
  
  Please see above.
  
  That's at least how I had envisioned it when v4l2_clk_get() was
  introduced. Let's remember that v4l2_clk was designed as a temporary
  workaround for platforms not implementing CCF yet. Is that still
  needed, or could be instead just get rid of it now ?
  
  I didn't check, but I don't think all platforms, handled by
  soc-camera, support CCF yet.
  
  After a quick check it looks like only OMAP1 and SH Mobile are
  missing. Atmel, MX2, MX3 and R-Car all support CCF. PXA27x has CCF
  support but doesn't enable it yet for an unknown (to me) reason.

On a side note, patches to enable CCF for PXA27x have been posted, they should 
be merged in v3.20.

  The CEU driver is used on both arch/sh and arch/arm/mach-shmobile. The
  former will most likely never receive CCF support, and the latter is
  getting fixed. As arch/sh isn't maintained anymore I would be fine
  with dropping CEU support for it.
  
  OMAP1 is thus the only long-term show-stopper. What should we do with
  it ?
  
  Indeed, what should we? :)
  
  You're listed as the soc-camera maintainer, so you should provide an
  answer to that question :-)
 
 Thanks for ar reminder;)
 
  I'll propose one, let's drop the omap1-camera driver (I've
  CC'ed the original author of the driver to this e-mail).
 
 Let's see what they reply, but I don't tgink Josh will want to wait that
 long. And until omap1 is in the mainline we cannot drop v4l2_clk.

Agreed, this was more of a question for the next step.

-- 
Regards,

Laurent Pinchart

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


kernel panic in dvb_demux.c:dvb_dmx_crc32

2014-12-30 Thread JPT
Hi,

I'm having kernel panics always at this exact location.

how can I find the cause?
how can I debug?
I already enabled DEBUG in kernel config, but haven't tried it yet.

this was with self built kernel 3.17.2, find the .config attached.
(jpt9 config was same except CONFIG_USB_*HCI_HCD=y)

Hardware: Technisat SkyStar USB HD (DVB-S/S2), Netgear ReadyNAS RN104,
Marvell Armada 370/XP (ARMv7)


Please CC, I'm not subscribed to the list.

thanks,

Jan

[37990.828635] Unable to handle kernel paging request at virtual address 13e09548
[37990.835879] pgd = c0004000
[37990.838590] [13e09548] *pgd=
[37990.842185] Internal error: Oops: 815 [#1] ARM
[37990.846635] Modules linked in: ir_lirc_codec ir_xmp_decoder lirc_dev ir_mce_kbd_decoder ir_sharp_decoder ir_sanyo_decoder ir_sony_decoder ir_jvc_decoder ir_rc6_decoder ir_rc5_decoder ir_nec_decoder rc_technisat_usb2 stv6110x dvb_usb_technisat_usb2 dvb_usb dvb_core stv090x rc_core ehci_orion xhci_hcd ehci_hcd
[37990.874197] CPU: 0 PID: 0 Comm: swapper Not tainted 3.17.2-rn104-jpt10 #7
[37990.880996] task: c081f8b0 ti: c0812000 task.ti: c0812000
[37990.886414] PC is at crc32_be+0x40/0x168
[37990.890345] LR is at 0xc0817ff0
[37990.893492] pc : [c02fc948]lr : [c0817ff0]psr: 2193
[37990.893492] sp : c0813d74  ip : 4544384d  fp : 00ac
[37990.904988] r10: e23a4770  r9 : 0086  r8 : a93252d6
[37990.910221] r7 : 139b5110  r6 : 6c7ae81a  r5 : e9279e7d  r4 : c0819ab0
[37990.916758] r3 : 31042148  r2 : 0001  r1 : e23a4640  r0 : 4faf1010
[37990.923296] Flags: nzCv  IRQs off  FIQs on  Mode SVC_32  ISA ARM  Segment kernel
[37990.930703] Control: 10c5387d  Table: 03330019  DAC: 0015
[37990.936457] Process swapper (pid: 0, stack limit = 0xc0812240)
[37990.942298] Stack: (0xc0813d74 to 0xc0814000)
[37990.946663] 3d60:  e23a462c e239430c 0561
[37990.954857] 3d80: e23a462c e23a562c   df5aa000 bf05dbc8 05bf bf05c7ec
[37990.963050] 3da0: 0020 c081a04c df5aa000 05bf 0001 e23a462c e23a562c 0012
[37990.971243] 3dc0: df414b78 e0830a34   df5aa000 bf05cef4 6c4a7895 228d
[37990.979437] 3de0: c08ab040 e0828981   0003 c005541c c08ab170 df414a78
[37990.987631] 3e00: 2193 0400 e0830800 02f0 01cc c081a04c df5aa000 bf05d14c
[37990.995824] 3e20: dd9d1a50 0006  df414ce0 dd9d1a00 e082e000 c081a04c bf07bebc
[37991.004018] 3e40: dd9d1a00  6193 dd9d1a00 df4af000 e0a4e620 c081a04c c0401e44
[37991.012212] 3e60: e0a4e690  0001 bf02189c c0813ebc  c08245a8 c003df38
[37991.020406] 3e80: def903c0 0040 0200 dda1d200 000d  0001 df4b59e0
[37991.028599] 3ea0: 020bf0b0 df4af1d4 0004 000d 0d000800 2291 75429efd 
[37991.036793] 3ec0: c0813ec0 de098e40 006b   006b defe1300 c085b13f
[37991.044987] 3ee0:  c0048f40 c0812000 c004e918 defe1300 006b  c0813f70
[37991.053180] 3f00:  dfffcc00 c08070d0 c0049030 006b c004b1d8 c0812028 006b
[37991.061373] 3f20: 006b c004886c c0837e6c c000f608 0011 0002 c08b5d40 c030bba0
[37991.069566] 3f40: c08b5d40 03ff c0813f70 c08b5d40 c085b13d c00084f0 c0040a54 6013
[37991.077760] 3f60:  c0813fa4 c085b13d c00126c0  c0824558  c001911c
[37991.085952] 3f80: c0812000 c081a0dc c0812038 c085b13d c085b13d dfffcc00 c08070d0 
[37991.094146] 3fa0: 0100 c0813fb8 c000f75c c0040a54 6013  c081a000 c07dec04
[37991.102339] 3fc0:   c07de610   c08070d0 c08660d4 c081a078
[37991.110532] 3fe0: c08070cc c0820920 4059 561f5811  8070  
[37991.118753] [c02fc948] (crc32_be) from [bf05dbc8] (dvb_dmx_crc32+0x10/0x18 [dvb_core])
[37991.127046] [bf05dbc8] (dvb_dmx_crc32 [dvb_core]) from [bf05c7ec] (dvb_dmx_swfilter_section_copy_dump+0x254/0x268 [dvb_core])
[37991.138728] [bf05c7ec] (dvb_dmx_swfilter_section_copy_dump [dvb_core]) from [bf05cef4] (dvb_dmx_swfilter_packet+0x45c/0x564 [dvb_core])
[37991.151280] [bf05cef4] (dvb_dmx_swfilter_packet [dvb_core]) from [bf05d14c] (dvb_dmx_swfilter+0xf4/0x164 [dvb_core])
[37991.162179] [bf05d14c] (dvb_dmx_swfilter [dvb_core]) from [bf07bebc] (usb_urb_complete+0xbc/0xe4 [dvb_usb])
[37991.172299] [bf07bebc] (usb_urb_complete [dvb_usb]) from [c0401e44] (__usb_hcd_giveback_urb+0x5c/0xe8)
[37991.181997] [c0401e44] (__usb_hcd_giveback_urb) from [bf02189c] (xhci_irq+0x8d8/0x1e08 [xhci_hcd])
[37991.191333] [bf02189c] (xhci_irq [xhci_hcd]) from [c0048f40] (handle_irq_event_percpu+0x78/0x140)
[37991.200570] [c0048f40] (handle_irq_event_percpu) from [c0049030] (handle_irq_event+0x28/0x38)
[37991.209461] [c0049030] (handle_irq_event) from [c004b1d8] (handle_simple_irq+0x64/0xa8)
[37991.217838] [c004b1d8] (handle_simple_irq) from [c004886c] (generic_handle_irq+0x2c/0x3c)
[37991.226385] [c004886c] (generic_handle_irq) from [c000f608] 

Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-30 Thread Josh Wu

Hi, Laurent

On 12/30/2014 8:15 AM, Laurent Pinchart wrote:

Hi Josh,

On Monday 29 December 2014 16:28:02 Josh Wu wrote:

On 12/26/2014 6:06 PM, Laurent Pinchart wrote:

On Friday 26 December 2014 10:14:26 Guennadi Liakhovetski wrote:

On Fri, 26 Dec 2014, Laurent Pinchart wrote:

On Friday 26 December 2014 14:37:14 Josh Wu wrote:

On 12/25/2014 6:39 AM, Guennadi Liakhovetski wrote:

On Mon, 22 Dec 2014, Josh Wu wrote:

On 12/20/2014 6:16 AM, Guennadi Liakhovetski wrote:

On Fri, 19 Dec 2014, Josh Wu wrote:

On 12/19/2014 5:59 AM, Guennadi Liakhovetski wrote:

On Thu, 18 Dec 2014, Josh Wu wrote:

To support async probe for ov2640, we need remove the code to get
'mclk' in ov2640_probe() function. oterwise, if soc_camera host
is not probed in the moment, then we will fail to get 'mclk' and
quit the ov2640_probe() function.

So in this patch, we move such 'mclk' getting code to
ov2640_s_power() function. That make ov2640 survive, as we can
pass a NULL (priv-clk) to soc_camera_set_power() function.

And if soc_camera host is probed, the when ov2640_s_power() is
called, then we can get the 'mclk' and that make us
enable/disable soc_camera host's clock as well.

Signed-off-by: Josh Wu josh...@atmel.com
---
v3 - v4:
v2 - v3:
v1 - v2:
   no changes.
   
drivers/media/i2c/soc_camera/ov2640.c | 31  ++---

1 file changed, 21 insertions(+), 10 deletions(-)

diff --git a/drivers/media/i2c/soc_camera/ov2640.c
b/drivers/media/i2c/soc_camera/ov2640.c
index 1fdce2f..9ee910d 100644
--- a/drivers/media/i2c/soc_camera/ov2640.c
+++ b/drivers/media/i2c/soc_camera/ov2640.c
@@ -739,6 +739,15 @@ static int ov2640_s_power(struct v4l2_subdev
*sd, int on)
struct i2c_client *client = v4l2_get_subdevdata(sd);
struct soc_camera_subdev_desc *ssdd =
soc_camera_i2c_to_desc(client);
struct ov2640_priv *priv = to_ov2640(client);
+   struct v4l2_clk *clk;
+
+   if (!priv-clk) {
+   clk = v4l2_clk_get(client-dev, mclk);
+   if (IS_ERR(clk))
+   dev_warn(client-dev, Cannot get the mclk.
maybe soc-camera host is not probed yet.\n);
+   else
+   priv-clk = clk;
+   }

return soc_camera_set_power(client-dev, ssdd, priv
-clk, on);
  }

Just let me explained a little more details at first:

As my understanding, current the priv-clk is a v4l2_clk: mclk, which
is a wrapper clock in soc_camera.c. it can make soc_camera to call
camera host's clock_start() clock_stop(). As in ov2640, the real mck
(pck1) is in ov2640 dt node (xvclk). So the camera host's
clock_start()/stop() only need to enable/disable his peripheral
clock.

I'm looking at the ov2640 datasheet. In the block diagram I only see
one input clock - the xvclk. Yes, it can be supplied by the camera
host controller, in which case it is natural for the camera host
driver to own and control it, or it can be a separate clock device -
either static or configurable. This is just a note to myself to
clarify, that it's one and the same clock pin we're talking about.

Now, from the hardware / DT PoV, I think, the DT should look like:

a) in the ov2640 I2C DT node we should have a clock consumer entry,
linking to a board-specific source.

That's what this patch series do right now.
In my patch 5/5 DT document said, ov2640 need a clock consumer which
refer to the xvclk input clock.
And it is a required property.


b) if the ov2640 clock is supplied by a camera host, its DT entry
should have a clock source subnode, to which ov2640 clock consumer
entry should link. The respective camera host driver should then parse
that clock subnode and register the respective clock with the V4L2
framework, by calling v4l2_clk_register().

Ok, So in this case, I need to wait for the mclk in probe of ov2640
driver. So that I can be compatible for the camera host which provide
the clock source.

Talking about mclk and xvclk is quite confusing. There's no mclk from an
ov2640 point of view. The ov2640 driver should call
v4l2_clk_get(xvclk).

Yes, I also was thinking about this, and yes, requesting a xvclk clock
would be more logical. But then, as you write below, if we let the
v4l2_clk wrapper first check for a CCF xvclk clock, say, none is found.
How do we then find the exported mclk V4L2 clock? Maybe v4l2_clk_get()
should use two names?..

Given that v4l2_clk_get() is only used by soc-camera drivers and that they
all call it with the clock name set to mclk, I wonder whether we
couldn't just get rid of struct v4l2_clk.id and ignore the id argument to
v4l2_clk_get() when CCF isn't available. Maybe we've overdesigned
v4l2_clk :-)

Sorry, I'm not clear about how to implement what you discussed here.

Do you mean, In the ov2640 driver:
1. need to remove the patch 4/5, add a master clock for sensor

No, the sensor has a clock input named xvclk, the ov2640 driver should thus
manage that clock. Patch 4/5 does the right thing.

However, I've just realized that it will cause 

Re: [PATCH 09/11 linux-next] [media] uvcvideo: remove unnecessary version.h inclusion

2014-12-30 Thread Fabian Frederick


 On 30 December 2014 at 00:42 Laurent Pinchart
 laurent.pinch...@ideasonboard.com wrote:


 Hi Fabian,

 Thank you for the patch.

 On Monday 29 December 2014 15:29:43 Fabian Frederick wrote:
  Based on versioncheck.
 
  Signed-off-by: Fabian Frederick f...@skynet.be

 Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

 Should I take the patch in my tree or do you plan to send a pull request for
 the whole series elsewhere ?

  ---
   drivers/media/usb/uvc/uvc_v4l2.c | 1 -
   1 file changed, 1 deletion(-)
 
  diff --git a/drivers/media/usb/uvc/uvc_v4l2.c
  b/drivers/media/usb/uvc/uvc_v4l2.c index 9c5cbcf..43e953f 100644
  --- a/drivers/media/usb/uvc/uvc_v4l2.c
  +++ b/drivers/media/usb/uvc/uvc_v4l2.c
  @@ -13,7 +13,6 @@
 
   #include linux/compat.h
   #include linux/kernel.h
  -#include linux/version.h
   #include linux/list.h
   #include linux/module.h
   #include linux/slab.h

 --
 Regards,

 Laurent Pinchart

Hi Laurent,

        Thanks for the ack, you can take the patch.
       
(Maybe Greg will try later to apply the whole patchset on linux-next.
It should not be a problem if some of them are already in.)

Regards,
Fabian
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-30 Thread Josh Wu

Hi, Guennadi

On 12/30/2014 4:36 PM, Guennadi Liakhovetski wrote:

Hi Laurent,

First of all, sorry, I am currently on a holiday, so, replies are delayed,
real work (reviewing or anything else) is impossible.


Thanks for your review in holiday. That's very helpful.


On Tue, 30 Dec 2014, Laurent Pinchart wrote:


Hi Guennadi,

On Friday 26 December 2014 11:38:11 Guennadi Liakhovetski wrote:

On Fri, 26 Dec 2014, Laurent Pinchart wrote:

On Friday 26 December 2014 10:14:26 Guennadi Liakhovetski wrote:

On Fri, 26 Dec 2014, Laurent Pinchart wrote:

On Friday 26 December 2014 14:37:14 Josh Wu wrote:

[snip]


Talking about mclk and xvclk is quite confusing. There's no mclk from
an ov2640 point of view. The ov2640 driver should call
v4l2_clk_get(xvclk).

Yes, I also was thinking about this, and yes, requesting a xvclk clock
would be more logical. But then, as you write below, if we let the
v4l2_clk wrapper first check for a CCF xvclk clock, say, none is
found. How do we then find the exported mclk V4L2 clock? Maybe
v4l2_clk_get() should use two names?..

Given that v4l2_clk_get() is only used by soc-camera drivers and that they
all call it with the clock name set to mclk, I wonder whether we
couldn't just get rid of struct v4l2_clk.id and ignore the id argument to
v4l2_clk_get() when CCF isn't available. Maybe we've overdesigned
v4l2_clk :-)

Sure, that'd be fine with me, if everyone else agrees.

Can you submit a patch ? That's the best way to find out if anyone objects.

Can do, sure, once I am back home and find time for this.


[snip]


v4l2_clk_get() should try to get the clock from CCF with a call to
clk_get() first, and then look at the list of v4l2-specific clocks.

Yes, how will it find the mclk when xvclk (or any other name) is
requested? We did discuss this in the beginning and agreed to use a
fixed clock name for the time being...

Please see above.


That's at least how I had envisioned it when v4l2_clk_get() was
introduced. Let's remember that v4l2_clk was designed as a temporary
workaround for platforms not implementing CCF yet. Is that still
needed,
or could be instead just get rid of it now ?

I didn't check, but I don't think all platforms, handled by soc-camera,
support CCF yet.

After a quick check it looks like only OMAP1 and SH Mobile are missing.
Atmel, MX2, MX3 and R-Car all support CCF. PXA27x has CCF support but
doesn't enable it yet for an unknown (to me) reason.

The CEU driver is used on both arch/sh and arch/arm/mach-shmobile. The
former will most likely never receive CCF support, and the latter is
getting fixed. As arch/sh isn't maintained anymore I would be fine with
dropping CEU support for it.

OMAP1 is thus the only long-term show-stopper. What should we do with it ?

Indeed, what should we? :)

You're listed as the soc-camera maintainer, so you should provide an answer to
that question :-)

Thanks for ar reminder;)


I'll propose one, let's drop the omap1-camera driver (I've
CC'ed the original author of the driver to this e-mail).

Let's see what they reply, but I don't tgink Josh will want to wait that
long.


Thanks. it's indeed true for me.


And until omap1 is in the mainline we cannot drop v4l2_clk.


So I think the better way right now for ov2640 driver is still request 
both the v4l2_clock: mclk, and the clock: xvclk in probe().
In that way,  we can take our time to introduce other patches to merged 
these two clocks.

Does it make sense?

Best Regards,
Josh Wu



Thanks
Guennadi


--
Regards,

Laurent Pinchart


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


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


Re: [PATCH v4 2/5] media: ov2640: add async probe function

2014-12-30 Thread Guennadi Liakhovetski
On Tue, 30 Dec 2014, Josh Wu wrote:

[snip]

  And until omap1 is in the mainline we cannot drop v4l2_clk.

s/until/as lonh as/

 So I think the better way right now for ov2640 driver is still request both
 the v4l2_clock: mclk, and the clock: xvclk in probe().
 In that way,  we can take our time to introduce other patches to merged these
 two clocks.
 Does it make sense?

How about this sequence, that we cat still try to get in on time for the 
next merge window:

1. stop using the clock name in v4l2_clk
2. add support for CCF to v4l2_clk, falling back to current behaviour if 
no clock is found
3. switch ov2640 to using xvclk

Otherwise I would propose to add asynchronous probing support to ov2640 
_without_ changing the clock name. Whether or not we changee clock's name 
isn't related directly to async support, since the driver already is using 
v4l2_clk with the standarf wrong name. So, I would consider these two 
changes separately - one is a new feature, another is a fix. The only 
question is in which order to apply them. In general fix-first is 
preferred, but since in this case the fix requires framework changes, we 
could go with feature-first. This also makes sense since features need to 
catch a merge window, whereas a fix can go in later too. So, if we don't 
manage the above 3-step plan, I would priposw the most primitive patch ro 
ov2640 just adding async support in line wuth other drivers and without 
changing the clock name at first.

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


Re: V4L2_CID_AUTO_FOCUS_START VS V4L2_CID_FOCUS_AUTO

2014-12-30 Thread Sakari Ailus
Hi Ben,

On Fri, Dec 19, 2014 at 11:48:58AM +0800, Bin Chen wrote:
 Hi,
 
 Can anyone explain what is the difference between setting control
 V4L2_CID_FOCUS_AUTO to 1 and and issuing V4L2_CID_AUTO_FOCUS_START?
 Confused...

V4L2_CID_AUTO_FOCUS_START starts a single-pass AF algorithm which ends after
reaching focus (or failing in trying that), whereas enabling the
V4L2_CID_FOCUS_AUTO control enables a contiguous AF algorithm.

-- 
Regards,

Sakari Ailus
e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


How to access DVB-onboard RC? (Technisat)

2014-12-30 Thread JPT
Hi,

I currently try to make my Technisat IR-RC work.
But nothing happens when I press a key.
What's wrong?

I checked the RC using a photo camera: at least it emits IR.


triggerhappy config:

DAEMON_OPTS=--triggers /etc/triggerhappy/triggers.d/ /dev/input/event0
/dev/input/event1

KEY_1   1   logger thd recieved KEY_1 from Technisat RC
KEY_2   1   logger thd recieved KEY_2 from Technisat RC
KEY_3   1   logger thd recieved KEY_3 from Technisat RC
...
(keys from rc-technisat-usb2.c see syslog below)

Triggerhappy works fine with events from event0
but not from event1

$ inputeventdaemon -l
/dev/input/event0:
  name : gpio-keys
  phys : gpio-keys/input0
  features : syn keys

/dev/input/event1:
  name : IR-receiver inside an USB DVB receiver
  phys : usb-:01:00.0-1/ir0
  features : syn keys reserved repeat

/dev/input/event2:
  name : MCE IR Keyboard/Mouse (technisat-usb2)
  phys : /input0
  features : syn keys relative reserved repeat


$ lsusb -t
...
/:  Bus 01.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 480M
|__ Port 1: Dev 3, If 0, Class=vend., Driver=dvb_usb_technisat_usb2,
480M


syslog:
dvb-usb: found a 'Technisat SkyStar USB HD (DVB-S/S2)' in warm state.
dvb-usb: will pass the complete MPEG2 transport stream to the software
demuxer.
DVB: registering new adapter (Technisat SkyStar USB HD (DVB-S/S2))
dvb-usb: MAC address: x
stv6110x_attach: Attaching STV6110x
technisat-usb2: i2c-error: 60 = 7
usb 1-1: DVB: registering adapter 0 frontend 0 (Technisat SkyStar USB HD
(DVB-S/S2))...
Registered IR keymap rc-technisat-usb2
input: IR-receiver inside an USB DVB receiver as
/devices/soc/soc:pcie-controller/pci:00/:00:01.0/:01:00.0/usb1/1-1/rc/rc0/input1
evbug: Connected device: input1 (IR-receiver inside an USB DVB receiver
at usb-:01:00.0-1/ir0)
rc0: IR-receiver inside an USB DVB receiver as
/devices/soc/soc:pcie-controller/pci:00/:00:01.0/:01:00.0/usb1/1-1/rc/rc0
IR NEC protocol handler initialized
IR RC5(x/sz) protocol handler initialized
IR RC6 protocol handler initialized
IR JVC protocol handler initialized
IR Sony protocol handler initialized
IR SANYO protocol handler initialized
IR Sharp protocol handler initialized
dvb-usb: schedule remote query interval to 100 msecs.

do I need those protocol handlers?

do I need lirc?


thanks,

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


Re: [RFC/PATCH] dvb-core: add template code for i2c binding model

2014-12-30 Thread Mauro Carvalho Chehab
Em Fri, 05 Dec 2014 19:49:33 +0900
tsk...@gmail.com escreveu:

 From: Akihiro Tsukada tsk...@gmail.com
 
 Define a standard interface for demod/tuner i2c driver modules.
 A module client calls dvb_i2c_attach_{fe,tuner}(),
 and a module driver defines struct dvb_i2c_module_param and
 calls DEFINE_DVB_I2C_MODULE() macro.
 
 This template provides implicit module requests and ref-counting,
 alloc/free's private data structures,
 fixes the usage of clientdata of i2c devices,
 and defines the platformdata structures for passing around
 device specific config/output parameters between drivers and clients.
 These kinds of code are common to (almost) all dvb i2c drivers/client,
 but they were scattered over adapter modules and demod/tuner modules.

The idea behind this patch is good. I didn't have time yet to test it on a
device that I have currently access.

I have a few comments below, mostly regards with naming convention.

By seeing the patches you converted a few drivers to use this, I'm a little
bit concern about the conversion. We need something that won't be hard to
convert to the new model without requiring to re-test everything.

Anyway, my plan is to take a deeper look on it next week and convert one
or two drivers to use the new I2C binding approach you're proposing.
 
 Signed-off-by: Akihiro Tsukada tsk...@gmail.com
 ---
  drivers/media/dvb-core/Makefile   |   4 +
  drivers/media/dvb-core/dvb_frontend.h |   1 +
  drivers/media/dvb-core/dvb_i2c.c  | 219 
 ++
  drivers/media/dvb-core/dvb_i2c.h  | 110 +
  4 files changed, 334 insertions(+)
  create mode 100644 drivers/media/dvb-core/dvb_i2c.c
  create mode 100644 drivers/media/dvb-core/dvb_i2c.h
 
 diff --git a/drivers/media/dvb-core/Makefile b/drivers/media/dvb-core/Makefile
 index 8f22bcd..271648d 100644
 --- a/drivers/media/dvb-core/Makefile
 +++ b/drivers/media/dvb-core/Makefile
 @@ -8,4 +8,8 @@ dvb-core-objs := dvbdev.o dmxdev.o dvb_demux.o dvb_filter.o   
 \
dvb_ca_en50221.o dvb_frontend.o\
$(dvb-net-y) dvb_ringbuffer.o dvb_math.o
  
 +ifneq ($(CONFIG_I2C)$(CONFIG_I2C_MODULE),)
 +dvb-core-objs += dvb_i2c.o
 +endif
 +
  obj-$(CONFIG_DVB_CORE) += dvb-core.o
 diff --git a/drivers/media/dvb-core/dvb_frontend.h 
 b/drivers/media/dvb-core/dvb_frontend.h
 index 816269e..41aae1b 100644
 --- a/drivers/media/dvb-core/dvb_frontend.h
 +++ b/drivers/media/dvb-core/dvb_frontend.h
 @@ -415,6 +415,7 @@ struct dtv_frontend_properties {
  struct dvb_frontend {
   struct dvb_frontend_ops ops;
   struct dvb_adapter *dvb;
 + struct i2c_client *fe_cl;

fe_cl doesn't seem to be a good name for something that should be
accessed by all drivers that use it. fe_i2c_client is likely a
better name for it.

   void *demodulator_priv;
   void *tuner_priv;
   void *frontend_priv;
 diff --git a/drivers/media/dvb-core/dvb_i2c.c 
 b/drivers/media/dvb-core/dvb_i2c.c
 new file mode 100644
 index 000..4ea4e5e
 --- /dev/null
 +++ b/drivers/media/dvb-core/dvb_i2c.c
 @@ -0,0 +1,219 @@
 +/*
 + * dvb_i2c.c
 + *
 + * Copyright 2014 Akihiro Tsukada tskd08 AT gmail DOT com
 + *
 + * This program is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License as published by
 + * the Free Software Foundation; either version 2 of the License, or
 + * (at your option) any later version.
 + *
 + * This program is distributed in the hope that it will be useful,
 + * but WITHOUT ANY WARRANTY; without even the implied warranty of
 + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + *
 + * You should have received a copy of the GNU General Public License
 + * along with this program; if not, see http://www.gnu.org/licenses/.
 + *
 + */
 +
 +#include dvb_i2c.h
 +
 +static struct i2c_client *
 +dvb_i2c_new_device(struct i2c_adapter *adap, struct i2c_board_info *info,
 +const unsigned short *probe_addrs)
 +{
 + struct i2c_client *cl;
 +
 + request_module(I2C_MODULE_PREFIX %s, info-type);
 + /* Create the i2c client */
 + if (info-addr == 0  probe_addrs)
 + cl = i2c_new_probed_device(adap, info, probe_addrs, NULL);
 + else
 + cl = i2c_new_device(adap, info);
 + if (!cl || !cl-dev.driver)
 + return NULL;
 + return cl;

The best would be to also register the device with the media controller,
if CONFIG_MEDIA_CONTROLLER is defined, just like v4l2_i2c_subdev_init()
does.

I would also try to use similar names for the function calls to the ones
that the v4l subsystem uses for subdevices.

 +}
 +
 +struct dvb_frontend *
 +dvb_i2c_attach_fe(struct i2c_adapter *adap, const struct i2c_board_info 
 *info,
 +   const void *cfg, void **out)
 +{
 + struct i2c_client *cl;
 + struct i2c_board_info bi;

Better to call this as tmp_info, as this seems to be just a temp var.

 

Re: [RFC PATCH 2/5] media: rcar_vin: Ensure all in-flight buffers are returned to error state before stopping.

2014-12-30 Thread Sakari Ailus
Hi Ben,

On Thu, Dec 18, 2014 at 02:49:33PM +, Ben Hutchings wrote:
 From: Ian Molton ian.mol...@codethink.co.uk
 
 Videobuf2 complains about buffers that are still marked ACTIVE (in use by the 
 driver) following a call to stop_streaming().
 
 This patch returns all active buffers to state ERROR prior to stopping.
 
 Note: this introduces a (non fatal) race condition as the stream is not 
 guaranteed to be stopped at this point.
 
 Signed-off-by: Ian Molton ian.mol...@codethink.co.uk
 Signed-off-by: William Towle william.to...@codethink.co.uk
 ---
  drivers/media/platform/soc_camera/rcar_vin.c |6 ++
  1 file changed, 6 insertions(+)
 
 diff --git a/drivers/media/platform/soc_camera/rcar_vin.c 
 b/drivers/media/platform/soc_camera/rcar_vin.c
 index 773de53..7069176 100644
 --- a/drivers/media/platform/soc_camera/rcar_vin.c
 +++ b/drivers/media/platform/soc_camera/rcar_vin.c
 @@ -516,8 +516,14 @@ static void rcar_vin_stop_streaming(struct vb2_queue *vq)
   struct soc_camera_host *ici = to_soc_camera_host(icd-parent);
   struct rcar_vin_priv *priv = ici-priv;
   struct list_head *buf_head, *tmp;
 + int i;
  
   spin_lock_irq(priv-lock);
 +
 + for (i = 0; i  vq-num_buffers; ++i)
 + if (vq-bufs[i]-state == VB2_BUF_STATE_ACTIVE)
 + vb2_buffer_done(vq-bufs[i], VB2_BUF_STATE_ERROR);
 +
   list_for_each_safe(buf_head, tmp, priv-capture)
   list_del_init(buf_head);
   spin_unlock_irq(priv-lock);

I'd use the driver's own queued buffer list to access the queued buffers,
you alread loop over that just below your own change.

-- 
Regards,

Sakari Ailus
e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: dvbv5-scan needs which channel file?

2014-12-30 Thread Olli Salonen
Hi David,

Well, the initial scan files need to be supplied for dvbv5-scan somehow.

The initial scan files that are maintained in the the git repo I
posted earlier are updated by users who notice differencies. Basically
I and some other users have created scripts that automatically
generate the files for my country, so it's rather easy. I don't know
how it works for other countries.

Anyway, if you prefer to generate the data yourself you can use w_scan
to generate it in DVBV3 format:
w_scan -ft -c FI -x  ~/initial_v3.conf

Then use the dvb-format-convert tool that comes in the v4l-utils package:
dvb-format-convert -I CHANNEL -O DVBV5 ~/initial_v3.conf ~/initial_data_v5.conf

Then you can run dvbv5-scan with this file:
dvbv5-scan ~/initial_data_v5.conf

Alternatively you can skip the whole conversion phase and run
dvbv5-scan with the DVBV3 initial tuning data:
dvbv5-scan -I CHANNEL ~/initial_v3.conf

Cheers,
-olli

On 30 December 2014 at 10:23, David Liontooth lionte...@cogweb.net wrote:

 Ah, thank you Olli -- much appreciated!

 If dvbv5-scan expects the initial scan files in the new DVBV5 format, does
 that mean that these still somewhat mysterious initial scan files have to
 be supplied, as in the link to the dtv-scan-tables? How are these initial
 scan files themselves generated?

 Surely there must be thousands of different dvb signal locations -- is
 linux-tv going to try to maintain these thousands of scan tables for
 download? What do users do when their particular location is not represented
 in the dtv-scan-tables.git?

 Finally, I'm using gnutv to record television; I imagine it still only
 accepts the old format? What's the new alternative?

 Cheers,
 David

 On 12/29/14, 11:55 PM, Olli Salonen wrote:

 Hello David,

 Coincidentally I was just yesterday working with dvbv5-scan and the
 initial scan files. dvbv5-scan expects the initial scan files in the
 new DVBV5 format. w_scan is not producing results in this format.

 The scan tables at
 http://git.linuxtv.org/cgit.cgi/dtv-scan-tables.git/ are in the new
 format. Some of them are a bit outdated though (send in a patch if you
 can update it for your area).

 The v4l-utils package also includes tools to convert between the old
 and the new format.

 Cheers,
 -olli


 On 29 December 2014 at 22:09, David Liontooth lionte...@cogweb.net
 wrote:

 Greetings --

 How do you actually use dvbv5-scan? It seems to require some kind of
 input
 file but there is no man page and the --help screen doesn't say anything
 about it.

 Could we document this? I tried

 $ dvbv5-scan
 Usage: dvbv5-scan [OPTION...] initial file
 scan DVB services using the channel file

 What is the channel file? Maybe the channels.conf file? (I created mine
 using w_scan -ft -A3 -X -cUS -o7 -a /dev/dvb/adapter0/)

 $ dvbv5-scan /etc/channels.conf
 ERROR key/value without a channel group while parsing line 1 of
 /etc/channels.conf

 So it knows what it wants -- but what is it? Or is this a matter of dvb
 versions, and my /etc/channels.conf is in the older format?

 Very mysterious.

 Cheers,
 David
 --
 To unsubscribe from this list: send the line unsubscribe linux-media in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html

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


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


Re: [RFC 1/6] cec: add new driver for cec support.

2014-12-30 Thread Sean Young
On Tue, Dec 23, 2014 at 03:32:17PM +0100, Kamil Debski wrote:
 +There are still a few todo's, the main one being the remote control support
 +feature of CEC. I need to research if that should be implemented via the
 +standard kernel remote control support.

I guess a new rc driver type RC_DRIVER_CEC should be introduced (existing
types are RC_DRIVER_IR_RAW and RC_DRIVER_SCANCODE). rc_register_device()
should not register the sysfs attributes specific for IR, but register
sysfs attributes for cec like a link to the device.

In addition there should be a new rc_type protocol RC_TYPE_CEC; now 
rc_keydown_notimeout() can be called for each key press.

I guess a new keymap should exist too.

HTH

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


Re: [PATCHv2] media: i2c/adp1653: devicetree support for adp1653

2014-12-30 Thread Sakari Ailus
Hi Pavel,

Thanks for the patch! A few comments below.

On Wed, Dec 24, 2014 at 11:34:34PM +0100, Pavel Machek wrote:
 
 We are moving to device tree support on OMAP3, but that currently
 breaks ADP1653 driver. This adds device tree support, plus required
 documentation.
 
 Signed-off-by: Pavel Machek pa...@ucw.cz
 
 ---
 
 Changed -microsec to -us, as requested by devicetree people.
 
 Fixed checkpatch issues.
 
 diff --git a/Documentation/devicetree/bindings/leds/common.txt 
 b/Documentation/devicetree/bindings/leds/common.txt
 index 2d88816..2c6c7c5 100644
 --- a/Documentation/devicetree/bindings/leds/common.txt
 +++ b/Documentation/devicetree/bindings/leds/common.txt
 @@ -14,6 +14,15 @@ Optional properties for child nodes:
   ide-disk - LED indicates disk activity
   timer - LED flashes at a fixed, configurable rate
  
 +- max-microamp : maximum intensity in microamperes of the LED
 +  (torch LED for flash devices)

s/torch LED/torch mode/

 +- flash-max-microamp : maximum intensity in microamperes of the
 +   flash LED; it is mandatory if the LED should
 +support the flash mode
 +- flash-timeout-microsec : timeout in microseconds after which the flash
 +   LED is turned off

These should go to a different patch.

 +
 +
  Examples:
  
  system-status {
 @@ -21,3 +30,10 @@ system-status {
   linux,default-trigger = heartbeat;
   ...
  };
 +
 +camera-flash {
 + label = Flash;
 + max-microamp = 5;
 + flash-max-microamp = 32;
 + flash-timeout-microsec = 50;
 +}
 diff --git a/Documentation/devicetree/bindings/media/i2c/adp1653.txt 
 b/Documentation/devicetree/bindings/media/i2c/adp1653.txt
 new file mode 100644
 index 000..3c7065f
 --- /dev/null
 +++ b/Documentation/devicetree/bindings/media/i2c/adp1653.txt
 @@ -0,0 +1,38 @@
 +* Analog Devices ADP1653 flash LED driver
 +
 +Required Properties:
 +
 +  - compatible: Must contain one of the following
 +- adi,adp1653

I doubt whether there are going to be more chips supported with the driver.
There hasn't been since the driver was written not I'm aware of one now.

 +  - reg: I2C slave address
 +
 +  - gpios: References to the GPIO that controls the power for the chip.
 +
 +There are two led outputs available - flash and indicator. One led is
 +represented by one child node, nodes need to be named flash and 
 indicator.

80 characters per line.

 +
 +Required properties of the LED child node:
 +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt
 +
 +Required properties of the flash LED child node:
 +
 +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt
 +- flash-timeout-us : see Documentation/devicetree/bindings/leds/common.txt
 +
 +Example:
 +
 +adp1653: led-controller@30 {
 +compatible = adi,adp1653;
 + reg = 0x30;
 +gpios = gpio3 24 GPIO_ACTIVE_HIGH; /* 88 */

Please use tabs for indentation above (and below).

 +
 + flash {
 +flash-timeout-us = 50;
 +flash-max-microamp = 32;
 +max-microamp = 5;
 + };
 +indicator {
 +max-microamp = 17500;
 + };
 +};
 diff --git a/arch/arm/boot/dts/omap3-n900.dts 
 b/arch/arm/boot/dts/omap3-n900.dts
 index bc82a12..d04e7cc 100644
 --- a/arch/arm/boot/dts/omap3-n900.dts
 +++ b/arch/arm/boot/dts/omap3-n900.dts
 @@ -553,6 +558,22 @@
  
   ti,usb-charger-detection = isp1704;
   };
 +
 + adp1653: led-controller@30 {
 + compatible = adi,adp1653;
 + reg = 0x30;
 + gpios = gpio3 24 GPIO_ACTIVE_HIGH; /* 88 */
 +
 + flash {
 + flash-timeout-us = 50;
 + flash-max-microamp = 32;
 + max-microamp = 5;
 + };
 +
 + indicator {
 + max-microamp = 17500;
 + };
 + };

This should go to a separate patch as well.

  };
  
  i2c3 {
 diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c
 index 873fe19..78341d0 100644
 --- a/drivers/media/i2c/adp1653.c
 +++ b/drivers/media/i2c/adp1653.c
 @@ -8,6 +8,7 @@
   * Contributors:
   *   Sakari Ailus sakari.ai...@iki.fi
   *   Tuukka Toivonen tuukka...@gmail.com
 + *  Pavel Machek pa...@ucw.cz
   *
   * This program is free software; you can redistribute it and/or
   * modify it under the terms of the GNU General Public License
 @@ -34,9 +35,12 @@
  #include linux/module.h
  #include linux/i2c.h
  #include linux/slab.h
 +#include linux/of_gpio.h
  #include media/adp1653.h
  #include media/v4l2-device.h
  
 +#include linux/gpio.h

Please arrange along with the rest of headers.

 +
  #define TIMEOUT_MAX  82
  #define TIMEOUT_STEP 54600
  #define TIMEOUT_MIN  (TIMEOUT_MAX - 

Re: dvbv5-scan needs which channel file?

2014-12-30 Thread David Liontooth


OK, perfect; thank you. This should be documented in dvbv5-scan. And we 
should have a man page for it.


Cheers,
David

On 12/30/14, 5:15 AM, Olli Salonen wrote:

Hi David,

Well, the initial scan files need to be supplied for dvbv5-scan somehow.

The initial scan files that are maintained in the the git repo I
posted earlier are updated by users who notice differencies. Basically
I and some other users have created scripts that automatically
generate the files for my country, so it's rather easy. I don't know
how it works for other countries.

Anyway, if you prefer to generate the data yourself you can use w_scan
to generate it in DVBV3 format:
w_scan -ft -c FI -x  ~/initial_v3.conf

Then use the dvb-format-convert tool that comes in the v4l-utils package:
dvb-format-convert -I CHANNEL -O DVBV5 ~/initial_v3.conf ~/initial_data_v5.conf

Then you can run dvbv5-scan with this file:
dvbv5-scan ~/initial_data_v5.conf

Alternatively you can skip the whole conversion phase and run
dvbv5-scan with the DVBV3 initial tuning data:
dvbv5-scan -I CHANNEL ~/initial_v3.conf

Cheers,
-olli

On 30 December 2014 at 10:23, David Liontooth lionte...@cogweb.net wrote:

Ah, thank you Olli -- much appreciated!

If dvbv5-scan expects the initial scan files in the new DVBV5 format, does
that mean that these still somewhat mysterious initial scan files have to
be supplied, as in the link to the dtv-scan-tables? How are these initial
scan files themselves generated?

Surely there must be thousands of different dvb signal locations -- is
linux-tv going to try to maintain these thousands of scan tables for
download? What do users do when their particular location is not represented
in the dtv-scan-tables.git?

Finally, I'm using gnutv to record television; I imagine it still only
accepts the old format? What's the new alternative?

Cheers,
David

On 12/29/14, 11:55 PM, Olli Salonen wrote:

Hello David,

Coincidentally I was just yesterday working with dvbv5-scan and the
initial scan files. dvbv5-scan expects the initial scan files in the
new DVBV5 format. w_scan is not producing results in this format.

The scan tables at
http://git.linuxtv.org/cgit.cgi/dtv-scan-tables.git/ are in the new
format. Some of them are a bit outdated though (send in a patch if you
can update it for your area).

The v4l-utils package also includes tools to convert between the old
and the new format.

Cheers,
-olli


On 29 December 2014 at 22:09, David Liontooth lionte...@cogweb.net
wrote:

Greetings --

How do you actually use dvbv5-scan? It seems to require some kind of
input
file but there is no man page and the --help screen doesn't say anything
about it.

Could we document this? I tried

$ dvbv5-scan
Usage: dvbv5-scan [OPTION...] initial file
scan DVB services using the channel file

What is the channel file? Maybe the channels.conf file? (I created mine
using w_scan -ft -A3 -X -cUS -o7 -a /dev/dvb/adapter0/)

$ dvbv5-scan /etc/channels.conf
ERROR key/value without a channel group while parsing line 1 of
/etc/channels.conf

So it knows what it wants -- but what is it? Or is this a matter of dvb
versions, and my /etc/channels.conf is in the older format?

Very mysterious.

Cheers,
David
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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



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


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


Re: [RFC/PATCH] dvb-core: add template code for i2c binding model

2014-12-30 Thread Mauro Carvalho Chehab
Em Tue, 30 Dec 2014 11:10:51 -0200
Mauro Carvalho Chehab mche...@osg.samsung.com escreveu:

 Em Fri, 05 Dec 2014 19:49:33 +0900
 tsk...@gmail.com escreveu:
 
  From: Akihiro Tsukada tsk...@gmail.com
  
  Define a standard interface for demod/tuner i2c driver modules.
  A module client calls dvb_i2c_attach_{fe,tuner}(),
  and a module driver defines struct dvb_i2c_module_param and
  calls DEFINE_DVB_I2C_MODULE() macro.
  
  This template provides implicit module requests and ref-counting,
  alloc/free's private data structures,
  fixes the usage of clientdata of i2c devices,
  and defines the platformdata structures for passing around
  device specific config/output parameters between drivers and clients.
  These kinds of code are common to (almost) all dvb i2c drivers/client,
  but they were scattered over adapter modules and demod/tuner modules.
 
 The idea behind this patch is good. I didn't have time yet to test it on a
 device that I have currently access.
 
 I have a few comments below, mostly regards with naming convention.
 
 By seeing the patches you converted a few drivers to use this, I'm a little
 bit concern about the conversion. We need something that won't be hard to
 convert to the new model without requiring to re-test everything.
 
 Anyway, my plan is to take a deeper look on it next week and convert one
 or two drivers to use the new I2C binding approach you're proposing.

Ok, did some tests and it worked. The issues I commented on my last email
may be fixed latter. I'm working on adding media controller support for
it.

The only thing I noticed is that it is causing some warnings at
dmesg about trying to create already created sysfs nodes, when the
driver is removed/reinserted.

Probably, the remove callback is called too soon or too late.

I'll do more tests latter (either this or the next week).

I used the enclosed patch on my tests.

Regards,
Mauro

[PATCH] mb86a20s: convert it to I2C binding model

Instead of using I2C raw API, use the standard I2C binding API,
with the DVB core support for it.

Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com

diff --git a/drivers/media/dvb-frontends/mb86a20s.c 
b/drivers/media/dvb-frontends/mb86a20s.c
index 8f54c39..8dd608b 100644
--- a/drivers/media/dvb-frontends/mb86a20s.c
+++ b/drivers/media/dvb-frontends/mb86a20s.c
@@ -18,6 +18,7 @@
 #include asm/div64.h
 
 #include dvb_frontend.h
+#include dvb_i2c.h
 #include mb86a20s.h
 
 #define NUM_LAYERS 3
@@ -35,12 +36,10 @@ static u8 mb86a20s_subchannel[] = {
 };
 
 struct mb86a20s_state {
-   struct i2c_adapter *i2c;
+   struct i2c_client *i2c;
const struct mb86a20s_config *config;
u32 last_frequency;
 
-   struct dvb_frontend frontend;
-
u32 if_freq;
enum mb86a20s_bandwidth bw;
bool inversion;
@@ -234,7 +233,7 @@ static int mb86a20s_i2c_writereg(struct mb86a20s_state 
*state,
};
int rc;
 
-   rc = i2c_transfer(state-i2c, msg, 1);
+   rc = i2c_transfer(state-i2c-adapter, msg, 1);
if (rc != 1) {
dev_err(state-i2c-dev,
%s: writereg error (rc == %i, reg == 0x%02x, data == 
0x%02x)\n,
@@ -269,7 +268,7 @@ static int mb86a20s_i2c_readreg(struct mb86a20s_state 
*state,
{ .addr = i2c_addr, .flags = I2C_M_RD, .buf = val, .len = 1 }
};
 
-   rc = i2c_transfer(state-i2c, msg, 2);
+   rc = i2c_transfer(state-i2c-adapter, msg, 2);
 
if (rc != 2) {
dev_err(state-i2c-dev, %s: reg=0x%x (error=%d)\n,
@@ -281,11 +280,11 @@ static int mb86a20s_i2c_readreg(struct mb86a20s_state 
*state,
 }
 
 #define mb86a20s_readreg(state, reg) \
-   mb86a20s_i2c_readreg(state, state-config-demod_address, reg)
+   mb86a20s_i2c_readreg(state, state-i2c-addr, reg)
 #define mb86a20s_writereg(state, reg, val) \
-   mb86a20s_i2c_writereg(state, state-config-demod_address, reg, val)
+   mb86a20s_i2c_writereg(state, state-i2c-addr, reg, val)
 #define mb86a20s_writeregdata(state, regdata) \
-   mb86a20s_i2c_writeregdata(state, state-config-demod_address, \
+   mb86a20s_i2c_writeregdata(state, state-i2c-addr, \
regdata, ARRAY_SIZE(regdata))
 
 /*
@@ -2058,41 +2057,34 @@ static int mb86a20s_tune(struct dvb_frontend *fe,
return rc;
 }
 
-static void mb86a20s_release(struct dvb_frontend *fe)
+static int mb86a20s_remove(struct i2c_client *i2c)
 {
-   struct mb86a20s_state *state = fe-demodulator_priv;
-
-   dev_dbg(state-i2c-dev, %s called.\n, __func__);
+   dev_dbg(i2c-dev, %s called.\n, __func__);
 
-   kfree(state);
+   return 0;
 }
 
 static struct dvb_frontend_ops mb86a20s_ops;
 
-struct dvb_frontend *mb86a20s_attach(const struct mb86a20s_config *config,
-   struct i2c_adapter *i2c)
+static int mb86a20s_probe(struct i2c_client *i2c,
+ const struct i2c_device_id *id)
 {
+   struct dvb_frontend *fe;
struct mb86a20s_state *state;

Re: mceusb: sysfs: cannot create duplicate filename '/class/rc/rc0' (race condition between multiple RC_CORE devices)

2014-12-30 Thread Stefan Lippers-Hollmann
Hi

Adding the maintainers for drivers/media/rc/rc-main.c into the loop.

This is a follow-up for:
http://lkml.kernel.org/r/201412181916.18051.s@gmx.de

On Thursday 18 December 2014, Stefan Lippers-Hollmann wrote:
 Occassionally, but not readily reproducably, I hit a race condition 
 between mceusb and other connected RC_CORE devices when mceusb tries 
 to create /class/rc/rc0, which is -by then- already taken by another 
 RC_CORE device. The other involved IR devices (physically only one)
 are part of a PCIe TeVii s480 s2.1 twin-tuner DVB-S2 card and aren't 
 actually supposed to receive IR signals (IR receiver not connected):
 
 mceusb device transceiver:
 Bus 002 Device 004: ID 0609:0334 SMK Manufacturing, Inc. eHome Infrared 
 Receiver
 
 DVB-T receiver (no RC_CORE device)
 Bus 001 Device 004: ID 0ccd:0069 TerraTec Electronic GmbH Cinergy T XE 
 (Version 2, AF9015)
 
 twin-tuner DVB-S2 PCIe device, TeVii s480 v2.1 (physically one IR 
 receiver (NEC protocol), logically recognized as two RC_CORE devices):
[...]
   Bus 006 Device 003: ID 9022:d660 TeVii Technology Ltd. DVB-S2 S660
   Bus 003 Device 003: ID 9022:d660 TeVii Technology Ltd. DVB-S2 S660

Today I got a new, similar, trace with kernel 3.18.1, but this is not a
regression and randomly happens with older kernels as well. The 
frequency of this occuring differs vastly, this time it was 12 days 
with probably 20 (re-)boots, before that it didn't happen for multiple
weeks. I can not totally rule out if it ever happened in the reverse
detection/ initialisation order, as I wouldn't notice the consequences 
of dvb_usb_dw2102's RC_CORE devices failing to initialise, but I think
the problem might be seated in the common core of rc-main.c.

usb 1-1.5: New USB device found, idVendor=0ccd, idProduct=0069
usb 1-1.5: New USB device strings: Mfr=1, Product=2, SerialNumber=3
usb 1-1.5: Product: Cinergy T USB XE Ver.2
usb 1-1.5: Manufacturer: TerraTec
usb 1-1.5: SerialNumber: 10012007
[...]
dvb-usb: found a 'TeVii S480.1 USB' in cold state, will try to load a firmware
dvb-usb: downloading firmware from file 'dvb-usb-s660.fw'
dw2102: start downloading DW210X firmware
usb 1-1.5: dvb_usb_v2: found a 'TerraTec Cinergy T USB XE' in cold state
usb 1-1.5: dvb_usb_v2: downloading firmware from file 'dvb-usb-af9015.fw'
[...]
usb 2-1.6: New USB device found, idVendor=0609, idProduct=0334
usb 2-1.6: New USB device strings: Mfr=1, Product=2, SerialNumber=3
usb 2-1.6: Product: MCE TRANCEIVR Emulator Device 2006
usb 2-1.6: Manufacturer: SMK CORPORATION
usb 2-1.6: SerialNumber: PA070620045513C
[...]
usb 3-1: USB disconnect, device number 2
usb 2-1.8: new full-speed USB device number 5 using ehci-pci
usb 1-1.5: dvb_usb_v2: found a 'TerraTec Cinergy T USB XE' in warm state
[...]
dvb-usb: found a 'TeVii S480.1 USB' in warm state.
dvb-usb: will pass the complete MPEG2 transport stream to the software demuxer.
DVB: registering new adapter (TeVii S480.1 USB)
dvb-usb: MAC address: 70:70:70:70:70:70
Invalid probe, probably not a DS3000
dvb-usb: no frontend was attached by 'TeVii S480.1 USB'
[...]
Registered IR keymap rc-tevii-nec
input: IR-receiver inside an USB DVB receiver as 
/devices/pci:00/:00:1c.5/:04:00.1/usb3/3-1/rc/rc0/input18
rc0: IR-receiver inside an USB DVB receiver as 
/devices/pci:00/:00:1c.5/:04:00.1/usb3/3-1/rc/rc0
dvb-usb: schedule remote query interval to 150 msecs.
dvb-usb: TeVii S480.1 USB successfully initialized and connected.
dvb-usb: found a 'TeVii S480.2 USB' in cold state, will try to load a firmware
dvb-usb: downloading firmware from file 'dvb-usb-s660.fw'
dw2102: start downloading DW210X firmware
[...]
Registered IR keymap rc-rc6-mce
[ cut here ]
WARNING: CPU: 1 PID: 311 at /tmp/buildd/linux-aptosid-3.18/fs/sysfs/dir.c:31 
sysfs_warn_dup+0x55/0x70()
sysfs: cannot create duplicate filename '/class/rc/rc0'
Modules linked in: rt2800usb(+) rt2x00usb rt2800lib rt2x00lib mac80211 cfg80211 
crc_ccitt rc_rc6_mce mceusb(+) rc_tevii_nec ds3000 nls_utf8 nls_cp437 vfat fat 
iTCO_wdt iTCO_vendor_support eeepc_wmi asus_wmi intel_rapl sparse_keymap rfkill 
x86_pkg_temp_thermal evdev intel_powerclamp coretemp kvm_intel kvm 
snd_hda_codec_hdmi crct10dif_pclmul crc32_pclmul ghash_clmulni_intel 
snd_hda_codec_realtek snd_hda_codec_generic aesni_intel aes_x86_64 lrw gf128mul 
glue_helper ablk_helper dvb_usb_af9015(+) cryptd dvb_usb_v2 dvb_usb_dw2102(+) 
dvb_usb dvb_core rc_core psmouse snd_hda_intel pcspkr snd_hda_controller 
serio_raw i2c_i801 snd_hda_codec snd_hwdep snd_pcm snd_timer snd soundcore 
lpc_ich mfd_core battery tpm_infineon i915 video i2c_algo_bit drm_kms_helper 
drm i2c_core intel_gtt ie31200_edac mei_me
 edac_core mei wmi button processor nct6775 hwmon_vid fuse parport_pc ppdev lp 
parport autofs4 ext4 crc16 jbd2 mbcache dm_mod sg sd_mod ohci_pci crc32c_intel 
ahci libahci libata scsi_mod xhci_pci ohci_hcd ehci_pci xhci_hcd ehci_hcd r8169 
mii usbcore usb_common fan thermal
CPU: 1 PID: 311 

Re: [PATCH/RFC v9 07/19] dt-binding: mfd: max77693: Add DT binding related macros

2014-12-30 Thread Sakari Ailus
Hi Jacek,

The driver depends on these so I'd rearrange this patch in the set before
the driver patch.

On Wed, Dec 03, 2014 at 05:06:42PM +0100, Jacek Anaszewski wrote:
 Add macros for max77693 led part related binding.
 
 Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
 Acked-by: Kyungmin Park kyungmin.p...@samsung.com
 Cc: Lee Jones lee.jo...@linaro.org
 Cc: Chanwoo Choi cw00.c...@samsung.com
 ---
  include/dt-bindings/mfd/max77693.h |   38 
 
  1 file changed, 38 insertions(+)
  create mode 100644 include/dt-bindings/mfd/max77693.h
 
 diff --git a/include/dt-bindings/mfd/max77693.h 
 b/include/dt-bindings/mfd/max77693.h
 new file mode 100644
 index 000..4011cb47
 --- /dev/null
 +++ b/include/dt-bindings/mfd/max77693.h
 @@ -0,0 +1,38 @@
 +/*
 + * This header provides macros for MAX77693 device binding
 + *
 + * Copyright (C) 2014, Samsung Electronics Co., Ltd.
 + *
 + * Author: Jacek Anaszewski j.anaszew...@samsung.com
 + */
 +
 +#ifndef __DT_BINDINGS_MAX77693_H__
 +#define __DT_BINDINGS_MAX77693_H
 +
 +/* External control pins */
 +#define MAX77693_LED_FLED_UNUSED 0
 +#define MAX77693_LED_FLED_USED   1
 +
 +/* FLED pins */
 +#define MAX77693_LED_FLED1   1
 +#define MAX77693_LED_FLED2   2

I'd personally simply use numbers for the above but I can't really say to be
an expert on the topic.

 +/* External trigger type */
 +#define MAX77693_LED_TRIG_TYPE_EDGE  0
 +#define MAX77693_LED_TRIG_TYPE_LEVEL 1
 +
 +/* Trigger flags */
 +#define MAX77693_LED_TRIG_FLASHEN(1  0)
 +#define MAX77693_LED_TRIG_TORCHEN(1  1)
 +#define MAX77693_LED_TRIG_SOFTWARE   (1  2)
 +
 +#define MAX77693_LED_TRIG_ALL(MAX77693_LED_TRIG_FLASHEN | \
 +  MAX77693_LED_TRIG_TORCHEN | \
 +  MAX77693_LED_TRIG_SOFTWARE)
 +
 +/* Boost modes */
 +#define MAX77693_LED_BOOST_OFF   0
 +#define MAX77693_LED_BOOST_ADAPTIVE  1
 +#define MAX77693_LED_BOOST_FIXED 2
 +
 +#endif /* __DT_BINDINGS_MAX77693_H */

-- 
Regards,

Sakari Ailus
e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC][PATCH] mn88472: add support for the mn88473 demod

2014-12-30 Thread Benjamin Larsson

On 12/21/2014 10:21 AM, Antti Palosaari wrote:

Moikka!


[...]



You patch looks rather good and these drivers should be merged to one if
possible, lets say registers are 80% same or something like that. Looks
like those are.


I've dropped this effort, the chips registers are not similar enough. 
The code that could be shared is not enough to give any advantage over 2 
drivers.


MvH
Benjamin Larsson
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] media: au0828 analog_register error path fixes to do proper cleanup

2014-12-30 Thread Shuah Khan
au0828_analog_register() doesn't release video and vbi queues
created by vb2_queue_init(). In addition, it doesn't unregister
vdev when vbi register fails. Add vb2_queue_release() calls to
release video and vbi queues to the failure path to be called
when vdev register fails. Add video_unregister_device() for
vdev when vbi register fails.

Signed-off-by: Shuah Khan shua...@osg.samsung.com
---
Please note that this patch is dependent on the au0828 vb2
conversion patch.

 drivers/media/usb/au0828/au0828-video.c | 10 +++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/drivers/media/usb/au0828/au0828-video.c 
b/drivers/media/usb/au0828/au0828-video.c
index 94b65b8..17450eb 100644
--- a/drivers/media/usb/au0828/au0828-video.c
+++ b/drivers/media/usb/au0828/au0828-video.c
@@ -1785,7 +1785,7 @@ int au0828_analog_register(struct au0828_dev *dev,
dprintk(1, unable to register video device (error = %d).\n,
retval);
ret = -ENODEV;
-   goto err_vbi_dev;
+   goto err_reg_vdev;
}
 
/* Register the vbi device */
@@ -1795,14 +1795,18 @@ int au0828_analog_register(struct au0828_dev *dev,
dprintk(1, unable to register vbi device (error = %d).\n,
retval);
ret = -ENODEV;
-   goto err_vbi_dev;
+   goto err_reg_vbi_dev;
}
 
-
dprintk(1, %s completed!\n, __func__);
 
return 0;
 
+err_reg_vbi_dev:
+   video_unregister_device(dev-vdev);
+err_reg_vdev:
+   vb2_queue_release(dev-vb_vidq);
+   vb2_queue_release(dev-vb_vbiq);
 err_vbi_dev:
video_device_release(dev-vbi_dev);
 err_vdev:
-- 
2.1.0

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


Re: dvbv5-scan needs which channel file?

2014-12-30 Thread Mauro Carvalho Chehab
Hi David,

Em Tue, 30 Dec 2014 07:49:05 -0800
David Liontooth lionte...@cogweb.net escreveu:

 
 OK, perfect; thank you. This should be documented in dvbv5-scan. And we 
 should have a man page for it.

This is documented, and there is a man page for it:

dvbv5-scan(1)User Commands   dvbv5-scan(1)



NAME
   dvbv5-scan - DVBv5 tool for frequency scanning

SYNOPSIS
   dvbv5-scan [OPTION]... initial-file

DESCRIPTION
   dvbv5-scan  is  a  command  line frequency scanning tool for digital TV
   services that is compliant with version 5 of the DVB API, and  backward
   compatible with the older v3 DVB API.

   dvbv5-scan  uses by default the new channel/service file format that it
   is capable of supporting all types of Digital TV standards. It can also
   support the legacy format used by the legacy dvb-apps.

   A single physical channel (also called as transponder) may have several
   virtual channels inside it, encapsulated via a MPEG Transport stream.

   Those virtual channels are called as 'service' at the MPEG-TS terminol‐
   ogy,  and may have one or more audio, video and other types of elements
   inside it.

   The  dvbv5-scan  goal  is  to  scan  for  a  list  of  physical   chan‐
   nels/transponders and identify there the MPEG-TS services available.

   The  dvbv5-scan tool is smart enough to retrieve the information at the
   MPEG-TS Network Information Table (NIT) about other channels  available
   on the stream.

OPTIONS
   The following options are valid:

   -3, --dvbv3
  Force dvbv5-scan to use DVBv3 only.

   -a, --adapter=adapter#
  Use the given adapter. Default value: 0.

   -d, --demux=demux#
  Use the given demux. Default value: 0.

   -f, --frontend=frontend#
  Use the given frontend. Default value: 0.

   -F, --file-freqs-only
  Don't  use  the  other  frequencies  discovered  during scan. By
  default, dvbv5-scan will find new transponder/physical  channels
  and  add  them at the internal frequency table it uses for scan.
  This option disables such feature.

   -G, --get_frontend
  Use data from get_frontend  on  the  output  file.  By  default,
  dvbv5-scan  will  repeat the same network parameters as found at
  the scan file. This should work fine if the output file will  be
  used  by  the  same  frontend. However, if you intend to use the
  generated file on another frontend, or  wants  a  faster  tuner,
  this option can be used to store the actual detected parameters,
  instead of the ones that came from the source channel file.

   -I, --input-format=format
  Format of the input file. Please notice that caps is ignored. It
  can be:

  channel - for dvb-apps compatible channel file;

  zap - for dvb-apps compatible zap file;

  dvbv5 (default) - for the dvbv5 apps format.

   -l, --lnbf=LNBf_type
  Type of LNBf to use 'help' lists the available ones.

   -N, --nit
  Use  data  from  NIT  table  on  the  output  file.  By default,
  dvbv5-scan will repeat the same network parameters as  found  at
  the  scan file. This should work fine if the output file will be
  used by the same frontend. However, if you  intend  to  use  the
  generated  file  on  another  frontend, or wants a faster tuner,
  this option can  be  used  to  store  the  parameters  that  are
  announced by the broadcaster via the MPEG-TS Network Information
  Table (NIT), instead of the ones that came from the source chan‐
  nel file.

   -o, --output=file
  output filename (default: dvb_channel.conf)

   -O, --output-format=format
  Output format:

  channel - for dvb-apps compatible channel file;

  zap - for dvb-apps compatible zap file;

  vdr - for vdr compatible zap file;

  dvbv5 (default) - for the dvbv5 apps format.

   -p, --parse-other-nit
  Parse  the  other  NIT/SDT  tables that could be found mainly on
  some DVB-C carriers.

   -S, --sat_number=satellite_number
  Satellite number.  Used only on satellite delivery systems.   If
  not specified, disable DISEqC satellite switch.

   -T, --timeout-multiply=factor
  Multiply  the  scan  lock wait time and MPEG-TS table parsing by
  this factor.

   -U, --freq_bpf=frequency
  SCR/Unicable band-pass filter frequency to use,  in  kHz.   Used
  only on satellite delivery systems.

   -v, --verbose
  

Re: dvbv5-scan needs which channel file?

2014-12-30 Thread Mauro Carvalho Chehab
Em Wed, 31 Dec 2014 00:11:34 -0200
Mauro Carvalho Chehab mche...@osg.samsung.com escreveu:

 Hi David,
 
 Em Tue, 30 Dec 2014 07:49:05 -0800
 David Liontooth lionte...@cogweb.net escreveu:
 
  
  OK, perfect; thank you. This should be documented in dvbv5-scan. And we 
  should have a man page for it.
 
 This is documented, and there is a man page for it:
 
 dvbv5-scan(1)User Commands   dvbv5-scan(1)
...

Forgot to mention, but, of course, if you think that the information there
is not enough, feel free to submit patches to improve it ;)

The documentation source file is at:

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/tree/utils/dvb/dvbv5-scan.1.in

Also, translations for other languages is also welcomed.
-- 

Cheers,
Mauro
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


cron job: media_tree daily build: ERRORS

2014-12-30 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for
the kernels and architectures in the list below.

Results of the daily build of media_tree:

date:   Wed Dec 31 04:00:09 CET 2014
git branch: test
git hash:   99f3cd52aee21091ce62442285a68873e3be833f
gcc version:i686-linux-gcc (GCC) 4.9.1
sparse version: v0.5.0-41-g6c2d743
smatch version: 0.4.1-3153-g7d56ab3
host hardware:  x86_64
host os:3.17-3.slh.2-amd64

linux-git-arm-at91: OK
linux-git-arm-davinci: OK
linux-git-arm-exynos: OK
linux-git-arm-mx: OK
linux-git-arm-omap: OK
linux-git-arm-omap1: OK
linux-git-arm-pxa: OK
linux-git-blackfin: OK
linux-git-i686: OK
linux-git-m32r: OK
linux-git-mips: OK
linux-git-powerpc64: OK
linux-git-sh: OK
linux-git-x86_64: OK
linux-2.6.32.27-i686: ERRORS
linux-2.6.33.7-i686: ERRORS
linux-2.6.34.7-i686: ERRORS
linux-2.6.35.9-i686: ERRORS
linux-2.6.36.4-i686: ERRORS
linux-2.6.37.6-i686: ERRORS
linux-2.6.38.8-i686: ERRORS
linux-2.6.39.4-i686: ERRORS
linux-3.0.60-i686: ERRORS
linux-3.1.10-i686: ERRORS
linux-3.2.37-i686: ERRORS
linux-3.3.8-i686: ERRORS
linux-3.4.27-i686: ERRORS
linux-3.5.7-i686: ERRORS
linux-3.6.11-i686: ERRORS
linux-3.7.4-i686: ERRORS
linux-3.8-i686: ERRORS
linux-3.9.2-i686: ERRORS
linux-3.10.1-i686: ERRORS
linux-3.11.1-i686: ERRORS
linux-3.12.23-i686: ERRORS
linux-3.13.11-i686: ERRORS
linux-3.14.9-i686: ERRORS
linux-3.15.2-i686: ERRORS
linux-3.16-i686: ERRORS
linux-3.17-i686: ERRORS
linux-3.18-i686: ERRORS
linux-2.6.32.27-x86_64: ERRORS
linux-2.6.33.7-x86_64: ERRORS
linux-2.6.34.7-x86_64: ERRORS
linux-2.6.35.9-x86_64: ERRORS
linux-2.6.36.4-x86_64: ERRORS
linux-2.6.37.6-x86_64: ERRORS
linux-2.6.38.8-x86_64: ERRORS
linux-2.6.39.4-x86_64: ERRORS
linux-3.0.60-x86_64: ERRORS
linux-3.1.10-x86_64: ERRORS
linux-3.2.37-x86_64: ERRORS
linux-3.3.8-x86_64: ERRORS
linux-3.4.27-x86_64: ERRORS
linux-3.5.7-x86_64: ERRORS
linux-3.6.11-x86_64: ERRORS
linux-3.7.4-x86_64: ERRORS
linux-3.8-x86_64: ERRORS
linux-3.9.2-x86_64: ERRORS
linux-3.10.1-x86_64: ERRORS
linux-3.11.1-x86_64: ERRORS
linux-3.12.23-x86_64: ERRORS
linux-3.13.11-x86_64: ERRORS
linux-3.14.9-x86_64: ERRORS
linux-3.15.2-x86_64: ERRORS
linux-3.16-x86_64: ERRORS
linux-3.17-x86_64: ERRORS
linux-3.18-x86_64: ERRORS
apps: OK
spec-git: OK
sparse: WARNINGS
smatch: ERRORS

Detailed results are available here:

http://www.xs4all.nl/~hverkuil/logs/Wednesday.log

Full logs are available here:

http://www.xs4all.nl/~hverkuil/logs/Wednesday.tar.bz2

The Media Infrastructure API from this daily build is here:

http://www.xs4all.nl/~hverkuil/spec/media.html
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html