Bug 761 - clean CSV files with instructions
Summary: clean CSV files with instructions
Status: RESOLVED INVALID
Alias: None
Product: Libre-SOC's first SoC
Classification: Unclassified
Component: Source Code (show other bugs)
Version: unspecified
Hardware: PC Linux
: --- enhancement
Assignee: Dmitry Selyutin
URL:
Depends on:
Blocks:
 
Reported: 2022-01-09 15:54 GMT by Dmitry Selyutin
Modified: 2022-01-09 16:58 GMT (History)
2 users (show)

See Also:
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:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.