Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1564447 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible revisions for proper detection Paul Kocialkowski <contact@paulk.fr> - 2017-01-22 17:00 +0100
    Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible  revisions for proper detection Thierry Reding <thierry.reding@gmail.com> - 2017-01-25 12:10 +0100
      Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible  revisions for proper detection Thierry Reding <thierry.reding@gmail.com> - 2017-01-25 12:20 +0100
        Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible  revisions for proper detection Paul Kocialkowski <contact@paulk.fr> - 2017-01-25 12:30 +0100
      Re: [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible  revisions for proper detection Paul Kocialkowski <contact@paulk.fr> - 2017-01-25 12:20 +0100

#1564447 — [PATCH v2 3/6] ARM: tegra: nyan-big: Include compatible revisions for proper detection

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@paulk.fr>
---
 arch/arm/boot/dts/tegra124-nyan-big.dts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

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",
+			"google,nyan-big-rev1", "google,nyan-big-rev0",
+			"google,nyan-big", "google,nyan", "nvidia,tegra124";
 
 	panel: panel {
 		compatible = "auo,b133xtn01";
-- 
2.11.0

[toc] | [next] | [standalone]

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

[Multipart message — attachments visible in raw view] — view raw

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@paulk.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

[toc] | [prev] | [next] | [standalone]

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

[Multipart message — attachments visible in raw view] — view raw

On Wed, Jan 25, 2017 at 12:08:08PM +0100, Paul Kocialkowski wrote:
> Le mercredi 25 janvier 2017 à 12:02 +0100, Thierry Reding a écrit :
> > 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@paulk.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.
> 
> Oops, sorry I missed that last faulty line. Actually, it should have contained:
> "google,nyan-blaze-rev1", "google,nyan-blaze-rev0";
> 
> instead so that all revision, from 7 down to 0 are covered.
> 
> Should I make another patch on top to fix that?

Here's what I've put into Tegra's for-next, that has all the necessary
revisions, doesn't it?

	https://git.kernel.org/cgit/linux/kernel/git/tegra/linux.git/commit/?h=for-next&id=bfa3c0d4e8ba8eff525d34dbde0e76a98ebfbade

If so, no need for another patch.

Thierry

[toc] | [prev] | [next] | [standalone]

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

[Multipart message — attachments visible in raw view] — view raw

Le mercredi 25 janvier 2017 à 12:14 +0100, Thierry Reding a écrit :
> On Wed, Jan 25, 2017 at 12:08:08PM +0100, Paul Kocialkowski wrote:
> > Le mercredi 25 janvier 2017 à 12:02 +0100, Thierry Reding a écrit :
> > > 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@paulk.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.
> > 
> > Oops, sorry I missed that last faulty line. Actually, it should have
> > contained:
> > "google,nyan-blaze-rev1", "google,nyan-blaze-rev0";
> > 
> > instead so that all revision, from 7 down to 0 are covered.
> > 
> > Should I make another patch on top to fix that?
> 
> Here's what I've put into Tegra's for-next, that has all the necessary
> revisions, doesn't it?
> 
> 	https://git.kernel.org/cgit/linux/kernel/git/tegra/linux.git/commit/?h=f
> or-next&id=bfa3c0d4e8ba8eff525d34dbde0e76a98ebfbade
> 
> If so, no need for another patch.

Excellent, it's all there! Thanks a bunch.

-- 
Paul Kocialkowski, developer of free digital technology at the lower levels

Website: https://www.paulk.fr/
Coding blog: https://code.paulk.fr/
Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/

[toc] | [prev] | [next] | [standalone]

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

[Multipart message — attachments visible in raw view] — view raw

Le mercredi 25 janvier 2017 à 12:02 +0100, Thierry Reding a écrit :
> 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@paulk.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.

Oops, sorry I missed that last faulty line. Actually, it should have contained:
"google,nyan-blaze-rev1", "google,nyan-blaze-rev0";

instead so that all revision, from 7 down to 0 are covered.

Should I make another patch on top to fix that?

-- 
Paul Kocialkowski, developer of free digital technology at the lower levels

Website: https://www.paulk.fr/
Coding blog: https://code.paulk.fr/
Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web