[PATCH] bsps/sparc: Add grlib_malloc(), grlib_calloc()

Daniel Hellstrom daniel at gaisler.com
Fri Dec 21 13:59:52 UTC 2018


Hi Sebastian,


On 2018-12-21 14:57, Sebastian Huber wrote:
> Hello Daniel,
>
> On 21/12/2018 14:42, Daniel Hellstrom wrote:
>> Hi Sebastian,
>>
>> Sorry for my very late response! I have reviewed the code but not 
>> executed it, it looks okay with me. I'm guessing it is also a 
>> positive thing to remove malloc for another approach to allocate 
>> memory when it comes to validation of the SW.
>
> thanks for the review.
>
> My next step is to move the grlib to bsps/shared/grlib so that it can 
> be used for RISC-V BSPs.


Great news! how will you do with the driver manager in this case, will 
you add a connection between FDT and the driver manager via FDT-bus 
driver? I think that would be of interest also for LEON, I would be 
happy to review such a solution or similar one.


>
>>
>> Another aspect of the same subject could be to use the RTEMS 
>> memalign() instead of custom alignment within the driver for 
>> descriptor tables etc. The history there is that memalign() did not 
>> exist when writing most of the drivers.
>
> Yes, this is also on my TODO list.
>
>>
>> Merry Christmas and a happy new year! 
>
> Thanks, Merry Christmas and Happy New Year too.
>


More information about the devel mailing list