From: Peter Schneider <pschneider1968@googlemail.com>
To: Mario Limonciello <superm1@kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
linux-pm@vger.kernel.org
Cc: "Linus Torvalds" <torvalds@linux-foundation.org>,
haojian.zhuang@marvell.com, mario.limonciello@amd.com,
"Thomas Gleixner" <tglx@linutronix.de>,
jirislaby@kernel.org,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Lee Jones" <lee@kernel.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
Subject: Re: Build error in 6.16-rc1 in drivers/mfd/88pm860x-core.c
Date: Mon, 9 Jun 2025 04:58:51 +0200 [thread overview]
Message-ID: <b765500b-cfde-45a8-8f7e-eead57db7dcf@googlemail.com> (raw)
In-Reply-To: <f656b948-aa3a-4811-b51f-64effa2001fc@kernel.org>
Am 09.06.2025 um 04:31 schrieb Mario Limonciello:
>
> On 6/8/25 8:58 PM, Peter Schneider wrote:
>>
>> I'm not a Kernel developer nor a C programmer, I only have superficial knowledge of C
>> (but I love to test and to tinker), so I don't quite understand this warning/error
>> message. Becauses some lines below, the node variable is used as a paramter to the
>> of_fwnode_handle macro (from include/linux/of.h):
>>
>> irq_domain_create_legacy(of_fwnode_handle(node), nr_irqs, chip- >irq_base, 0,
>> &pm860x_irq_domain_ops, chip);
>>
>> Does that not count as variable usage to the compiler?
> It's not because there is a macro when CONFIG_OF is not defined that assigns it as NULL.
> So to the compiler there is no variable use.
Ah, thanks for the explanation! I didn't quite see and grasp the nested #ifdefs depending
on CONFIG_OF and the alternate macro definition for of_fwnode_handle when CONFIG_OF is not
set. Now as I do, it makes perfect sense.
Beste Grüße,
Peter Schneider
--
Climb the mountain not to plant your flag, but to embrace the challenge,
enjoy the air and behold the view. Climb it so you can see the world,
not so the world can see you. -- David McCullough Jr.
OpenPGP: 0xA3828BD796CCE11A8CADE8866E3A92C92C3FF244
Download: https://www.peters-netzplatz.de/download/pschneider1968_pub.asc
https://keys.mailvelope.com/pks/lookup?op=get&search=pschneider1968@googlemail.com
https://keys.mailvelope.com/pks/lookup?op=get&search=pschneider1968@gmail.com
prev parent reply other threads:[~2025-06-09 2:58 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-09 1:58 Build error in 6.16-rc1 in drivers/mfd/88pm860x-core.c Peter Schneider
2025-06-09 2:31 ` Mario Limonciello
2025-06-09 2:58 ` Peter Schneider [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=b765500b-cfde-45a8-8f7e-eead57db7dcf@googlemail.com \
--to=pschneider1968@googlemail.com \
--cc=gregkh@linuxfoundation.org \
--cc=haojian.zhuang@marvell.com \
--cc=jirislaby@kernel.org \
--cc=lee@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mario.limonciello@amd.com \
--cc=superm1@kernel.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=u.kleine-koenig@pengutronix.de \
/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