From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: mchehab@osg.samsung.com, linux-media@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/6] radio: Utilize the module_isa_driver macro
Date: Wed, 20 Jul 2016 08:45:25 -0400 [thread overview]
Message-ID: <20160720124502.GA1119@sophia> (raw)
In-Reply-To: <6e689553-b3bc-c5d1-1246-ff109d2bba17@xs4all.nl>
On Wed, Jul 20, 2016 at 12:37:31PM +0200, Hans Verkuil wrote:
>On 07/18/2016 04:45 PM, William Breathitt Gray wrote:
>> The module_isa_driver macro is a helper macro for ISA drivers which do
>> not do anything special in module init/exit. This patchset eliminates a
>> lot of ISA driver registration boilerplate code by utilizing
>> module_isa_driver, which replaces module_init and module_exit.
>>
>> William Breathitt Gray (6):
>> radio: terratec: Utilize the module_isa_driver macro
>> radio: rtrack2: Utilize the module_isa_driver macro
>> radio: trust: Utilize the module_isa_driver macro
>> radio: zoltrix: Utilize the module_isa_driver macro
>> radio: aztech: Utilize the module_isa_driver macro
>> radio: aimslab: Utilize the module_isa_driver macro
>
>Good idea, but it doesn't compile:
>
>module_isa_driver(terratec_driver.driver, 1);
>
>expands to:
>
>static int __init terratec_driver.driver_init(void)
>{
> return isa_register_driver(&(terratec_driver.driver), 1);
>}
>
>So now the function name contains a '.' and it won't compile.
>
>Regards,
>
> Hans
Oops, looks like I was a bit on autopilot there. I'll have to rethink
this patchset at a later point to overcome the symbol naming issue.
Thank you,
William Breathitt Gray
prev parent reply other threads:[~2016-07-20 12:45 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-18 14:45 [PATCH 0/6] radio: Utilize the module_isa_driver macro William Breathitt Gray
2016-07-18 14:45 ` [PATCH 1/6] radio: terratec: " William Breathitt Gray
2016-07-18 14:46 ` [PATCH 2/6] radio: rtrack2: " William Breathitt Gray
2016-07-18 14:46 ` [PATCH 3/6] radio: trust: " William Breathitt Gray
2016-07-18 14:46 ` [PATCH 4/6] radio: zoltrix: " William Breathitt Gray
2016-07-18 14:46 ` [PATCH 5/6] radio: aztech: " William Breathitt Gray
2016-07-18 14:47 ` [PATCH 6/6] radio: aimslab: " William Breathitt Gray
2016-07-20 10:37 ` [PATCH 0/6] radio: " Hans Verkuil
2016-07-20 12:45 ` William Breathitt Gray [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=20160720124502.GA1119@sophia \
--to=vilhelm.gray@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@osg.samsung.com \
/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.