Re: [Qemu-devel] [PATCH RFC 01/11] TCG translation

[Yoshinori Sato <ysato@users.sourceforge.jp>]

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 <ysato@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