| Message ID | 20231018124007.1306159-3-sunilvl@ventanamicro.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | a06835227280436c1aae021a3f43d3abfcba3835 |
| Headers | show |
| Series | RISC-V: ACPI improvements | expand |
On Wed, Oct 18, 2023 at 06:10:05PM +0530, Sunil V L wrote: > acpi_get_rhct() currently returns pointer to acpi_table_header > structure. But since this is specific to RHCT, return pointer to > acpi_table_rhct structure itself. > > Suggested-by: Andrew Jones <ajones@ventanamicro.com> > Signed-off-by: Sunil V L <sunilvl@ventanamicro.com> > --- > drivers/acpi/riscv/rhct.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/acpi/riscv/rhct.c b/drivers/acpi/riscv/rhct.c > index b280b3e9c7d9..489b0e93b1e5 100644 > --- a/drivers/acpi/riscv/rhct.c > +++ b/drivers/acpi/riscv/rhct.c > @@ -9,7 +9,7 @@ > > #include <linux/acpi.h> > > -static struct acpi_table_header *acpi_get_rhct(void) > +static struct acpi_table_rhct *acpi_get_rhct(void) > { > static struct acpi_table_header *rhct; > acpi_status status; > @@ -26,7 +26,7 @@ static struct acpi_table_header *acpi_get_rhct(void) > } > } > > - return rhct; > + return (struct acpi_table_rhct *)rhct; > } > > /* > @@ -48,7 +48,7 @@ int acpi_get_riscv_isa(struct acpi_table_header *table, unsigned int cpu, const > BUG_ON(acpi_disabled); > > if (!table) { > - rhct = (struct acpi_table_rhct *)acpi_get_rhct(); > + rhct = acpi_get_rhct(); > if (!rhct) > return -ENOENT; > } else { > -- > 2.39.2 > Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
diff --git a/drivers/acpi/riscv/rhct.c b/drivers/acpi/riscv/rhct.c index b280b3e9c7d9..489b0e93b1e5 100644 --- a/drivers/acpi/riscv/rhct.c +++ b/drivers/acpi/riscv/rhct.c @@ -9,7 +9,7 @@ #include <linux/acpi.h> -static struct acpi_table_header *acpi_get_rhct(void) +static struct acpi_table_rhct *acpi_get_rhct(void) { static struct acpi_table_header *rhct; acpi_status status; @@ -26,7 +26,7 @@ static struct acpi_table_header *acpi_get_rhct(void) } } - return rhct; + return (struct acpi_table_rhct *)rhct; } /* @@ -48,7 +48,7 @@ int acpi_get_riscv_isa(struct acpi_table_header *table, unsigned int cpu, const BUG_ON(acpi_disabled); if (!table) { - rhct = (struct acpi_table_rhct *)acpi_get_rhct(); + rhct = acpi_get_rhct(); if (!rhct) return -ENOENT; } else {
acpi_get_rhct() currently returns pointer to acpi_table_header structure. But since this is specific to RHCT, return pointer to acpi_table_rhct structure itself. Suggested-by: Andrew Jones <ajones@ventanamicro.com> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com> --- drivers/acpi/riscv/rhct.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)