[spi_device] Change addr_latched as a pulse

Problem:

    `addr_latched_i` in `spid_readsram` module is expected to be a pulse
    signal. The `spi_readcmd` module generates the signal as a level by
    comparing `addr_cnt_d` with all zero value.

    As a result, the `strb` register in `spid_readsram` follows the
    current address, which is increased by when a byte is sent to the
    host system. However, the `spid_readsram` logic pushes the data into
    the FIFO already. As the FIFO depth is 2, one more entry has been
    added to the FIFO, which results the host system sees the a byte has
    been shifted.

Resolution:

    Revised the `addr_latched` logic to be a pulse. Either `addr_cnt_d`
    or `addr_latched` can be revised. I chose the latter. Latching the
    latched signal and generated a pulse.

Signed-off-by: Eunchan Kim <eunchan@opentitan.org>
2 files changed
tree: c464d498b6943fbf238cccb9e810ce15e5833668
  1. .github/
  2. ci/
  3. doc/
  4. hw/
  5. rules/
  6. site/
  7. sw/
  8. test/
  9. third_party/
  10. util/
  11. .bazelignore
  12. .bazelrc
  13. .bazelversion
  14. .clang-format
  15. .dockerignore
  16. .flake8
  17. .gitignore
  18. .style.yapf
  19. .svlint.toml
  20. .svls.toml
  21. _index.md
  22. apt-requirements.txt
  23. azure-pipelines.yml
  24. bazelisk.sh
  25. BUILD.bazel
  26. check_tool_requirements.core
  27. CLA
  28. COMMITTERS
  29. CONTRIBUTING.md
  30. LICENSE
  31. meson-config.txt
  32. meson.build
  33. meson_init.sh
  34. meson_options.txt
  35. python-requirements.txt
  36. README.md
  37. tool_requirements.py
  38. toolchain.txt
  39. topgen-reg-only.core
  40. topgen.core
  41. WORKSPACE
  42. yum-requirements.txt
README.md

OpenTitan

OpenTitan logo

About the project

OpenTitan is an open source silicon Root of Trust (RoT) project. OpenTitan will make the silicon RoT design and implementation more transparent, trustworthy, and secure for enterprises, platform providers, and chip manufacturers. OpenTitan is administered by lowRISC CIC as a collaborative project to produce high quality, open IP for instantiation as a full-featured product. See the OpenTitan site and OpenTitan docs for more information about the project.

About this repository

This repository contains hardware, software and utilities written as part of the OpenTitan project. It is structured as monolithic repository, or “monorepo”, where all components live in one repository. It exists to enable collaboration across partners participating in the OpenTitan project.

Documentation

The project contains comprehensive documentation of all IPs and tools. You can access it online at docs.opentitan.org.

How to contribute

Have a look at [CONTRIBUTING]({{< relref “CONTRIBUTING.md” >}}) and our documentation on project organization and processes for guidelines on how to contribute code to this repository.

Licensing

Unless otherwise noted, everything in this repository is covered by the Apache License, Version 2.0 (see LICENSE for full text).