Add OPCodes package with sprite operation constants

.devcontainer/devcontainer.json

@@ -4,20 +4,27 @@
   "runArgs": [
     "--privileged",
     "--cap-add=SYS_ADMIN",
-    "--shm-size=2g"
+    "--shm-size=2g",
+    "-v",
+    "/run/user/1000/gnupg/S.gpg-agent:/run/user/1000/gnupg/S.gpg-agent"
   ],
   "customizations": {
     "vscode": {
+      "extensions": [
+        "/home/xilinx/vsxirepo/vhdl-by-hgb.vsix",
+        "eamodio.gitlens"
+      ],
       "settings": {
         "terminal.integrated.defaultProfile.linux": "bash"
       }
     }
   },
   "remoteUser": "xilinx",
-  "mounts": [
-    "source=${localWorkspaceFolder},target=/workspaces/project,type=bind"
-  ],
-  "workspaceFolder": "/workspaces/project",
+  "workspaceMount": "source=${localWorkspaceFolder},target=/workspaces/${localWorkspaceFolderBasename},type=bind",
+  "workspaceFolder": "/workspaces/${localWorkspaceFolderBasename}",
   "features": {},
-  "forwardPorts": [10000]
-}
+  "forwardPorts": [
+    10000
+  ],
+  "postStartCommand": "git config --global user.signingkey 87C8A5DD5C14DF55DBE1DB4199AC216D447E61C0 && git config --global gpg.format openpgp && git config --global commit.gpgsign true && git config --global tag.forceSignAnnotated true && sudo apt update && sudo apt upgrade -y"
+}

.github/workflows/build.yml

@@ -1,4 +1,4 @@
-name: Manuelles Build mit ISE
+name: Build with HDLBuild
 
 on:
   workflow_dispatch:
@@ -9,9 +9,12 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v2
+        with:
+            submodules: true
 
       - name: Build
         run: |
-          cd "$GITHUB_WORKSPACE/build"
-          make
+          cd "$GITHUB_WORKSPACE"
+          hdlbuild dep
+          hdlbuild build
 

.gitignore

@@ -1 +1,6 @@
+.hdlbuild_deps/
+.working/
+reports/
+output/
 .locale/
+vhdl_ls.toml

.gitmodules

@@ -1,9 +0,0 @@
-[submodule "libs/AXI-HS-MUX"]
-	path = libs/AXI-HS-MUX
-	url = ssh://git@git.0xmax42.io:33612/maxp/AXI-HS-MUX.git
-[submodule "libs/AXI-HS-Scheduler"]
-	path = libs/AXI-HS-Scheduler
-	url = ssh://git@git.0xmax42.io:33612/maxp/AXI-HS-Scheduler.git
-[submodule "build"]
-	path = build
-	url = git@github.com:0xMax42/Xilinx-ISE-Build.git

project.cfg

@@ -1,344 +0,0 @@
-## Main settings.. ##
-
-# Project name
-# @remark The name of the project is used as default name for the top module and the ucf file
-PROJECT = SpriteChannel
-
-# Target device
-# @example xc3s1200e-4-fg320 | xc5vlx50t-1-ff1136
-TARGET_PART = xc3s1200e-4-fg320
-
-# Path to the Xilinx ISE installation
-XILINX = /opt/Xilinx/14.7/ISE_DS/ISE
-
-# Optional the name of the top module (default is the project name)
-TOPLEVEL = SpriteChannel
-
-# Optional the path/name of the ucf file (default is the project name)
-CONSTRAINTS = src/SpriteChannel.ucf
-
-# Optional a target to copy the bit file to (make copy)
-# COPY_TARGET_DIR = 
-
-## ## ## ## ## ## ## ##
-# ---------------------
-
-## Source files settings.. ##
-# The source files to be compiled
-# @example `VSOURCE += src/main.v` (add a single Verilog file per line)
-# @example `VHDSOURCE += src/main.vhd` (add a single VHDL file per line)
-VHDSOURCE += ../Asynchronous-FIFO-AXI-Handshake/libs/Pipeline-AXI-Handshake/src/PipelineRegister.vhd
-VHDSOURCE += ../Asynchronous-FIFO-AXI-Handshake/libs/Pipeline-AXI-Handshake/src/PipelineController.vhd
-VHDSOURCE += ../Asynchronous-FIFO-AXI-Handshake/libs/Pipeline-AXI-Handshake/src/PipelineStage.vhd
-VHDSOURCE += ../Asynchronous-FIFO-AXI-Handshake/libs/Pipeline-AXI-Handshake/src/PipelineFilter.vhd
-# VHDSOURCE += libs/PriorityEncoders.vhd
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_2.vhdl
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_4.vhdl
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_8.vhdl
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_16.vhdl
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_32.vhdl
-# VHDSOURCE += libs/AXI-HS-Scheduler/build/AXI_Handshaking_Scheduler_64.vhdl
-# VHDSOURCE += libs/AXI-HS-MUX/AXI_HS_MUX.vhd
-VHDSOURCE += libs/OPCodes.vhd
-
-VHDSOURCE += src/VerticalSpritePipeline.vhd
-VHDSOURCE += src/RegisterFile.vhd
-VHDSOURCE += src/OPDecoder.vhd
-VHDSOURCE += src/Rom.vhd
-VHDSOURCE += src/RomPackage.vhd
-VHDSOURCE += src/HorizontalSpritePipeline.vhd
-VHDSOURCE += src/SpriteChannel.vhd
-
-
-## Test files settings.. ##
-# The testbench files to be compiled
-# @example `VTEST += tests/main_tb.v` (add a single Verilog testbench file per line)
-# @example `VHDTEST += tests/main_tb.vhd` (add a single VHDL testbench file per line)
-
-# VHDTEST += test/Scheduler_tb.vhd
-# VHDTEST += test/OPDecoder_tb.vhd
-# VHDTEST += test/YHitCheck_tb.vhd
-# VHDTEST += test/SpriteChannel_tb.vhd
-# VHDTEST += test/RegisterFile_tb.vhd
-
-## ## ## ## ## ## ## ##
-# ---------------------
-
-## ISE executable settings.. ##
-
-# General command line options to be passed to all ISE executables (default is `-intstyle xflow`)
-# COMMON_OPTS = 
-
-# Options for the XST synthesizer
-#### Synthese Options (XST) #####
-
-# Optimization goal: prioritize speed or area.
-# Values: Speed | Area
-XST_OPTS += -opt_mode Speed
-
-# Optimization level: more aggressive optimizations at level 2.
-# Values: 1 | 2
-XST_OPTS += -opt_level 2
-
-# Use the new XST parser (recommended for modern designs).
-# Values: yes | no
-XST_OPTS += -use_new_parser yes
-
-# Preserve design hierarchy or allow flattening for optimization.
-# Values: Yes | No | Soft
-XST_OPTS += -keep_hierarchy No
-
-# Determines how hierarchy is preserved in the netlist.
-# Values: As_Optimized | Rebuilt
-XST_OPTS += -netlist_hierarchy As_Optimized
-
-# Global optimization strategy for nets.
-# Values: AllClockNets | Offset_In_Before | Offset_Out_After | Inpad_To_Outpad | Max_Delay
-XST_OPTS += -glob_opt AllClockNets
-
-## Misc ##
-
-# Enable reading of IP cores.
-# Values: YES | NO
-XST_OPTS += -read_cores YES
-
-# Do not write timing constraints into synthesis report.
-# Values: YES | NO
-XST_OPTS += -write_timing_constraints NO
-
-# Analyze paths across different clock domains.
-# Values: YES | NO
-XST_OPTS += -cross_clock_analysis NO
-
-# Character used to separate hierarchy levels in instance names.
-# Default: /
-XST_OPTS += -hierarchy_separator /
-
-# Delimiters used for bus signals.
-# Values: <> | [] | () | {}
-XST_OPTS += -bus_delimiter <>
-
-# Maintain original case of identifiers.
-# Values: Maintain | Upper | Lower
-XST_OPTS += -case Maintain
-
-# Target maximum utilization ratio for slices.
-# Values: 1–100
-XST_OPTS += -slice_utilization_ratio 100
-
-# Target maximum utilization ratio for BRAMs.
-# Values: 1–100
-XST_OPTS += -bram_utilization_ratio 100
-
-# Use Verilog 2001 syntax features.
-# Values: YES | NO
-XST_OPTS += -verilog2001 YES
-
-#### HDL Options ####
-
-## FSM ##
-
-# Extract FSMs (Finite State Machines) from HDL code.
-# Values: YES | NO
-XST_OPTS += -fsm_extract YES
-
-# Encoding strategy for FSMs.
-# Values: Auto | Gray | One-Hot | Johnson | Compact | Sequential | Speed1 | User
-XST_OPTS += -fsm_encoding Auto
-
-# Add safe logic for undefined FSM states.
-# Values: Yes | No
-XST_OPTS += -safe_implementation No
-
-# Structure used to implement FSMs.
-# Values: LUT | BRAM
-XST_OPTS += -fsm_style LUT
-
-## RAM/ROM ##
-
-# Extract RAM inference from HDL.
-# Values: Yes | No
-XST_OPTS += -ram_extract Yes
-
-# Style used to implement RAM.
-# Values: Auto | Block | Distributed
-XST_OPTS += -ram_style Auto
-
-# Extract ROM inference from HDL.
-# Values: Yes | No
-XST_OPTS += -rom_extract Yes
-
-# Style used for implementing ROM.
-# Values: Auto | Distributed | Block
-XST_OPTS += -rom_style Auto
-
-# Enable or disable automatic BRAM packing.
-# Values: YES | NO
-XST_OPTS += -auto_bram_packing NO
-
-## MUX/Decoder/Shift Register ##
-
-# Extract multiplexers where possible.
-# Values: Yes | No | Force
-XST_OPTS += -mux_extract Yes
-
-# Style used for implementing MUX logic.
-# Values: Auto | MUXCY | MUXF
-XST_OPTS += -mux_style Auto
-
-# Extract decoder logic from behavioral code.
-# Values: YES | NO
-XST_OPTS += -decoder_extract YES
-
-# Extract and optimize priority encoder structures.
-# Values: Yes | No | Force
-XST_OPTS += -priority_extract Yes
-
-# Extract shift register logic.
-# Values: YES | NO
-XST_OPTS += -shreg_extract YES
-
-# Extract simple shift operations into dedicated hardware.
-# Values: YES | NO
-XST_OPTS += -shift_extract YES
-
-## Multiplier ##
-
-# Style for implementing multipliers.
-# Values: Auto | LUT | Pipe_LUT | Pipe_Block | Block
-XST_OPTS += -mult_style Auto
-
-## Misc ##
-
-# Collapse XOR trees where beneficial.
-# Values: YES | NO
-XST_OPTS += -xor_collapse YES
-
-# Share resources like adders or multipliers between logic blocks.
-# Values: YES | NO | Force
-XST_OPTS += -resource_sharing YES
-
-# Convert asynchronous resets to synchronous where possible.
-# Values: YES | NO
-XST_OPTS += -async_to_sync NO
-
-#### Xilinx Specific Options ####
-
-## Optimization ##
-
-# Enable removal of logically equivalent registers.
-# Values: YES | NO
-XST_OPTS += -equivalent_register_removal YES
-
-# Duplicate registers to reduce fanout or improve timing.
-# Values: YES | NO
-XST_OPTS += -register_duplication YES
-
-# Move registers across logic to balance timing.
-# Values: Yes | No | Forward | Backward
-XST_OPTS += -register_balancing No
-
-# Use clock enable signals where possible.
-# Values: Auto | Yes | No
-XST_OPTS += -use_clock_enable Yes
-
-# Use synchronous set (preset) signals when available.
-# Values: Auto | Yes | No
-XST_OPTS += -use_sync_set Yes
-
-# Use synchronous reset signals where possible.
-# Values: Auto | Yes | No
-XST_OPTS += -use_sync_reset Yes
-
-## I/O ##
-
-# Insert IO buffers for top-level ports.
-# Values: YES | NO
-XST_OPTS += -iobuf YES
-
-# Placement strategy for IOB registers (Auto = let tools decide).
-# Values: Auto | YES | NO
-XST_OPTS += -iob Auto
-
-## Misc ##
-
-# Maximum allowed fanout for a net.
-# Values: integer (e.g., 500)
-XST_OPTS += -max_fanout 500
-
-# Maximum number of BUFGs (global buffers) to use.
-# Values: 0–32 (device-dependent)
-XST_OPTS += -bufg 24
-
-# Enable logic packing into slices.
-# Values: YES | NO
-XST_OPTS += -slice_packing YES
-
-# Try to reduce the number of primitive instances used.
-# Values: YES | NO
-XST_OPTS += -optimize_primitives NO
-
-# Margin in percent beyond the target slice utilization.
-# Values: 0–100
-XST_OPTS += -slice_utilization_ratio_maxmargin 5
-
-
-# Options for the NGDBuild tool
-# NGDBUILD_OPTS =
-
-# Options for the MAP tool
-# @example -mt 2 (multi-threading with 2 threads)
-# @example -cm speed (speed optimization)
-# @example -ol high
-# @example -detail
-# @example -timing
-MAP_OPTS = -detail -timing -ol high
-
-# Options for the PAR tool
-# @example -mt 2 (multi-threading with 2 threads)
-# @example -ol high
-PAR_OPTS = -ol high
-
-# Options for the BitGen tool
-# @example -g Compress (compress bitstream)
-# @example -g StartupClk:Cclk (specify the startup clock to onboard clock)
-# @example -g StartupClk:JtagClk (specify the startup clock to JTAG clock)
-BITGEN_OPTS = -g StartupClk:JtagClk
-
-# Options for the Trace tool
-# TRACE_OPTS =
-
-# Options for the Fuse tool
-# FUSE_OPTS = 
-
-# Options for the ISim simulator
-# @example -gui (start the simulator in GUI mode)
-# ISIM_OPTS =
-
-# Options for the ISim batch file
-# @example vcd dumpfile $@.vcd \n vcd dumpvars -m /UUT \n run 1 sec \n vcd dumpflush \n quit
-# ISIM_CMD = vcd dumpfile $@.vcd \n vcd dumpvars -m /UUT \n run 1 sec \n vcd dumpflush \n quit
-
-## ## ## ## ## ## ## ##
-# ---------------------
-
-## Programmer settings.. ##
-
-# The programmer to use
-# @example impact | digilent | xc3sprog
-# @remark impact is the default Xilinx programmer and you must create a impact.cmd file in the root directory..
-PROGRAMMER = 
-
-## Digilent JTAG cable settings
-
-# @remark Use the `djtgcfg enum` command to list all available devices
-# DJTG_DEVICE = DOnbUsb
-
-# The index of the JTAG device for the `prog` target
-# DJTG_INDEX = 0
-
-# The index of the flash device for the `flash` target
-# DJTG_FLASH_INDEX = 1
-
-## ## ## ## ## ## ## ##
-# ---------------------

project.yml

@@ -0,0 +1,270 @@
+name: SpriteChannel
+topmodule: SpriteChannel
+target_device: xc3s1200e-4-fg320
+xilinx_path: /opt/Xilinx/14.7/ISE_DS/ISE
+
+constraints: src/SpriteChannel.ucf
+
+sources:
+  vhdl:
+    - path: src/*.vhd
+      library: work
+    - path: libs/*.vhd
+      library: work
+
+testbenches:
+  vhdl:
+    - path: tests/*.vhd
+      library: work
+
+dependencies:
+   - git: "https://git.0xmax42.io/HDL/Pipeline-AXI-Handshake.git"
+     rev: "main"
+
+build:
+  build_dir: working
+  report_dir: reports
+  copy_target_dir: output
+
+# Tool Optionen
+tool_options:
+  common: 
+    - "-intstyle"
+    - "xflow"
+
+  ngdbuild: []
+
+  map: 
+    - "-detail"
+    - "-timing"
+    - "-ol"
+    - "high"
+
+  par: []
+
+  bitgen: 
+    - "-g"
+    - "StartupClk:JtagClk"
+
+  trace:
+    - "-v"
+    - "3"
+    - "-n"
+    - "3"
+
+  fuse: 
+    - "-incremental"
+
+  isim: 
+    - "-gui"
+
+  xst: 
+    # Optimization goal: prioritize speed or area.
+    # Values: Speed | Area
+    - "-opt_mode Speed"
+
+    # Optimization level: more aggressive optimizations at level 2.
+    # Values: 1 | 2
+    - "-opt_level 2"
+
+    # Use the new XST parser (recommended for modern designs).
+    # Values: yes | no
+    - "-use_new_parser yes"
+
+    # Preserve design hierarchy or allow flattening for optimization.
+    # Values: Yes | No | Soft
+    - "-keep_hierarchy No"
+
+    # Determines how hierarchy is preserved in the netlist.
+    # Values: As_Optimized | Rebuilt
+    - "-netlist_hierarchy As_Optimized"
+
+    # Global optimization strategy for nets.
+    # Values: AllClockNets | Offset_In_Before | Offset_Out_After | Inpad_To_Outpad | Max_Delay
+    - "-glob_opt AllClockNets"
+
+    ## Misc ##
+
+    # Enable reading of IP cores.
+    # Values: YES | NO
+    - "-read_cores YES"
+
+    # Do not write timing constraints into synthesis report.
+    # Values: YES | NO
+    - "-write_timing_constraints NO"
+
+    # Analyze paths across different clock domains.
+    # Values: YES | NO
+    - "-cross_clock_analysis NO"
+
+    # Character used to separate hierarchy levels in instance names.
+    # Default: /
+    - "-hierarchy_separator /"
+
+    # Delimiters used for bus signals.
+    # Values: <> | [] | () | {}
+    - "-bus_delimiter <>"
+
+    # Maintain original case of identifiers.
+    # Values: Maintain | Upper | Lower
+    - "-case Maintain"
+
+    # Target maximum utilization ratio for slices.
+    # Values: 1–100
+    - "-slice_utilization_ratio 100"
+
+    # Target maximum utilization ratio for BRAMs.
+    # Values: 1–100
+    - "-bram_utilization_ratio 100"
+
+    # Use Verilog 2001 syntax features.
+    # Values: YES | NO
+    - "-verilog2001 YES"
+
+    #### HDL Options ####
+
+    ## FSM ##
+
+    # Extract FSMs (Finite State Machines) from HDL code.
+    # Values: YES | NO
+    - "-fsm_extract YES"
+
+    # Encoding strategy for FSMs.
+    # Values: Auto | Gray | One-Hot | Johnson | Compact | Sequential | Speed1 | User
+    - "-fsm_encoding Auto"
+
+    # Add safe logic for undefined FSM states.
+    # Values: Yes | No
+    - "-safe_implementation No"
+
+    # Structure used to implement FSMs.
+    # Values: LUT | BRAM
+    - "-fsm_style LUT"
+
+    ## RAM/ROM ##
+
+    # Extract RAM inference from HDL.
+    # Values: Yes | No
+    - "-ram_extract Yes"
+
+    # Style used to implement RAM.
+    # Values: Auto | Block | Distributed
+    - "-ram_style Auto"
+
+    # Extract ROM inference from HDL.
+    # Values: Yes | No
+    - "-rom_extract Yes"
+
+    # Style used for implementing ROM.
+    # Values: Auto | Distributed | Block
+    - "-rom_style Auto"
+
+    # Enable or disable automatic BRAM packing.
+    # Values: YES | NO
+    - "-auto_bram_packing NO"
+
+    ## MUX/Decoder/Shift Register ##
+
+    # Extract multiplexers where possible.
+    # Values: Yes | No | Force
+    - "-mux_extract Yes"
+
+    # Style used for implementing MUX logic.
+    # Values: Auto | MUXCY | MUXF
+    - "-mux_style Auto"
+
+    # Extract decoder logic from behavioral code.
+    # Values: YES | NO
+    - "-decoder_extract YES"
+
+    # Extract and optimize priority encoder structures.
+    # Values: Yes | No | Force
+    - "-priority_extract Yes"
+
+    # Extract shift register logic.
+    # Values: YES | NO
+    - "-shreg_extract YES"
+
+    # Extract simple shift operations into dedicated hardware.
+    # Values: YES | NO
+    - "-shift_extract YES"
+
+    ## Multiplier ##
+
+    # Style for implementing multipliers.
+    # Values: Auto | LUT | Pipe_LUT | Pipe_Block | Block
+    - "-mult_style Auto"
+
+    ## Misc ##
+
+    # Collapse XOR trees where beneficial.
+    # Values: YES | NO
+    - "-xor_collapse YES"
+
+    # Share resources like adders or multipliers between logic blocks.
+    # Values: YES | NO | Force
+    - "-resource_sharing YES"
+
+    # Convert asynchronous resets to synchronous where possible.
+    # Values: YES | NO
+    - "-async_to_sync NO"
+
+    #### Xilinx Specific Options ####
+
+    ## Optimization ##
+
+    # Enable removal of logically equivalent registers.
+    # Values: YES | NO
+    - "-equivalent_register_removal YES"
+
+    # Duplicate registers to reduce fanout or improve timing.
+    # Values: YES | NO
+    - "-register_duplication YES"
+
+    # Move registers across logic to balance timing.
+    # Values: Yes | No | Forward | Backward
+    - "-register_balancing No"
+
+    # Use clock enable signals where possible.
+    # Values: Auto | Yes | No
+    - "-use_clock_enable Yes"
+
+    # Use synchronous set (preset) signals when available.
+    # Values: Auto | Yes | No
+    - "-use_sync_set Yes"
+
+    # Use synchronous reset signals where possible.
+    # Values: Auto | Yes | No
+    - "-use_sync_reset Yes"
+
+    ## I/O ##
+
+    # Insert IO buffers for top-level ports.
+    # Values: YES | NO
+    - "-iobuf YES"
+
+    # Placement strategy for IOB registers (Auto = let tools decide).
+    # Values: Auto | YES | NO
+    - "-iob Auto"
+
+    ## Misc ##
+
+    # Maximum allowed fanout for a net.
+    # Values: integer (e.g., 500)
+    - "-max_fanout 500"
+
+    # Maximum number of BUFGs (global buffers) to use.
+    # Values: 0–32 (device-dependent)
+    - "-bufg 24"
+
+    # Enable logic packing into slices.
+    # Values: YES | NO
+    - "-slice_packing YES"
+
+    # Try to reduce the number of primitive instances used.
+    # Values: YES | NO
+    - "-optimize_primitives NO"
+
+    # Margin in percent beyond the target slice utilization.
+    # Values: 0–100
+    - "-slice_utilization_ratio_maxmargin 5"

src/VerticalSpritePipeline.vhd

@@ -82,7 +82,8 @@ begin
 
     INST0_VSpritePipeline_Y_Sprite : entity work.PipelineRegister
         generic map(
-            G_Width => G_Y_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Y_Width
             )
         port map(
             I_CLK    => I_CLK,
@@ -93,7 +94,8 @@ begin
 
     INST0_VSpritePipeline_Y_Request : entity work.PipelineRegister
         generic map(
-            G_Width => G_Y_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Y_Width
             )
         port map(
             I_CLK    => I_CLK,
@@ -109,7 +111,8 @@ begin
 
     INST_VSpritePipeline_Y_Bottom_Sprite : entity work.PipelineRegister
         generic map(
-            G_Width => G_Y_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Y_Width
             )
         port map(
             I_CLK    => I_CLK,
@@ -120,7 +123,8 @@ begin
 
     INST1_VSpritePipeline_Y_Sprite : entity work.PipelineRegister
         generic map(
-            G_Width => G_Y_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Y_Width
             )
         port map(
             I_CLK    => I_CLK,
@@ -131,7 +135,8 @@ begin
 
     INST1_VSpritePipeline_Y_Request : entity work.PipelineRegister
         generic map(
-            G_Width => G_Y_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Y_Width
             )
         port map(
             I_CLK    => I_CLK,
@@ -147,15 +152,26 @@ begin
         ) else '0';
 
     --@ Calculate the offset address of the sprite
-    C_Offset <= std_logic_vector(
-        to_unsigned(
-            K_SPRITE_ROW_OFFSETS(to_integer(unsigned(R1_Y_Request) - unsigned(R1_Y_Sprite))),
-            C_Offset'length)
-        );
+    process (R1_Y_Request, R1_Y_Sprite)
+        variable V_SPRITE_ROW_OFFSETS_Address : integer := 0;
+    begin
+        V_SPRITE_ROW_OFFSETS_Address := to_integer(unsigned(R1_Y_Request) - unsigned(R1_Y_Sprite));
+
+        if V_SPRITE_ROW_OFFSETS_Address > 0 and V_SPRITE_ROW_OFFSETS_Address < G_Sprite_Height then
+            C_Offset <= std_logic_vector(
+                to_unsigned(
+                    K_SPRITE_ROW_OFFSETS(V_SPRITE_ROW_OFFSETS_Address),
+                    C_Offset'length)
+                );
+        else
+            C_Offset <= (others => '0');
+        end if;
+    end process;
 
     INST_IsVisible_OutputRegister : entity work.PipelineRegister
         generic map(
-            G_Width => 1
+            G_PipelineStages => 1,
+            G_Width          => 1
             )
         port map(
             I_CLK     => I_CLK,
@@ -166,7 +182,8 @@ begin
 
     INST_Offset_OutputRegister : entity work.PipelineRegister
         generic map(
-            G_Width => G_Offset_Width
+            G_PipelineStages => 1,
+            G_Width          => G_Offset_Width
             )
         port map(
             I_CLK    => I_CLK,