Re: [U-Boot] [PATCH 02/14] spl: mmc: Fix build without LIBCOMMON_SUPPORT
On 11/26/2016 09:45 PM, Tom Rini wrote: > On Fri, Nov 25, 2016 at 11:32:23PM +0100, Marek Vasut wrote: >> If CONFIG_SPL_LIBCOMMON_SUPPORT is undefined, the following error >> will happen, so fix it. >> >> In file included from common/spl/spl_mmc.c:11:0: >> common/spl/spl_mmc.c: In function ‘spl_mmc_load_image’: >> include/spl.h:18:30: error: label at end of compound statement >> #define MMCSD_MODE_UNDEFINED 0 >> ^ >> common/spl/spl_mmc.c:335:7: note: in expansion of macro >> ‘MMCSD_MODE_UNDEFINED’ >> case MMCSD_MODE_UNDEFINED: >>^ >> Signed-off-by: Marek Vasut>> Cc: Pantelis Antoniou >> Cc: Tom Rini >> --- >> common/spl/spl_mmc.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c >> index 0b681c2..43c1030 100644 >> --- a/common/spl/spl_mmc.c >> +++ b/common/spl/spl_mmc.c >> @@ -343,10 +343,12 @@ static int spl_mmc_load_image(struct spl_image_info >> *spl_image, >> >> break; >> case MMCSD_MODE_UNDEFINED: >> -#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT >> default: >> +{ >> +#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT >> puts("spl: mmc: wrong boot mode\n"); >> #endif >> +} >> } >> >> return err; > > I think we have something slightly off here. Is > CONFIG_SPL_SERIAL_SUPPORT also enabled? That would turn this into a > no-op as puts becomes a do { } while(0), if disabled. So I think we > should just remove the ifdef here. No, series does not fit into SPL on this platform. -- Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-rockchip (take 2)
On Fri, Nov 25, 2016 at 07:02:54PM -0700, Simon Glass wrote: > Hi Tom, > > I fixed up the checking of voltage and current to correct the test > failure. Sorry for not spotting this correctly. > > > The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32: > > MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-rockchip.git > > for you to fetch changes up to 6b388f0bed0e3d021128b4fc3c41da4572984e3b: > > rockchip: configs: correct partitions 'boot' size (2016-11-25 17:59:32 > -0700) > Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 04/14] serial: 16550: Add getfcr accessor
On Fri, Nov 25, 2016 at 11:32:25PM +0100, Marek Vasut wrote: > Add function which allows fetching the default FCR register setting > from platform data for DM , while retaining old behavior for non-DM > by returning UART_FCRVAL. > > Signed-off-by: Marek Vasut> Cc: Tom Rini > Cc: Simon Glass Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 14/14] mips: jz47xx: Add Creator CI20 platform
On Fri, Nov 25, 2016 at 11:32:35PM +0100, Marek Vasut wrote: > From: Paul Burton> > Add support for the Creator CI20 platform based on the JZ4780 SoC. > The DTS file comes from Linux 4.6 as of revision > 78800558d104e003f9ae92e0107f1de39cf9de9f > > So far, there are still a few details which will have to be fixed > once they are fleshed out in Linux: > - pinmux: Thus far, this board just pokes the pinmux registers to > set the pinmux. For MMC in SPL, this will have to stay. > But for full u-boot a proper pinmux driver will have to > be added once the pinmux semantics in DT are in mainline > Linux. > - ethernet,efuse: DT bindings are missing from mainline Linux. > > Signed-off-by: Marek Vasut > Cc: Daniel Schwierzeck > Cc: Paul Burton This needs to be rebased on top of master as there's a lot of config.h things that should be in the defconfig. -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 02/14] spl: mmc: Fix build without LIBCOMMON_SUPPORT
On Fri, Nov 25, 2016 at 11:32:23PM +0100, Marek Vasut wrote: > If CONFIG_SPL_LIBCOMMON_SUPPORT is undefined, the following error > will happen, so fix it. > > In file included from common/spl/spl_mmc.c:11:0: > common/spl/spl_mmc.c: In function ‘spl_mmc_load_image’: > include/spl.h:18:30: error: label at end of compound statement > #define MMCSD_MODE_UNDEFINED 0 > ^ > common/spl/spl_mmc.c:335:7: note: in expansion of macro ‘MMCSD_MODE_UNDEFINED’ > case MMCSD_MODE_UNDEFINED: >^ > Signed-off-by: Marek Vasut> Cc: Pantelis Antoniou > Cc: Tom Rini > --- > common/spl/spl_mmc.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c > index 0b681c2..43c1030 100644 > --- a/common/spl/spl_mmc.c > +++ b/common/spl/spl_mmc.c > @@ -343,10 +343,12 @@ static int spl_mmc_load_image(struct spl_image_info > *spl_image, > > break; > case MMCSD_MODE_UNDEFINED: > -#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT > default: > + { > +#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT > puts("spl: mmc: wrong boot mode\n"); > #endif > + } > } > > return err; I think we have something slightly off here. Is CONFIG_SPL_SERIAL_SUPPORT also enabled? That would turn this into a no-op as puts becomes a do { } while(0), if disabled. So I think we should just remove the ifdef here. -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 03/14] mmc: Fix warning if debug() is not used
On Fri, Nov 25, 2016 at 11:32:24PM +0100, Marek Vasut wrote: > If debug() is not used, then the whole content of debug(...) will > be removed by the preprocessor, which will result in the following > warning. This patch adds __maybe_unused annotation to fix this. > > drivers/mmc/mmc.c: In function ‘mmc_init’: > drivers/mmc/mmc.c:1685:11: warning: variable ‘start’ set but not used > [-Wunused-but-set-variable] > unsigned start; > > Signed-off-by: Marek Vasut> Cc: Pantelis Antoniou > Cc: Tom Rini Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 11/11] configs: dra7xx: Enable lp873x options
On Wed, Nov 23, 2016 at 01:25:34PM +0530, Lokesh Vutla wrote: > From: Keerthy> > DRA71-evm uses LP873x regulator. Enable lp873x PMIC config options. > > Signed-off-by: Keerthy > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 10/11] configs: dra7xx: Enable pmic/regulator options
On Wed, Nov 23, 2016 at 01:25:33PM +0530, Lokesh Vutla wrote: > Enable pmic/regulator config options. > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 09/11] configs: dra7xx: hs: Enable DM_ETH
On Wed, Nov 23, 2016 at 01:25:32PM +0530, Lokesh Vutla wrote: > Enable DM_ETH for hs boards. > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 08/11] configs: ti_omap5_common: Select dtb name for dra71x
On Wed, Nov 23, 2016 at 01:25:31PM +0530, Lokesh Vutla wrote: > From: Nishanth Menon> > Select dtb name for dra71x-evm. > > Signed-off-by: Nishanth Menon Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 07/11] ARM: dts: dra71x-evm: Add DT support
On Wed, Nov 23, 2016 at 01:25:30PM +0530, Lokesh Vutla wrote: > Add DT support for dra71-evm and built it as part of FIT image. > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 06/11] ARM: dts: dra7xx: sync DT with latest Linux
On Wed, Nov 23, 2016 at 01:25:29PM +0530, Lokesh Vutla wrote: > Sync all dra7xx based dts files with latest Linux > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 05/11] ARM: OMAP4+: Add support for getting pbias info from board
On Wed, Nov 23, 2016 at 01:25:28PM +0530, Lokesh Vutla wrote: > Palmas driver assumes it is always TPS659xx regulator on all DRA7xx based > boards to enable mmc regulator. This is not true always like in case of > DRA71x-evm. So get this information based on the board. > > Signed-off-by: Lokesh Vutla> Signed-off-by: Vignesh R > Signed-off-by: Nishanth Menon Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 04/11] board: ti: dra71x-evm: Add PMIC support
On Wed, Nov 23, 2016 at 01:25:27PM +0530, Lokesh Vutla wrote: > From: Keerthy> > Add the pmic_data for LP873x PMIC which is used to power > up dra71x-evm. > > Note: As per the DM[1] DRA71x supports only OP_NOM. So, updating > the efuse registers only to use OPP_NOM irrespective of any > CONFIG_DRA7__OPP_{NOM,od,high} is defined. > > [1] http://www.ti.com/product/DRA718/technicaldocuments > > Signed-off-by: Keerthy > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 03/11] board: ti: dra72: Introduce optimization for rgmii timing for rev C
On Wed, Nov 23, 2016 at 01:25:26PM +0530, Lokesh Vutla wrote: > From: Nishanth Menon> > Rev C version of EVM does require IODelay to be configured for RGMII > pins in MANUAL_1 configuration. Update the same based on PG2.0 initial > simulation values. > Data based on PCT_DRA72x_SR2.0_SR1.0_v1.3.0.7 > > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 02/11] board: ti: dra71x-evm: Add mux settings
On Wed, Nov 23, 2016 at 01:25:25PM +0530, Lokesh Vutla wrote: > Add mux and iodelay settings for dra71x-evm. > Data generated using PCT_DRA71x_SR2.0_v1.0.0.0 version (June 2016). > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 01/11] board: ti: dra71x-evm: Add epprom support
On Wed, Nov 23, 2016 at 01:25:24PM +0530, Lokesh Vutla wrote: > The dra71x-evm is a board based on TI's DRA718 processor targeting > BOM-optimized > entry infotainment systems such as display audio and is a software compatible > derivative of the highly successful DRA74 and DRA72 processor families. > More information can be found here[1]. > > Add epprom detection for dra71-evm. > > [1] http://www.ti.com/product/dra718 > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v3 2/2] ARM: am57xx_evm: enable DFU support
On Fri, Nov 25, 2016 at 02:25:55PM +0530, Sekhar Nori wrote: > AM57xx GP EVM has USB2 port of the SoC exposed as > USB client port. > > It is useful to be able to use this port for USB > DFU downloads. > > Enable USB DFU support. Tested on AM57x GP EVM Rev > A3 using DFU to download to connected SD card. > > configs for HS version of the AM57x EVM are > included in the patch but not really tested. > > Signed-off-by: Sekhar NoriReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v3 1/2] ARM: ti: consolidate dfu environment variables
On Fri, Nov 25, 2016 at 02:25:54PM +0530, Sekhar Nori wrote: > Introduce include/environment/ti/dfu.h that > consolidates environment variable definitions > for various TI boards that support DFU today. > > Tested on AM335x EVM, AM437x SK EVM and DRA74x > EVM by using DFU to write to SD card. > > Signed-off-by: Sekhar NoriReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 9/9] ti_armv7_common: env: Increase IO buffer size
On Fri, Nov 25, 2016 at 11:14:26AM +0530, Lokesh Vutla wrote: > There are certain environment variables whose length is greater than > the defined IO buffer size. So, increase the IO buffer size to print the > entire variables. > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 8/9] ARM: dts: AM571x-IDK Initial Support
On Fri, Nov 25, 2016 at 11:14:25AM +0530, Lokesh Vutla wrote: > From: Schuyler Patton> > Add initial DTS support for AM571-IDK evm. > > Signed-off-by: Schuyler Patton > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/9] board: ti: am57xx: Add support for the am571x idk
On Fri, Nov 25, 2016 at 11:14:24AM +0530, Lokesh Vutla wrote: > From: Steve Kipisz> > The AM571x Industrial Development Kit (IDK) is a board based on TI's > AM571x SoC which has a single core 1.5GHz Cortex-A15processor. This > board is a development platform for the Industrial Market with: > > - 1GB of DDR3L > - Dual 1Gbps Ethernet > - HDMI > - PRU-ICSS > - uSD > - 16GB eMMC > - CAN > - RS-485 > - PCIe > - USB3.0 > - Video Input Port > - Industrial IO port and expansion connector > > The PRU/ICSS will be supported by 3rd party software for EtherCat, > Profibus, and other Industrial protocols. > > The link to the data sheet and TRM can be found here: > http://www.ti.com/product/AM5718 > > Signed-off-by: Steve Kipisz > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 6/9] board: ti: am572x-idk: Update pinmux using latest PMT
On Fri, Nov 25, 2016 at 11:14:23AM +0530, Lokesh Vutla wrote: > Update the board pinmux for AM572x-IDK board using latest PMT[1] and the > board files named am572x_idk_v1p3b_sr2p0 that were autogenerated on > 20th October, 2016 by "Steve Kipisz" and > "Tom Johnson ". > > [1] https://dev.ti.com/pinmux/app.html#/default/ > > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 5/9] board: ti: am572x: Add pinmux for X15/GPEVM SR2.0 using latest PMT
On Fri, Nov 25, 2016 at 11:14:22AM +0530, Lokesh Vutla wrote: > From: Nishanth Menon> > Update the board pinmux for AM572x-IDK board using latest PMT[1] and the > board files named am572x_gp_evm_A3a_sr2p0 that were autogenerated on > 19th October, 2016 by "Ahmad Rashed ". > > [1] https://dev.ti.com/pinmux/app.html#/default/ > > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 4/9] board: ti: am57xx: Update SR1.1 RGMII0 iodelay timings for x15/GPEVM
On Fri, Nov 25, 2016 at 11:14:21AM +0530, Lokesh Vutla wrote: > From: Nishanth Menon> > Update the timing for RGMII0 interface based on > PCT_DRA75x_DRA74x_SR1.1_v1.3.10 version (Jan 2016). This update > is for SR1.1 > > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 3/9] board: ti: am57xx: Add support for detection of X15 revb1
On Fri, Nov 25, 2016 at 11:14:20AM +0530, Lokesh Vutla wrote: > BeagleBoard-X15 Rev B1 with SR1.1 platform have incompatible changes for HDMI > GPIO requiring new dtb support. This implies we have to properly identify > the platform now as well. Hence provide a different board name for the > Rev B1 variants. > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/9] board: ti: am57xx: Add support for detection of reva3 variations for GPEVM
On Fri, Nov 25, 2016 at 11:14:19AM +0530, Lokesh Vutla wrote: > From: Nishanth Menon> > AM57xx evm Rev A3 with SR2.0 platform have incompatible changes for HDMI > GPIO requiring new dtb support. This implies we have to properly identify > the platform now as well. Hence provide a different board name for the > Rev A3 variations. > > Signed-off-by: Nishanth Menon > Signed-off-by: Lokesh Vutla Reviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/9] ARM: dts: am57xx: sync DT with latest Linux
On Fri, Nov 25, 2016 at 11:14:18AM +0530, Lokesh Vutla wrote: > Sync all am57xx based dts files with latest Linux > > Signed-off-by: Lokesh VutlaReviewed-by: Tom Rini -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-rockchip
On Fri, Nov 25, 2016 at 12:39:13PM -0700, Simon Glass wrote: > Hi Tom, > > This includes support for minnie and mickey Chromebooks, rk3036 USB > support, some power improvements and other minor things. > > > The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32: > > MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-rockchip.git > > for you to fetch changes up to fc334a074b7309e08c8f9571f4e5025f345a: > > rockchip: configs: correct partitions 'boot' size (2016-11-25 10:03:32 > -0700) > Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-fdt
On Fri, Nov 25, 2016 at 09:51:02AM -0700, Simon Glass wrote: > Hi Tom. > > The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32: > > MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-fdt.git > > for you to fetch changes up to 643f8d4c07788f28ad283e3fcb29d88a4b16a4be: > > MAINTAINERS: Fix syntax and update filename for FDT (2016-11-25 > 09:48:09 -0700) > Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] u-boot 2016.11 and A33 1G ram detection.
Uboot reports 512Mb on my ET_Q8_V2.0 allwinner A33 tablet but it have 1G Ram (android and my eyes on pcb see it!), kernel(4.9-rc6) report 512Mb too. Is ram autodetected or i can change something in dts(i suppose) to permit correct detection? Uboot pass to kernel ram size or kernel detect itself ? Thank ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] arm: Remove smdk2410 board
On Sat, Nov 26, 2016 at 07:30:36PM +0530, Jagan Teki wrote: > On Sat, Nov 26, 2016 at 7:14 PM, Tom Riniwrote: > > On Sat, Nov 26, 2016 at 07:03:33PM +0530, Jagan Teki wrote: > >> On Wed, Nov 23, 2016 at 6:31 PM, Simon Glass wrote: > >> > This board has not been converted to DM_SERIAL by the deadline. > >> > Remove it. > >> > > >> > Signed-off-by: Simon Glass > >> > --- > >> > >> < snip> > >> > >> > CONFIG_ARCH_SHARK > >> > -CONFIG_ARCH_SMDK2410 > >> > >> Code relate to this still there in arch/arm/include/asm/mach-types.h, > > > > That's a file we sync(ed) from the kernel and should leave alone, imho. > > I always think that the code related to the CONFIG_* which are listed > in config_whitelist.txt should be dropped as there is no usage on the > tree. Yes, but that's a special case of a file that we should just not touch. I will however accept the argument that we should re-sync the file with a generated copy from v4.8 for example but only take the MACH_TYPE_xxx part and remove all of the machine_is_xxx parts of the file. -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] arm: Remove smdk2410 board
On Sat, Nov 26, 2016 at 7:14 PM, Tom Riniwrote: > On Sat, Nov 26, 2016 at 07:03:33PM +0530, Jagan Teki wrote: >> On Wed, Nov 23, 2016 at 6:31 PM, Simon Glass wrote: >> > This board has not been converted to DM_SERIAL by the deadline. >> > Remove it. >> > >> > Signed-off-by: Simon Glass >> > --- >> >> < snip> >> >> > CONFIG_ARCH_SHARK >> > -CONFIG_ARCH_SMDK2410 >> >> Code relate to this still there in arch/arm/include/asm/mach-types.h, > > That's a file we sync(ed) from the kernel and should leave alone, imho. I always think that the code related to the CONFIG_* which are listed in config_whitelist.txt should be dropped as there is no usage on the tree. thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] arm: Remove smdk2410 board
On Sat, Nov 26, 2016 at 07:03:33PM +0530, Jagan Teki wrote: > On Wed, Nov 23, 2016 at 6:31 PM, Simon Glasswrote: > > This board has not been converted to DM_SERIAL by the deadline. > > Remove it. > > > > Signed-off-by: Simon Glass > > --- > > < snip> > > > CONFIG_ARCH_SHARK > > -CONFIG_ARCH_SMDK2410 > > Code relate to this still there in arch/arm/include/asm/mach-types.h, That's a file we sync(ed) from the kernel and should leave alone, imho. -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 4/4] serial: Drop the s3c24x0 serial driver
On Sat, Nov 26, 2016 at 12:55 PM, David Müller (ELSOFT AG)wrote: > Simon Glass wrote: >> This is not used by any boards. Drop it. >> >> Signed-off-by: Simon Glass > > Acked-by: David Müller Reviewed-by: Jagan Teki thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 3/4] arm: Remove VCMA9 board
On Sat, Nov 26, 2016 at 12:55 PM, David Müller (ELSOFT AG)wrote: > Simon Glass wrote: >> This board has not been converted to DM_SERIAL by the deadline. >> Remove it. >> >> Signed-off-by: Simon Glass > > Acked-by: David Müller Reviewed-by: Jagan Teki thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/4] arm: Remove smdk2410 board
On Wed, Nov 23, 2016 at 6:31 PM, Simon Glasswrote: > This board has not been converted to DM_SERIAL by the deadline. > Remove it. > > Signed-off-by: Simon Glass > --- < snip> > CONFIG_ARCH_SHARK > -CONFIG_ARCH_SMDK2410 Code relate to this still there in arch/arm/include/asm/mach-types.h, better to remove. thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] armv8/spin_table.c: fix spin table release address
Oops, Seems like I misunderstood this approach. Thanks for the clarification! Regards, 2016-11-26 11:17 GMT+09:00 Masahiro Yamada: > 2016-11-25 7:38 GMT+09:00 Tsung-Han Lin : > > Since only the master core will relocate itself, the spin-table release > > address seen by master core and other slave cores is different. > > This commit fixes this issue by recalculating the spin-table release > > address' offset instead of leaving it to compiler. > > No. > You should bring the slaves cores in after relocation > so that the slaves enter the *relocated* _start. > > -- > Best Regards > Masahiro Yamada > -- Tsung-Han "*Johnny*" Lin Page: http://tsunghanlin.github.com/ Email: tsunghan...@gmail.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot