* [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning
@ 2021-10-29 0:58 frowand.list
2021-10-29 2:07 ` Rob Herring
0 siblings, 1 reply; 5+ messages in thread
From: frowand.list @ 2021-10-29 0:58 UTC (permalink / raw)
To: Rob Herring; +Cc: devicetree, linux-kernel
From: Frank Rowand <frank.rowand@sony.com>
Fix kernel build warning:
drivers/of/unittest-data/tests-interrupts.dtsi:32.26-35.6: Warning (interrupt_map): /testcase-data/interrupts/intmap1: Missing '#address-cells' in interrupt-map provider
A recently implemented dtc compiler warning reported the dts problem.
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
---
drivers/of/unittest-data/tests-interrupts.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/of/unittest-data/tests-interrupts.dtsi b/drivers/of/unittest-data/tests-interrupts.dtsi
index 9b60a549f502..8c2b91b998aa 100644
--- a/drivers/of/unittest-data/tests-interrupts.dtsi
+++ b/drivers/of/unittest-data/tests-interrupts.dtsi
@@ -31,6 +31,7 @@ test_intmap0: intmap0 {
test_intmap1: intmap1 {
#interrupt-cells = <2>;
+ #address-cells = <1>;
interrupt-map = <0x5000 1 2 &test_intc0 15>;
};
--
Frank Rowand <frank.rowand@sony.com>
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning
2021-10-29 0:58 [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning frowand.list
@ 2021-10-29 2:07 ` Rob Herring
2021-10-29 3:07 ` Frank Rowand
0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2021-10-29 2:07 UTC (permalink / raw)
To: Frank Rowand; +Cc: devicetree, linux-kernel@vger.kernel.org
On Thu, Oct 28, 2021 at 7:58 PM <frowand.list@gmail.com> wrote:
>
> From: Frank Rowand <frank.rowand@sony.com>
>
> Fix kernel build warning:
> drivers/of/unittest-data/tests-interrupts.dtsi:32.26-35.6: Warning (interrupt_map): /testcase-data/interrupts/intmap1: Missing '#address-cells' in interrupt-map provider
>
> A recently implemented dtc compiler warning reported the dts problem.
>
> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
> ---
> drivers/of/unittest-data/tests-interrupts.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/of/unittest-data/tests-interrupts.dtsi b/drivers/of/unittest-data/tests-interrupts.dtsi
> index 9b60a549f502..8c2b91b998aa 100644
> --- a/drivers/of/unittest-data/tests-interrupts.dtsi
> +++ b/drivers/of/unittest-data/tests-interrupts.dtsi
> @@ -31,6 +31,7 @@ test_intmap0: intmap0 {
>
> test_intmap1: intmap1 {
> #interrupt-cells = <2>;
> + #address-cells = <1>;
Notice that we have 2 nodes with interrupt-map here. One has
'#address-cells' and one doesn't. Why? Because we need to test that
the code can handle both cases.
The dtc warnings are more what should 'new' users do. I don't know
what DTs don't have #address-cells, but my guess is ancient ones.
Rob
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning
2021-10-29 2:07 ` Rob Herring
@ 2021-10-29 3:07 ` Frank Rowand
2021-10-29 13:16 ` Rob Herring
0 siblings, 1 reply; 5+ messages in thread
From: Frank Rowand @ 2021-10-29 3:07 UTC (permalink / raw)
To: Rob Herring; +Cc: devicetree, linux-kernel@vger.kernel.org
On 10/28/21 9:07 PM, Rob Herring wrote:
> On Thu, Oct 28, 2021 at 7:58 PM <frowand.list@gmail.com> wrote:
>>
>> From: Frank Rowand <frank.rowand@sony.com>
>>
>> Fix kernel build warning:
>> drivers/of/unittest-data/tests-interrupts.dtsi:32.26-35.6: Warning (interrupt_map): /testcase-data/interrupts/intmap1: Missing '#address-cells' in interrupt-map provider
>>
>> A recently implemented dtc compiler warning reported the dts problem.
>>
>> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
>> ---
>> drivers/of/unittest-data/tests-interrupts.dtsi | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/of/unittest-data/tests-interrupts.dtsi b/drivers/of/unittest-data/tests-interrupts.dtsi
>> index 9b60a549f502..8c2b91b998aa 100644
>> --- a/drivers/of/unittest-data/tests-interrupts.dtsi
>> +++ b/drivers/of/unittest-data/tests-interrupts.dtsi
>> @@ -31,6 +31,7 @@ test_intmap0: intmap0 {
>>
>> test_intmap1: intmap1 {
>> #interrupt-cells = <2>;
>> + #address-cells = <1>;
>
> Notice that we have 2 nodes with interrupt-map here. One has
> '#address-cells' and one doesn't. Why? Because we need to test that
> the code can handle both cases.>
> The dtc warnings are more what should 'new' users do. I don't know
> what DTs don't have #address-cells, but my guess is ancient ones.
>
> Rob
>
I had hoped to build all of the .dts files in the Linux tree, with the
new dtc, but did not get to that today. That should flush out any
cases that would result in build fail from the new approach of treating
all warnings as errors. I may get to that tomorrow.
If there any any existing .dts files that will trigger the interrupt
map warning, will we require that they be fixed so that the build will
not fail?
-Frank
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning
2021-10-29 3:07 ` Frank Rowand
@ 2021-10-29 13:16 ` Rob Herring
2021-10-29 17:33 ` Frank Rowand
0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2021-10-29 13:16 UTC (permalink / raw)
To: Frank Rowand; +Cc: devicetree, linux-kernel@vger.kernel.org
On Thu, Oct 28, 2021 at 10:07 PM Frank Rowand <frowand.list@gmail.com> wrote:
>
> On 10/28/21 9:07 PM, Rob Herring wrote:
> > On Thu, Oct 28, 2021 at 7:58 PM <frowand.list@gmail.com> wrote:
> >>
> >> From: Frank Rowand <frank.rowand@sony.com>
> >>
> >> Fix kernel build warning:
> >> drivers/of/unittest-data/tests-interrupts.dtsi:32.26-35.6: Warning (interrupt_map): /testcase-data/interrupts/intmap1: Missing '#address-cells' in interrupt-map provider
> >>
> >> A recently implemented dtc compiler warning reported the dts problem.
> >>
> >> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
> >> ---
> >> drivers/of/unittest-data/tests-interrupts.dtsi | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/of/unittest-data/tests-interrupts.dtsi b/drivers/of/unittest-data/tests-interrupts.dtsi
> >> index 9b60a549f502..8c2b91b998aa 100644
> >> --- a/drivers/of/unittest-data/tests-interrupts.dtsi
> >> +++ b/drivers/of/unittest-data/tests-interrupts.dtsi
> >> @@ -31,6 +31,7 @@ test_intmap0: intmap0 {
> >>
> >> test_intmap1: intmap1 {
> >> #interrupt-cells = <2>;
> >> + #address-cells = <1>;
> >
> > Notice that we have 2 nodes with interrupt-map here. One has
> > '#address-cells' and one doesn't. Why? Because we need to test that
> > the code can handle both cases.>
> > The dtc warnings are more what should 'new' users do. I don't know
> > what DTs don't have #address-cells, but my guess is ancient ones.
> >
> > Rob
> >
>
> I had hoped to build all of the .dts files in the Linux tree, with the
> new dtc, but did not get to that today. That should flush out any
> cases that would result in build fail from the new approach of treating
> all warnings as errors. I may get to that tomorrow.
They are still just warnings. You mean the requirement to be warning
free? That's not new for dts files.
> If there any any existing .dts files that will trigger the interrupt
> map warning, will we require that they be fixed so that the build will
> not fail?
I already submitted patches for them.
Rob
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning
2021-10-29 13:16 ` Rob Herring
@ 2021-10-29 17:33 ` Frank Rowand
0 siblings, 0 replies; 5+ messages in thread
From: Frank Rowand @ 2021-10-29 17:33 UTC (permalink / raw)
To: Rob Herring; +Cc: devicetree, linux-kernel@vger.kernel.org
On 10/29/21 8:16 AM, Rob Herring wrote:
> On Thu, Oct 28, 2021 at 10:07 PM Frank Rowand <frowand.list@gmail.com> wrote:
>>
>> On 10/28/21 9:07 PM, Rob Herring wrote:
>>> On Thu, Oct 28, 2021 at 7:58 PM <frowand.list@gmail.com> wrote:
>>>>
>>>> From: Frank Rowand <frank.rowand@sony.com>
>>>>
>>>> Fix kernel build warning:
>>>> drivers/of/unittest-data/tests-interrupts.dtsi:32.26-35.6: Warning (interrupt_map): /testcase-data/interrupts/intmap1: Missing '#address-cells' in interrupt-map provider
>>>>
>>>> A recently implemented dtc compiler warning reported the dts problem.
>>>>
>>>> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
>>>> ---
>>>> drivers/of/unittest-data/tests-interrupts.dtsi | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/of/unittest-data/tests-interrupts.dtsi b/drivers/of/unittest-data/tests-interrupts.dtsi
>>>> index 9b60a549f502..8c2b91b998aa 100644
>>>> --- a/drivers/of/unittest-data/tests-interrupts.dtsi
>>>> +++ b/drivers/of/unittest-data/tests-interrupts.dtsi
>>>> @@ -31,6 +31,7 @@ test_intmap0: intmap0 {
>>>>
>>>> test_intmap1: intmap1 {
>>>> #interrupt-cells = <2>;
>>>> + #address-cells = <1>;
>>>
>>> Notice that we have 2 nodes with interrupt-map here. One has
>>> '#address-cells' and one doesn't. Why? Because we need to test that
>>> the code can handle both cases.>
>>> The dtc warnings are more what should 'new' users do. I don't know
>>> what DTs don't have #address-cells, but my guess is ancient ones.
>>>
>>> Rob
>>>
>>
>> I had hoped to build all of the .dts files in the Linux tree, with the
>> new dtc, but did not get to that today. That should flush out any
>> cases that would result in build fail from the new approach of treating
>> all warnings as errors. I may get to that tomorrow.
>
> They are still just warnings. You mean the requirement to be warning
> free? That's not new for dts files.
>
>> If there any any existing .dts files that will trigger the interrupt
>> map warning, will we require that they be fixed so that the build will
>> not fail?
>
> I already submitted patches for them.
OK, I'll drop this patchk, and instead create a patch that documents
why the unittest .dts is missing #address-cells.
-Frank
>
> Rob
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-10-29 17:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-29 0:58 [PATCH 1/1] of: unittest: fix dts for interrupt-map provider build warning frowand.list
2021-10-29 2:07 ` Rob Herring
2021-10-29 3:07 ` Frank Rowand
2021-10-29 13:16 ` Rob Herring
2021-10-29 17:33 ` Frank Rowand
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).