linux-kernel - Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible revisions for proper detection

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: <20170125110246.GD9344@ulmo.ba.sec>
Date:   Wed, 25 Jan 2017 12:02:46 +0100
From:   Thierry Reding <thierry.reding@...il.com>
To:     Paul Kocialkowski <contact@...lk.fr>
Cc:     linux-tegra@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
        Jon Hunter <jonathanh@...dia.com>,
        Stephen Warren <swarren@...dotorg.org>,
        Alexandre Courbot <gnurou@...il.com>,
        Mikko Perttunen <cyndis@...si.fi>,
        Peter Rosin <peda@...ntia.se>
Subject: Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible
 revisions for proper detection

On Sun, Jan 22, 2017 at 04:47:50PM +0100, Paul Kocialkowski wrote:
> Depthcharge (the payload used with cros devices) will attempt to detect
> boards using their revision. This includes all the known revisions for
> the nyan-big board so that the dtb can be selected preferably.
> 
> Defining compatibly revisions allows depthcharge to select the kernel
> via the revision it detects instead of using the default kernel. This
> allows having a FIT image with multiple kernels for multiple devices.
> 
> Signed-off-by: Paul Kocialkowski <contact@...lk.fr>
> ---
>  arch/arm/boot/dts/tegra124-nyan-big.dts | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)

I've applied this, with a small change, see below.

> diff --git a/arch/arm/boot/dts/tegra124-nyan-big.dts b/arch/arm/boot/dts/tegra124-nyan-big.dts
> index 67d7cfb32541..f12ece9b76a6 100644
> --- a/arch/arm/boot/dts/tegra124-nyan-big.dts
> +++ b/arch/arm/boot/dts/tegra124-nyan-big.dts
> @@ -6,7 +6,12 @@
>  
>  / {
>  	model = "Acer Chromebook 13 CB5-311";
> -	compatible = "google,nyan-big", "nvidia,tegra124";
> +	compatible = "google,nyan-big-rev7", "google,nyan-big-rev6",
> +			"google,nyan-big-rev5", "google,nyan-big-rev4",
> +			"google,nyan-big-rev3", "google,nyan-big-rev2",
> +			"google,nyan-big-rev5", "google,nyan-big-rev4",

This line is the same as two lines above, so I've removed this one. I've
also used slightly different padding (two tabs and align to the first
string in the first line with spaces) to make it consistent with other
string lists in the file.

Thanks,
Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)