| Message ID | 20230327132512.84540-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | Mainlined, archived |
| Headers | show |
| Series | [v2,1/1] ACPI: SPCR: Amend indentation | expand |
On Mon, Mar 27, 2023 at 3:25 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > > 1) Remove unnecessary blank lines. > 2) Reformat one comment for consistency. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > v2: avoided changind double spaces (Rafael) > drivers/acpi/spcr.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/acpi/spcr.c b/drivers/acpi/spcr.c > index 01d29ebdb9c2..cd36a97b0ea2 100644 > --- a/drivers/acpi/spcr.c > +++ b/drivers/acpi/spcr.c > @@ -71,7 +71,6 @@ static bool xgene_8250_erratum_present(struct acpi_table_spcr *tb) > > /** > * acpi_parse_spcr() - parse ACPI SPCR table and add preferred console > - * > * @enable_earlycon: set up earlycon for the console specified by the table > * @enable_console: setup the console specified by the table. > * > @@ -82,7 +81,6 @@ static bool xgene_8250_erratum_present(struct acpi_table_spcr *tb) > * > * When CONFIG_ACPI_SPCR_TABLE is defined, this function should be called > * from arch initialization code as soon as the DT/ACPI decision is made. > - * > */ > int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) > { > @@ -97,9 +95,7 @@ int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) > if (acpi_disabled) > return -ENODEV; > > - status = acpi_get_table(ACPI_SIG_SPCR, 0, > - (struct acpi_table_header **)&table); > - > + status = acpi_get_table(ACPI_SIG_SPCR, 0, (struct acpi_table_header **)&table); > if (ACPI_FAILURE(status)) > return -ENOENT; > > @@ -202,7 +198,8 @@ int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) > if (xgene_8250_erratum_present(table)) { > iotype = "mmio32"; > > - /* for xgene v1 and v2 we don't know the clock rate of the > + /* > + * For xgene v1 and v2 we don't know the clock rate of the > * UART so don't attempt to change to the baud rate state > * in the table because driver cannot calculate the dividers > */ > -- Applied as 6.4 material, thanks!
diff --git a/drivers/acpi/spcr.c b/drivers/acpi/spcr.c index 01d29ebdb9c2..cd36a97b0ea2 100644 --- a/drivers/acpi/spcr.c +++ b/drivers/acpi/spcr.c @@ -71,7 +71,6 @@ static bool xgene_8250_erratum_present(struct acpi_table_spcr *tb) /** * acpi_parse_spcr() - parse ACPI SPCR table and add preferred console - * * @enable_earlycon: set up earlycon for the console specified by the table * @enable_console: setup the console specified by the table. * @@ -82,7 +81,6 @@ static bool xgene_8250_erratum_present(struct acpi_table_spcr *tb) * * When CONFIG_ACPI_SPCR_TABLE is defined, this function should be called * from arch initialization code as soon as the DT/ACPI decision is made. - * */ int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) { @@ -97,9 +95,7 @@ int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) if (acpi_disabled) return -ENODEV; - status = acpi_get_table(ACPI_SIG_SPCR, 0, - (struct acpi_table_header **)&table); - + status = acpi_get_table(ACPI_SIG_SPCR, 0, (struct acpi_table_header **)&table); if (ACPI_FAILURE(status)) return -ENOENT; @@ -202,7 +198,8 @@ int __init acpi_parse_spcr(bool enable_earlycon, bool enable_console) if (xgene_8250_erratum_present(table)) { iotype = "mmio32"; - /* for xgene v1 and v2 we don't know the clock rate of the + /* + * For xgene v1 and v2 we don't know the clock rate of the * UART so don't attempt to change to the baud rate state * in the table because driver cannot calculate the dividers */
1) Remove unnecessary blank lines. 2) Reformat one comment for consistency. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- v2: avoided changind double spaces (Rafael) drivers/acpi/spcr.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-)