From: Antti Palosaari <crope@iki.fi>
To: Andy Walls <awalls@md.metrocast.net>
Cc: James <bjlockie@lockie.ca>,
linux-media Mailing List <linux-media@vger.kernel.org>
Subject: Re: boot slow down
Date: Mon, 06 Aug 2012 14:41:44 +0300 [thread overview]
Message-ID: <501FAD78.6090205@iki.fi> (raw)
In-Reply-To: <8a09e594-04b7-41b7-88a7-9877fb20810d@email.android.com>
On 08/06/2012 01:57 PM, Andy Walls wrote:
> Antti Palosaari <crope@iki.fi> wrote:
>
>> On 08/06/2012 11:37 AM, Andy Walls wrote:
>>> James <bjlockie@lockie.ca> wrote:
>>>
>>>> On 08/04/12 13:42, Andy Walls wrote:
>>>>> James <bjlockie@lockie.ca> wrote:
>>>>>
>>>>>> There's a big pause before the 'unable'
>>>>>>
>>>>>> [ 2.243856] usb 4-1: Manufacturer: Logitech
>>>>>> [ 62.739097] cx25840 6-0044: unable to open firmware
>>>>>> v4l-cx23885-avcore-01.fw
>>>>>>
>>>>>>
>>>>>> I have a cx23885
>>>>>> cx23885[0]: registered device video0 [v4l2]
>>>>>>
>>>>>> Is there any way to stop it from trying to load the firmware?
>>>>>> What is the firmware for, analog tv? Digital works fine and analog
>>>> is
>>>>>> useless to me.
>>>>>> I assume it is timing out there.
>>>>>> --
>>>>>> To unsubscribe from this list: send the line "unsubscribe
>>>> linux-media"
>>>>>> in
>>>>>> the body of a message to majordomo@vger.kernel.org
>>>>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>>>
>>>>> The firmware is for the analog broadcast audio standard (e.g. BTSC)
>>>> detection microcontroller.
>>>>>
>>>>> The A/V core of the CX23885/7/8 chips is for analog vidoe and audio
>>>> processing (broadcast, CVBS, SVideo, audio L/R in).
>>>>>
>>>>> The A/V core of the CX23885 provides the IR unit and the Video PLL
>>>> provides the timing for the IR unit.
>>>>>
>>>>> The A/V core of the CX23888 provides the Video PLL which is the
>>>> timing for the IR unit in the CX23888.
>>>>>
>>>>> Just grab the firmware and be done with it. Don't waste time with
>>>> trying to make the cx23885 working properly but halfway.
>>>>>
>>>>> Regards,
>>>>> Andy
>>>>
>>>> I already have the firmware.
>>>> # ls -l /lib/firmware/v4l-cx23885-avcore-01.fw
>>>> -rw-r--r-- 1 root root 16382 Oct 15 2011
>>>> /lib/firmware/v4l-cx23885-avcore-01.fw
>>>>
>>>>
>>>> --
>>>> To unsubscribe from this list: send the line "unsubscribe
>> linux-media"
>>>> in
>>>> the body of a message to majordomo@vger.kernel.org
>>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>
>>>
>>> Hmm. The firmware file size and location look right.
>>>
>>> The 60 second delay is the default kernel delay waiting for the
>> userspace firmware loader to fetch th file amd provide it to the kernel
>> via sysfs.
>>>
>>> That doesn't appear to be happening. I know udev runs some script to
>> accomplish that. I'm away from my development system, so I can't
>> investigate further.
>>>
>>> Regards,
>>> Andy
>>
>> I suspect it could be the firmware download issue with udev. Recent
>> udev
>> versions doesn't allow firmware download during module init path as
>> module init should not be blocked such long period.
>>
>> I did quite much work for resolving that issue for the dvb usb by
>> deferring device init in probe using work-queue. It is not good looking
>>
>> solution and Mauro is still trying to found out more general solution.
>>
>> regards
>> Antti
>>
>> --
>> http://palosaari.fi/
>
> So the cx25840 module scheduling the firmware load to be done by a worker thread is not enough to satisfy the new udev rule?
>
> http://git.linuxtv.org/media_tree.git/blob/staging/for_v3.7:/drivers/media/video/cx25840/cx25840-core.c#l628
>
> Regards,
> Andy
If it blocks module init, or probe which is called in module init
context, it timeouts by udev when request_firmware() is called.
There seems to be workqueue, but I am not enough familiar with cx25840
driver to say how code path actually runs.
I suspect cx25840_initialize() will not return until firmware is loaded?
And that function is called when module is loaded?
regards
Antti
--
http://palosaari.fi/
prev parent reply other threads:[~2012-08-06 11:42 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-04 15:52 boot slow down James
2012-08-04 17:05 ` Ezequiel Garcia
2012-08-04 17:42 ` Andy Walls
2012-08-04 22:21 ` James
2012-08-04 22:28 ` James
2012-08-05 21:20 ` Sakari Ailus
2012-08-06 4:30 ` James
2012-08-06 4:38 ` James
2012-08-07 11:27 ` Sakari Ailus
2012-08-07 13:04 ` bjlockie
2012-08-07 13:53 ` Andy Walls
2012-08-07 20:33 ` bjlockie
2012-08-07 21:42 ` James
2012-08-07 21:49 ` James
2012-08-08 8:24 ` Sakari Ailus
2012-08-08 17:18 ` bjlockie
2012-08-11 15:20 ` Sakari Ailus
2012-08-12 0:06 ` Andy Walls
2012-08-12 19:56 ` Ezequiel Garcia
2012-08-13 12:25 ` Mauro Carvalho Chehab
2012-08-07 21:40 ` James
2012-08-06 8:37 ` Andy Walls
2012-08-06 9:19 ` Antti Palosaari
2012-08-06 10:57 ` Andy Walls
2012-08-06 11:41 ` Antti Palosaari [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=501FAD78.6090205@iki.fi \
--to=crope@iki.fi \
--cc=awalls@md.metrocast.net \
--cc=bjlockie@lockie.ca \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.