[U-Boot] bav335x support broken

[Gilles]

Hi Anish,

Yes, actually my board support is based on an older sitara patch and it works when applied to v2015.01. But I was mistaken about the problem.
I went back to my original patch applied to v2015.01 and it gives the same error but then moves on to SPL and boots ok.

http://pastebin.com/A23qKkj8

So I guess it's safe to say that the "fat" error is not the problem. The problem when I compile a74ef40a471d9d4bffb36a8c89744cf6fd631e6f is that it restarts over and over.

http://pastebin.com/wvjx4mB8

Do you have any suggestions how how to find the cause of the constant restart? I actually don't have a debugger for this hardware :-(

Cheers,
Gilles
.



> On Mar 16, 2015, at 09:14 , Anish Khurana <anish.khurana130181 at gmail.com> wrote:
> 
> Hi Giles,
> 
> I was browsing the code and it seems it is failing in spl_fat.c file
> in function spl_load_image_fat_os() and internally calls
> do_fat_read_at() function. Also I noticed that configuration is very
> near to am335x( ti sitara) architecture, so probably can try some more
> configure similar to am335x .
> 
> Thanks,
> Anish
> 
> 
> On Mon, Mar 16, 2015 at 12:02 PM, Gilles <gilles at gigadevices.com> wrote:
>> Hi Anish,
>> 
>> Thanks for pointing that out. I ran menuconfig and that fixed the compilation issue however, now I'm getting this error when I try to run it:
>> 
>> spl_load_image_fat_os: error reading image args, err - -1
>> 
>> I guess I'll fool around with menuconfig see if there is something that should be enabled and isn't. (maybe you have a suggestion on this?)
>> 
>> Anyway, I'll post updated defconfigs when I figure it out. Hopefully this is just a config issue.
>> 
>> Thanks,
>> Gilles
>> .
>> 
>> 
>>> On Mar 14, 2015, at 10:55 , Anish Khurana <anish.khurana130181 at gmail.com> wrote:
>>> 
>>> Hi Gilles,
>>> 
>>> This u-boot version is having lot of changes , they have enabled
>>> Kconfig similar to Kernel and having GUI support ( make menuconfig) .
>>> so if you are gettting error as you mentioned , try to add Device
>>> model configurations  and SYS malloc() as :
>>> 
>>> Steps 1 make <birdland_bav335x config>
>>> Step 2 make menuconfig
>>> Step 3 Device driver -> Enable driver model, enable Driver model for serial)
>>> Step 4 malloc() pool , General setup --> Enable malloc() pool.
>>> 
>>> Please try this , I think it will work.
>>> 
>>> Thanks,
>>> Anish
>>> 
>>> 
>>> On Sat, Mar 14, 2015 at 10:36 AM, Gilles <gilles at gigadevices.com> wrote:
>>>> Folks,
>>>> 
>>>> I posted a patch to add support for bav335x boards (a322aad99de4). The patch was tested against v2015.04-rc1 and worked perfectly but somehow, something was introduced since then which breaks the board support.
>>>> 
>>>> There are two main errors I need to fix before posting another patch but need help with the second error.
>>>> 
>>>> The first error is easy. Compilation throws:
>>>> #error "Please define NS16550 registers size."
>>>> which I can simply fix by defining CONFIG_SYS_NS16550_REG_SIZE in include/configs/bav335x.h
>>>> ((( Altough I am stil puzzled as to why this was defined as -4 (where?) on v2015.04-rc1 )))
>>>> 
>>>> After adding the define, everything compiles up to failing on the final link with:
>>>> 
>>>> drivers/serial/built-in.o: In function `get_current':
>>>> /home/gilles/bbdev/u-boot/drivers/serial/serial.c:389: undefined reference to `default_serial_console'
>>>> drivers/serial/built-in.o: In function `serial_initialize':
>>>> 
>>>> Can anyone please explain what changed between 2015.04-rc1 and 2015.04-rc2 which could cause such a behavior? I have spent the last couple hours re-basing a branch to see where it breaks as to maybe get a clue on what changed but no luck so far. Any tips would be appreciated.
>>>> 
>>>> Thanks,
>>>> Gilles
>>>> .
>>>> 
>>>> 
>>>> 
>>>> _______________________________________________
>>>> U-Boot mailing list
>>>> U-Boot at lists.denx.de
>>>> http://lists.denx.de/mailman/listinfo/u-boot
>>