Re: [PATCH v14 1/5] x86/boot: Introduce get_acpi_rsdp() to parse RSDP in cmdline from KEXEC

From: Ingo Molnar
Date: Mon Dec 17 2018 - 13:48:06 EST



* Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:

> On Mon, 17 Dec 2018, Ingo Molnar wrote:
> > * Chao Fan <fanc.fnst@xxxxxxxxxxxxxx> wrote:
> > > + val[len] = '\0';
> > > + return (acpi_physical_address)simple_strtoull(val, &e, 16);
> >
> > 'return' is not a function - no need for the parenthesis.
>
> It's a typecast which better has parenthesis....

Yeah, see my followup to the other reply where I withdraw the objection
and point out the ugliness of the various typecasts.

Thanks,

Ingo