Re: [PATCH 5/7] fs: Calculate new position before validtiy check in lseek()

2019-01-23 Thread Sascha Hauer
On Tue, Jan 22, 2019 at 05:13:36PM -0800, Andrey Smirnov wrote: > Calculate new position before validtiy check in lseek() to simplify > code a bit as well as make following commit simpler. This should be > harmless thing to do, since we don't actually use calculated value > unless it passes the

Re: [PATCH 4/7] fs: Change error checking logic for fsdrv->lseek() call

2019-01-23 Thread Sascha Hauer
On Tue, Jan 22, 2019 at 05:13:35PM -0800, Andrey Smirnov wrote: > On 32-bit systems, cheking for IS_ERR_VALUE(pos) is not > correct. Expanding that code we get (loff_t cast is added for clarity): > > (loff_t)pos >= (unsigned long)-MAX_ERRNO > > given that loff_t is a 64-bit signed value, any

Re: [PATCH v2 1/2] ARM: zii-vf610-dev: fix boot from SD

2019-01-23 Thread Sascha Hauer
On Wed, Jan 23, 2019 at 10:16:19AM -0500, Vivien Didelot wrote: > Since 5f99a8d40305 ("dts: VFxxx: Add aliases for ESDHC controllers"), > the SD card slot has a consistent name across all ZII Dev platforms, > including CFU1. They all use mmc1 instead of mci0 or mci1. > > This allows us to

Re: [PATCH v2 0/3] ath79: enhancements

2019-01-23 Thread Sascha Hauer
On Wed, Jan 23, 2019 at 08:24:00AM +0100, Oleksij Rempel wrote: > changes v2 20190123: > - squash bootstrap helper patches to one patch > - rename helper macro > - Add more comments > > Oleksij Rempel (3): > MIPS: tplink-mr3020: use ar9331_pbl_generic_start >

Re: [PATCH v2] ARM: zynq: fix "'IO_SPACE_LIMIT' redefined" warning

2019-01-23 Thread Sascha Hauer
On Tue, Jan 22, 2019 at 09:38:25PM +0300, Antony Pavlov wrote: > The patch fixes the following compiler's warning: > > In file included from include/common.h:33:0, > from arch/arm/mach-zynq/zynq.c:18: > arch/arm/include/asm/io.h:4:0: warning: "IO_SPACE_LIMIT" redefined > #define

Re: [PATCH] unlink_recursive: Drop struct data

2019-01-23 Thread Sascha Hauer
On Thu, Jan 17, 2019 at 04:41:55PM -0800, Andrey Smirnov wrote: > Drop struct data which doesn't seem to serve any purpose in the code > and looks like a leftover. > > Signed-off-by: Andrey Smirnov > --- > lib/unlink-recursive.c | 27 +++ > 1 file changed, 7

Re: [PATCH] ls: Adjust amount of space allocated for filesize

2019-01-23 Thread Sascha Hauer
On Thu, Jan 17, 2019 at 04:41:13PM -0800, Andrey Smirnov wrote: > Maximum file size on 64-bit system is 2^63-1 (9223372036854775807), > which takes more that 14 characters of space. In order to keep things > properly aligned adjust that spacing to 20 on 64-bit platforms. > > Before: > > ls -l

Re: [PATCH v2 2/2] ARM: zii-vf610-dev: boot initrd from SD

2019-01-23 Thread Andrey Smirnov
On Wed, Jan 23, 2019 at 7:16 AM Vivien Didelot wrote: > > As for tftp, make use of the initramfs if one is present on the SD card. > Makes sense: Reviewed-by: Andrey Smirnov > Signed-off-by: Vivien Didelot > --- > .../boards/zii-vf610-dev/defaultenv-zii-vf610-dev/boot/sd| 5 + > 1

Re: [PATCH v2 1/2] ARM: zii-vf610-dev: fix boot from SD

2019-01-23 Thread Andrey Smirnov
On Wed, Jan 23, 2019 at 7:16 AM Vivien Didelot wrote: > > Since 5f99a8d40305 ("dts: VFxxx: Add aliases for ESDHC controllers"), > the SD card slot has a consistent name across all ZII Dev platforms, > including CFU1. They all use mmc1 instead of mci0 or mci1. > > This allows us to completely drop

[PATCH] ARM: aarch64: Avoid relocations in runtime-offset.S

2019-01-23 Thread Andrey Smirnov
Since get_runtime_offset() is executed as a part of reloaction logic, it cannot have code dependend on any kind of relocation. Unfortunately, current codebase violates this rule and linkadr: .quad get_runtime_offset ends up producing R_AARCH64_RELATIVE relocation that has to be resolved at

Re: powerpc-linux-gnu-gcc: error: unrecognized command line option ‘-mno-spe’

2019-01-23 Thread Andrey Smirnov
On Wed, Jan 23, 2019 at 4:02 AM Antony Pavlov wrote: > > Hi Sascha! > > I'm playing with MAKEALL script to build as many defconfigs as possible. > My build environment is base on Debian Buster. > Debian Buster has cross compiler for PowerPC32. > > Alas I have no success with debian

[PATCH v2 2/2] ARM: zii-vf610-dev: boot initrd from SD

2019-01-23 Thread Vivien Didelot
As for tftp, make use of the initramfs if one is present on the SD card. Signed-off-by: Vivien Didelot --- .../boards/zii-vf610-dev/defaultenv-zii-vf610-dev/boot/sd| 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/boards/zii-vf610-dev/defaultenv-zii-vf610-dev/boot/sd

[PATCH v2 1/2] ARM: zii-vf610-dev: fix boot from SD

2019-01-23 Thread Vivien Didelot
Since 5f99a8d40305 ("dts: VFxxx: Add aliases for ESDHC controllers"), the SD card slot has a consistent name across all ZII Dev platforms, including CFU1. They all use mmc1 instead of mci0 or mci1. This allows us to completely drop the init scripts from the default ZII VF610 Dev environment, and

Re: [PATCH] imd: model and compatible missing in metadata

2019-01-23 Thread Scherer, Thorsten
Hello, On Tue, Jan 22, 2019 at 09:53:37AM +0100, Uwe Kleine-König wrote: > Hello, > > On Tue, Jan 22, 2019 at 09:10:40AM +0100, Scherer, Thorsten wrote: > > i spent some time trying to understand the usage of imd_used etc. > > > > In another reply I stated: > > > > 'I misread the documenation

powerpc-linux-gnu-gcc: error: unrecognized command line option ‘-mno-spe’

2019-01-23 Thread Antony Pavlov
Hi Sascha! I'm playing with MAKEALL script to build as many defconfigs as possible. My build environment is base on Debian Buster. Debian Buster has cross compiler for PowerPC32. Alas I have no success with debian powerpc-linux-gnu-gcc 8.2.0-2. After CROSS_COMPILE=powerpc-linux-gnu-

Re: signed HDMI firmware in imx8mq image

2019-01-23 Thread Yazdani, Reyhaneh
Hi everyone, I found the reason of the problem. It is all related to the HDMI FW itself. The one that I took from imx-mkimage repository is different from the one, which is used via Yocto to build final image. Now HDMI is working :) Best, Reyhaneh On 1/16/19 9:30 PM, Andrey Smirnov wrote: > On

Re: [HELP] Barebox porting

2019-01-23 Thread Sascha Hauer
On Wed, Jan 23, 2019 at 01:01:46PM +0300, Seraphim Dolbilov wrote: > 23.01.2019, 12:11, "Sascha Hauer" : > > What are you trying to archieve? > > My board contains mcp79411 - it's an I2C RTC with EEPROM, which contains > global-registered MAC address. I want this MAC address to be read by Barebox

Re: [HELP] Barebox porting

2019-01-23 Thread Seraphim Dolbilov
23.01.2019, 12:11, "Sascha Hauer" : > What are you trying to archieve? My board contains mcp79411 - it's an I2C RTC with EEPROM, which contains global-registered MAC address. I want this MAC address to be read by Barebox and passed to Linux kernel FEC driver through bootargs. I was trying to find

Re: [HELP] Barebox porting

2019-01-23 Thread Sascha Hauer
On Tue, Jan 22, 2019 at 07:12:16PM +0300, Seraphim Dolbilov wrote: > How can I get an output of a shell command in hush shell? > Backticks and $() cannot do not work :( You can't get the output of a command, that's not implemented. Some commands work around this in the way that you can pass a

Re: [PATCH v2 2/3] MIPS: black-swift: use ar9331_pbl_generic_start

2019-01-23 Thread Sam Ravnborg
On Wed, Jan 23, 2019 at 08:24:02AM +0100, Oleksij Rempel wrote: > From: Oleksij Rempel > > most ar9331 boards are almost identical. Remove as match duplicate code Here too. ___ barebox mailing list barebox@lists.infradead.org

Re: [PATCH v2 1/3] MIPS: tplink-mr3020: use ar9331_pbl_generic_start

2019-01-23 Thread Sam Ravnborg
Hi Oleksij. On Wed, Jan 23, 2019 at 08:24:01AM +0100, Oleksij Rempel wrote: > From: Oleksij Rempel > > most ar9331 boards are almost identical. Remove as match duplicate code s/match/much/ ___ barebox mailing list barebox@lists.infradead.org