On 02.04.2012 00:07, Hans-Frieder Vogt wrote:
Support of AVerMedia AVerTV HD Volar, with tuner MxL5007t, second version of
patch
(usage of clock_adc_lut instead of adc config variable)
Signed-off-by: Hans-Frieder Vogt<hfv...@gmx.net>
Patch does not apply.
wget -O - http://patchwork.linuxtv.org/patch/10536/mbox/ | git am -s
[...]
Applying: AF9035/AF9033/TUA9001 support for AverTV A867R (mxl5007t),
version 2
error: drivers/media/dvb/dvb-usb/af9033.c: does not exist in index
error: drivers/media/dvb/dvb-usb/af9033.h: does not exist in index
error: drivers/media/dvb/dvb-usb/af9033_priv.h: does not exist in index
How is it possible you have the af9033 demod driver inside dvb-usb
directory? Demod drivers are inside drivers/media/dvb/frontends/. DVB
USB interface drivers are inside drivers/media/dvb/dvb-usb/.
Now it looks still much better than first version.
Here are still some comments of quick visual review:
+ for (i = 0; i< ARRAY_SIZE(clock_adc_lut); i++) {
+ if (clock_adc_lut[i].clock == state->cfg.clock)
+ break;
+ }
+ if (i>= ARRAY_SIZE(clock_adc_lut)) {
+ ret = -EINVAL;
+ goto err;
+ }
That error check is useless in my understanding. It is never taken.
Likely some Kernel semantic error checker will report it later...
+ adc_freq = clock_adc_lut[i].adc;
- for (i = 0; i< af9035_properties[0].num_adapters; i++)
+ for (i = 0; i< af9035_properties[0].num_adapters; i++) {
af9035_af9033_config[i].clock = clock_lut[tmp];
+ }
No braces allowed here as it is single line. You did not ran
checkpatch.pl as it will report those.
Hans-Frieder, fix those findings and remember ran also Kernel
checkpatch.pl - it may report some more findings.
You can ran it like that:
git diff drivers/media/dvb/frontends/af9033.c | ./scripts/checkpatch.pl -
Or if you have already added those files as git add then you can:
git diff --cached | ./scripts/checkpatch.pl -
Good work still! Next try should be OK :)
regards
Antti
--
http://palosaari.fi/
--
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
