mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Wadim Egorov <w.egorov@phytec.de>
To: barebox@lists.infradead.org
Subject: Re: [PATCH v2] image: am335x: Generate SPI MLO images
Date: Fri, 05 Jun 2015 09:06:31 +0200	[thread overview]
Message-ID: <55714A77.8090009@phytec.de> (raw)
In-Reply-To: <1433486348-29192-1-git-send-email-w.egorov@phytec.de>



On 05.06.2015 08:39, Wadim Egorov wrote:
> Generate *.spi.img files for SPI loading on AM335x.
>
> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> Please ignore the first patch. That was a bad approach.
>
> I have two questions.
> 1. Does the af inventions GF board support SPI boot? If not, we can drop the
>     spi image generation.
> 2. Building the barebox with more than one core will produce errors:

Oh, missed a dependency here.

$(obj)/%.mlospi: $(obj)/% $(obj)/%.mlo FORCE
         $(call if_changed,mlo_spi_image)


>
> fopen: No such file or directory
>    SPI-IMG images/start_am33xx_phytec_phycore_sram_128mb.pblx.mlospi
>    SPI-IMG images/start_am33xx_phytec_phycore_sram_512mb.pblx.mlospi
> make[1]: *** [images/start_am33xx_afi_gf_sram.pblx.mlospi] Fehler 1
> make[1]: *** Auf noch nicht beendete Prozesse wird gewartet …fopen: No such file or directory
>
> make[1]: *** [images/start_am33xx_phytec_phycore_sram_256mb.pblx.mlospi] Fehler 1
>    SPI-IMG images/start_am33xx_phytec_phycore_sram_2x512mb.pblx.mlospi
> fopen: No such file or directory
> make[1]: fopen: No such file or directory
> *** [images/start_am33xx_phytec_phycore_sram_128mb.pblx.mlospi] Fehler 1
> make[1]: *** [images/start_am33xx_phytec_phycore_sram_512mb.pblx.mlospi] Fehler 1
> fopen: No such file or directory
> make[1]: *** [images/start_am33xx_phytec_phycore_sram_2x512mb.pblx.mlospi] Fehler 1
> make: *** [images] Fehler 2
>
> This seems to be a makefile dependency problem. Any ideas how to fix this?
>
> ---
>   images/Makefile.am33xx | 28 ++++++++++++++++++++++++++++
>   1 file changed, 28 insertions(+)
>
> diff --git a/images/Makefile.am33xx b/images/Makefile.am33xx
> index eae2a6a..e9b211d 100644
> --- a/images/Makefile.am33xx
> +++ b/images/Makefile.am33xx
> @@ -7,13 +7,23 @@ quiet_cmd_mlo_image = MLO     $@
>   $(obj)/%.mlo: $(obj)/% FORCE
>   	$(call if_changed,mlo_image)
>   
> +# %.mlospi - convert MLO image for SPI loading
> +# ----------------------------------------------------------------
> +quiet_cmd_mlo_spi_image = SPI-IMG $@
> +      cmd_mlo_spi_image = scripts/mk-omap-image -s -a 0x402f0400 $<.mlo > $@
> +
> +$(obj)/%.mlospi: $(obj)/% FORCE
> +	$(call if_changed,mlo_spi_image)
> +
>   pblx-$(CONFIG_MACH_AFI_GF) += start_am33xx_afi_gf_sdram
>   FILE_barebox-am33xx-afi-gf.img = start_am33xx_afi_gf_sdram.pblx
>   am33xx-barebox-$(CONFIG_MACH_AFI_GF) += barebox-am33xx-afi-gf.img
>   
>   pblx-$(CONFIG_MACH_AFI_GF) += start_am33xx_afi_gf_sram
>   FILE_barebox-am33xx-afi-gf-mlo.img = start_am33xx_afi_gf_sram.pblx.mlo
> +FILE_barebox-am33xx-afi-gf-mlo.spi.img = start_am33xx_afi_gf_sram.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_AFI_GF) += barebox-am33xx-afi-gf-mlo.img
> +am33xx-mlospi-$(CONFIG_MACH_AFI_GF) += barebox-am33xx-afi-gf-mlo.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycore_sdram
>   FILE_barebox-am33xx-phytec-phycore.img = start_am33xx_phytec_phycore_sdram.pblx
> @@ -25,19 +35,27 @@ am33xx-barebox-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycore_sram_256mb
>   FILE_barebox-am33xx-phytec-phycore-mlo-256mb.img = start_am33xx_phytec_phycore_sram_256mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phycore-mlo-256mb.spi.img = start_am33xx_phytec_phycore_sram_256mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-256mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-256mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycore_sram_128mb
>   FILE_barebox-am33xx-phytec-phycore-mlo-128mb.img = start_am33xx_phytec_phycore_sram_128mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phycore-mlo-128mb.spi.img = start_am33xx_phytec_phycore_sram_128mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-128mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-128mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycore_sram_512mb
>   FILE_barebox-am33xx-phytec-phycore-mlo-512mb.img = start_am33xx_phytec_phycore_sram_512mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phycore-mlo-512mb.spi.img = start_am33xx_phytec_phycore_sram_512mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-512mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-512mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycore_sram_2x512mb
>   FILE_barebox-am33xx-phytec-phycore-mlo-2x512mb.img = start_am33xx_phytec_phycore_sram_2x512mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phycore-mlo-2x512mb.spi.img = start_am33xx_phytec_phycore_sram_2x512mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-2x512mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycore-mlo-2x512mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phyflex_sdram
>   FILE_barebox-am33xx-phytec-phyflex.img = start_am33xx_phytec_phyflex_sdram.pblx
> @@ -45,11 +63,15 @@ am33xx-barebox-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phyflex
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phyflex_sram_256mb
>   FILE_barebox-am33xx-phytec-phyflex-mlo-256mb.img = start_am33xx_phytec_phyflex_sram_256mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phyflex-mlo-256mb.spi.img = start_am33xx_phytec_phyflex_sram_256mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phyflex-mlo-256mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phyflex-mlo-256mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phyflex_sram_512mb
>   FILE_barebox-am33xx-phytec-phyflex-mlo-512mb.img = start_am33xx_phytec_phyflex_sram_512mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phyflex-mlo-512mb.spi.img = start_am33xx_phytec_phyflex_sram_512mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phyflex-mlo-512mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phyflex-mlo-512mb.spi.img
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycard_sdram
>   FILE_barebox-am33xx-phytec-phycard.img = start_am33xx_phytec_phycard_sdram.pblx
> @@ -57,7 +79,9 @@ am33xx-barebox-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycard
>   
>   pblx-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += start_am33xx_phytec_phycard_sram_256mb
>   FILE_barebox-am33xx-phytec-phycard-mlo-256mb.img = start_am33xx_phytec_phycard_sram_256mb.pblx.mlo
> +FILE_barebox-am33xx-phytec-phycard-mlo-256mb.spi.img = start_am33xx_phytec_phycard_sram_256mb.pblx.mlospi
>   am33xx-mlo-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycard-mlo-256mb.img
> +am33xx-mlospi-$(CONFIG_MACH_PHYTEC_SOM_AM335X) += barebox-am33xx-phytec-phycard-mlo-256mb.spi.img
>   
>   pblx-$(CONFIG_MACH_BEAGLEBONE) += start_am33xx_beaglebone_sdram
>   FILE_barebox-am33xx-beaglebone.img = start_am33xx_beaglebone_sdram.pblx
> @@ -72,3 +96,7 @@ image-y += $(am33xx-mlo-y)
>   else
>   image-y += $(am33xx-barebox-y)
>   endif
> +
> +ifdef CONFIG_OMAP_BUILD_SPI
> +image-y += $(am33xx-mlospi-y)
> +endif


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2015-06-05  7:07 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-05  6:39 Wadim Egorov
2015-06-05  7:06 ` Wadim Egorov [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55714A77.8090009@phytec.de \
    --to=w.egorov@phytec.de \
    --cc=barebox@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox