Hello Antti,
On 2014-10-31 11:13:59, Antti Palosaari wrote:
>
>
>On 10/31/2014 04:55 AM, Antti Palosaari wrote:
>>
>>
>> On 10/31/2014 04:34 AM, Nibble Max wrote:
>>> Hello Antti,
>>>
>>> On 2014-10-31 01:36:14, Antti Palosaari wrote:
On 10/30/2014 06:38 AM, Nibble Max wrote:
>>
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: Fri Oct 31 04:00:17 CET 2014
git branch: test
git hash: 37fa8716e2d4c4155205aa4a904835de09edbb88
gcc versio
On 10/31/2014 04:55 AM, Antti Palosaari wrote:
On 10/31/2014 04:34 AM, Nibble Max wrote:
Hello Antti,
On 2014-10-31 01:36:14, Antti Palosaari wrote:
On 10/30/2014 06:38 AM, Nibble Max wrote:
-if (tab_len > 83) {
+if (tab_len > 86) {
That is not nice, but I could try find bett
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3a2f22b7d0cc64482a91529e23c2570aa0602fa6
commit: 7b34be71db533f3e0cf93d53cf62d036cdb5418a [media] use IS_ENABLED() macro
date: 1 year, 9 months ago
config: x86_64-randconfig-iv0-10311039 (attached as .config)
On 10/31/2014 04:34 AM, Nibble Max wrote:
Hello Antti,
On 2014-10-31 01:36:14, Antti Palosaari wrote:
On 10/30/2014 06:38 AM, Nibble Max wrote:
- if (tab_len > 83) {
+ if (tab_len > 86) {
That is not nice, but I could try find better solution and fix it later.
What is the
Hello Antti,
On 2014-10-31 01:36:14, Antti Palosaari wrote:
>
>
>On 10/30/2014 06:38 AM, Nibble Max wrote:
>
- if (tab_len > 83) {
+ if (tab_len > 86) {
>>>
>>> That is not nice, but I could try find better solution and fix it later.
>>
>> What is the reason to check this parameter?
>>
Hi Mauro,
this one is now based on your cx231xx branch.
Regards
Matthias
--
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
The only remaining place that modifies the relevant bit is in function
cx231xx_set_Colibri_For_LowIF
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-avcore.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/media/usb/cx231xx/cx231xx-avcore.c
Is there any update on DVB-C support? It seems that the patch that
ZZram made, works on lower versions, but not on the current branch.
I have looked in the media_tree.git branch, and see that the (needed)
flags are their, but still it says 'DVB-C not supported'. Would be
great if you guys could im
On Thu, 30 Oct 2014, Olli Salonen wrote:
TechnoTrend CT2-4500 CI is a PCIe device with DVB-T2/C tuner. It is
similar to DVBSky T980C, just with different PCI ID and remote
controller.
Additional note, this should be applied on top of Max Nibble's commits:
http://www.mail-archive.com/linux-med
TechnoTrend CT2-4500 CI is a PCIe device with DVB-T2/C tuner. It is similar to
DVBSky T980C, just with different PCI ID and remote controller.
Signed-off-by: Olli Salonen
---
drivers/media/pci/cx23885/cx23885-cards.c | 14 ++
drivers/media/pci/cx23885/cx23885-dvb.c | 8 +---
Em Thu, 30 Oct 2014 21:12:21 +0100
Matthias Schwarzott escreveu:
> Hi!
>
> This time the series got only small updates from Antti's review.
Oh, no! I just merged the v3 patch series. I manually fixed the
points that Antti has made.
I'm also testing it already.
Could you, instead, rebase it on
Replace numbers by the constants of same value and same meaning.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-cards.c | 62 +++
1 file changed, 31 insertions(+), 31 deletions(-)
diff --git a/drivers/media/usb/
change almost all instances of I2C_1 to I2C_1_MUX_3
Only these cases are changed to I2C_1_MUX_1:
* All that have dont_use_port_3 set.
* CX231XX_BOARD_HAUPPAUGE_EXETER, old code did explicitly not switch to port3.
* eeprom access for 930C
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palo
Change to I2C_0 ... I2C_2 for the master ports
and add I2C_1_MUX_1 and I2C_1_MUX_3 for the muxed ones.
V2: Renamed mux adapters to seperate them from master adapters.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx.h | 9 +
1 file c
This is a preparation for deleting the otherwise useless i2c_clients
that are allocated for all the i2c master adapters.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 17 +++--
drivers/media/usb/cx231xx/cx231xx.h |
V2: Use snprintf to construct the complete name
V3: Remove unneeded variable.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/usb/cx231xx/cx231xx-i2c.c
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/Kconfig| 1 +
drivers/media/usb/cx231xx/cx231xx-core.c | 5
drivers/media/usb/cx231xx/cx231xx-i2c.c | 44 +++-
drivers/media/usb/cx231xx/cx231xx.h | 4 +
Get used i2c port from bus_nr and status of port_3 switch.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/cx231xx/cx231xx-i2c.c
b/drivers/media/usb/cx231xx/cx231xx-i
The only remaining place that modifies the relevant bit is in function
cx231xx_set_Colibri_For_LowIF
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-avcore.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/media/usb/cx231xx/cx231xx-avcore.c
All switching should be done by i2c mux adapters.
Drop explicit dont_use_port_3 flag.
Drop info message about switch.
Only the removed code in start_streaming is questionable:
It did switch the port_3 flag without accessing i2c in between.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Pa
This is a preparation for mapping I2C_1_MUX_1 and I2C_1_MUX_3 later to the
seperate
muxed i2c adapters.
Map mux adapters to I2C_1 for now.
Add local variables for i2c_adapters in dvb_init to get line lengths
shorter.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers
The scanning itself just fails (as before this series) but now the correct
busses are scanned.
V2: Changed to symbolic names where muxed adapters can be seen directly.
V3: Comment about scanning busses ordered by physical port numbers.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palos
This is used later for is_tuner function that switches i2c behaviour for
some tuners.
V2: Add comments about possible improvements for port_3 switch function.
V3: Fix coding style.
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-avcore.c | 12
drivers/media
For each i2c master there is a i2c_client allocated that could be
deleted now that its only two users have been changed to use their
own i2c_client.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 7 ---
drivers/media/usb/cx231xx
Hi!
This time the series got only small updates from Antti's review.
Additionally I added a patch to no longer directly modify
the content of the port3 switch bit in PWR_CTL_EN (from function
cx231xx_set_power_mode).
Now there are two places where I wonder what happens:
1. cx231xx_set_Colibri_F
This is a preparation for deleting the otherwise useless i2c_clients
that are allocated for all the i2c master adapters.
Signed-off-by: Matthias Schwarzott
Reviewed-by: Antti Palosaari
---
drivers/media/usb/cx231xx/cx231xx-cards.c | 24 +---
1 file changed, 13 insertions(+),
On Thu, Oct 30, 2014 at 3:52 PM, Michael Ira Krufky wrote:
> On Thu, Oct 30, 2014 at 9:37 AM, Mauro Carvalho Chehab
> wrote:
>> Hi Devin,
>>
>> Em Thu, 30 Oct 2014 09:15:31 -0400
>> Devin Heitmueller escreveu:
>>
>>> Hi Mauro,
>>>
>>> > Syncronize it and put them on the same order as found at au
On Thu, Oct 30, 2014 at 9:37 AM, Mauro Carvalho Chehab
wrote:
> Hi Devin,
>
> Em Thu, 30 Oct 2014 09:15:31 -0400
> Devin Heitmueller escreveu:
>
>> Hi Mauro,
>>
>> > Syncronize it and put them on the same order as found at au0828
>> > driver, as all the au0828 devices with analog TV need the
>> >
Reviewed-by: Olli Salonen
On Thu, 23 Oct 2014, Nibble Max wrote:
DVBSky s950ci dvb-s/s2 ci PCIe card:
1>dvb frontend: M88TS2022(tuner),M88DS3103(demod)
2>ci controller: CIMAX SP2 or its clone.
3>PCIe bridge: CX23885
The patchs are based on the following patchs.
Olli Salonen submit:
https://pa
Reviewed-by: Olli Salonen
On Thu, 23 Oct 2014, Nibble Max wrote:
DVBSky s950ci dvb-s/s2 ci PCIe card:
1>dvb frontend: M88TS2022(tuner),M88DS3103(demod)
2>ci controller: CIMAX SP2 or its clone.
3>PCIe bridge: CX23885
The patchs are based on the following patchs.
Olli Salonen submit:
https://pa
Em Thu, 2 Oct 2014 07:21:02 +0200
Matthias Schwarzott escreveu:
> change almost all instances of I2C_1 to I2C_1_MUX_3
So far, this is likely the most dangerous patch on this series ;)
>
> Only these cases are changed to I2C_1_MUX_1:
> * All that have dont_use_port_3 set.
> * CX231XX_BOARD_HAU
Dear Purchasing Manage,
Good morning., How are you?
As knowing that you are in the LED lighting market . Now I am writing to
establish business relationship with you . The following RGB flood light,
sensor flood lighting has been shipped to Germany , Switzerland, Romania ,
Russian , France ,
On 10/30/2014 06:38 AM, Nibble Max wrote:
- if (tab_len > 83) {
+ if (tab_len > 86) {
That is not nice, but I could try find better solution and fix it later.
What is the reason to check this parameter?
How about remove this check?
It is just to check you will not overwrite b
Acked-by: Antti Palosaari
Reviewed-by: Antti Palosaari
Antti
On 10/30/2014 12:43 PM, Olli Salonen wrote:
Set the property for delivery system also in case of SYS_DVBC_ANNEX_B. This
behaviour is observed in the sniffs taken with Hauppauge HVR-955Q Windows
driver.
Signed-off-by: Olli Salonen
As pointed by smatch:
drivers/media/pci/cx23885/cx23885-dvb.c:1066 dvb_register() error: we
previously assumed 'fe0->dvb.frontend' could be null (see line 1060)
drivers/media/pci/cx23885/cx23885-dvb.c:1990 cx23885_dvb_register()
error: we previously assumed 'fe0' could be null (se
On Thu, Oct 30, 2014 at 9:37 AM, Mauro Carvalho Chehab
wrote:
> Yeah, if nobody is using such devices, then we should get rid of them,
> but I prefer to have this on a separate patch, in order to give
> people the opportunity to complain.
Yes, I would definitely suggest a separate patch. There's
Now that all platforms instantiate the VSP1 through DT, platform data
support isn't needed anymore.
Signed-off-by: Laurent Pinchart
---
drivers/media/platform/Kconfig | 2 +-
drivers/media/platform/vsp1/vsp1.h | 14 +-
drivers/media/platform/vsp1/vsp1_drv.c | 81
Hi Devin,
Em Thu, 30 Oct 2014 09:15:31 -0400
Devin Heitmueller escreveu:
> Hi Mauro,
>
> > Syncronize it and put them on the same order as found at au0828
> > driver, as all the au0828 devices with analog TV need the
> > same quirks.
>
> The MXL and Woodbury boards don't support analog under L
On Oct 30 Mauro Carvalho Chehab wrote:
> on avr32 arch, those warnings happen:
> drivers/media/firewire/firedtv-fw.c: In function 'node_update':
> drivers/media/firewire/firedtv-fw.c:329: warning: comparison is always
> true due to limited range of data type
>
> In this particular cas
On Sat, Oct 25, 2014 at 4:17 PM, Michael Krufky wrote:
> From: Richard Vollkommer
>
> Adds control of the IF output level to the xc5000 tuner
> configuration structure. Increases the IF level to the
> demodulator to fix failure to lock and picture breakup
> issues (with the au8522 demodulator, i
> Well, perhaps you could add a printk message warning the user that
> the driver is not using the latest firmware and performance/quality
> could be badly affected.
I wouldn't use the term "badly affected". There are tens of thousands
of units out there for which users are quite happy with the c
Hi Mauro,
> Syncronize it and put them on the same order as found at au0828
> driver, as all the au0828 devices with analog TV need the
> same quirks.
The MXL and Woodbury boards don't support analog under Linux, so
probably shouldn't be included in the list of quirks.
That said, the MXL and Woo
Em Fri, 26 Sep 2014 18:11:45 -0700
Behan Webster escreveu:
> Replace the use of nested functions where a normal function will suffice.
>
> Nested functions are not liked by upstream kernel developers in general. Their
> use breaks the use of clang as a compiler, and doesn't make the code any
> b
There are several Auvitek au0828 listed at sound quirks. However,
the table there is incomplete. Simplify the table using a macro
for au0828 devices and update it to reflect the devices currently
-
v2: Fix SOB, au0828 driver reference and add a warning at au0828-cards.c.
Mauro Carvalho Chehab (2
Add a macro to simplify au0828 quirk table. That makes easier
to check it against the USB IDs at drivers/media/usb/au0828/au0828-cards.c.
Cc: sta...@vger.kernel.org
Signed-off-by: Mauro Carvalho Chehab
diff --git a/drivers/media/usb/au0828/au0828-cards.c
b/drivers/media/usb/au0828/au0828-cards.
The au0828 quirks table is currently not in sync with the au0828
media driver.
Syncronize it and put them on the same order as found at au0828
driver, as all the au0828 devices with analog TV need the
same quirks.
Cc: sta...@vger.kernel.org
Signed-off-by: Mauro Carvalho Chehab
diff --git a/soun
Em Thu, 30 Oct 2014 12:16:13 +0100
Clemens Ladisch escreveu:
> Mauro Carvalho Chehab wrote:
> > Add a macro to simplify au0828 quirk table. That makes easier
> > to check it against the USB IDs at drivers/media/usb/au0828-card.c
> >
> > +++ b/sound/usb/quirks-table.h
> > ...
> > + * This should b
Mauro Carvalho Chehab wrote:
> Add a macro to simplify au0828 quirk table. That makes easier
> to check it against the USB IDs at drivers/media/usb/au0828-card.c
>
> +++ b/sound/usb/quirks-table.h
> ...
> + * This should be kept in sync with drivers/media/usb/au0828-card.c
The file does not exist
Em Thu, 30 Oct 2014 12:09:07 +0100
Takashi Iwai escreveu:
> At Thu, 30 Oct 2014 08:53:04 -0200,
> Mauro Carvalho Chehab wrote:
> >
> > From: Mauro Carvalho Chehab
> >
> > Add a macro to simplify au0828 quirk table. That makes easier
> > to check it against the USB IDs at drivers/media/usb/au08
At Thu, 30 Oct 2014 08:53:04 -0200,
Mauro Carvalho Chehab wrote:
>
> From: Mauro Carvalho Chehab
>
> Add a macro to simplify au0828 quirk table. That makes easier
> to check it against the USB IDs at drivers/media/usb/au0828-card.c
>
> Cc: sta...@vger.kernel.org
> Signed-off-by: Mauro Carvalho
Hi Olli,
Em Thu, 30 Oct 2014 12:32:33 +0200
Olli Salonen escreveu:
> On Thu, 30 Oct 2014, Mauro Carvalho Chehab wrote:
>
> > Ah, ok. Are you planning to submit a patch for it, and the patches adding
> > support for HVR-955Q?
>
> I can submit a patch for that, no problem.
Yes, please do that.
From: Mauro Carvalho Chehab
The au0828 quirks table is currently not in sync with the au0828
media driver.
Syncronize it and put them on the same order as found at au0828
driver, as all the au0828 devices with analog TV need the
same quirks.
Cc: sta...@vger.kernel.org
Signed-off-by: Mauro Carva
There are several Auvitek au0828 listed at sound quirks. However,
the table there is incomplete. Simplify the table using a macro
for au0828 devices and update it to reflect the devices currently
supported at the Linux Kernel by drivers/media/usb/au0828 driver.
Mauro Carvalho Chehab (2):
[media]
From: Mauro Carvalho Chehab
Add a macro to simplify au0828 quirk table. That makes easier
to check it against the USB IDs at drivers/media/usb/au0828-card.c
Cc: sta...@vger.kernel.org
Signed-off-by: Mauro Carvalho Chehab
Signed-off-by: Mauro Carvalho Chehab
diff --git a/sound/usb/quirks-table
Set the property for delivery system also in case of SYS_DVBC_ANNEX_B. This
behaviour is observed in the sniffs taken with Hauppauge HVR-955Q Windows
driver.
Signed-off-by: Olli Salonen
---
drivers/media/tuners/si2157.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/tuner
On 2014-10-30 10:54, Mauro Carvalho Chehab wrote:
changeset e87b540be2dd broke RC5-SZ decoding, as it forgot to add
the extra bit check for the enabled protocols at the beginning of
the logic.
Signed-off-by: Mauro Carvalho Chehab
Acked-by: David Härdeman
diff --git a/drivers/media/rc/ir-rc
On Thu, 30 Oct 2014, Mauro Carvalho Chehab wrote:
Ah, ok. Are you planning to submit a patch for it, and the patches adding
support for HVR-955Q?
I can submit a patch for that, no problem. However, I'm not working with
HVR-955Q at the moment. I don't have access to ATSC/ClearQAM signal.
If
on avr32 arch, those warnings happen:
drivers/media/firewire/firedtv-fw.c: In function 'node_update':
drivers/media/firewire/firedtv-fw.c:329: warning: comparison is always
true due to limited range of data type
In this particular case, the signal is desired, as the isochannel
var
dib7000p is for DVB, and not ISDB. So, there's no layer.
That removes this compilation warning:
drivers/media/dvb-frontends/dib7000p.c:1972: warning: 'i' is used
uninitialized in this function
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-frontends/dib7000p.c | 9 -
Hello.
I have:
Bus 001 Device 004: ID 048d:9135 Integrated Technology Express, Inc.
Zolid Mini DVB-T Stick
This is well supported DVB-T USB card - used successfully with
dvb-usb-af9035 module.
Till now.
After every kernel update I have to rebuild v4l driver.
After last time (with kernel 3.
changeset e87b540be2dd broke RC5-SZ decoding, as it forgot to add
the extra bit check for the enabled protocols at the beginning of
the logic.
Signed-off-by: Mauro Carvalho Chehab
diff --git a/drivers/media/rc/ir-rc5-decoder.c
b/drivers/media/rc/ir-rc5-decoder.c
index 2ef763928ca4..84fa6e9b59a1
Em Thu, 30 Oct 2014 08:04:29 +0200
Olli Salonen escreveu:
> Hi Mauro,
>
> No, for ClearQAM the delivery_system should be set to 0x10 and this
> patch does not include that. At the time of submission of that patch I
> only had the trace from the ATSC case.
Ah, ok. Are you planning to submit a pa
v3:
-config demod mclk in "set_params" call back.
-remove "set_config".
v2:
-make demod mclk selection logic simple.
-merge demod mclk and ts mclk into one call back.
-make code clean.
M88RS6000 is the integrated chip, which includes tuner and demod.
Here splite its tuner as a standalone driver.
v3:
-reconstruct the codes in "set_frontend" to keep the same logic for M88RS6000
and M88DS3103.
-remove calling "set_config" of tuner to set demod mclk.
-demod mclk will be set in "set_params" of tuner call back.
v2:
-make demod mclk selection logic simple.
-merge demod mclk and ts mclk into one
Hi Kamil,
Can you make a patch to fix this sparse error?
/home/hans/work/build/media-git/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c:1178:25:
error: incompatible types in conditional expression (different base types)
s5p_mfc_hw_call should probably be s5p_mfc_hw_call_void.
Regards,
H
66 matches
Mail list logo
