cron job: media_tree daily build: OK
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: Mon Jan 4 04:00:16 CET 2016 git branch: test git hash: 768acf46e1320d6c41ed1b7c4952bab41c1cde79 gcc version:i686-linux-gcc (GCC) 5.1.0 sparse version: v0.5.0-51-ga53cea2 smatch version: v0.5.0-3228-g5cf65ab host hardware: x86_64 host os:4.3.0-164 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-bf561: 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.34.7-i686: OK linux-2.6.35.9-i686: OK linux-2.6.36.4-i686: OK linux-2.6.37.6-i686: OK linux-2.6.38.8-i686: OK linux-2.6.39.4-i686: OK linux-3.0.60-i686: OK linux-3.1.10-i686: OK linux-3.2.37-i686: OK linux-3.3.8-i686: OK linux-3.4.27-i686: OK linux-3.5.7-i686: OK linux-3.6.11-i686: OK linux-3.7.4-i686: OK linux-3.8-i686: OK linux-3.9.2-i686: OK linux-3.10.1-i686: OK linux-3.11.1-i686: OK linux-3.12.23-i686: OK linux-3.13.11-i686: OK linux-3.14.9-i686: OK linux-3.15.2-i686: OK linux-3.16.7-i686: OK linux-3.17.8-i686: OK linux-3.18.7-i686: OK linux-3.19-i686: OK linux-4.0-i686: OK linux-4.1.1-i686: OK linux-4.2-i686: OK linux-4.3-i686: OK linux-4.4-rc1-i686: OK linux-2.6.34.7-x86_64: OK linux-2.6.35.9-x86_64: OK linux-2.6.36.4-x86_64: OK linux-2.6.37.6-x86_64: OK linux-2.6.38.8-x86_64: OK linux-2.6.39.4-x86_64: OK linux-3.0.60-x86_64: OK linux-3.1.10-x86_64: OK linux-3.2.37-x86_64: OK linux-3.3.8-x86_64: OK linux-3.4.27-x86_64: OK linux-3.5.7-x86_64: OK linux-3.6.11-x86_64: OK linux-3.7.4-x86_64: OK linux-3.8-x86_64: OK linux-3.9.2-x86_64: OK linux-3.10.1-x86_64: OK linux-3.11.1-x86_64: OK linux-3.12.23-x86_64: OK linux-3.13.11-x86_64: OK linux-3.14.9-x86_64: OK linux-3.15.2-x86_64: OK linux-3.16.7-x86_64: OK linux-3.17.8-x86_64: OK linux-3.18.7-x86_64: OK linux-3.19-x86_64: OK linux-4.0-x86_64: OK linux-4.1.1-x86_64: OK linux-4.2-x86_64: OK linux-4.3-x86_64: OK linux-4.4-rc1-x86_64: OK apps: OK spec-git: OK sparse: WARNINGS smatch: ERRORS Detailed results are available here: http://www.xs4all.nl/~hverkuil/logs/Monday.log Full logs are available here: http://www.xs4all.nl/~hverkuil/logs/Monday.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
[PATCH] modified: drivers/media/tuners/si2157.c fixed frequency range
From: koshkoshka2 --- drivers/media/tuners/si2157.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c index 0e1ca2b..5da5b42 100644 --- a/drivers/media/tuners/si2157.c +++ b/drivers/media/tuners/si2157.c @@ -364,8 +364,8 @@ static int si2157_get_if_frequency(struct dvb_frontend *fe, u32 *frequency) static const struct dvb_tuner_ops si2157_ops = { .info = { .name = "Silicon Labs Si2146/2147/2148/2157/2158", - .frequency_min = 5500, - .frequency_max = 86200, + .frequency_min = 4200, + .frequency_max = 87000, }, .init = si2157_init, -- 1.9.1 -- 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
Elgato Game Capture HD on Linux
Hi Steven, I wish you a happy new year, it's been a while. I've made great success on the Elgato Game Capture HD driver for Linux - the major features are working. Currently, it's supporting capturing / streaming HDMI with 1080p30 and 720p60, and component with 1080p30, 1080i60, 720p60 and 576p50 (PAL). Eventhough I've just quickly hacked together everything, I've learned alot. The official Mac OS X drivers included files with comments. They were my cheatsheet. One important thing, I've just recently figured out: the official driver is not doing stuff sequentially, like I've always assumed. It's doing many things in parallel using threads. This is the reason, why reverse engineering has been a nightmare for me, cause lots of unrelated, random packets showed up and I couldn't interpret them. No two USB logs looked nearly the same - there were always differences. Sometimes major, sometimes minor. There seems to be one thread, which is trying to receive bulk data on endpoints 0x04 and 0x01 (maybe even one thread per endpoint?), without any sleep. This was causing LOTS of USB log entries in conjunction with my USB hardware analyzer - talking about 5 - 8 million lines for 20 seconds. Another thread is constantly checking the input source's information, like resolution (small sleeps, unlike bulk receive). If it changes, this thread writes new, customized configuration to the device. I have logged lots of different traffic and am sure, that I can track this down. There seems to be another thread, which seems to do just the things, I was actually trying to track down in the first place: plain simple device setup / initialization. Just sending over some packets, like bitrate configuration and video output resolution. Luckily, most of these values are commented in Elgato's official Mac OS X driver and I've also figured out, how most of them work (https://github.com/tolga9009/elgato-gchd/blob/master/commands.h). Eventhough lots and lots of refactoring is urgently waiting for me (honestly, I'm ashamed of it), I think it's atleast a good point to start. I'm gonna work on refactoring / tidying up next major hacking session in about 2 - 3 months. You can find the latest code here: https://github.com/tolga9009/elgato-gchd. Looking forward to port this over to V4L2, once the refactoring is done. If you're interested in how things turned out, I've even uploaded a small video presentation on YouTube: https://youtu.be/rpm6TJu6HkE?t=6m34s. You can skip it to around 6:30, cause I'm just showing how to compile & firmware extraction first. Cheers, Tolga -- 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] drivers/media/usb/dvb-usb-v2: constify mxl111sf_tuner_config structure
On Sun, Jan 3, 2016 at 7:11 AM, Julia Lawall wrote: > This mxl111sf_tuner_config structure is never modified, so declare it as > const. > > There are some indentation changes to remain within 80 columns. > > Done with the help of Coccinelle. > > Signed-off-by: Julia Lawall Thank you for this, Julia Reviewed-by: Michael Ira Krufky > > --- > drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c |6 +++--- > drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h |8 > drivers/media/usb/dvb-usb-v2/mxl111sf.c |2 +- > 3 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c > b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c > index 444579b..7d16252 100644 > --- a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c > +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c > @@ -36,7 +36,7 @@ MODULE_PARM_DESC(debug, "set debugging level (1=info > (or-able))."); > struct mxl111sf_tuner_state { > struct mxl111sf_state *mxl_state; > > - struct mxl111sf_tuner_config *cfg; > + const struct mxl111sf_tuner_config *cfg; > > enum mxl_if_freq if_freq; > > @@ -489,8 +489,8 @@ static struct dvb_tuner_ops mxl111sf_tuner_tuner_ops = { > }; > > struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, > - struct mxl111sf_state *mxl_state, > - struct mxl111sf_tuner_config *cfg) > + struct mxl111sf_state *mxl_state, > + const struct mxl111sf_tuner_config *cfg) > { > struct mxl111sf_tuner_state *state = NULL; > > diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h > b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h > index e6caab2..509b550 100644 > --- a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h > +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h > @@ -63,13 +63,13 @@ struct mxl111sf_tuner_config { > #if IS_ENABLED(CONFIG_DVB_USB_MXL111SF) > extern > struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, > - struct mxl111sf_state *mxl_state, > - struct mxl111sf_tuner_config *cfg); > + struct mxl111sf_state *mxl_state, > + const struct mxl111sf_tuner_config *cfg); > #else > static inline > struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, > - struct mxl111sf_state *mxl_state, > - struct mxl111sf_tuner_config *cfg) > + struct mxl111sf_state *mxl_state, > + const struct mxl111sf_tuner_config *cfg) > { > printk(KERN_WARNING "%s: driver disabled by Kconfig\n", __func__); > return NULL; > diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf.c > b/drivers/media/usb/dvb-usb-v2/mxl111sf.c > index b669dec..b586e17 100644 > --- a/drivers/media/usb/dvb-usb-v2/mxl111sf.c > +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf.c > @@ -856,7 +856,7 @@ static int mxl111sf_ant_hunt(struct dvb_frontend *fe) > return 0; > } > > -static struct mxl111sf_tuner_config mxl_tuner_config = { > +static const struct mxl111sf_tuner_config mxl_tuner_config = { > .if_freq = MXL_IF_6_0, /* applies to external IF output, only > */ > .invert_spectrum = 0, > .read_reg= mxl111sf_read_reg, > > -- > 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 v0] Add tw5864 driver
On Sun, Jan 3, 2016 at 5:47 AM, Joe Perches wrote: > several of these have unnecessary parentheses Thanks, fixed. > Maybe use bool a bit more Thanks, fixed. > or maybe just use fls Thanks, fls() fit greatly, rewritten the function with compatibility testing. >> +static inline int bs_size_ue(unsigned int val) >> +{ >> + int i_size = 0; >> + static const int i_size0_254[255] = { > > Same sort of thing Dropped this procedure because it is not used. Thanks. >> diff --git a/drivers/staging/media/tw5864/tw5864-config.c >> b/drivers/staging/media/tw5864/tw5864-config.c > [] >> +u8 tw_indir_readb(struct tw5864_dev *dev, u16 addr) >> +{ >> + int timeout = 3; > > misleading name, retries would be more proper, > or maybe use real timed loops. Thanks, renamed to "retries". > This seems a little repetitive. Thanks, reworked. > u16? Thanks, fixed. > odd indentation Indeed. For some mysterious reason, vim + syntastic insists on this way. Fixed. >> +#ifdef DEBUG >> + dev_dbg(&input->root->pci->dev, >> + "input %d, frame md stats: min %u, max %u, avg %u, cells above >> threshold: %u\n", >> + input->input_number, min, max, sum / md_cells, >> + cnt_above_thresh); >> +#endif > > unnecessary #ifdef Not quite. This debug printout works with variables which are declared in another "#ifdef DEBUG" clause. And it turns out that dev_dbg is compiled not only if DEBUG is declared, so when I remove this ifdef, I get "undefined variable" errors. It seems it is compiled if this condition is met: #if (defined DEBUG) || (defined CONFIG_DYNAMIC_DEBUG) so I can wrap my stats variables into this statement instead. But such change is not equivalent - I guess CONFIG_DYNAMIC_DEBUG is common to be enabled, so debug stats will be always calculated, even when module is not under debug. Except if I use DEFINE_DYNAMIC_DEBUG_METADATA etc. in my code. Please let me know if this can be sorted out in cleaner way. On Sun, Jan 3, 2016 at 7:38 AM, Leon Romanovsky wrote: > On Sun, Jan 03, 2016 at 03:41:42AM +0200, Andrey Utkin wrote: > >> +/* >> + * TW5864 driver - Exp-Golomb code functions >> + * >> + * Copyright (C) 2015 Bluecherry, LLC >> + * Copyright (C) 2015 Andrey Utkin > > I doubt that you have contract with your employer which permits you to > claim copyright on the work/product. Thank you for commenting. I have previously asked my employer to review copyright statment, and he told this is fine. Now I have requrested him again with reference to your comment. -- Bluecherry developer. -- 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: bt8xx: constify sp887x_config structure
This sp887x_config structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- This patch and the previous one on the same file can be applied in any order. drivers/media/pci/bt8xx/dvb-bt8xx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/bt8xx/dvb-bt8xx.c b/drivers/media/pci/bt8xx/dvb-bt8xx.c index d407244..a3aa0cd 100644 --- a/drivers/media/pci/bt8xx/dvb-bt8xx.c +++ b/drivers/media/pci/bt8xx/dvb-bt8xx.c @@ -318,7 +318,7 @@ static int microtune_mt7202dtf_request_firmware(struct dvb_frontend* fe, const s return request_firmware(fw, name, &bt->bt->dev->dev); } -static struct sp887x_config microtune_mt7202dtf_config = { +static const struct sp887x_config microtune_mt7202dtf_config = { .demod_address = 0x70, .request_firmware = microtune_mt7202dtf_request_firmware, }; -- 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: bt8xx: constify or51211_config structure
The or51211_config structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/pci/bt8xx/dvb-bt8xx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/bt8xx/dvb-bt8xx.c b/drivers/media/pci/bt8xx/dvb-bt8xx.c index d407244..8aeb14c 100644 --- a/drivers/media/pci/bt8xx/dvb-bt8xx.c +++ b/drivers/media/pci/bt8xx/dvb-bt8xx.c @@ -458,7 +458,7 @@ static void or51211_sleep(struct dvb_frontend * fe) bttv_write_gpio(bt->bttv_nr, 0x0001, 0x); } -static struct or51211_config or51211_config = { +static const struct or51211_config or51211_config = { .demod_address = 0x15, .request_firmware = or51211_request_firmware, .setmode = or51211_setmode, -- 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] drivers/media/usb/dvb-usb-v2: constify mxl111sf_tuner_config structure
This mxl111sf_tuner_config structure is never modified, so declare it as const. There are some indentation changes to remain within 80 columns. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c |6 +++--- drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h |8 drivers/media/usb/dvb-usb-v2/mxl111sf.c |2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c index 444579b..7d16252 100644 --- a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.c @@ -36,7 +36,7 @@ MODULE_PARM_DESC(debug, "set debugging level (1=info (or-able))."); struct mxl111sf_tuner_state { struct mxl111sf_state *mxl_state; - struct mxl111sf_tuner_config *cfg; + const struct mxl111sf_tuner_config *cfg; enum mxl_if_freq if_freq; @@ -489,8 +489,8 @@ static struct dvb_tuner_ops mxl111sf_tuner_tuner_ops = { }; struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, - struct mxl111sf_state *mxl_state, - struct mxl111sf_tuner_config *cfg) + struct mxl111sf_state *mxl_state, + const struct mxl111sf_tuner_config *cfg) { struct mxl111sf_tuner_state *state = NULL; diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h index e6caab2..509b550 100644 --- a/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf-tuner.h @@ -63,13 +63,13 @@ struct mxl111sf_tuner_config { #if IS_ENABLED(CONFIG_DVB_USB_MXL111SF) extern struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, - struct mxl111sf_state *mxl_state, - struct mxl111sf_tuner_config *cfg); + struct mxl111sf_state *mxl_state, + const struct mxl111sf_tuner_config *cfg); #else static inline struct dvb_frontend *mxl111sf_tuner_attach(struct dvb_frontend *fe, - struct mxl111sf_state *mxl_state, - struct mxl111sf_tuner_config *cfg) + struct mxl111sf_state *mxl_state, + const struct mxl111sf_tuner_config *cfg) { printk(KERN_WARNING "%s: driver disabled by Kconfig\n", __func__); return NULL; diff --git a/drivers/media/usb/dvb-usb-v2/mxl111sf.c b/drivers/media/usb/dvb-usb-v2/mxl111sf.c index b669dec..b586e17 100644 --- a/drivers/media/usb/dvb-usb-v2/mxl111sf.c +++ b/drivers/media/usb/dvb-usb-v2/mxl111sf.c @@ -856,7 +856,7 @@ static int mxl111sf_ant_hunt(struct dvb_frontend *fe) return 0; } -static struct mxl111sf_tuner_config mxl_tuner_config = { +static const struct mxl111sf_tuner_config mxl_tuner_config = { .if_freq = MXL_IF_6_0, /* applies to external IF output, only */ .invert_spectrum = 0, .read_reg= mxl111sf_read_reg, -- 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] av7110: constify sp8870_config structure
This sp8870_config structure is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/pci/ttpci/av7110.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/ttpci/av7110.c b/drivers/media/pci/ttpci/av7110.c index a69dc6a..18d229f 100644 --- a/drivers/media/pci/ttpci/av7110.c +++ b/drivers/media/pci/ttpci/av7110.c @@ -1739,7 +1739,7 @@ static int alps_tdlb7_request_firmware(struct dvb_frontend* fe, const struct fir #endif } -static struct sp8870_config alps_tdlb7_config = { +static const struct sp8870_config alps_tdlb7_config = { .demod_address = 0x71, .request_firmware = alps_tdlb7_request_firmware, -- 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
Charity Donation
Hi, My name is Jeffrey Skoll, a philanthropist and the founder of one of the largest private foundations in the world. I believe strongly in ‘giving while living.’ I had one idea that never changed in my mind — that you should use your wealth to help people and I have decided to secretly give USD2.498 Million to a randomly selected individual. On receipt of this email, you should count yourself as the individual. Kindly get back to me at your earliest convenience, so I know your email address is valid. Visit the web page to know more about me: http://www.theglobeandmail.com/news/national/meet-the-canadian-billionaire-whos-giving-it-all-away/article4209888/ or you can read an article of me on Wikipedia. Regards, Jeffrey Skoll. -- 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