Re: [PATCH v2 1/2] arm/cpu/start.c: Distil some common code in __start().

2015-10-30 Thread Andrey Smirnov
> > Previously we knew that the pointer barebox_arm_machine() works on is a > machine type entry and we knew that the pointer returned from > barebox_arm_boot_dtb() is a dtb. With this patch both are share the same > pointer so we would have to check if the result returned from > barebox_arm_boot_d

Re: [PATCH v2 1/2] arm/cpu/start.c: Distil some common code in __start().

2015-10-30 Thread Sascha Hauer
On Wed, Oct 28, 2015 at 11:34:37AM -0700, Andrey Smirnov wrote: > Both barebox_boarddata and barebox_boot_dtb perform essentially the > same function -- hold a pointer to a chunk of private data. Since only > one variable is ever used at any given time we may as well merge those > two variable into

[PATCH v2 1/2] arm/cpu/start.c: Distil some common code in __start().

2015-10-28 Thread Andrey Smirnov
Both barebox_boarddata and barebox_boot_dtb perform essentially the same function -- hold a pointer to a chunk of private data. Since only one variable is ever used at any given time we may as well merge those two variable into one. This also allows us to share more code between two boot paths (boa