spl: Add support to load FIT from Filesystem
v4 cover spl: Add support to load FIT from Filesystem
Some devices like MMC, USB can be formatted to a FS and can act as a boot media. Given that FIT image load support in SPL support only raw devices, SPL should also be able to support loading FIT image from a File system.
This series add support to load FIT image from a filesystem and also adding hooks to FAT FS.
Changes since v3:
- Use the same function for reading the fit from FS or raw device.
Lokesh Vutla (3):
spl: fit: Fix the number of bytes read when reading fdt from fit
spl: Allow to load a FIT containing U-Boot from FS
spl: Support loading a FIT from FAT FS
common/spl/spl_fat.c | 31 ++++++++++++++++++++-- common/spl/spl_fit.c | 75 ++++++++++++++++++++++++++++++++++++++++++---------- include/spl.h | 10 +++++++ 3 files changed, 100 insertions(+), 16 deletions(-)
Signature · 1 line
2.1.4
PATCH v4 1/3 spl: fit: Fix the number of bytes read when reading fdt from fit
sectors field is not being updated when reading fdt from fit image. Because of this size_of(u-boot.bin) is being read when reading fdt. Fixing it by updating the sectors field properly.
--- common/spl/spl_fit.c | 1 + 1 file changed, 1 insertion(+)
common/spl/spl_fit.c +1 -0
@@ -176,6 +176,7 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)*/ dst = load_ptr + data_size; fdt_offset += base_offset; + sectors = (fdt_len + info->bl_len -1 ) / info->bl_len; count = info->read (info, sector + fdt_offset / info->bl_len, sectors, dst); debug("fit read %x sectors to %x, dst %p, data_offset %x\n",
Signature · 1 line
2.1.4
PATCH v4 2/3 spl: Allow to load a FIT containing U-Boot from FS
In reply to Lokesh VutlaThis provides a way to load a FIT containing U-Boot and a selection of device tree files from a File system. Making sure that all the reads and writes are aligned to their respective needs.
---
- Assuming info->priv is used for passing filename for now and using this for detecting if it a fs read. If this is not preferred I can create a new field for filename.
common/spl/spl_fit.c | 76 +++++++++++++++++++++++++++++++++++++++++----------- include/spl.h | 10 +++++++ 2 files changed, 71 insertions(+), 15 deletions(-)
common/spl/spl_fit.c +61 -15
@@ -82,6 +82,42 @@ static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp)return -ENOENT ; } +static int get_aligned_image_offset (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the first address before offset which is +* aligned to ARCH_DMA_MINALIGN. If it is raw read return the +* block number to which offset belongs. +*/ +if (info->priv) +return offset & ~(ARCH_DMA_MINALIGN -1 ); +else +return offset / info->bl_len; +} + +static int get_aligned_image_overhead (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the difference between the offset and +* the first address before offset which is aligned to +* ARCH_DMA_MINALIGN. If it is raw read return the offset within the +* block. +*/ +if (info->priv) +return offset & (ARCH_DMA_MINALIGN -1 ); +else +return offset % info->bl_len; +} + +static int get_aligned_image_size (struct spl_load_info *info,int data_size, +int offset) +{ +if (info->priv) +return data_size +get_aligned_image_overhead (info, offset); +else +return (data_size + info->bl_len -1 ) / info->bl_len; +} +int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fit) {int sectors;
@@ -91,7 +127,7 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)void *load_ptr;int fdt_offset, fdt_len;int data_offset, data_size; -int base_offset; +int base_offset, align_len;int src_sector;void *dst;
@@ -117,8 +153,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* In fact the FIT has its own load address, but we assume it cannot * be before CONFIG_SYS_TEXT_BASE. */ - fit = (void *)(CONFIG_SYS_TEXT_BASE - size - info->bl_len); - sectors = (size + info->bl_len -1 ) / info->bl_len; + align_len =ARCH_DMA_MINALIGN -1 ; + fit = (void *)((CONFIG_SYS_TEXT_BASE - size - info->bl_len - + align_len) & ~align_len); + sectors =get_aligned_image_size (info, size,0 ); count = info->read (info, sector, sectors, fit);debug ("fit read sector %lx, sectors=%d, dst=%p, count=%lu\n" , sector, sectors, fit, count);
@@ -151,19 +189,23 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* byte will be at 'load'. This may mean we need to load it starting * before then, since we can only read whole blocks. */ - sectors = (data_size + info->bl_len -1 ) / info->bl_len; + sectors =get_aligned_image_size (info, data_size, data_offset); data_offset += base_offset; load_ptr = (void *)load;debug ("U-Boot size %x, data %p\n" , data_size, load_ptr); - dst = load_ptr - (data_offset % info->bl_len); + dst = load_ptr;/* Read the image */ - src_sector = sector + data_offset / info->bl_len; -debug ("image: data_offset=%x, dst=%p, src_sector=%x, sectors=%x\n" , - data_offset, dst, src_sector, sectors); + src_sector = sector +get_aligned_image_offset (info, data_offset); +debug ("Aligned image read: dst=%p, src_sector=%x, sectors=%x\n" , + dst, src_sector, sectors); count = info->read (info, src_sector, sectors, dst);if (count != sectors)return -EIO ; +debug ("image: dst=%p, data_offset=%x, size=%x\n" , dst, data_offset, + data_size); +memcpy (dst, dst +get_aligned_image_overhead (info, data_offset), + data_size);/* Figure out which device tree the board wants to use */ fdt_len =spl_fit_select_fdt (fit, images, &fdt_offset);
@@ -173,14 +215,15 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)/* * Read the device tree and place it after the image. There may be * some extra data before it since we can only read entire blocks. +* And also align the destination address to ARCH_DMA_MINALIGN. */ - dst = load_ptr + data_size; + dst = (void *)((load + data_size + align_len) & ~align_len); fdt_offset += base_offset; - sectors = (fdt_len + info->bl_len -1 ) / info->bl_len; - count = info->read (info, sector + fdt_offset / info->bl_len, sectors, - dst); -debug ("fit read %x sectors to %x, dst %p, data_offset %x\n" , - sectors, spl_image.load_addr, dst, fdt_offset); + sectors =get_aligned_image_size (info, fdt_len, fdt_offset); + src_sector = sector +get_aligned_image_offset (info, fdt_offset); + count = info->read (info, src_sector, sectors, dst); +debug ("Aligned fdt read: dst %p, src_sector = %x, sectors %x\n" , + dst, src_sector, sectors);if (count != sectors)return -EIO ;
@@ -189,7 +232,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* After this we will have the U-Boot image and its device tree ready * for us to start. */ -memcpy (dst, dst + fdt_offset % info->bl_len, fdt_len); +debug ("fdt: dst=%p, data_offset=%x, size=%x\n" , dst, fdt_offset, + fdt_len); +memcpy (load_ptr + data_size, + dst +get_aligned_image_overhead (info, fdt_offset), fdt_len);return 0 ; }
include/spl.h +10 -0
@@ -45,6 +45,16 @@ struct spl_load_info { void *buf); }; +/** +* spl_load_simple_fit() - Loads a fit image from a device. +* @info: Structure containing the information required to load data. +* @sector: Sector number where FIT image is located in the device +* @fdt: Pointer to the copied FIT header. +* +* Reads the FIT image @sector in the device. Loads u-boot image to +* specified load address and copies the dtb to end of u-boot image. +* Returns 0 on success. +*/ int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fdt);#define SPL_COPY_PAYLOAD_ONLY 1
Signature · 1 line
2.1.4
PATCH v4 2/3 spl: Allow to load a FIT containing U-Boot from FS
Hi,
Quoted 158 lines · Lokesh Vutla
This provides a way to load a FIT containing U-Boot and a selection of device tree files from a File system. Making sure that all the reads and writes are aligned to their respective needs.
Signed-off-by Lokesh Vutla <[email protected]>---
- Assuminginfo->privis used for passing filename for now and using this for detecting if it a fs read. If this is not preferred I can create a new field for filename.common/spl/spl_fit.c | 76 +++++++++++++++++++++++++++++++++++++++++----------- include/spl.h | 10 +++++++ 2 files changed, 71 insertions(+), 15 deletions(-)common/spl/spl_fit.c +61
-15@@ -82,6 +82,42 @@ static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp)return -ENOENT ; } +static int get_aligned_image_offset (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the first address before offset which is +* aligned to ARCH_DMA_MINALIGN. If it is raw read return the +* block number to which offset belongs. +*/ +if (info->priv) +return offset & ~(ARCH_DMA_MINALIGN -1 ); +else +return offset / info->bl_len; +} + +static int get_aligned_image_overhead (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the difference between the offset and +* the first address before offset which is aligned to +* ARCH_DMA_MINALIGN. If it is raw read return the offset within the +* block. +*/ +if (info->priv) +return offset & (ARCH_DMA_MINALIGN -1 ); +else +return offset % info->bl_len; +} + +static int get_aligned_image_size (struct spl_load_info *info,int data_size, +int offset) +{ +if (info->priv) +return data_size +get_aligned_image_overhead (info, offset); +else +return (data_size + info->bl_len -1 ) / info->bl_len; +} +int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fit) {int sectors;@@ -91,7 +127,7 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)void *load_ptr;int fdt_offset, fdt_len;int data_offset, data_size; -int base_offset; +int base_offset, align_len;int src_sector;void *dst;@@ -117,8 +153,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* In fact the FIT has its own load address, but we assume it cannot * be before CONFIG_SYS_TEXT_BASE. */ - fit = (void *)(CONFIG_SYS_TEXT_BASE - size - info->bl_len); - sectors = (size + info->bl_len -1 ) / info->bl_len; + align_len =ARCH_DMA_MINALIGN -1 ; + fit = (void *)((CONFIG_SYS_TEXT_BASE - size - info->bl_len - + align_len) & ~align_len); + sectors =get_aligned_image_size (info, size,0 ); count = info->read (info, sector, sectors, fit);debug ("fit read sector %lx, sectors=%d, dst=%p, count=%lu\n" , sector, sectors, fit, count);@@ -151,19 +189,23 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* byte will be at 'load'. This may mean we need to load it starting * before then, since we can only read whole blocks. */ - sectors = (data_size + info->bl_len -1 ) / info->bl_len; + sectors =get_aligned_image_size (info, data_size, data_offset); data_offset += base_offset; load_ptr = (void *)load;debug ("U-Boot size %x, data %p\n" , data_size, load_ptr); - dst = load_ptr - (data_offset % info->bl_len); + dst = load_ptr;/* Read the image */ - src_sector = sector + data_offset / info->bl_len; -debug ("image: data_offset=%x, dst=%p, src_sector=%x, sectors=%x\n" , - data_offset, dst, src_sector, sectors); + src_sector = sector +get_aligned_image_offset (info, data_offset); +debug ("Aligned image read: dst=%p, src_sector=%x, sectors=%x\n" , + dst, src_sector, sectors); count = info->read (info, src_sector, sectors, dst);if (count != sectors)return -EIO ; +debug ("image: dst=%p, data_offset=%x, size=%x\n" , dst, data_offset, + data_size); +memcpy (dst, dst +get_aligned_image_overhead (info, data_offset), + data_size);/* Figure out which device tree the board wants to use */ fdt_len =spl_fit_select_fdt (fit, images, &fdt_offset);@@ -173,14 +215,15 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)/* * Read the device tree and place it after the image. There may be * some extra data before it since we can only read entire blocks. +* And also align the destination address to ARCH_DMA_MINALIGN. */ - dst = load_ptr + data_size; + dst = (void *)((load + data_size + align_len) & ~align_len); fdt_offset += base_offset; - sectors = (fdt_len + info->bl_len -1 ) / info->bl_len; - count = info->read (info, sector + fdt_offset / info->bl_len, sectors, - dst); -debug ("fit read %x sectors to %x, dst %p, data_offset %x\n" , - sectors, spl_image.load_addr, dst, fdt_offset); + sectors =get_aligned_image_size (info, fdt_len, fdt_offset); + src_sector = sector +get_aligned_image_offset (info, fdt_offset); + count = info->read (info, src_sector, sectors, dst); +debug ("Aligned fdt read: dst %p, src_sector = %x, sectors %x\n" , + dst, src_sector, sectors);if (count != sectors)return -EIO ;@@ -189,7 +232,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* After this we will have the U-Boot image and its device tree ready * for us to start. */ -memcpy (dst, dst + fdt_offset % info->bl_len, fdt_len); +debug ("fdt: dst=%p, data_offset=%x, size=%x\n" , dst, fdt_offset, + fdt_len); +memcpy (load_ptr + data_size, + dst +get_aligned_image_overhead (info, fdt_offset), fdt_len);return 0 ; }include/spl.h +3
-0@@ -45,6 +45,16 @@ struct spl_load_info { void *buf); }; +/** +* spl_load_simple_fit() - Loads a fit image from a device. +* @info: Structure containing the information required to load data.
include/spl.h struct spl_load_info { · line 60
Quoted 10 lines
include/spl.h +7
-0+* @sector: Sector number where FIT image is located in the device +* @fdt: Pointer to the copied FIT header. +* +* Reads the FIT image @sector in the device. Loads u-boot image to +* specified load address and copies the dtb to end of u-boot image. +* Returns 0 on success. +*/ int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fdt);#define SPL_COPY_PAYLOAD_ONLY 1
this is not working for me. I haven't had a time to look at it but you might know where the problem is.
Thanks,
Michal
U-Boot SPL 2016.05-rc3-00067-g01d410633461-dirty (May 03 2016 - 14:43:30) Trying to boot from MMC1
reading u-boot.bin
reading u-boot.img
Found FITspl_fit_read: name u-boot.img, file_offset 0, size 44c, buf 7fffb40 reading u-boot.imgspl_fit_read: error reading image u-boot.img, ret - 0
fit read sector 0, sectors=1100, dst=0000000007fffb40, count=1100 data_offset=0, data_size=80310
U-Boot size 80310, data 0000000008000000
Aligned image read: dst=0000000008000000, src_sector=440, sectors=80310spl_fit_read: name u-boot.img, file_offset 440, size 80310, buf 8000000 reading u-boot.imgspl_fit_read: error reading image u-boot.img, ret - 0
image: dst=0000000008000000, data_offset=44c, size=80310 Selecting config 'zynqmp-zcu102', fdt 'fdt at 1'
FIT: Selected 'zynqmp-zcu102'spl_fit_read: name u-boot.img, file_offset 80740, size 6603, buf 8080340 reading u-boot.img
** Unable to read file u-boot.img **spl_fit_read: error reading image u-boot.img, ret - -1 Aligned fdt read: dst 0000000008080340, src_sector = 80740, sectors 6603 reading u-boot.imgspl_load_image_fat: error reading image u-boot.img, err - -1
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###
PATCH v4 2/3 spl: Allow to load a FIT containing U-Boot from FS
Quoted 1 line · Michal Simek
Hi,
[..snip..]
Quoted 22 lines
this is not working for me. I haven't had a time to look at it but you might know where the problem is.
Thanks,
MichalU-Boot SPL 2016.05-rc3-00067-g01d410633461-dirty (May 03 2016 - 14:43:30) Trying to boot from MMC1
reading u-boot.bin
reading u-boot.img
Found FITspl_fit_read: name u-boot.img,file_offset0, size 44c, buf 7fffb40 reading u-boot.imgspl_fit_read: error reading image u-boot.img, ret - 0
fit read sector 0, sectors=1100, dst=0000000007fffb40, count=1100 data_offset=0, data_size=80310
U-Boot size 80310, data 0000000008000000
Aligned image read: dst=0000000008000000, src_sector=440, sectors=80310spl_fit_read: name u-boot.img,file_offset440, size 80310, buf 8000000 reading u-boot.img
Quoted 8 lines
spl_fit_read: error reading image u-boot.img, ret - 0
image: dst=0000000008000000, data_offset=44c, size=80310 Selecting config 'zynqmp-zcu102', fdt 'fdt at 1'
FIT: Selected 'zynqmp-zcu102'spl_fit_read: name u-boot.img,file_offset80740, size 6603, buf 8080340 reading u-boot.img
** Unable to read file u-boot.img **spl_fit_read: error reading image u-boot.img, ret - -1
The error started here. It is able to detect fdt from fit header but not able to read it from file. If you don't mind can you tell me what is the size of the file u-boot.img?
A dumb question: Can you confirm that on this same HEAD the RFC patch from you works fine?
I verified this on 2 different platforms and worked fine. Lets see If I am missing any corner case.
Thanks and regards,
Lokesh
Quoted 7 lines
Aligned fdt read: dst 0000000008080340,
src_sector= 80740, sectors 6603 reading u-boot.imgspl_load_image_fat: error reading image u-boot.img, err - -1
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###
PATCH v4 2/3 spl: Allow to load a FIT containing U-Boot from FS
In reply to Lokesh VutlaQuoted 22 lines · Lokesh Vutla
This provides a way to load a FIT containing U-Boot and a selection of device tree files from a File system. Making sure that all the reads and writes are aligned to their respective needs.
Signed-off-by Lokesh Vutla <[email protected]>---
- Assuminginfo->privis used for passing filename for now and using this for detecting if it a fs read. If this is not preferred I can create a new field for filename.common/spl/spl_fit.c | 76 +++++++++++++++++++++++++++++++++++++++++----------- include/spl.h | 10 +++++++ 2 files changed, 71 insertions(+), 15 deletions(-)common/spl/spl_fit.c +0
-0@@ -82,6 +82,42 @@ static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp)return -ENOENT ; }
common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 94
Quoted 10 lines
common/spl/spl_fit.c +10
-0+static int get_aligned_image_offset (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the first address before offset which is +* aligned to ARCH_DMA_MINALIGN. If it is raw read return the +* block number to which offset belongs. +*/ +if (info->priv) +return offset & ~(ARCH_DMA_MINALIGN -1 ); +else
remove this else.
Quoted 4 lines
common/spl/spl_fit.c +4
-0+return offset / info->bl_len; +} + +static int get_aligned_image_overhead (struct spl_load_info *info,int offset)
common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 108
Quoted 10 lines
common/spl/spl_fit.c +10
-0+{ +/* +* If it is a FS read, get the difference between the offset and +* the first address before offset which is aligned to +* ARCH_DMA_MINALIGN. If it is raw read return the offset within the +* block. +*/ +if (info->priv) +return offset & (ARCH_DMA_MINALIGN -1 ); +else
remove this else.
common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 117
Quoted 9 lines
common/spl/spl_fit.c +9
-0+return offset % info->bl_len; +} + +static int get_aligned_image_size (struct spl_load_info *info,int data_size, +int offset) +{ +if (info->priv) +return data_size +get_aligned_image_overhead (info, offset); +else
remove this else.
Quoted 16 lines
common/spl/spl_fit.c +4
-1+return (data_size + info->bl_len -1 ) / info->bl_len; +} +int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fit) {int sectors;@@ -91,7 +127,7 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)void *load_ptr;int fdt_offset, fdt_len;int data_offset, data_size; -int base_offset; +int base_offset, align_len;int src_sector;void *dst;
common/spl/spl_fit.c int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit) · line 158
Quoted 8 lines
common/spl/spl_fit.c +3
-2@@ -117,8 +153,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* In fact the FIT has its own load address, but we assume it cannot * be before CONFIG_SYS_TEXT_BASE. */ - fit = (void *)(CONFIG_SYS_TEXT_BASE - size - info->bl_len); - sectors = (size + info->bl_len -1 ) / info->bl_len; + align_len =ARCH_DMA_MINALIGN -1 ; + fit = (void *)((CONFIG_SYS_TEXT_BASE - size - info->bl_len - + align_len) & ~align_len);
Don't you want to use ALIGN macro here?
Quoted 34 lines
common/spl/spl_fit.c +10
-5+ sectors =get_aligned_image_size (info, size,0 ); count = info->read (info, sector, sectors, fit);debug ("fit read sector %lx, sectors=%d, dst=%p, count=%lu\n" , sector, sectors, fit, count);@@ -151,19 +189,23 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* byte will be at 'load'. This may mean we need to load it starting * before then, since we can only read whole blocks. */ - sectors = (data_size + info->bl_len -1 ) / info->bl_len; + sectors =get_aligned_image_size (info, data_size, data_offset); data_offset += base_offset; load_ptr = (void *)load;debug ("U-Boot size %x, data %p\n" , data_size, load_ptr); - dst = load_ptr - (data_offset % info->bl_len); + dst = load_ptr;/* Read the image */ - src_sector = sector + data_offset / info->bl_len; -debug ("image: data_offset=%x, dst=%p, src_sector=%x, sectors=%x\n" , - data_offset, dst, src_sector, sectors); + src_sector = sector +get_aligned_image_offset (info, data_offset); +debug ("Aligned image read: dst=%p, src_sector=%x, sectors=%x\n" , + dst, src_sector, sectors); count = info->read (info, src_sector, sectors, dst);if (count != sectors)return -EIO ; +debug ("image: dst=%p, data_offset=%x, size=%x\n" , dst, data_offset, + data_size); +memcpy (dst, dst +get_aligned_image_overhead (info, data_offset), + data_size);/* Figure out which device tree the board wants to use */ fdt_len =spl_fit_select_fdt (fit, images, &fdt_offset);
common/spl/spl_fit.c int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit) · line 220
Quoted 7 lines
common/spl/spl_fit.c +2
-1@@ -173,14 +215,15 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)/* * Read the device tree and place it after the image. There may be * some extra data before it since we can only read entire blocks. +* And also align the destination address to ARCH_DMA_MINALIGN. */ - dst = load_ptr + data_size; + dst = (void *)((load + data_size + align_len) & ~align_len);
Same here.
Thanks,
Michal
PATCH v4 2/3 spl: Allow to load a FIT containing U-Boot from FS
Quoted 23 lines · Michal Simek
Quoted 22 lines · Lokesh Vutla
This provides a way to load a FIT containing U-Boot and a selection of device tree files from a File system. Making sure that all the reads and writes are aligned to their respective needs.
Signed-off-by Lokesh Vutla <[email protected]>---
- Assuminginfo->privis used for passing filename for now and using this for detecting if it a fs read. If this is not preferred I can create a new field for filename.common/spl/spl_fit.c | 76 +++++++++++++++++++++++++++++++++++++++++----------- include/spl.h | 10 +++++++ 2 files changed, 71 insertions(+), 15 deletions(-)common/spl/spl_fit.c +0
-0@@ -82,6 +82,42 @@ static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp)return -ENOENT ; }
Quoted 12 lines
common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 94
Quoted 10 lines
common/spl/spl_fit.c +10
-0+static int get_aligned_image_offset (struct spl_load_info *info,int offset) +{ +/* +* If it is a FS read, get the first address before offset which is +* aligned to ARCH_DMA_MINALIGN. If it is raw read return the +* block number to which offset belongs. +*/ +if (info->priv) +return offset & ~(ARCH_DMA_MINALIGN -1 ); +else remove this else.
Sure.
Quoted 17 lines
Quoted 5 lines
common/spl/spl_fit.c +4
-0+return offset / info->bl_len; +} + +static int get_aligned_image_overhead (struct spl_load_info *info,int offset)common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 108
Quoted 10 lines
common/spl/spl_fit.c +10
-0+{ +/* +* If it is a FS read, get the difference between the offset and +* the first address before offset which is aligned to +* ARCH_DMA_MINALIGN. If it is raw read return the offset within the +* block. +*/ +if (info->priv) +return offset & (ARCH_DMA_MINALIGN -1 ); +else remove this else.
Sure.
Quoted 12 lines
common/spl/spl_fit.c static int spl_fit_select_fdt(const void *fdt, int images, int *fdt_offsetp) · line 117
Quoted 10 lines
common/spl/spl_fit.c +9
-0+return offset % info->bl_len; +} + +static int get_aligned_image_size (struct spl_load_info *info,int data_size, +int offset) +{ +if (info->priv) +return data_size +get_aligned_image_overhead (info, offset); +else remove this else.
Sure.
Quoted 17 lines
Quoted 17 lines
common/spl/spl_fit.c +4
-1+return (data_size + info->bl_len -1 ) / info->bl_len; +} +int spl_load_simple_fit (struct spl_load_info *info,ulong sector,void *fit) {int sectors;@@ -91,7 +127,7 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)void *load_ptr;int fdt_offset, fdt_len;int data_offset, data_size; -int base_offset; +int base_offset, align_len;int src_sector;void *dst;
Quoted 11 lines
common/spl/spl_fit.c int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit) · line 158
Quoted 9 lines
common/spl/spl_fit.c +3
-2@@ -117,8 +153,10 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* In fact the FIT has its own load address, but we assume it cannot * be before CONFIG_SYS_TEXT_BASE. */ - fit = (void *)(CONFIG_SYS_TEXT_BASE - size - info->bl_len); - sectors = (size + info->bl_len -1 ) / info->bl_len; + align_len =ARCH_DMA_MINALIGN -1 ; + fit = (void *)((CONFIG_SYS_TEXT_BASE - size - info->bl_len - + align_len) & ~align_len);Don't you want to use ALIGN macro here?
ALIGN() will return the next aligned address which is equal to:
fit = (void *)((CONFIG_SYS_TEXT_BASE - size - info->bl_len + align_len) & ~align_len);
But here we need the previous aligned address so that we do not loose any data.
Thanks and regards,
Lokesh
Quoted 48 lines
Quoted 35 lines
common/spl/spl_fit.c +10
-5+ sectors =get_aligned_image_size (info, size,0 ); count = info->read (info, sector, sectors, fit);debug ("fit read sector %lx, sectors=%d, dst=%p, count=%lu\n" , sector, sectors, fit, count);@@ -151,19 +189,23 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)* byte will be at 'load'. This may mean we need to load it starting * before then, since we can only read whole blocks. */ - sectors = (data_size + info->bl_len -1 ) / info->bl_len; + sectors =get_aligned_image_size (info, data_size, data_offset); data_offset += base_offset; load_ptr = (void *)load;debug ("U-Boot size %x, data %p\n" , data_size, load_ptr); - dst = load_ptr - (data_offset % info->bl_len); + dst = load_ptr;/* Read the image */ - src_sector = sector + data_offset / info->bl_len; -debug ("image: data_offset=%x, dst=%p, src_sector=%x, sectors=%x\n" , - data_offset, dst, src_sector, sectors); + src_sector = sector +get_aligned_image_offset (info, data_offset); +debug ("Aligned image read: dst=%p, src_sector=%x, sectors=%x\n" , + dst, src_sector, sectors); count = info->read (info, src_sector, sectors, dst);if (count != sectors)return -EIO ; +debug ("image: dst=%p, data_offset=%x, size=%x\n" , dst, data_offset, + data_size); +memcpy (dst, dst +get_aligned_image_overhead (info, data_offset), + data_size);/* Figure out which device tree the board wants to use */ fdt_len =spl_fit_select_fdt (fit, images, &fdt_offset);common/spl/spl_fit.c int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit) · line 220
Quoted 7 lines
common/spl/spl_fit.c +2
-1@@ -173,14 +215,15 @@ int spl_load_simple_fit(struct spl_load_info *info, ulong sector, void *fit)/* * Read the device tree and place it after the image. There may be * some extra data before it since we can only read entire blocks. +* And also align the destination address to ARCH_DMA_MINALIGN. */ - dst = load_ptr + data_size; + dst = (void *)((load + data_size + align_len) & ~align_len);Same here.
Thanks,
Michal
PATCH v4 3/3 spl: Support loading a FIT from FAT FS
In reply to Lokesh VutlaDetect a FIT when loading from a FAT File system and handle it using the new FIT SPL support.
--- common/spl/spl_fat.c | 31 +++++++++++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-)
common/spl/spl_fat.c +29 -2
@@ -15,6 +15,7 @@#include <fat.h> #include <errno.h> #include <image.h> +#include <libfdt.h> static int fat_registered;
@@ -39,6 +40,20 @@ static int spl_register_fat_device(struct blk_desc *block_dev, int partition)return err; } +static ulong spl_fit_read (struct spl_load_info *load,ulong file_offset, +ulong size,void *buf) +{ +loff_t actread; +int ret; +char *filename = (char *)load->priv; + + ret =fat_read_file (filename, buf, file_offset, size, &actread); +if (ret) +return ret; +else +return actread; +} +int spl_load_image_fat (struct blk_desc *block_dev,int partition,const char *filename)
@@ -57,9 +72,21 @@ int spl_load_image_fat(struct blk_desc *block_dev,if (err <=0 ) goto end; -spl_parse_image_header (header); +if (IS_ENABLED (CONFIG_SPL_LOAD_FIT ) && +image_get_magic (header) ==FDT_MAGIC ) { +struct spl_load_info load; - err =file_fat_read (filename, (u8 *)spl_image.load_addr,0 ); +debug ("Found FIT\n" ); + load.read = spl_fit_read; + load.bl_len =1 ; + load.priv = (void *)filename; + +return spl_load_simple_fit (&load,0 , header); + }else { +spl_parse_image_header (header); + + err =file_fat_read (filename, (u8 *)spl_image.load_addr,0 ); + } end: #ifdef CONFIG_SPL_LIBCOMMON_SUPPORT
Signature · 1 line
2.1.4
PATCH v4 3/3 spl: Support loading a FIT from FAT FS
Quoted 25 lines · Lokesh Vutla
Detect a FIT when loading from a FAT File system and handle it using the new FIT SPL support.
Signed-off-by Lokesh Vutla <[email protected]>--- common/spl/spl_fat.c | 31 +++++++++++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-)common/spl/spl_fat.c +2
-0@@ -15,6 +15,7 @@#include <fat.h> #include <errno.h> #include <image.h> +#include <libfdt.h> static int fat_registered;@@ -39,6 +40,20 @@ static int spl_register_fat_device(struct blk_desc *block_dev, int partition)return err; } +static ulong spl_fit_read(struct spl_load_info *load, ulong file_offset,
common/spl/spl_fat.c static int spl_register_fat_device(struct blk_desc *block_dev, int partition) · line 53
Quoted 10 lines
common/spl/spl_fat.c +10
-0+ ulong size, void *buf) +{ + loff_t actread; +int ret; +char *filename = (char *)load->priv; + + ret =fat_read_file (filename, buf, file_offset, size, &actread); +if (ret) +return ret; +else
you can remove this else.
Thanks,
Michal
PATCH v4 3/3 spl: Support loading a FIT from FAT FS
Quoted 25 lines · Michal Simek
Quoted 24 lines · Lokesh Vutla
Detect a FIT when loading from a FAT File system and handle it using the new FIT SPL support.
Signed-off-by Lokesh Vutla <[email protected]>--- common/spl/spl_fat.c | 31 +++++++++++++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-)common/spl/spl_fat.c +1
-0@@ -15,6 +15,7 @@#include <fat.h> #include <errno.h> #include <image.h> +#include <libfdt.h> static int fat_registered;@@ -39,6 +40,20 @@ static int spl_register_fat_device(struct blk_desc *block_dev, int partition)return err; }
Quoted 13 lines
common/spl/spl_fat.c static int spl_register_fat_device(struct blk_desc *block_dev, int partition) · line 53
Quoted 11 lines
common/spl/spl_fat.c +11
-0+static ulong spl_fit_read (struct spl_load_info *load,ulong file_offset, +ulong size,void *buf) +{ +loff_t actread; +int ret; +char *filename = (char *)load->priv; + + ret =fat_read_file (filename, buf, file_offset, size, &actread); +if (ret) +return ret; +else you can remove this else.
Sure. Will fix it in the next version.
Thanks and regards,
Lokesh
Quoted 4 lines
Thanks,
Michal