Re: [PATCH] regulators: fix typo in Kconfig and max8973-regulator

From: Mark Brown
Date: Wed Sep 15 2021 - 09:16:09 EST


On Wed, Sep 15, 2021 at 02:06:27PM +0300, Ramona Alexandra Nechita wrote:
> MAX8973 is supposed to be MAX8973A. Kconfig and the
> initial comment of max8973-regulator.c were modified
> accordingly.

It is?

> config REGULATOR_MAX8973

Why not update the Kconfig symbol as well? If there's a danger of
collisions that's probably the main thing people will be using.

> - tristate "Maxim MAX8973 voltage regulator "
> + tristate "Maxim MAX8973A voltage regulator "

There's an extra space there while you're fixing stuff up.

> @@ -1,7 +1,7 @@
> /*
> * max8973-regulator.c -- Maxim max8973
> *
> - * Regulator driver for MAXIM 8973 DC-DC step-down switching regulator.
> + * Regulator driver for MAXIM 8973A DC-DC step-down switching regulator.

What about the first line?

Attachment: signature.asc
Description: PGP signature