All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@xilinx.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4] zynqmp: Add generic target
Date: Thu, 10 May 2018 11:10:01 +0200	[thread overview]
Message-ID: <6453a02f-31a4-e39c-1790-07ff47e68cec@xilinx.com> (raw)
In-Reply-To: <20180425074352.57148-1-agraf@suse.de>

Hi,

On 25.4.2018 09:43, Alexander Graf wrote:
> I would like to create a generic U-Boot build that adapts itself completely
> based on the DT passed in. That way we can potentially support running
> random board configurations with a single U-Boot binary built as part of
> the distribution.
> 
> Currently a few things are still missing to make it a full reality. The
> most obvious one I think is the EEPROM location. This would need to also
> move into something described by DT.
> 
> Apart from that, we're almost there. This patch adds a defconfig that simply
> contains all drivers we could make use of. We can then enable individual
> boards along the way and slowly adapt everything to be fully DT described
> while we identify each missing bit.
> 
> Signed-off-by: Alexander Graf <agraf@suse.de>
> 
> ---
> 
> v1 -> v2:
> 
>   - Remove debug uart
> 
> v2 -> v3
> 
>   - show model information instead of custom IDENT_STRING
> 
> v3 -> v4:
> 
>   - reintroduce CONFIG_IDENT_STRING
>   - remove CONFIG_DEFAULT_DEVICE_TREE
>   - run through savedefconfig
>   - replace CONFIG_SYS_I2C_ZYNQ with CONFIG_SYS_I2C_CADENCE
>   - remove CONFIG_ZYNQ_SDHCI{0,1} - needs to be solved generically
>   - add xhci usb1 to xhci controller list
> ---
>  configs/xilinx_zynqmp_generic_defconfig | 86 +++++++++++++++++++++++++++++++++
>  include/configs/xilinx_zynqmp_generic.h | 21 ++++++++
>  2 files changed, 107 insertions(+)
>  create mode 100644 configs/xilinx_zynqmp_generic_defconfig
>  create mode 100644 include/configs/xilinx_zynqmp_generic.h
> 

I still have the same concern as I have told you here.
https://lists.denx.de/pipermail/u-boot/2018-April/326477.html

It is not that generic as it is suggest to be.
Resolving that issues first is IMHO the best way to go instead of
pretending that we are there by new target.

Thanks,
Michal

      reply	other threads:[~2018-05-10  9:10 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-25  7:43 [U-Boot] [PATCH v4] zynqmp: Add generic target Alexander Graf
2018-05-10  9:10 ` 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=6453a02f-31a4-e39c-1790-07ff47e68cec@xilinx.com \
    --to=michal.simek@xilinx.com \
    --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.