[PATCH 3/6] shgen: Import from RTEMS

[Chris Johns]

On 9/6/18 9:48 am, Joel Sherrill wrote:
> On Fri, Jun 8, 2018 at 6:28 PM, Chris Johns <chrisj at rtems.org
> <mailto:chrisj at rtems.org>> wrote:
> 
>     On 8/6/18 10:32 pm, Gedare Bloom wrote:
>     > On Thu, Jun 7, 2018 at 10:19 PM, Chris Johns <chrisj at rtems.org
>     <mailto:chrisj at rtems.org>> wrote:
>     >> On 07/06/2018 16:11, Sebastian Huber wrote:
>     >>> On 07/06/18 07:53, Chris Johns wrote:
>     >>>> On 07/06/2018 15:39, Sebastian Huber wrote:
>     >>>>> Corresponding RTEMS commit is 75933d5d25cd50f80162b7a0d2f66a5534e1763f.
>     >>>>>
>     >>>>> Update #3443.
>     >>>>> ---
>     >>>>>   misc/shgen/AUTHORS |   3 +
>     >>>>>   misc/shgen/COPYING | 340
>     +++++++++++++++++++++++++++++++++++++++++++++++++++++
>     >>>>>   misc/shgen/TODO    |  13 ++
>     >>>>>   misc/shgen/sci.c   | 177 ++++++++++++++++++++++++++++
>     >>>>>   misc/shgen/sci.h   |  11 ++
>     >>>>>   misc/shgen/shgen.c | 114 ++++++++++++++++++
>     >>>>>   misc/wscript       |   9 ++
>     >>>>>   7 files changed, 667 insertions(+)
>     >>>>>   create mode 100644 misc/shgen/AUTHORS
>     >>>>>   create mode 100644 misc/shgen/COPYING
>     >>>>>   create mode 100644 misc/shgen/TODO
>     >>>>>   create mode 100644 misc/shgen/sci.c
>     >>>>>   create mode 100644 misc/shgen/sci.h
>     >>>>>   create mode 100644 misc/shgen/shgen.c
>     >>>>>
>     >>>>> diff --git a/misc/shgen/AUTHORS b/misc/shgen/AUTHORS
>     >>>>> new file mode 100644
>     >>>>> index 0000000..225c2fa
>     >>>>> --- /dev/null
>     >>>>> +++ b/misc/shgen/AUTHORS
>     >>>>> @@ -0,0 +1,3 @@
>     >>>>> +Ralf Corsepius (corsepiu at faw.uni-ulm.de <mailto:corsepiu at faw.uni-ulm.de>)
>     >>>>> +    * Initial implementation
>     >>>>> +    * generator for sci bitrate table
>     >>>>> diff --git a/misc/shgen/COPYING b/misc/shgen/COPYING
>     >>>>> new file mode 100644
>     >>>>> index 0000000..8cc2ef7
>     >>>>> --- /dev/null
>     >>>>> +++ b/misc/shgen/COPYING
>     >>>>> @@ -0,0 +1,340 @@
>     >>>>> +
>     >>>>> +            GNU GENERAL PUBLIC LICENSE
>     >>>> The RTEMS tools is almost clean of GPL code. There is a small piece in the
>     >>>> rtemstoolkit I would to replace but I have not done that yet.
>     >>>
>     >>> The nios2gen is also GPL (the RTEMS GPL with linking exception). Is this a
>     >>> problem?
>     >>
>     >> We are moving to a BSD-2 license so I assumed this is part of that change?
>     >>
>     >>> The GCC, GDB and Binutils are also GPL.
>     >>
>     >> They are not in this repo. It is about our repos and the compliance
>     obligations
>     >> users have with our code base.
>     >>
>     > No, I think  Sebastian's goal is to remove all host tools from the rtems.git.
>     >
> 
>     Sorry, I was meaning the license and not the patch. Moving the tools is a good
>     idea but the RTEMS Tools is licensed BSD-2 or similar so moving GPL code into
>     that repo changes this.
> 
> 
> Is this tool actually used as part of building any SH BSP? I don't recall
> ever seeing a rule using it. I suspect it was used to generate headers
> initially manually and that was never part of the RTEMS build process.
> The tools was added just in case but never actually used.

Same, I have not seen it being used.

> IMO if that's the case, then we don't need it.

Agreed and if it turns out it is needed we can revisit this then.

> Do we even know what headers were generated by it?

Sorry, I do not know.

Chris