All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <monstr@monstr.eu>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] zynq: treat ps7_init.c/h as external files to ignore them
Date: Thu, 08 May 2014 15:13:27 +0200	[thread overview]
Message-ID: <536B82F7.4020003@monstr.eu> (raw)
In-Reply-To: <1399552248-10142-1-git-send-email-yamada.m@jp.panasonic.com>

On 05/08/2014 02:30 PM, Masahiro Yamada wrote:
> ps7_init.c and ps7_init.h are supposed to be exported by hw project
> and copied to board/xilinx/zynq/ directory.
> 
> We want them to be ignored by git.
> So what we should do is to always treat them as external files
> rather than replacing ps7_init.c
> 
> This commit does:
> 
>  - Move a weak function ps7_init() to arch/arm/cpu/armv7/zynq/spl.c
>    and delete board/xilinx/zynq/ps7_init.c
> 
>  - Compile board/xilinx/zynq/ps7_init.c only when it exists
> 
>  - Add .gitignore to ignore ps7_init.c/h
> 
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Cc: Michal Simek <michal.simek@xilinx.com>
> ---
>  arch/arm/cpu/armv7/zynq/spl.c |  8 ++++++++
>  board/xilinx/zynq/.gitignore  |  1 +
>  board/xilinx/zynq/Makefile    |  5 ++++-
>  board/xilinx/zynq/ps7_init.c  | 12 ------------
>  4 files changed, 13 insertions(+), 13 deletions(-)
>  create mode 100644 board/xilinx/zynq/.gitignore
>  delete mode 100644 board/xilinx/zynq/ps7_init.c
> 

Signed-off-by: Michal Simek <monstr@monstr.eu>

Masahiro: I have just tried to run this on the top of my zynq branch
with fpga changes I have sent and I can't see any problem in MMC boot mode.

As we discussed earlier - I am doing these changes in zynq-common
not to deal with dts files now.

diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h
index 8e0f93599fc4..fb08a978019a 100644
--- a/include/configs/zynq-common.h
+++ b/include/configs/zynq-common.h
@@ -226,15 +226,6 @@
 #define CONFIG_FIT
 #define CONFIG_FIT_VERBOSE     1 /* enable fit_format_{error,warning}() */

-/* FDT support */
-#define CONFIG_OF_CONTROL
-#define CONFIG_OF_SEPARATE
-#define CONFIG_DISPLAY_BOARDINFO_LATE
-
-/* RSA support */
-#define CONFIG_FIT_SIGNATURE
-#define CONFIG_RSA
-

And I get this (I am sending generic board enabling for Zynq in a minute).

U-Boot SPL 2014.04-00236-gb861b285b4e9-dirty (May 08 2014 - 15:08:00)
mmc boot
reading system.dtb
spl_load_image_fat_os: error reading image system.dtb, err - -1
reading u-boot.img
reading u-boot.img


U-Boot 2014.04-00236-gb861b285b4e9-dirty (May 08 2014 - 15:08:00)

I2C:   ready
DRAM:  Memory: ECC disabled
1 GiB
MMC:   zynq_sdhci: 0
Using default environment

In:    serial
Out:   serial
Err:   serial
Net:   Gem.e000b000
Warning: failed to set MAC address

Hit any key to stop autoboot:  0
zynq-uboot>


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: <http://lists.denx.de/pipermail/u-boot/attachments/20140508/4e9a0a2f/attachment.pgp>

      reply	other threads:[~2014-05-08 13:13 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-08 12:30 [U-Boot] [PATCH] zynq: treat ps7_init.c/h as external files to ignore them Masahiro Yamada
2014-05-08 13:13 ` Michal Simek [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=536B82F7.4020003@monstr.eu \
    --to=monstr@monstr.eu \
    --cc=u-boot@lists.denx.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.