Added script to generate c header from yaml and generated header #1733
Conversation
Signed-off-by: Henrik Fegran <[email protected]>
| ################################################################################ | ||
| # | ||
| # Copyright 2020 OpenHW Group | ||
| # Copyright 2020 Silicon Labs, Inc. |
There was a problem hiding this comment.
Should be 2022 or 2023.
And I think @MikeOpenHWGroup said before that the openhw group copyright line isn't really necessary?
There was a problem hiding this comment.
Correct. I will add it if I create a file, but other members are not expected to. There is no significant impact if do grant OpenHW copyright (I am sure a lawyer would point out that assigning copyright to OpenHW means that OpenHW will have standing as an "injured party" should somebody violate the license.)
...and yes, 2020 s.b. 2023.
| ** Copyright 2021 OpenHW Group | ||
| ** Copyright 2021 Silicon Labs | ||
| ** | ||
| ** Licensed under the Solderpad Hardware Licence, Version 2.0 (the "License"); |
There was a problem hiding this comment.
Should probably have an SPDX line too.
| @@ -0,0 +1,955 @@ | |||
| /* | |||
There was a problem hiding this comment.
Really nice! I like this new bsp header.
| } __attribute__((packed)) volatile timeh_t; | ||
|
|
||
|
|
||
| typedef union { |
There was a problem hiding this comment.
script issue here - should not have multiple mhpmcounter-type defs
| #define MSCRATCHCSW 0x348 | ||
| #define MSCRATCHCSWL 0x349 | ||
| #define TSELECT 0x7A0 | ||
| #define TDATA1 0x7A1 |
There was a problem hiding this comment.
What can we do about tdata1? It doesn't seem to have a struct.
There was a problem hiding this comment.
Yet another corner case... I need to add a list of corner cases to the script - generally I do not want multiple definitions for registers that have n-number of instances, but there are some that actually require this. I will add a do not merge to this PR and handle that.
No description provided.