* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
@ 2011-10-25 21:48 Olof Johansson
2011-10-25 22:00 ` Rob Herring
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2011-10-25 21:48 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
Looks like Marc added new references to irq_start in his PPI patch,
and Rob removed it in parallel. Also, irq_offset is no longer
available. Looks like you need to respin your patch, Marc.
arch/arm/common/gic.c: In function 'gic_dist_init':
arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
this function)
arch/arm/common/gic.c:290: error: (Each undeclared identifier is
reported only once
arch/arm/common/gic.c:290: error: for each function it appears in.)
arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
named 'irq_offset'
The second error is because of the change from Paul that removes
module.h from irq.h:
arch/arm/common/gic.c: In function 'gic_init':
arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
in this function)
Looks odd that I should need to include module.h on my own just to use
irq defines. Paul?
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
2011-10-25 21:48 linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal Olof Johansson
@ 2011-10-25 22:00 ` Rob Herring
2011-10-25 22:01 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2011-10-25 22:00 UTC (permalink / raw)
To: linux-arm-kernel
Olof,
On 10/25/2011 04:48 PM, Olof Johansson wrote:
> Hi,
>
> Looks like Marc added new references to irq_start in his PPI patch,
> and Rob removed it in parallel. Also, irq_offset is no longer
> available. Looks like you need to respin your patch, Marc.
>
> arch/arm/common/gic.c: In function 'gic_dist_init':
> arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
> this function)
> arch/arm/common/gic.c:290: error: (Each undeclared identifier is
> reported only once
> arch/arm/common/gic.c:290: error: for each function it appears in.)
> arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
> named 'irq_offset'
I fixed this and sent a pull request yesterday to Arnd.
> The second error is because of the change from Paul that removes
> module.h from irq.h:
>
> arch/arm/common/gic.c: In function 'gic_init':
> arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
> in this function)
>
> Looks odd that I should need to include module.h on my own just to use
> irq defines. Paul?
>
THIS_MODULE is needed by irq.h itself in irq_alloc_descs, so it probably
needs to be added back.
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
2011-10-25 22:00 ` Rob Herring
@ 2011-10-25 22:01 ` Olof Johansson
2011-10-26 14:27 ` Paul Gortmaker
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2011-10-25 22:01 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 25, 2011 at 3:00 PM, Rob Herring <robherring2@gmail.com> wrote:
> Olof,
>
> On 10/25/2011 04:48 PM, Olof Johansson wrote:
>> Hi,
>>
>> Looks like Marc added new references to irq_start in his PPI patch,
>> and Rob removed it in parallel. Also, irq_offset is no longer
>> available. Looks like you need to respin your patch, Marc.
>>
>> arch/arm/common/gic.c: In function 'gic_dist_init':
>> arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
>> this function)
>> arch/arm/common/gic.c:290: error: (Each undeclared identifier is
>> reported only once
>> arch/arm/common/gic.c:290: error: for each function it appears in.)
>> arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
>> named 'irq_offset'
>
> I fixed this and sent a pull request yesterday to Arnd.
Ah, oops, forgot to check for already-posted-patches. :) Excellent.
>> The second error is because of the change from Paul that removes
>> module.h from irq.h:
>>
>> arch/arm/common/gic.c: In function 'gic_init':
>> arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
>> in this function)
>>
>> Looks odd that I should need to include module.h on my own just to use
>> irq defines. Paul?
>>
>
> THIS_MODULE is needed by irq.h itself in irq_alloc_descs, so it probably
> needs to be added back.
Well, the new export.h that defines THIS_MODULE probably needs to be
added to irq.h, yes.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
2011-10-25 22:01 ` Olof Johansson
@ 2011-10-26 14:27 ` Paul Gortmaker
2011-10-26 14:35 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Paul Gortmaker @ 2011-10-26 14:27 UTC (permalink / raw)
To: linux-arm-kernel
On 11-10-25 06:01 PM, Olof Johansson wrote:
> On Tue, Oct 25, 2011 at 3:00 PM, Rob Herring <robherring2@gmail.com> wrote:
>> Olof,
>>
>> On 10/25/2011 04:48 PM, Olof Johansson wrote:
>>> Hi,
>>>
>>> Looks like Marc added new references to irq_start in his PPI patch,
>>> and Rob removed it in parallel. Also, irq_offset is no longer
>>> available. Looks like you need to respin your patch, Marc.
>>>
>>> arch/arm/common/gic.c: In function 'gic_dist_init':
>>> arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
>>> this function)
>>> arch/arm/common/gic.c:290: error: (Each undeclared identifier is
>>> reported only once
>>> arch/arm/common/gic.c:290: error: for each function it appears in.)
>>> arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
>>> named 'irq_offset'
>>
>> I fixed this and sent a pull request yesterday to Arnd.
>
> Ah, oops, forgot to check for already-posted-patches. :) Excellent.
>
>
>>> The second error is because of the change from Paul that removes
>>> module.h from irq.h:
>>>
>>> arch/arm/common/gic.c: In function 'gic_init':
>>> arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
>>> in this function)
>>>
>>> Looks odd that I should need to include module.h on my own just to use
>>> irq defines. Paul?
>>>
>>
>> THIS_MODULE is needed by irq.h itself in irq_alloc_descs, so it probably
>> needs to be added back.
>
> Well, the new export.h that defines THIS_MODULE probably needs to be
> added to irq.h, yes.
No. The whole point of the commit was to avoid nested includes
as per the commit log:
Also convert the irq_alloc_descs variants to macros, since all
they really do is is call the __irq_alloc_descs primitive.
This avoids including export.h and no debug info is lost.
If your file is relating to modular usage such that it is a modular
provider of interrupt sources and hence uses "THIS_MODULE", then it
will need export.h in the future. And that macro has nothing whatsoever
to do with any irq defines; it is used all over the kernel to tie
structures back to a (possibly modular) code block, just as per the
use case here in irq.h is.
Thanks,
Paul.
>
>
> -Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
2011-10-26 14:27 ` Paul Gortmaker
@ 2011-10-26 14:35 ` Olof Johansson
2011-10-26 14:58 ` Paul Gortmaker
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2011-10-26 14:35 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 26, 2011 at 7:27 AM, Paul Gortmaker
<paul.gortmaker@windriver.com> wrote:
> On 11-10-25 06:01 PM, Olof Johansson wrote:
>> On Tue, Oct 25, 2011 at 3:00 PM, Rob Herring <robherring2@gmail.com> wrote:
>>> Olof,
>>>
>>> On 10/25/2011 04:48 PM, Olof Johansson wrote:
>>>> Hi,
>>>>
>>>> Looks like Marc added new references to irq_start in his PPI patch,
>>>> and Rob removed it in parallel. Also, irq_offset is no longer
>>>> available. Looks like you need to respin your patch, Marc.
>>>>
>>>> arch/arm/common/gic.c: In function 'gic_dist_init':
>>>> arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
>>>> this function)
>>>> arch/arm/common/gic.c:290: error: (Each undeclared identifier is
>>>> reported only once
>>>> arch/arm/common/gic.c:290: error: for each function it appears in.)
>>>> arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
>>>> named 'irq_offset'
>>>
>>> I fixed this and sent a pull request yesterday to Arnd.
>>
>> Ah, oops, forgot to check for already-posted-patches. :) Excellent.
>>
>>
>>>> The second error is because of the change from Paul that removes
>>>> module.h from irq.h:
>>>>
>>>> arch/arm/common/gic.c: In function 'gic_init':
>>>> arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
>>>> in this function)
>>>>
>>>> Looks odd that I should need to include module.h on my own just to use
>>>> irq defines. Paul?
>>>>
>>>
>>> THIS_MODULE is needed by irq.h itself in irq_alloc_descs, so it probably
>>> needs to be added back.
>>
>> Well, the new export.h that defines THIS_MODULE probably needs to be
>> added to irq.h, yes.
>
> No. ?The whole point of the commit was to avoid nested includes
> as per the commit log:
>
> ? ?Also convert the irq_alloc_descs variants to macros, since all
> ? ?they really do is is call the __irq_alloc_descs primitive.
> ? ?This avoids including export.h and no debug info is lost.
>
> If your file is relating to modular usage such that it is a modular
> provider of interrupt sources and hence uses "THIS_MODULE", then it
> will need export.h in the future. And that macro has nothing whatsoever
> to do with any irq defines; it is used all over the kernel to tie
> structures back to a (possibly modular) code block, just as per the
> use case here in irq.h is.
I strongly disagree.
Why should I have to include export.h to a file that is not ever going
to be built as a module just because I need to include irq.h? That's
nonsense. If irq.h needs it, it should include it on its own.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal
2011-10-26 14:35 ` Olof Johansson
@ 2011-10-26 14:58 ` Paul Gortmaker
0 siblings, 0 replies; 6+ messages in thread
From: Paul Gortmaker @ 2011-10-26 14:58 UTC (permalink / raw)
To: linux-arm-kernel
On 11-10-26 10:35 AM, Olof Johansson wrote:
> On Wed, Oct 26, 2011 at 7:27 AM, Paul Gortmaker
> <paul.gortmaker@windriver.com> wrote:
>> On 11-10-25 06:01 PM, Olof Johansson wrote:
>>> On Tue, Oct 25, 2011 at 3:00 PM, Rob Herring <robherring2@gmail.com> wrote:
>>>> Olof,
>>>>
>>>> On 10/25/2011 04:48 PM, Olof Johansson wrote:
>>>>> Hi,
>>>>>
>>>>> Looks like Marc added new references to irq_start in his PPI patch,
>>>>> and Rob removed it in parallel. Also, irq_offset is no longer
>>>>> available. Looks like you need to respin your patch, Marc.
>>>>>
>>>>> arch/arm/common/gic.c: In function 'gic_dist_init':
>>>>> arch/arm/common/gic.c:290: error: 'irq_start' undeclared (first use in
>>>>> this function)
>>>>> arch/arm/common/gic.c:290: error: (Each undeclared identifier is
>>>>> reported only once
>>>>> arch/arm/common/gic.c:290: error: for each function it appears in.)
>>>>> arch/arm/common/gic.c:296: error: 'struct gic_chip_data' has no member
>>>>> named 'irq_offset'
>>>>
>>>> I fixed this and sent a pull request yesterday to Arnd.
>>>
>>> Ah, oops, forgot to check for already-posted-patches. :) Excellent.
>>>
>>>
>>>>> The second error is because of the change from Paul that removes
>>>>> module.h from irq.h:
>>>>>
>>>>> arch/arm/common/gic.c: In function 'gic_init':
>>>>> arch/arm/common/gic.c:620: error: 'THIS_MODULE' undeclared (first use
>>>>> in this function)
>>>>>
>>>>> Looks odd that I should need to include module.h on my own just to use
>>>>> irq defines. Paul?
>>>>>
>>>>
>>>> THIS_MODULE is needed by irq.h itself in irq_alloc_descs, so it probably
>>>> needs to be added back.
>>>
>>> Well, the new export.h that defines THIS_MODULE probably needs to be
>>> added to irq.h, yes.
>>
>> No. The whole point of the commit was to avoid nested includes
>> as per the commit log:
>>
>> Also convert the irq_alloc_descs variants to macros, since all
>> they really do is is call the __irq_alloc_descs primitive.
>> This avoids including export.h and no debug info is lost.
>>
>> If your file is relating to modular usage such that it is a modular
>> provider of interrupt sources and hence uses "THIS_MODULE", then it
>> will need export.h in the future. And that macro has nothing whatsoever
>> to do with any irq defines; it is used all over the kernel to tie
>> structures back to a (possibly modular) code block, just as per the
>> use case here in irq.h is.
>
> I strongly disagree.
>
> Why should I have to include export.h to a file that is not ever going
> to be built as a module just because I need to include irq.h? That's
You won't. Unless you use irq_alloc_descs. Also export.h isn't just
for modules. It is for code that in some way is exporting infrastructure
to allow modular infrastructure to exist. So there is lots of files that
will include export.h that in turn are not modules themselves. That was
largely the point.
Thanks,
Paul.
> nonsense. If irq.h needs it, it should include it on its own.
>
>
> -Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-10-26 14:58 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-25 21:48 linux-next failure on tegra due to conflicts between PPI and irq domain patches and irq.h->module.h include removal Olof Johansson
2011-10-25 22:00 ` Rob Herring
2011-10-25 22:01 ` Olof Johansson
2011-10-26 14:27 ` Paul Gortmaker
2011-10-26 14:35 ` Olof Johansson
2011-10-26 14:58 ` Paul Gortmaker
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).