All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vicente Bergas <vicencb@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/4] rockchip: Reduce prerequisites from the host
Date: Mon, 18 Jun 2018 22:07:32 +0200	[thread overview]
Message-ID: <CAAMcf8CKWsZLcyYNR_zsZE9=38ZLyEk395WDsgeDVBGrtmZmQQ@mail.gmail.com> (raw)
In-Reply-To: <8c5acb36-c6cb-5aea-7ab1-52b08c153f20@rock-chips.com>

On Wed, 13 Jun 2018 11:51:13 +0800, kever.yang at rock-chips.com wrote:
>(snip)
>I just not understand why remove the dependency on python is so important,
>there already many modules depend on python.

On Wed, 13 Jun 2018 11:57:48 +0800, kever.yang at rock-chips.com wrote:
>(snip)
> Could you share why you don't want to use python, can convert the
>script to C?

Hi Kever,
there are several reasons, here are the ones I can think of:
 1.- Python is a ~100MB weight dependency.
 2.- Because of (1) I don not have it installed by default.
 3.- My Linux distribution of choice defaults to python3 but
     u-boot requires 'python' to be python2.
 4.- "Shall use C language whenever possible."
     http://www.denx.de/wiki/U-Boot/DesignRequirements
 5.- It is not just a dependency on what needs to be installed
     on the host, is also a dependency on which programming
     languages u-boot's contributors need to know about.
     So, it can be considered an entry barrier.

I don not have strong feelings on this getting merged and certainly
will not get upset if it does not. It is entirely your decision.

I have submitted this patch series mostly "for your information" and
because Philipp thought that it "looks worthwhile":
On Fri, 1 Jun 2018 00:37:17 +0200, philipp.tomsich at
theobroma-systems.com wrote:
>(snip)
>> On 1 Jun 2018, at 00:28, Vicente Bergas <vicencb@gmail.com> wrote:
>(snip)
>> SPL_FIT_GENERATOR and SPL_OF_PLATDATA require python.
>> In order to remove this dependency:
>> 1.- I have written a C version for SPL_FIT_GENERATOR.
>> 2.- Disabled SPL_OF_PLATDATA, it just works.
>>
>> MKIMAGE_DTC_PATH requires dtc in the PATH.
>> In order to remove this dependency, I have changed it to use the built-in one.
>>
>> If there is interest in those changes, I can post the full patch.
>
>Please submit a patch--it certainly looks worthwhile.

Regards,
  Vicenç.

  reply	other threads:[~2018-06-18 20:07 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-01 16:46 [U-Boot] [PATCH 0/4] rockchip: Reduce prerequisites from the host Vicente Bergas
2018-06-01 16:46 ` [U-Boot] [PATCH 1/4] add make_fit_atf in tools Vicente Bergas
2018-06-01 16:46 ` [U-Boot] [PATCH 2/4] rockchip: rk3399: use make_fit_atf instead of make_fit_atf.py Vicente Bergas
2018-06-01 16:46 ` [U-Boot] [PATCH 3/4] rockchip: rk3399: disable CONFIG_SPL_OF_PLATDATA Vicente Bergas
2018-06-13  3:51   ` Kever Yang
2018-06-01 16:46 ` [U-Boot] [PATCH 4/4] rockchip: rk3399: set CONFIG_MKIMAGE_DTC_PATH Vicente Bergas
2018-06-13  3:52   ` Kever Yang
2018-06-13  3:57 ` [U-Boot] [PATCH 0/4] rockchip: Reduce prerequisites from the host Kever Yang
2018-06-18 20:07   ` Vicente Bergas [this message]
2018-06-18 20:14     ` Dr. Philipp Tomsich

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='CAAMcf8CKWsZLcyYNR_zsZE9=38ZLyEk395WDsgeDVBGrtmZmQQ@mail.gmail.com' \
    --to=vicencb@gmail.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.