From: Tony Lindgren <tony@atomide.com>
To: Felipe Balbi <balbi@ti.com>
Cc: Linux OMAP Mailing List <linux-omap@vger.kernel.org>,
Linux ARM Kernel Mailing List
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 0/5] arm: omap: debug-leds: few changes
Date: Fri, 7 Sep 2012 15:32:59 -0700 [thread overview]
Message-ID: <20120907223259.GT1303@atomide.com> (raw)
In-Reply-To: <1343120042-16695-1-git-send-email-balbi@ti.com>
* Felipe Balbi <balbi@ti.com> [120724 02:02]:
> Hi Tony,
>
> here are a few changes for the debug-leds driver. This series tries to get the
> driver closer to be moved away from arch/arm/plat-omap/ and under drivers/leds.
> There's only the dependency with <plat/fpga.h> now.
>
> I tried moving the needed definitions into the C file, but there's another file
> which uses the same structure (arch/arm/mach-omap1/leds-h2p2-debug.c) so I left
> it for another series. Ideally, omap1 would be using the same debug-leds driver
> for that purpose and we could drop the dependency on <plat/fpga.h>.
>
> BTW, patches were compile tested only due to the lack of an H4 board.
>
> Felipe Balbi (5):
> arm: omap: debug-leds: drop machine_is_* checks
> arm: omap: debug-leds: move initialization to debug-leds
> arm: omap: debug-leds: add section annotation to probe
> arm: omap: debug-leds: add a remove method
> arm: omap: debug-leds: switch to resource_size()
>
> arch/arm/plat-omap/debug-leds.c | 37 +++++++++++++++++++++++--------------
> 1 file changed, 23 insertions(+), 14 deletions(-)
Sorry for forgetting this series.. Found it after shrinking my inbox
to a more readable size :)
There are some good changes in linux-next that remove most of the
debug-leds code. Maybe take a quick look what can be rebased on
those to make it into a reald driver?
Regards,
Tony
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/5] arm: omap: debug-leds: few changes
Date: Fri, 7 Sep 2012 15:32:59 -0700 [thread overview]
Message-ID: <20120907223259.GT1303@atomide.com> (raw)
In-Reply-To: <1343120042-16695-1-git-send-email-balbi@ti.com>
* Felipe Balbi <balbi@ti.com> [120724 02:02]:
> Hi Tony,
>
> here are a few changes for the debug-leds driver. This series tries to get the
> driver closer to be moved away from arch/arm/plat-omap/ and under drivers/leds.
> There's only the dependency with <plat/fpga.h> now.
>
> I tried moving the needed definitions into the C file, but there's another file
> which uses the same structure (arch/arm/mach-omap1/leds-h2p2-debug.c) so I left
> it for another series. Ideally, omap1 would be using the same debug-leds driver
> for that purpose and we could drop the dependency on <plat/fpga.h>.
>
> BTW, patches were compile tested only due to the lack of an H4 board.
>
> Felipe Balbi (5):
> arm: omap: debug-leds: drop machine_is_* checks
> arm: omap: debug-leds: move initialization to debug-leds
> arm: omap: debug-leds: add section annotation to probe
> arm: omap: debug-leds: add a remove method
> arm: omap: debug-leds: switch to resource_size()
>
> arch/arm/plat-omap/debug-leds.c | 37 +++++++++++++++++++++++--------------
> 1 file changed, 23 insertions(+), 14 deletions(-)
Sorry for forgetting this series.. Found it after shrinking my inbox
to a more readable size :)
There are some good changes in linux-next that remove most of the
debug-leds code. Maybe take a quick look what can be rebased on
those to make it into a reald driver?
Regards,
Tony
next prev parent reply other threads:[~2012-09-07 22:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-24 8:53 [PATCH 0/5] arm: omap: debug-leds: few changes Felipe Balbi
2012-07-24 8:53 ` Felipe Balbi
2012-07-24 8:53 ` [PATCH 1/5] arm: omap: debug-leds: drop machine_is_* checks Felipe Balbi
2012-07-24 8:53 ` Felipe Balbi
2012-07-24 8:53 ` [PATCH 2/5] arm: omap: debug-leds: move initialization to debug-leds Felipe Balbi
2012-07-24 8:53 ` Felipe Balbi
2012-07-24 8:54 ` [PATCH 3/5] arm: omap: debug-leds: add section annotation to probe Felipe Balbi
2012-07-24 8:54 ` Felipe Balbi
2012-07-24 8:54 ` [PATCH 4/5] arm: omap: debug-leds: add a remove method Felipe Balbi
2012-07-24 8:54 ` Felipe Balbi
2012-07-24 8:54 ` [PATCH 5/5] arm: omap: debug-leds: switch to resource_size() Felipe Balbi
2012-07-24 8:54 ` Felipe Balbi
2012-09-07 22:32 ` Tony Lindgren [this message]
2012-09-07 22:32 ` [PATCH 0/5] arm: omap: debug-leds: few changes Tony Lindgren
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=20120907223259.GT1303@atomide.com \
--to=tony@atomide.com \
--cc=balbi@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
/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.