public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Patch to enable LEDs on sunxi/Cubietruck
Date: Tue, 12 May 2015 17:15:26 +0200	[thread overview]
Message-ID: <5552190E.9090505@redhat.com> (raw)
In-Reply-To: <20150512153931.6fd4a240@maestro.informatik.hs-augsburg.de>

Hi,

On 05/12/2015 03:39 PM, Prof. Dr. Gundolf Kiefer wrote:
> Dear developers,
>
> the attached patch adds LED support for the Cubietruck board.
>
> I found that the support for the onboard leds is missing for
> sunxi-based boards in the latest Debian unstable and perhaps
> upstream version of U-Boot. IMHO LEDs are very helpful for headless
> systems. Perhaps the patch can be helpful somehow.
>
> The code has basically been ported from the U-Boot version maintained at
> linux-sunxi.org. As in the sunxi version, the blue led is switched on
> by default.
>
> Kind regards,
>
> Gundolf Kiefer

Thanks for the patch, but we really do not want to add new options to
CONFIG_SYS_EXTRA_OPTIONS, at a minimum you need to add new Kconfig
options for the leds, using gpio names for them, and sunxi_name_to_gpio
in some place to translate to gpio numbers.

But we really do not want that either for upstream u-boot master we've
switched to using devicetree for all sunxi boards, and we really want
to use devicetree for this, which already has the led info, rather then
duplicating the led info in sunxi defconfig files.

Simon, do we already have support for gpio-leds through devicetree
in the device-model code somewhere?

Regards,

Hans



>
>
> Begin forwarded message:
>
> Date: Tue, 05 May 2015 09:46:39 -0700
> From: Vagrant Cascadian <vagrant@aikidev.net>
> To: "Prof. Dr. Gundolf Kiefer" <gundolf.kiefer@hs-augsburg.de>,
> 783846 at bugs.debian.org Subject: Re: Bug#783846: Patch to enable LEDs on
> sunix/Cubietruck
>
>
> On 2015-04-30, Prof. Dr. Gundolf Kiefer wrote:
>> It seems that the support for the onboard leds is missing for
>> sunxi-based boards even in the latest version (Debian unstable). IMHO
>> LEDs are very helpful for headless systems.
>
> Thanks for the patches!
>
> If you could submit these patches to u-boot upstream, that would be
> best; we're trying to minimize the patches that are not already included
> upstream.
>
>
> live well,
>    vagrant
>
>
>
> ---------------------------------------------------------------
> Prof. Dr. Gundolf Kiefer
> Effiziente Eingebettete Systeme  -  Efficient Embedded Systems
> Fakult?t f?r Informatik          -  Faculty of Computer Science
> Hochschule Augsburg - University of Applied Sciences
> http://www.hs-augsburg.de/~kiefer
> ---------------------------------------------------------------
>

  reply	other threads:[~2015-05-12 15:15 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-12 13:39 [U-Boot] Patch to enable LEDs on sunxi/Cubietruck Prof. Dr. Gundolf Kiefer
2015-05-12 15:15 ` Hans de Goede [this message]
2015-05-13  8:04   ` Prof. Dr. Gundolf Kiefer
2015-05-13 13:52   ` Simon Glass

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=5552190E.9090505@redhat.com \
    --to=hdegoede@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox