Hi Ohad,<br><br><div class="gmail_quote">2012/5/22 Ohad Ben-Cohen <span dir="ltr"><<a href="mailto:ohad@wizery.com" target="_blank">ohad@wizery.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Michal,<br>
<div class="im"><br>
On Tue, May 22, 2012 at 8:51 AM, Michal Simek <<a href="mailto:monstr@monstr.eu">monstr@monstr.eu</a>> wrote:<br>
> Simple enabling RSC_VDEV in rproc_handle_rsc doesn't work.<br>
<br>
</div>Sure - you'll need to actually plug the vrings allocation code there,<br>
too (i.e. this requires some coding, it's not a 1-liner).<br>
<div class="im"><br>
> BTW: I am using kernel modules and there is no dependency for<br>
> virtio_rpmsg_bus in remoteproc module<br>
> in sense if you load remoteproc module you also need to load<br>
> virtio_rpmsg_bus to get firmware to work.<br>
<br>
</div>This is expected: remoteproc only provides means for booting and<br>
controlling remote processors and it doesn't boot them upfront.<br>
<br>
To actually boot a remote processor, and do something useful with it,<br>
one needs to load a module which utilizes remoteproc's API (e.g.<br>
virtio_rpmsg_bus, but YMMV).<br>
<div class="im"><br>
> ~ # lsmod<br>
> ~ # modprobe zynq_remoteproc<br>
> zynq_remoteproc: Unknown symbol rproc_register (err 0)<br>
> zynq_remoteproc: Unknown symbol rproc_alloc (err 0)<br>
> zynq_remoteproc: Unknown symbol rproc_vq_interrupt (err 0)<br>
> zynq_remoteproc: Unknown symbol rproc_free (err 0)<br>
> zynq_remoteproc: Unknown symbol rproc_unregister (err 0)<br>
<br>
</div>We don't have these errors - have you tried looking into them ?<br></blockquote><div><br>It is probably any problem with modules. If I load them in sequence<br>there is no error.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
Btw do you plan on submitting the code for zynq_remoteproc ?<br></blockquote><div><br>It is still under developing and there are some core changes in ARM low level code<br>which need to be applied to mainline before this driver. I haven't sent these patches<br>
to mainline for review but I expect they won't be accepted but maybe I am wrong.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im"><br>
> Can you see the same behavior on omap?<br>
<br>
</div>Except the "Unknown symbol" errors, everything else looks good.<br></blockquote><br></div>Ok. Good.<br><br>Thanks,<br>Michal<br><br>