From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Paul Bolle <pebolle@tiscali.nl>
Cc: Randy Dunlap <rdunlap@xenotime.net>,
linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro
Date: Thu, 03 Nov 2011 16:34:30 -0200 [thread overview]
Message-ID: <4EB2DEB6.7030508@infradead.org> (raw)
In-Reply-To: <1319995903.14409.42.camel@x61.thuisdomein>
Em 30-10-2011 15:31, Paul Bolle escreveu:
> On Sun, 2011-10-30 at 10:24 -0700, Randy Dunlap wrote:
>> On 10/30/11 10:12, Paul Bolle wrote:
>>> 2) I'm not sure why things are done that way. Why can't builtin drivers
>>> and loadable drivers default to identical values? But perhaps I'm just
>>> misunderstanding the code.
>>
>> They could default to identical values.
>
> That would make the cleaning up I'm trying to do now somewhat easier. It
> would allow to simplify the drivers a bit too.
>
>> Maybe someone thinks that
>> it's more difficult to pass parameters to builtin drivers so they
>> just try to use some sane defaults for them instead, whereas it's
>> easy (easier) to pass parameters to loadable modules. ??
>
> Perhaps Mauro or the people at linux-media know the reasoning here. Or
> they can show us that I didn't parse the code correctly, of course.
I can't remember the dirty details about this driver, sorry. The first
patch on it might shed some light:
commit 46a60cfef581307d8273919182ae939d44ff7cca
Author: Fabio Belavenuto <belavenuto@gmail.com>
Date: Tue Dec 30 19:27:09 2008 -0300
V4L/DVB (10155): Add TEA5764 radio driver
Add support for radio driver TEA5764 from NXP.
This chip is connected in pxa I2C bus in EZX phones
from Motorola, the chip is used in phone model A1200.
This driver is for OpenEZX project (www.openezx.org)
Tested with A1200 phone, openezx kernel and fm-tools
[mchehab@redhat.com: Fixed CodingStyle and solved some merge conflicts]
Signed-off-by: Fabio Belavenuto <belavenuto@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
>From the above, I _suspect_ that the default (whatever it is) is due to
the Motorola A1200 phone. Not sure if it is compiled as module or as builtin
at OpenEZX.
>
>
> Paul Bolle
>
> --
> 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
prev parent reply other threads:[~2011-11-03 18:34 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-30 12:08 [PATCH] [RESEND] media: tea5764: reconcile Kconfig symbol and macro Paul Bolle
2011-10-30 16:43 ` Randy Dunlap
2011-10-30 17:12 ` Paul Bolle
2011-10-30 17:24 ` Randy Dunlap
[not found] ` <1319995903.14409.42.camel@x61.thuisdomein>
2011-11-03 18:34 ` Mauro Carvalho Chehab [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=4EB2DEB6.7030508@infradead.org \
--to=mchehab@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=pebolle@tiscali.nl \
--cc=rdunlap@xenotime.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox