linux-kernel - Re: [RFC PATCH v1 2/2] powerpc/86xx: Introduce and use common dtsi

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1456785480.5360.60.camel@buserror.net>
Date:	Mon, 29 Feb 2016 16:38:00 -0600
From:	Scott Wood <oss@...error.net>
To:	Alessio Igor Bogani <alessio.bogani@...ttra.eu>,
	Kumar Gala <galak@...nel.crashing.org>
Cc:	linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH v1 2/2] powerpc/86xx: Introduce and use common dtsi

On Thu, 2016-02-25 at 11:57 +0100, Alessio Igor Bogani wrote:
> Signed-off-by: Alessio Igor Bogani <alessio.bogani@...ttra.eu>
> ---
>  arch/powerpc/boot/dts/fsl/gef_ppc9a.dts        | 284 +++-------------------
>  arch/powerpc/boot/dts/fsl/gef_sbc310.dts       | 276 +++------------------
>  arch/powerpc/boot/dts/fsl/gef_sbc610.dts       | 284 +++-------------------
>  arch/powerpc/boot/dts/fsl/mpc8641_hpcn.dts     | 318 ++++------------------
> ---
>  arch/powerpc/boot/dts/fsl/mpc8641_hpcn_36b.dts | 317 ++++------------------
> --
>  arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi  | 124 ++++++++++
>  arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi   |  59 +++++
>  arch/powerpc/boot/dts/fsl/sbc8641d.dts         | 314 +++-------------------
> --
>  8 files changed, 400 insertions(+), 1576 deletions(-)
>  create mode 100644 arch/powerpc/boot/dts/fsl/mpc8641si-post.dtsi
>  create mode 100644 arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi

Could you post a diff of what the decompiled trees look like before and after
this change (e.g. interrupts went from 2-cell to 4-cell)?  It is very hard to
review in this form.  Or better, like the config change, have a commit that
first makes changes to what the unified trees will be, and then a second
commit that generates the same output using includes.

> +/include/ "qoriq-mpic.dtsi"
> +	pic@...00 {
> +		compatible = "chrp,open-pic";
> +	};

This is removing the fsl,mpic compatible, which is required for 4-cell
interrupt specifiers.

> +&pci0 {
> +	compatible = "fsl,mpc8641-pcie";
> +	device_type = "pci";
> +	#size-cells = <2>;
> +	#address-cells = <3>;
> +	bus-range = <0x0 0xff>;
> +	clock-frequency = <33333333>;

The clock frequency of PCI Express is not 33 MHz.


> diff --git a/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi
> b/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi
> new file mode 100644
> index 0000000..026fa58
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/fsl/mpc8641si-pre.dtsi
> @@ -0,0 +1,59 @@
> +/*
> + * MPC8641 Silicon/SoC Device Tree Source (pre include)
> + *
> + * Copyright 2016 Elettra-Sincrotrone Trieste S.C.p.A.
> + *
> + * This program is free software; you can redistribute  it and/or modify it
> + * under  the terms of  the GNU General  Public License as published by the
> + * Free Software Foundation;  either version 2 of the  License, or (at your
> + * option) any later version.
> + *
> + */
> +
> +/dts-v1/;
> +
> +/ {
> +	compatible = "fsl,MPC8641";

This compatible is pointless -- it will be overwritten by the board
compatible.

-Scott