From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Date: Thu, 06 Jul 2017 12:44:09 +0300 Subject: [U-Boot] [PATCH v2 3/3] x86: Add Intel Edison board files In-Reply-To: References: <20170705205628.52620-1-andriy.shevchenko@linux.intel.com> <20170705205628.52620-4-andriy.shevchenko@linux.intel.com> Message-ID: <1499334249.22624.285.camel@linux.intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Thu, 2017-07-06 at 12:07 +0800, Bin Meng wrote: > Hi Andy, > > On Thu, Jul 6, 2017 at 4:56 AM, Andy Shevchenko > wrote: > > Add Intel Edison board which is using U-Boot. > Thanks for review, my answers below. > Thank you for your efforts to bring edition support upstream! Looks > quite clean. A few comments below. I hope to see it soon there! > > +config SYS_USB_OTG_BASE > > +       hex > > +       default 0xf9100000 > > Can this otg controller be put into device tree, like other > peripherals? I will check the possibility. > > diff --git a/board/intel/edison/edison_start.S > > b/board/intel/edison/edison_start.S > > This file should be renamed to just start.S Done. > > +U-boot is a main bootloader on Intel Edison board. > > U-Boot Fixed. > Are there instructions on how to program U-Boot on a board that does > not have pre-flashed U-Boot? or in a situation that users flashed a > bad image that makes the board brick. We assume that board is supplied with official image. There is official documentation how to "unbrick" board to the stock state. So, I consider it out of scope of U-Boot documentation. > > +/*----------------------------------------------------------------- > > ------ > > nits: /* Boot */ Fixed (all cases). -- Andy Shevchenko Intel Finland Oy