From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nikita Kiryanov Date: Thu, 7 Jul 2016 11:53:41 +0300 Subject: [U-Boot] [PATCH 2/3] ARM: board: cm_fx6: fixup mtd partitions in the fdt In-Reply-To: <576ABA31.5040905@compulab.co.il> References: <20160619154456.29027-1-christopher.spinrath@rwth-aachen.de> <84159f3220054bfa87ce9a72443afa1a@rwthex-s1-b.rwth-ad.de> <576ABA31.5040905@compulab.co.il> Message-ID: <20160707085341.GB28953@arkadi-linux.compulab.local> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Jun 22, 2016 at 07:17:53PM +0300, Igor Grinberg wrote: > On 06/19/2016 06:44 PM, Christopher Spinrath wrote: > > The cm-fx6 module has an on-board st,m25p compatible spi flash chip > > used for u-boot (binary & environment). Overwrite the partitions in > > the device tree by the partition table provided in the mtdparts > > environment variable, if it is set. > > > > This allows to specify a kernel independent partitioning in the > > environment and provides a convient way for the user to adapt the > > partition table. > > > > Signed-off-by: Christopher Spinrath > > --- > > board/compulab/cm_fx6/cm_fx6.c | 16 +++++++++++++++- > > 1 file changed, 15 insertions(+), 1 deletion(-) > > > > diff --git a/board/compulab/cm_fx6/cm_fx6.c b/board/compulab/cm_fx6/cm_fx6.c > > index 712057a..81a7ae2 100644 > > --- a/board/compulab/cm_fx6/cm_fx6.c > > +++ b/board/compulab/cm_fx6/cm_fx6.c > > [...] > > > +#ifdef CONFIG_FDT_FIXUP_PARTITIONS > > +struct node_info nodes[] = { > > + { "st,m25p", MTD_DEV_TYPE_NOR, }, > > Nikita, is this enough for all flashes we assemble on cm-fx6? Yes, CM-FX6 is using M25PX16 and SST25VF016B, both of which are supported by the m25p80.c driver. However, on the mainline branch I don't see "m25p" in the list of device ids, and IIRC the request is to favor "jedec,spi-nor" as compatible string over device specific ones. > > > +}; > > +#endif > > + > > [...] > > -- > Regards, > Igor. >