missing pin definition, is this on purpose?

[maciejopalinski]

LiteX-CNC/docs/src/examples/json/5a-75e_v6.0_i48o56.json

Lines 23 to 24 in eee501c

	{"direction": "in", "pin":"j1:6", "name":"j1:7"},
	{"direction": "out", "pin":"j1:9" , "name":"j1:10"},

Hello, I noticed that one pin definition is missing: J1:7

https://github.com/cleverfox/chubby75/blob/215b5bd561d9e25dd296f03bae392f257a0d0937/5a-75e/hardware_V8.2.md?plain=1#L168

This is a pin responsible for address E. Is it unavailable or just accidentally ommited?

If it is accidental, then I propose the following change:

                { "direction": "in", "pin": "j1:6", "name": "j1:7" },
++++            { "direction": "out", "pin": "j1:7", "name": "j1:8" },
                { "direction": "out", "pin": "j1:9", "name": "j1:10" },

[Peter-van-Tol]

This pin is accidently omitted. The background is that these files were based on the HUB75 layout, which indicates that pin E is not necessarily a data-pin. But the given repo, as well as litex-boards, show it is actually a pin used. So let's use it.

Would you like to provide a merge request for this issue?

[maciejopalinski]

Sure, I will make a pull request. I'm happy to contribute