Re: [Qemu-devel] [PATCH] hw/ppc/spapr, e500: Use new property "stdout-path" for boot console

[David Gibson <david@gibson.dropbear.id.au>]

[-- Attachment #1: Type: text/plain, Size: 2526 bytes --]

On Thu, Mar 01, 2018 at 11:35:50AM +0530, Nikunj A Dadhania wrote:
> Linux kernel commit 2a9d832cc9aae21ea827520fef635b6c49a06c6d
> (of: Add bindings for chosen node, stdout-path) deprecated chosen property
> "linux,stdout-path" and "stdout".
> 
> Introduce the new property "stdout-path" and continue supporting the older
> property to remain compatible with existing/older firmware. This older property
> can be deprecated after 5 years.
> 
> Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>

Applied to ppc-for-2.12, thanks.

> ---
>  hw/ppc/e500.c  | 7 +++++++
>  hw/ppc/spapr.c | 7 +++++++
>  2 files changed, 14 insertions(+)
> 
> diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c
> index a40d3ec3e3..a325a95015 100644
> --- a/hw/ppc/e500.c
> +++ b/hw/ppc/e500.c
> @@ -119,7 +119,14 @@ static void dt_serial_create(void *fdt, unsigned long long offset,
>      qemu_fdt_setprop_string(fdt, "/aliases", alias, ser);
>  
>      if (defcon) {
> +        /*
> +         * "linux,stdout-path" and "stdout" properties are deprecated by linux
> +         * kernel. New platforms should only use the "stdout-path" property. Set
> +         * the new property and continue using older property to remain
> +         * compatible with the existing firmware.
> +         */
>          qemu_fdt_setprop_string(fdt, "/chosen", "linux,stdout-path", ser);
> +        qemu_fdt_setprop_string(fdt, "/chosen", "stdout-path", ser);
>      }
>  }
>  
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 83c9d66dd5..58a44edc4a 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -1062,7 +1062,14 @@ static void spapr_dt_chosen(sPAPRMachineState *spapr, void *fdt)
>      }
>  
>      if (!spapr->has_graphics && stdout_path) {
> +        /*
> +         * "linux,stdout-path" and "stdout" properties are deprecated by linux
> +         * kernel. New platforms should only use the "stdout-path" property. Set
> +         * the new property and continue using older property to remain
> +         * compatible with the existing firmware.
> +         */
>          _FDT(fdt_setprop_string(fdt, chosen, "linux,stdout-path", stdout_path));
> +        _FDT(fdt_setprop_string(fdt, chosen, "stdout-path", stdout_path));
>      }
>  
>      spapr_dt_ov5_platform_support(fdt, chosen);

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]