Re: [PATCH 2/2] x86/platform/geode: reference the real node of the cs5535 GPIO controller
From: Dave Hansen
Date: Thu May 21 2026 - 11:50:20 EST
On 4/29/26 05:23, Bartosz Golaszewski wrote:
> We now can access the address of the software node associated with the
> GPIO controller cell of the cs5535 MFD device. Make it the target of the
> GPIO software node references in geode-common.c. Make sure the cs5535
> driver is built-in for all boards selecting GEODE_COMMON.
This looks like a good cleanup, but the changelog has me scratching my
head a bit.
I'm not quite sure what problem this is solving. It's also a bit unusual
to have a driver depend on another not being built as a module. There's
also not really a mention of the:
+MODULE_IMPORT_NS("CS5535");
in the changelog.
Oh, and on a minor note, it might all be more clear if the changelog was
written in imperative voice and removed the "We".
Honestly, this is old hardware so I don't want to fret about this too
much. I'd be happy to ack this, but I don't really know what I'm acking. ;)