From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Date: Thu, 15 May 2014 10:41:15 +0200 Subject: [U-Boot] [RFC PATCH] fpga: Added support to load bit stream from SD/MMC In-Reply-To: <20140514212652.GR22182@bill-the-cat> References: <10a1bf356ea627253cef0177c6e1f1d5d03e3d7c.1399039799.git.michal.simek@xilinx.com> <537352B9.8040903@monstr.eu> <20140514212652.GR22182@bill-the-cat> Message-ID: <53747DAB.8080309@monstr.eu> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05/14/2014 11:26 PM, Tom Rini wrote: > On Wed, May 14, 2014 at 01:25:45PM +0200, Michal Simek wrote: >> Hi Tom, >> >> On 05/02/2014 04:10 PM, Michal Simek wrote: >>> From: Siva Durga Prasad Paladugu >>> >>> Added support to load a bitstream image in chunks by reading it in >>> chunks from SD/MMC. >>> Command format: >>> loadfs [dev] [address] [image size] [blocksize] >>> [] >>> Example: fpga loadfs 0 1000000 3dbafc 4000 mmc 0 fpga.bin >>> >>> Signed-off-by: Siva Durga Prasad Paladugu >>> Signed-off-by: Michal Simek >>> --- >>> >>> We have tested just FAT case but I am sending this as RFC >>> to give some inputs from you. >>> The reason for this command is that when u-boot running >>> from OCM on zynq there is just 192kB for u-boot itself >>> and there must be an option to load bitstream to PL >>> because there full DDR controller. >>> Bitstreams are bigger and bigger and reading it with chunks >>> is only one way how to do it. >>> This is also necessary to do it for u-boot SPL when >>> we add fpga support there. >>> >>> The question is if this is the way how to extend fpga command >>> because loadfs should be fpga subcommand. >>> >>> This patch is based on this branch. >>> http://git.denx.de/?p=u-boot/u-boot-microblaze.git;a=shortlog;h=refs/heads/xnext/zynq >>> >>> Thanks for your comments, >>> Michal >>> >>> --- >>> common/cmd_fpga.c | 41 +++++++++++++++++++++ >>> drivers/fpga/fpga.c | 28 ++++++++++++++ >>> drivers/fpga/xilinx.c | 16 ++++++++ >>> drivers/fpga/zynqpl.c | 85 +++++++++++++++++++++++++++++++++++++++++++ >>> include/configs/zynq-common.h | 1 + >>> include/fpga.h | 9 +++++ >>> include/xilinx.h | 3 ++ >>> 7 files changed, 183 insertions(+) >> >> Are you OK with this patch? >> I would expect that you will have some comments regarding this because it is >> command extension. >> If you are OK with this I will add it to my next fpga pull request with >> partial bitstream support. > > It's a new subcommand, seems like a fine idea. Can we leverage > FS_TYPE_ANY so it's not limited to FAT? > Ok. Fixed. I don't know how to use sandbox fs but I have tried it on EXT4. It ended in error message from ext4 but first reading was fine that's why there shouldn't be any problem in flow. Here is v2. http://lists.denx.de/pipermail/u-boot/2014-May/179570.html Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 263 bytes Desc: OpenPGP digital signature URL: