Skip to content
Snippets Groups Projects
  1. Aug 18, 2021
  2. Aug 17, 2021
  3. Aug 16, 2021
  4. Aug 13, 2021
  5. Aug 12, 2021
    • Raul E Rangel's avatar
      drivers/storage/sdhci: Add probe_pci_sdhci_host · 0656da87
      Raul E Rangel authored
      
      Currently the established pattern of adding PCI SDHCI controllers is:
      
        if (pci_find_device(GENESYS_PCI_VID, GL9755S_PCI_DID, &pci_dev))
            sd = new_pci_sdhci_host(pci_dev, SDHCI_PLATFORM_REMOVABLE, 0, 0);
      
      This has two downsides:
      1) It violates the depthcharge philosophy that we only access hardware
         when it's about to be used.
      2) pci_find_device is very slow on the zork platform. It takes 7ms.
         This is due to the fact that the PCI devices on the buses enter low
         power mode. The Genesys SDHCI controller has a 3ms PortTPowerOnTime
         with a 4uS sleep time. So it quickly adds up when scanning the bus
         multiple times.
      
      This change adds a new method of adding SDHCI controllers. It defers
      any probing until the device is actually needed. This means that
      for removable SDHCI controllers, they never get accessed in normal
      mode. probe_pci_sdhci_host can also accept a bridge device and the
      first device connected will be checked. This is exactly how
      new_nvme_ctrlr works. It saves on having to scan the entire PCI space
      looking for a specific VID/DID.
      
      I created a sdhci_pci_table with vid/did pairs so we can reduce the
      duplication across the boards. This way we can centralize all the
      quirks.
      
      BUG=b:179303870, b:171470014
      BRANCH=zork
      TEST=Boot ezkinil and verify SD card still functions
      Logs show:
      > Found SDHCI 17a0:9750 at 2:0.0
      
      Signed-off-by: default avatarRaul E Rangel <rrangel@chromium.org>
      Change-Id: Ife2fd883dec8b41bf198324fbc25c31579a41e45
      Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/depthcharge/+/2676807
      
      
      Reviewed-by: default avatarMathew King <mathewk@chromium.org>
      Reviewed-by: default avatarRob Barnes <robbarnes@google.com>
      Commit-Queue: Tim Wawrzynczak <twawrzynczak@chromium.org>
      0656da87
    • Yu-Ping Wu's avatar
      tests: Fix heap declaration · 085e9e48
      Yu-Ping Wu authored
      
      The current code
      
       char *_eheap = _heap + sizeof(_heap);
      
      sets the "value" of the _eheap pointer. However, to work with
      libpayload's libc/malloc.c it is the "address" of _eheap that needs to
      be set.
      
      Add TEST_SYMBOL() and TEST_REGION() macros to tests/test.h. Fix heap
      declaration by using TEST_SYMBOL(heap, size).
      
      BUG=b:172343578
      TEST=make unit-tests
      BRANCH=none
      
      Change-Id: I34f96ea4565cb06dee0b2a46c0a0ff44a597182e
      Signed-off-by: default avatarYu-Ping Wu <yupingso@chromium.org>
      Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/depthcharge/+/3088401
      
      
      Tested-by: default avatarJakub Czapiga <czapiga@google.com>
      Reviewed-by: default avatarJakub Czapiga <czapiga@google.com>
      Reviewed-by: default avatarJulius Werner <jwerner@chromium.org>
      085e9e48
  6. Aug 11, 2021
  7. Aug 10, 2021
  8. Aug 09, 2021
  9. Aug 07, 2021
  10. Aug 04, 2021
  11. Jul 29, 2021
  12. Jul 28, 2021
  13. Jul 26, 2021
  14. Jul 22, 2021
  15. Jul 21, 2021
  16. Jul 20, 2021
  17. Jul 16, 2021
  18. Jul 15, 2021
  19. Jul 14, 2021
  20. Jul 13, 2021
Loading