Hi,
On 12.03.19 02:37, Andrey Smirnov wrote:
On Mon, Mar 11, 2019 at 6:41 AM David Brandt <d.brandt@xxxxxxxxx> wrote:
This patch adds the bootaux command which starts
auxiliary cores of iMX processors, currently only
for iMX8MQ. This is based on the u-boot-imx
command and shares the behaviour.
The currently unnecessary parameter for the core
to start is necessary for the iMX8QM and other
processors having multiple auxiliary cores.
Not saying that there's anything wrong with this patch, but I am
guessing that upstream Linux will eventually implement this
functionality using remoteproc framework. Maybe now would be a good
time to add a very basic version of it to Barebox? Another option to
make this a little bit more flexible would be to implement this using
firmwaremgr framework similar to
drivers/frimware/{socfpga.c,altera_serial.c}. Anyway, just a couple of
suggestions that I had.
I agree with remoteproc here.
The task of remoteproc on linux is:
- parse specially formated ELF file:
- Register needed resources extracted from ELF
- prepare clocks
- do address translation
- kick on the CPU
The main issue of linux part is to dunamically patch devicetree and tell master linux
instance about IP Cores which was migrated for master CPU to the slave CPU.
With barebox this issues can be solved. First of all, barebox already has ELF parser. So
it can be reused. The drivers/remoteproc/imx_rproc.c from linux can probably be ported as
is. As soon as first step done, we would be able to patch devicetree in barebox and gave
it over to linux. So, we will be able to tell linux to let some clocks on and do not touch
reserved memory areas.
Kind regards,
Oleksij Rempel
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox