PipelineC icon indicating copy to clipboard operation
PipelineC copied to clipboard

Clean and standardize PipelineC repo.

Open bartokon opened this issue 2 years ago • 4 comments

Repo readability is really low. I suggest to move all .h files to some kind of headers folder and iee, io, axi folders to merge them. Do we really need one extra layer-directory for dvi/dvi_pmod.c? fixed/q0_23.h?

bartokon avatar Oct 14 '22 11:10 bartokon

Making a headers directory sounds fine to me And then move dirs+files like you said headers/axi/axis.h for ex. dvi/ dir was to match vga/ which did end up having a few files in it fixed/ is supposed to be filled in with various fixed point type implementations (if they dont get autogenerated otherwise :-/)

JulianKemmerer avatar Oct 14 '22 19:10 JulianKemmerer

I might prefer include as dir name over headers ex. include/axi/axis.h

JulianKemmerer avatar Oct 17 '22 01:10 JulianKemmerer

Can we roll this out?

ghost avatar Jan 18 '23 15:01 ghost

I think it needs some clarification still @Datavenia

Is the goal to move directories with .h files into include/<name>/<file>.h ?

I am not in a rush to get to this

But if you put together a PR with the change I can try and test it after the big cleanup PR thats also pending probably

JulianKemmerer avatar Jan 18 '23 22:01 JulianKemmerer