From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 28 May 2013 12:00:15 -0400 Subject: [U-Boot] dfu: dfu and UBI Volumes In-Reply-To: <51A44D86.2050108@denx.de> References: <519F97CA.4060901@denx.de> <6AD958CB-3CFC-4362-B72D-511147D041AC@antoniou-consulting.com> <20130524171213.GT17119@bill-the-cat> <51A1B522.2050405@denx.de> <20130527192135.GX17119@bill-the-cat> <51A42CCD.1020607@denx.de> <20130528055824.3606538116A@gemini.denx.de> <51A44D86.2050108@denx.de> Message-ID: <20130528160015.GE5829@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, May 28, 2013 at 08:24:06AM +0200, Heiko Schocher wrote: > Hello Wolfgang, > > Am 28.05.2013 07:58, schrieb Wolfgang Denk: > > Dear Heiko, > > > > In message <51A42CCD.1020607@denx.de> you wrote: > >> > >>> I would imagine, but testing and implementation might show a better way, > >>> we do UBI as ubi ubiN volume-name, ie: > >>> rootfs ubi ubi0 rootfs > >>> user ubi ubi0 user-data > >>> and so forth, and augment dfu_nand.c with UBI knowledge, ala dfu_mmc and > >>> fat/ext knowledge. > >> > >> Yes, I think, thats the way to go ... > > > > I doubt that dfu_nand.c is the right place for this. What if I start > > using DFU on NOR flash? The decisions which device type (NAND, MMC, > > NOR, USB mass storage, ...) to habdle on one side, and which partition > > type / image type (raw, UBI volume, file, ...) on the other side are > > fully orthogonal to each other. They should be handled by independent > > code, and not one of them repeated for all implementations of the > > other. > > Yes, exactly ... We can see about what re-org makes sense once we've got another implementation here. We might be able to share the filesystem-based write code, but that in part might cause some screams since it'll depend on our continued (ab)use of run_command. The device-specific code does end up being the device-specific API part. Unless we're adding ubifs in addition to ubi volume support to DFU, we don't have real shared filesystem stuff, yet. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: