Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] blackfin and micropython
Date: Mon, 21 Sep 2015 09:13:56 +0200	[thread overview]
Message-ID: <20150921091356.17452d5c@free-electrons.com> (raw)
In-Reply-To: <CAFOYHZCPCAEdq_wjxY+P=U+5cmthuEy_TkTzu6HNsLLcXHhBVg@mail.gmail.com>

Chris,

On Mon, 21 Sep 2015 16:25:54 +1200, Chris Packham wrote:

> I'm just looking at the micropython build failures and the blackfin
> ones have have me a bit stumped.
> 
> The ones currently showing up[1] are format specifier errors. The code
> in question[2] doesn't look particularly bad and other
> architectures/compilers aren't complaining I guess it is getting
> confused with the math around const byte * and unsigned int. I can
> work around that issue by adding -Wno-error=format to CFLAGS for
> BR2_bfin. But that leads to the next issue.
> 
>   modffi.c:87: error: expected specifier-qualifier-list before ?ffi_closure?
>   modffi.c: In function ?mod_ffi_callback?:
>   modffi.c:248: error: ?mp_obj_fficallback_t? has no member named ?clo?
> 
> I think these are because nothing defines FFI_CLOSURES. ffitarget.h
> for bfin seems a lot sparser than for other architectures so I'm
> thinking that's the problem.
> 
> I notice python, python3 and guile are not available for bfin. Not
> that they explicitly exclude it but probably because of on of the
> other dependencies they all have.

Python is not available because it depends on BR2_USE_MMU as it uses
fork().

Guile is not available, because it depends on libatomic, which is not
available on Blackfin.

None of these seem to apply to Micropython. However, it would be fine
for me if you did:

	# libffi doesn't provide the closure implementation on Blackfin
	depends on !BR2_bfin

And that's it. The number of Blackfin developers using the mainline
Buildroot seems very limited, so I don't think it's worth the effort
fixing all packages, especially when the fix is non-trivial.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2015-09-21  7:13 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-21  4:25 [Buildroot] blackfin and micropython Chris Packham
2015-09-21  7:13 ` Thomas Petazzoni [this message]
2015-09-21  8:51   ` [Buildroot] [PATCH] micropython: Disable for Blackfin Chris Packham
2015-09-21  9:22     ` Baruch Siach
2015-09-22  4:18       ` [Buildroot] [PATCHv2] " Chris Packham

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=20150921091356.17452d5c@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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