Bug 761

Summary: clean CSV files with instructions
Product: Libre-SOC's first SoC Reporter: Dmitry Selyutin <ghostmansd>
Component: Source CodeAssignee: Dmitry Selyutin <ghostmansd>
Status: RESOLVED INVALID    
Severity: enhancement CC: libre-soc-bugs, lkcl
Priority: ---    
Version: unspecified   
Hardware: PC   
OS: Linux   
NLnet milestone: --- total budget (EUR) for completion of task and all subtasks: 0
budget (EUR) for this task, excluding subtasks' budget: 0 parent task for budget allocation:
child tasks for budget allocation: The table of payments (in EUR) for this task; TOML format:

Description Dmitry Selyutin 2022-01-09 15:54:45 GMT
https://bugs.libre-soc.org/show_bug.cgi?id=550#c72

Problem statement, briefly: we have been using the "comment" field in major.csv, minor_*.csv and extra.csv as a name placeholder. However, this field comes from microwatt sources, and some fields contain stuff we don't need. Let's clean these up.
Comment 1 Luke Kenneth Casson Leighton 2022-01-09 16:58:53 GMT
(In reply to dmitry.selyutin from comment #0)
> https://bugs.libre-soc.org/show_bug.cgi?id=550#c72
> 
> Problem statement, briefly: we have been using the "comment" field in
> major.csv, minor_*.csv and extra.csv as a name placeholder. 

not quite: microwatt's *developers* use the comment field to contain the
instruction name.

> However, this
> field comes from microwatt sources, and some fields contain stuff we don't
> need. Let's clean these up.

this would deviate significantly from microwatt, and would lose information
that is useful to them.  the reason i have left them in is so that the
microwatt team could, at some point in the future, consider using the CSV
files (directly).

if those fields are removed, then they cannot do that.

bottom line is, we can add fields, but not take them away.