Enable OpenAMP build

65 views
Skip to first unread message

Xiang Xiao

unread,
Nov 10, 2019, 7:48:45 AM11/10/19
to NuttX
Fix the build issue, the upcoming patch will add sim demo and README.

Thanks
Xiang
0001-openamp-add-make-file-defines.patch

Alan Carvalho de Assis

unread,
Nov 10, 2019, 7:58:58 AM11/10/19
to nu...@googlegroups.com
Hi Xiang,

I think you missed the README, it is not in the patch.

BR,

Alan
> --
> You received this message because you are subscribed to the Google Groups
> "NuttX" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to nuttx+un...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/nuttx/80e88cce-42b9-40ba-ab11-ada427a3d197%40googlegroups.com.
>

Xiang Xiao

unread,
Nov 10, 2019, 8:17:26 AM11/10/19
to NuttX
Will provide in the sim demo, please wait a second.


On Sunday, November 10, 2019 at 8:58:58 PM UTC+8, Alan Carvalho de Assis wrote:
Hi Xiang,

I think you missed the README, it is not in the patch.

BR,

Alan

On 11/10/19, Xiang Xiao <xiaoxia...@gmail.com> wrote:
> Fix the build issue, the upcoming patch will add sim demo and README.
>
> Thanks
> Xiang
>
> --
> You received this message because you are subscribed to the Google Groups
> "NuttX" group.
> To unsubscribe from this group and stop receiving emails from it, send an

Xiang Xiao

unread,
Nov 10, 2019, 8:18:18 AM11/10/19
to NuttX
Minor patch fix the compiler warning.
0001-fix-warning-openamp-Makefile-linker-input-file-unuse.patch

Gregory Nutt

unread,
Nov 10, 2019, 9:08:07 AM11/10/19
to nu...@googlegroups.com
Minor patch fix the compiler warning.

> Fix the build issue, the upcoming patch will add sim demo and README.

Both patches committed.  Thanks!

Greg



Xiang Xiao

unread,
Nov 10, 2019, 9:26:09 AM11/10/19
to NuttX
Change all sprintf to snprintf and refine the rptun naming rule
0001-Refine-the-rptun-naming.patch

Gregory Nutt

unread,
Nov 10, 2019, 9:45:45 AM11/10/19
to nu...@googlegroups.com

> Change all sprintf to snprintf and refine the rptun naming rule

Committed!

Greg


Xiang Xiao

unread,
Nov 11, 2019, 9:25:20 AM11/11/19
to NuttX
Again minor fix for drivers/rptun/rptun.c.
0001-rptun-refine-the-start-stop-flow-to-support-the-repe.patch
0002-rptun-vring-size-should-also-be-aligned.patch

Gregory Nutt

unread,
Nov 11, 2019, 10:12:18 AM11/11/19
to nu...@googlegroups.com

> Again minor fix for drivers/rptun/rptun.c.

And again committed.. Thanks!

Greg


Xiang Xiao

unread,
Nov 12, 2019, 2:50:39 AM11/12/19
to NuttX
Greg, this patch enable OpenAMP on simulator, README.txt contain the detailed HOWTO instruction.
we will provide the final patch in the next couple of day to help people donwload source code from the official release and apply some patch(pending upstram) on it.
0001-Support-OpenAMP-between-two-simulator-instances.patch

Gregory Nutt

unread,
Nov 12, 2019, 9:04:29 AM11/12/19
to nu...@googlegroups.com
> Greg, this patch enable OpenAMP on simulator, README.txt contain the
> detailed HOWTO instruction.
> we will provide the final patch in the next couple of day to help
> people donwload source code from the official release and apply some
> patch(pending upstram) on it.

Committed.  Nice job!  I will try to make some time to experiment with this.

Greg


Xiang Xiao

unread,
Nov 12, 2019, 9:30:00 AM11/12/19
to NuttX
Sure, before we provide the patch against the official mainline, please try with these two git instead:

Xiang Xiao

unread,
Nov 12, 2019, 9:34:51 AM11/12/19
to NuttX
BTW, our codebase still lack one patch for simulator and will upstream the patch tomorrow.
So please apply the attached patch to nuttx/openamp/libmetal manually before that. 
0001-nuttx-correctly-handle-the-hardware-which-hasn-t-the.patch

Gregory Nutt

unread,
Nov 12, 2019, 9:43:31 AM11/12/19
to nu...@googlegroups.com

> BTW, our codebase still lack one patch for simulator and will upstream
> the patch tomorrow.
> So please apply the attached patch to nuttx/openamp/libmetal manually
> before that.

I have plenty to do today so I can wait for the official patch against
the mainline.

Thanks,

Greg


Reply all
Reply to author
Forward
0 new messages