Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute

Path	csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From	Aaro Koskinen <aaro.koskinen@iki.fi>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute
Date	Mon, 15 Feb 2016 21:00:01 +0100
Message-ID	<r2xtn-5CC-5@gated-at.bofh.it> (permalink)
References	<r2jTs-4Xm-11@gated-at.bofh.it> <r2pma-aS-25@gated-at.bofh.it> <r2vBg-4if-9@gated-at.bofh.it> <r2wdZ-4Rn-29@gated-at.bofh.it> <r2x0m-5sb-19@gated-at.bofh.it> <r2xjI-5z4-9@gated-at.bofh.it> <r2xtn-5CC-7@gated-at.bofh.it>
X-Original-To	Heinrich Schuchardt <xypron.glpk@gmx.de>
MIME-Version	1.0
Content-Type	text/plain; charset=us-ascii
Content-Disposition	inline
User-Agent	Mutt/1.5.24 (2015-08-30)
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	42
Organization	linux.* mail to news gateway
X-Original-Cc	Arnd Bergmann <arnd@arndb.de>, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Ralf Baechle <ralf@linux-mips.org>, Andrew Bresticker <abrestic@chromium.org>, devicetree@vger.kernel.org, linux-mips@linux-mips.org, linux-kernel@vger.kernel.org
X-Original-Date	Mon, 15 Feb 2016 21:59:48 +0200
X-Original-Message-ID	<20160215195948.GF1640@darkstar.musicnaut.iki.fi>
X-Original-References	<1455513977-934-1-git-send-email-xypron.glpk@gmx.de> <56C1B3A0.4090301@cogentembedded.com> <56C21054.4070702@gmx.de> <20160215183838.GD1640@darkstar.musicnaut.iki.fi> <56C2253D.50101@gmx.de> <20160215194056.GE1640@darkstar.musicnaut.iki.fi> <56C22C18.5090608@gmx.de>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1334769

Show key headers only | View raw

Hi,

On Mon, Feb 15, 2016 at 08:50:48PM +0100, Heinrich Schuchardt wrote:
> On 02/15/2016 08:40 PM, Aaro Koskinen wrote:
> > On Mon, Feb 15, 2016 at 08:21:33PM +0100, Heinrich Schuchardt wrote:
> >> flash-kernel has a database /usr/share/flash-kernel/db/all.db with
> >> entries like:
> >>
> >> Machine: LeMaker Banana Pi
> >> Kernel-Flavors: armmp armmp-lpae
> >> Boot-Script-Path: /boot/boot.scr
> >> DTB-Id: sun7i-a20-bananapi.dtb
> >> U-Boot-Script-Name: bootscr.sunxi
> >> Required-Packages: u-boot-tools
> >>
> >> Machine refers to the value of /proc/device-tree/model.
> >> DTB-Id is the dtb to be installed.
> >>
> >> So what flash-kernel does is:
> >> - look up value of /proc/device-tree/model
> >> - retrieve correlated dtb file name from database
> >> - install dtb with this name
> >> - create symbolic links for the dtb
> >>
> >> If multiple boards use the the same dtb that is fine with flash-kernel
> >> as long as the value of model is unique per dtb.
> > 
> > OCTEON does not work like this. The file you are modifying
> > (octeon_3xxx.dts) is compiled into the kernel, and there is no external
> > DTB file. So the model string will be always the same regardless on
> > which board you have booted the kernel.
> 
> you are right DTBs are built in for MIPS systems.
> 
> Still it would be useful to be able to use the same property 'model' to
> determine which u-boot script (boot.scr) to install.

But that cannot be solved by adding a static model string to
octeon_3xxx.dts, because different OCTEON boards need different u-boot
commands (some boot from flash, some from mmc, some from USB, etc.).

A.

Thread

[PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Heinrich Schuchardt <xypron.glpk@gmx.de> - 2016-02-15 06:30 +0100
  Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> - 2016-02-15 12:20 +0100
    Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Arnd Bergmann <arnd@arndb.de> - 2016-02-15 12:30 +0100
    Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Heinrich Schuchardt <xypron.glpk@gmx.de> - 2016-02-15 19:00 +0100
      Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> - 2016-02-15 19:20 +0100
        [PATCH 1/1 v2] MIPS: DTS: cavium-octeon: provide model attribute Heinrich Schuchardt <xypron.glpk@gmx.de> - 2016-02-15 19:40 +0100
          Re: [PATCH 1/1 v2] MIPS: DTS: cavium-octeon: provide model attribute David Daney <ddaney.cavm@gmail.com> - 2016-02-17 01:30 +0100
      Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Aaro Koskinen <aaro.koskinen@iki.fi> - 2016-02-15 19:40 +0100
        Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Heinrich Schuchardt <xypron.glpk@gmx.de> - 2016-02-15 20:30 +0100
          Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Aaro Koskinen <aaro.koskinen@iki.fi> - 2016-02-15 20:50 +0100
            Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Aaro Koskinen <aaro.koskinen@iki.fi> - 2016-02-15 21:00 +0100
            Re: [PATCH 1/1] MIPS: DTS: cavium-octeon: provide model attribute Heinrich Schuchardt <xypron.glpk@gmx.de> - 2016-02-15 21:00 +0100

csiph-web