Merge pull request #65 from antonblanchard/loadstore-opt

A small loadstore optimisation, and some reformatting

Reformat loadstore1

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat loadstore2

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

loads don't do both byte reversal and sign extension

Give the synthesis tools a clue that we don't need to do both byte reversal
and sign extension.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #64 from antonblanchard/reformat-3

Reformat some more files

Merge pull request #63 from antonblanchard/multiply-cleanup

Multiply cleanup

Reformat wishbone code

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat glibc_random

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat simple_ram_behavioural

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat sim_console

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat multiply_tb

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat execute2

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat CR file

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat register file

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reformat multiply code

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Don't use VHDL 2008 condition operator in multiply

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #62 from antonblanchard/byte-reverse-store-opt

Move byte reversal of stores to first cycle

Merge pull request #61 from antonblanchard/execute-cleanup

execute1 no longer needs sim_console

Move byte reversal of stores to first cycle

We are seeing some timing issues with the second cycle of loadstore,
and  we aren't doing much in the first cycle, so move it here.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

execute1 no longer needs sim_console

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #60 from antonblanchard/testbenches

Add a few more test benches

Fix multiply_tb

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Add an icache testbench

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #56 from antonblanchard/writeback-fix3

Remove cycle in writeback

Remove cycle in writeback

The pipeline had a cycle in writeback. Writeback is pretty
simple and unlikely to be a bottleneck, so lets remove it.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #59 from antonblanchard/trap-decode

Fix make check

Merge pull request #58 from antonblanchard/decode2-assert

Fix spurious outstanding assert

Fix make check

We need to finish support for all the trap instructions, but for now
we at least need a decode entry for tw, so we know to stall until the
previous instruction completes. Some of our test cases were failing
because the trap executed before the previous instruction completed.

All these trap instructions need to be resolved at completion, not
in execute.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Fix spurious outstanding assert

Check it in the sequential process, not the combinatorial one.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #57 from antonblanchard/add-nop

Add a decode for the nop instruction

Add a decode for the nop instruction

We want these to go out without any GPR dependencies, so add
a specific entry in decode for them.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #55 from antonblanchard/fetch-fix

Add a default value for RESET_ADDRESS

Add a default value for RESET_ADDRESS

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #51 from antonblanchard/writeback-fix

Some writeback updates

Reformat writeback.vhdl

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Exit if we try to write more than one GPR or CR in a cycle

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #50 from antonblanchard/decode1-opt

No need to gate nia or insn in decode1

No need to gate nia or insn in decode1

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #49 from antonblanchard/icache-2

Add a simple direct mapped icache

Add a simple direct mapped icache

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

SOC memory wishbone should clear ACK regardless of STB

The memory wishbone doesn't clear ACK and move the state machine on
until STB is de-asserted. This seems like it isn't compliant with
the spec and results in a maximum throughput of 1 transfer every
3 cycles.

Fixing this improves the situation to one transfer every 2 cycles.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #48 from antonblanchard/clk_gen_bypass

Fix clk_gen_bypass

Fix clk_gen_bypass

I broke clk_gen_bypass when updating the SOC reset code.

Fixes 03fd06deaf9f ("Rework SOC reset")
Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #47 from antonblanchard/if-fix

Explicitly check against '1' in if statements

Merge pull request #46 from antonblanchard/record-fix

Remove names from end record statements

Explicitly check against '1' in if statements

nvc doesn't like what I think is a VHDL 2008 construct. Lets just
check against '1' explicitly.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Remove names from end record statements

These are optional, and vhdlpp from iverilog barfs on them.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #45 from antonblanchard/fixes

Fix a couple of issues in the recent pipelining merge

Fix issue in loadstore1

We weren't using the register in this stage.

Fixes: 819f8200905f ("Register outputs on loadstore1")
Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Fix issue in execute2

We weren't using the register in this stage.

Fixes: c7aa683ba8aa ("Register outputs on execute2")
Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #44 from antonblanchard/nia-remove

Remove nia from loadstore and multiply

Remove nia from loadstore and multiply

Neither unit needs the NIA, so remove it.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #43 from mikey/trivial

Remove FIXME comment

Remove FIXME comment

This was mistakenly left behind in 4d5abfb430d1 ("Remove dynamic
ranges from code")

Signed-off-by: Michael Neuling <mikey@neuling.org>

Merge pull request #41 from mikey/travis

Allow a full make check on Travis

Merge pull request #42 from antonblanchard/fetch-rework-v2

Fetch rework

Reformat core.vhdl

Remove sim console

We can force all existing code to use the UART console
by passing 0 in bit zero of the sim config register.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Reduce multiply to 2 cycles

We want all non load/store ops to take 2 cycles to make
tracking write back easier.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Register outputs on writeback

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Register outputs on execute2

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Register outputs on loadstore1

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Move debug execute output into decode2

This covers all units, and we avoid double printing.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Rework pipeline, add stall and flush signals

This adds stall and flush signals to the pipeline.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Allow a full make check on Travis

Some Travis instances allow more CPU time. On these we can perform the
full 'make check'.

This patch allows this longer `make check`. To enable it you need to
go into your Travis configuration and add a TRAVIS_FULL_CHECK
environment variable.

If you don't add this environment, the shorter make check_light is
still run.

Signed-off-by: Michael Neuling <mikey@neuling.org>

Merge pull request #40 from antonblanchard/makefile-dependencies

Update Makefile dependencies

Update Makefile dependencies

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Switch soc to use std_ulogic

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Share soc.vhdl between FPGA and sim

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Pass wishbone record to bram memory module

(And rename it to mw_soc_memory).

This makes soc.vhdl simpler and provides the same interface as
the simulated memory, which will help when sharing soc.vhdl
with sim later

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Rework wishbone slave address decoding

Don't make it synchronous, no latches

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Move wishbone arbiter out of the core

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Re-indent and reformat soc.vhdl

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Split FPGA toplevel from soc

This will be useful when we start needing different toplevels for
different boards.

We keep the reset and clock generators in the toplevel as they will
eventually be taken over by litedram when we integrate it, and they
are more likely to change on different system types.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Merge pull request #39 from antonblanchard/no-x-state

Don't send out X state from the memory behavioural

Don't send out X state from the memory behavioural

Just send out all 1s.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #36 from mikey/gitignore

Add new files to git ignore

Merge pull request #38 from antonblanchard/multiply-warn

Quieten multiply warning

Quieten multiply warning

We no longer gate multiply with the valid signal, so it's complaining
a lot. Comment out the warning.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Add new files to git ignore

Signed-off-by: Michael Neuling <mikey@neuling.org>

Merge pull request #35 from antonblanchard/multiply-opt

Simplify multiply

Simplify multiply

No need to gate everything with the valid bit.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #34 from antonblanchard/decode-table

Decode table

Add a decode bit to mark an instruction as single through the pipeline

This is used by the pipelining patches. Mark everyone as single through
the pipeline to start.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

decode1 array fix header

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Merge pull request #33 from antonblanchard/cr-fix

Fix CR forwarding

Merge pull request #32 from antonblanchard/register-file-forwarding

Add forwarding in the register file

Use simulated UART in core test bench

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Make sim poll non-blocking

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Add simulated UART design

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Fix CR forwarding

We weren't actually forwarding writes in the same cycle. Not a
problem right now, but noticed when testing the pipelining series.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Add forwarding in the register file

We need this for the upcoming pipelining patches.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #31 from antonblanchard/no-second-write-port-2

More second write port removal

Merge pull request #30 from antonblanchard/writeback-assert

Add some assertions to writeback

More second write port removal

I missed the register file updates for the second write port
removal.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Add some assertions to writeback

We want to make sure we never complete more than one
instruction per cycle, or write back more than one GPR
or CR per cycle.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Merge pull request #29 from antonblanchard/no-second-write-port

Remove second write port

Merge pull request #28 from antonblanchard/loadstore-cleanup

Remove some more loadstore debug

Remove second write port

We only need two write ports for load with update instructions.
Having two write ports just for this instruction is expensive.

For now we will force them to be the only instruction in the
pipeline, and take two cycles of writeback.

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>

Remove some more loadstore debug

Signed-off-by: Anton Blanchard <anton@linux.ibm.com>