Skip to content
Snippets Groups Projects
  1. Feb 05, 2009
    • Stefan Roese's avatar
      cfi_flash: Fix typo in cfi_flash.c · e1fb6d0d
      Stefan Roese authored
      
      Patch "flash/cfi_flash: Use virtual sector start address, not phys"
      introduced a small typo and compilation warning for systems with CFI
      legacy support (e.g. hcu4). This patch fixes it.
      
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      e1fb6d0d
    • Stefan Roese's avatar
      jedec_flash: Only use manufacturer defines from common flash.h · 28745db9
      Stefan Roese authored
      
      This patch removes the double defined manufacturer defines from
      jedec_flash.c. Since the common defines in flash.h are 32bit
      we now need the (16) cast. This patch also removes the compilation
      warning (e.g. seen on hcu5):
      
      ./MAKEALL hcu5
      Configuring for hcu5 board...
      jedec_flash.c:219: warning: large integer implicitly truncated to unsigned type
      
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      28745db9
    • Stefan Roese's avatar
      cfi_flash: Silence compilation warning · ec21d5cf
      Stefan Roese authored
      
      Patch "flash/cfi_flash: Use virtual sector start address, not phys"
      introduced a small compilation warning. This patch fixes it.
      
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      ec21d5cf
    • Becky Bruce's avatar
      flash/cfi_flash: Use virtual sector start address, not phys · 09ce9921
      Becky Bruce authored
      
      include/flash.h was commented to say that the address in
      flash_info->start was a physical address.  However, from u-boot's
      point of view, and looking at most flash code, it makes more
      sense for this to be a virtual address.  So I corrected the
      comment to indicate that this was a virtual address.
      
      The only flash driver that was actually treating the address
      as physical was the mtd/cfi_flash driver.  However, this code
      was using it inconsistently as it actually directly dereferenced
      the "start" element, while it used map_physmem to get a
      virtual address in other places.  I changed this driver so
      that the code which initializes the info->start field calls
      map_physmem to get a virtual address, eliminating the need for
      further map_physmem calls.  The code is now consistent.
      
      The *only* place a physical address should be used is when defining the
      flash banks list that is used to initialize the flash_info struct,
      usually found in the board config file.
      
      Signed-off-by: default avatarBecky Bruce <beckyb@kernel.crashing.org>
      Signed-off-by: default avatarStefan Roese <sr@denx.de>
      09ce9921
  2. Feb 03, 2009
  3. Feb 02, 2009
  4. Feb 01, 2009
  5. Jan 31, 2009
Loading