Fix for issue #58

[Silabs-ArjanB]

Fix for #58

• Added bus error for non-supported sbaccess values
• Corrected bus error code for non-supported sbaccess values
• Added bus error for misaligned SBA access attempts

Signed-off-by: Arjan Bink [email protected]

[Silabs-ArjanB]

Hi, could the CI please be started again? (The failure seems unrelated to the committed code)

[bluewww]

Needs a rebase

[Silabs-ArjanB]

@bluewww Merged in master

[bluewww]

I thought about this again, I think its best if we just now allow 8-bit and 16-bit accesses since the code is already in place and seems correct to me see #57

[Silabs-ArjanB]

I thought about this again, I think its best if we just now allow 8-bit and 16-bit accesses since the code is already in place and seems correct to me see #57

Hi @bluewww Fully agreed that that would be the ideal fix. I went down that path first and quickly realized that this is more work than I wanted to invest in this, so instead of adding (not officially required support for) byte and halfword accesses I chose to at least fix the bugs in the currently supported access types. Note that the debugger knows which access types are supported via sbcs (and not supporting all of them is allowed). The code for the byte enables is largely there, but also the wdata, rdata, and address lsb will need fixes to support byte and halfword accesses.

[noytzach]

Hi @bluewww and @Silabs-ArjanB,
Please see #106, for adding 8b and 16b accesses

[omerguzelelectronicguy]

This merge request doesn't need to be merged bcs the including changes are already added to main. So I thought it can be closed to prevent misunderstanding. @bluewww