[flash_ctrl] Correct behavior when buffer not enabled.

When buffer is not enabled, the flash_phy_rd may erroneously return data
to back to back transactions when it is not supposed to.

This happens because even when buffers are not enabled, the read data is
written into the holding FIFO between read and descramble stages.  As a
result, the return path falsely thinks the data is available and returns
it.

This causes an issue because even though the data is returned, the front door
logic has already created 2 transactions to the flash, and a result, we have
extra data returning.

The buffer not enabled case can be caused by otp_ctrl not returning the
flash controller's request for a key.  This in turn can happen because
entropy is not yet enabled.  This latter point deserves a wider discussion
as to the right solution.

To fix this, the forward hint is used to distinguish when the data in the
FIFO is valid vs when it is not.

Signed-off-by: Timothy Chen <timothytim@google.com>
1 file changed
tree: b660fd2669546a66b5fa47e8cdae59c3f7ce4026
  1. .github/
  2. ci/
  3. doc/
  4. hw/
  5. site/
  6. sw/
  7. test/
  8. util/
  9. .clang-format
  10. .dockerignore
  11. .flake8
  12. .gitignore
  13. .style.yapf
  14. .svlint.toml
  15. .svls.toml
  16. _index.md
  17. apt-requirements.txt
  18. azure-pipelines.yml
  19. check_tool_requirements.core
  20. CLA
  21. COMMITTERS
  22. CONTRIBUTING.md
  23. LICENSE
  24. meson.build
  25. meson_init.sh
  26. meson_options.txt
  27. python-requirements.txt
  28. README.md
  29. tool_requirements.py
  30. toolchain.txt
  31. topgen-generator.core
  32. topgen-reg-only.core
  33. topgen.core
  34. 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 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).