From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from asavdk3.altibox.net ([109.247.116.14]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dUbFW-0006ig-Oz for barebox@lists.infradead.org; Mon, 10 Jul 2017 16:09:28 +0000 Date: Mon, 10 Jul 2017 18:09:02 +0200 From: Sam Ravnborg Message-ID: <20170710160902.GA24241@ravnborg.org> References: <20170710152108.23601-1-l.stach@pengutronix.de> <20170710152706.GA12108@ravnborg.org> <1499701357.18894.1.camel@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1499701357.18894.1.camel@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] ARM: imx: phyCORE i.MX7 on phyBOARD-Zeta support To: Lucas Stach Cc: barebox@lists.infradead.org On Mon, Jul 10, 2017 at 05:42:37PM +0200, Lucas Stach wrote: > Am Montag, den 10.07.2017, 17:27 +0200 schrieb Sam Ravnborg: > > Hi Lucas > > > > > index 000000000000..b1608dd9c730 > > > --- /dev/null > > > +++ b/arch/arm/boards/phytec-phycore-imx7/flash-header-phytec-phycore-imx7.imxcfg > > > @@ -0,0 +1,76 @@ > > > +/* > > > + * Copyright (C) 2016 NXP Semiconductors > > > + * > > > + * SPDX-License-Identifier: GPL-2.0 > > > + * > > > + * Refer docs/README.imxmage for more details about how-to configure > > > + * and create imximage boot image > > > + * > > > + * The syntax is taken as close as possible with the kwbimage > > > + */ > > > + > > > +soc imx7 > > > +loadaddr 0x80000000 > > > +dcdofs 0x400 > > > + > > > +wm 32 0x30340004 0x4F400005 > > > +/* Clear then set bit30 to ensure exit from DDR retention */ > > > +wm 32 0x30360388 0x40000000 > > > +wm 32 0x30360384 0x40000000 > > > + > > > +wm 32 0x30391000 0x00000002 > > > +wm 32 0x307a0000 0x01040001 > > > > > > Uwe posted a patch the other day to replace all these > > constants with defines across the tree. > > Looks like a step backward to introduce a new file where this approach is not used. > > Uwes patch replaces raw values in the i.MX6 files (where we have defines > for the registers). This is a new i.MX7 board, where no such defines > exists for now. OK, I had missed that "little" detail. > > I'm not saying that we shouldn't try to do the same thing for i.MX7, but > it is considerably more work than for MX6. Nice janitorial task for someone. Sam _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox