Messages in this thread | | | From | Rob Herring <> | Date | Thu, 9 Mar 2023 07:28:59 -0600 | Subject | Re: [PATCH v2 03/21] of: Rename of_modalias_node() |
| |
On Wed, Mar 8, 2023 at 4:06 PM Sebastian Reichel <sebastian.reichel@collabora.com> wrote: > > Hi, > > On Tue, Mar 07, 2023 at 06:19:03PM -0600, Rob Herring wrote: > > On Tue, Mar 07, 2023 at 05:53:41PM +0100, Miquel Raynal wrote: > > > This helper does not produce a real modalias, but tries to get the > > > "product" compatible part of the "vendor,product" compatibles only. It > > > is far from creating a purely useful modalias string and does not seem > > > to be used like that directly anyway, so let's try to give this helper a > > > more meaningful name before moving there a real modalias helper (already > > > existing under of/device.c). > > > > > > Also update the various documentations to refer to the strings as > > > "aliases" rather than "modaliases" which has a real meaning in the Linux > > > kernel. > > > > > > There is no functional change. > > > > > > Cc: Rafael J. Wysocki <rafael@kernel.org> > > > Cc: Len Brown <lenb@kernel.org> > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > > > Cc: Maxime Ripard <mripard@kernel.org> > > > Cc: Thomas Zimmermann <tzimmermann@suse.de> > > > Cc: Sebastian Reichel <sre@kernel.org> > > > Cc: Wolfram Sang <wsa@kernel.org> > > > Cc: Mark Brown <broonie@kernel.org> > > > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> > > > --- > > > drivers/acpi/bus.c | 7 ++++--- > > > drivers/gpu/drm/drm_mipi_dsi.c | 2 +- > > > drivers/hsi/hsi_core.c | 2 +- > > > > These should not have been using this function. The matching on just the > > product was a relic from I2C and SPI which we don't want to propogate. > > No clue why ACPI needed it... > > > > If you respin or want to fixup while applying, can you add a kerneldoc > > comment to not add new users of the function. Not that anyone will > > follow that... :( > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > I just checked and HSI is not using the data for matching. Instead > it uses the returned data to set the device name. Matching happens > using the full compatible. > > FWIW the MIPI HSI standard never became a big thing, so we have only > one HSI DT driver upstream and that is the Nokia N900 modem driver.
Can we add a patch in removing the call then.
I'm pretty sure MIPI stands for Must Invent Peripheral Interfaces.
Rob
| |