On Mon, 21 Jan 2019 22:35:43 +0900, Thomas Huth wrote: > > On 2019-01-21 14:15, Yoshinori Sato wrote: > > This part only supported RXv1 instructions. > > Instruction manual. > > https://www.renesas.com/us/en/doc/products/mpumcu/doc/rx_family/r01us0032ej0120_rxsm.pdf > > > > Signed-off-by: Yoshinori Sato <ys...@users.sourceforge.jp> > > --- > [...] > > diff --git a/target/rx/op_helper.c b/target/rx/op_helper.c > > new file mode 100644 > > index 0000000000..c9c87dc308 > > --- /dev/null > > +++ b/target/rx/op_helper.c > > @@ -0,0 +1,548 @@ > > +/* > > + * RX helper functions > > + * > > + * Copyright (c) 2018 Yoshinori Sato > > + * > > + * This library is free software; you can redistribute it and/or > > + * modify it under the terms of the GNU Lesser General Public > > + * License as published by the Free Software Foundation; either > > + * version 2 of the License, or (at your option) any later version. > > Hi, > > just a short meta comment for all the new files in your patch series: > It's either > > GNU *Library* General Public License, version 2 > > or > > GNU Lesser General Public License, version *2.1* > > I'd suggest to go with "Lesser" and "2.1". > > Apart from that, it might be a good idea to CC: some people who are > familiar with TCG if you want to make sure to get some review (see the > MAINTAINERS file for example).
Thanks comment. OK. I will update the license text and resend it. > Thanks, > Thomas > > > PS: Seems like we've got this "Lesser" vs. "Library" wrong in a couple > of other source files in QEMU ... I'll try to come up with a patch to > fix this in the files that we've already got in the repo. > -- Yosinori Sato