From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Tue, 06 Jun 2023 16:54:28 +0200 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1q6Y4s-00DA6p-4k for lore@lore.pengutronix.de; Tue, 06 Jun 2023 16:54:28 +0200 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1q6Y4p-0007Vh-1o; Tue, 06 Jun 2023 16:54:27 +0200 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1q6Y4l-0007Uf-EL; Tue, 06 Jun 2023 16:54:23 +0200 Message-ID: <682aecd7-d997-a17e-5883-05a54506e45c@pengutronix.de> Date: Tue, 6 Jun 2023 16:54:22 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Content-Language: en-US To: Johannes Zink , distrokit@pengutronix.de References: <20230606-default_enable_fastboot_rock3a-v1-0-42195d538df2@pengutronix.de> <20230606-default_enable_fastboot_rock3a-v1-3-42195d538df2@pengutronix.de> From: Ahmad Fatoum In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [DistroKit] [PATCH 3/5] v8a: images: generate sparse images from rock3a initramfs, kernel and dtb X-BeenThere: distrokit@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: DistroKit Mailinglist List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: patchwork-jzi@pengutronix.de Sender: "DistroKit" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: distrokit-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false On 06.06.23 16:49, Johannes Zink wrote: > On 6/6/23 16:42, Ahmad Fatoum wrote: >> On 06.06.23 16:41, Ahmad Fatoum wrote: >>> On 06.06.23 16:31, Johannes Zink wrote: >>>> This allows to load the images to barebox using android fastboot, since >>>> otherwise the sparse header is missing and the image size is not >>>> properly aligned as required by fastboot. >>> >>> IME fastboot just complains and transfers the files anyway. In that >>> case building sparse images just adds complication. Did your fastboot >>> command fail without this change? >>> >>>> >>>> Signed-off-by: Johannes Zink >>>> --- >>>>   configs/platform-v8a/config/images/rock3a.config | 21 +++++++++++++++++++++ >>>>   1 file changed, 21 insertions(+) >>>> >>>> diff --git a/configs/platform-v8a/config/images/rock3a.config b/configs/platform-v8a/config/images/rock3a.config >>>> index 522d82cdc9c9..cb0c7c4e575d 100644 >>>> --- a/configs/platform-v8a/config/images/rock3a.config >>>> +++ b/configs/platform-v8a/config/images/rock3a.config >>>> @@ -44,3 +44,24 @@ image barebox-rock3a.norimg { >>>>           image = "barebox-rock3a.img" >>>>       } >>>>   } >>>> + >>>> +image initramfs-sparse.img { >>>> +    android-sparse { >>>> +        image = "root.cpio.gz" >>>> +        block-size = 4k >>>> +    } >>>> +} >>>> + >>>> +image kernel-sparse.img { >>>> +    android-sparse { >>>> +        image = "linuximage" >>>> +        block-size = 4k >>>> +    } >>>> +} >> >> Also, these two aren't rock3a specific, so they shouldn't be here >> especially not with such generic sounding names. > > ack, gonna move them to a separate config for v2 of this series. Though, I think they should keep their generic names, as they can be used across all other v8a targets. Their name should be aligned with the file they are in unless they are removed. > > Johannes > >> >>>> + >>>> +image dtb-rock3a-sparse.img { >>>> +    android-sparse { >>>> +        image = "rk3568-rock-3a.dtb" >>>> +        block-size = 4k >>>> +    } >>>> +} >>>> >>> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |