Re: [GIT PULL FOR 3.3 RESEND] AF9015/AF9013 changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 30-12-2011 14:50, Mauro Carvalho Chehab wrote:
> On 24-12-2011 14:53, Antti Palosaari wrote:
>> Hello
>> I just looked latest for 3.3 and there was 3 patch missing I have already PULL requested.
>>
>> Could you PULL those ASAP from that tree:
>> http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/misc
>>
>> 2011-11-19 tda18218: fix 6 MHz default IF frequency
>> 2011-11-19 af9015: limit I2C access to keep FW happy
>> 2011-11-28 af9013: rewrite whole driver
> 
> Antti,
> 
> Please, don't send pull requests like that. They won't go to my queue, as patchwork
> won't get it.
> 
> For patchwork to get a git pull request, it has to contain the strings bellow:
> 	The following changes since commit 1a5cd29631a6b75e49e6ad8a770ab9d69cda0fa2:
> 
> 	[media] tda10021: Add support for DVB-C Annex C (2011-12-20 14:01:08 -0200)
> 
> 	are available in the git repository at:
> 	git://linuxtv.org/gliakhovetski/v4l-dvb.git for-3.3
> 
> (those are from a real example).
> 
> E. g. the pull request should be generated with git request-pull.
> 
> This time, I'll get it by hand. I'm assuming that you want me to pull from:
> 	git://linuxtv.org/anttip/media_tree.git misc
> 
> based on the URL you've sent me.

Still, the same problem as before:

drivers/media/dvb/dvb-usb/af9015.c: In function ‘af9015_rc_query’:
drivers/media/dvb/dvb-usb/af9015.c:1091:6: error: ‘adap’ undeclared (first use in this function)
drivers/media/dvb/dvb-usb/af9015.c:1091:6: note: each undeclared identifier is reported only once for each function it appears in
drivers/media/dvb/dvb-usb/af9015.c:1094:12: error: ‘struct af9015_state’ has no member named ‘sleep’
drivers/media/dvb/dvb-usb/af9015.c:1094:30: error: ‘fe’ undeclared (first use in this function)

As all patches that went to 3.1-rc7 were merged back, this means
that there's something else missing there (maybe that patch
you asked me to revert).

After digging for a while, it seems that this patch is dependent
on the one you asked me to revert:

Author: Antti Palosaari <crope@xxxxxx>
Date:   Sat Nov 12 22:33:30 2011 -0300

    [media] af9015: limit I2C access to keep FW happy
    
    AF9015 firmware does not like if it gets interrupted by I2C adapter
    request on some critical phases. During normal operation I2C adapter
    is used only 2nd demodulator and tuner on dual tuner devices.
    
    Override demodulator callbacks and use mutex for limit access to
    those "critical" paths to keep AF9015 happy.
    
    Signed-off-by: Antti Palosaari <crope@xxxxxx>
    Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>

Applying it before this patch made it to apply. Please check if
my merge conflict didn't break anything.

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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux