public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h
@ 2012-07-11 12:26 Michal Simek
  2012-07-18 22:56 ` Mike Frysinger
  2012-08-10 19:56 ` Anatolij Gustschin
  0 siblings, 2 replies; 5+ messages in thread
From: Michal Simek @ 2012-07-11 12:26 UTC (permalink / raw)
  To: u-boot

Include arch specific gpio.h instead of asm-generic/gpio.h
because several architectures (Microblaze, Blackfin, Nios2, OpenRISC)
define gpio functions in header file.
asm-generic/gpio.h can be included in arch specific gpio.h
(For example: ARM)

Signed-off-by: Michal Simek <monstr@monstr.eu>
CC: Simon Glass <sjg@chromium.org>

---
v2: Use only arch specific gpio.h
    Remove commentary
---
 lib/fdtdec.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/lib/fdtdec.c b/lib/fdtdec.c
index cc09e06..af17ac1 100644
--- a/lib/fdtdec.c
+++ b/lib/fdtdec.c
@@ -24,8 +24,7 @@
 #include <libfdt.h>
 #include <fdtdec.h>
 
-/* we need the generic GPIO interface here */
-#include <asm-generic/gpio.h>
+#include <asm/gpio.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h
  2012-07-11 12:26 [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h Michal Simek
@ 2012-07-18 22:56 ` Mike Frysinger
  2012-08-06  7:37   ` Michal Simek
  2012-08-10 19:56 ` Anatolij Gustschin
  1 sibling, 1 reply; 5+ messages in thread
From: Mike Frysinger @ 2012-07-18 22:56 UTC (permalink / raw)
  To: u-boot

On Wednesday 11 July 2012 08:26:38 Michal Simek wrote:
> Include arch specific gpio.h instead of asm-generic/gpio.h
> because several architectures (Microblaze, Blackfin, Nios2, OpenRISC)
> define gpio functions in header file.
> asm-generic/gpio.h can be included in arch specific gpio.h
> (For example: ARM)

Acked-by: Mike Frysinger <vapier@gentoo.org>
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120718/fd25106d/attachment.pgp>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h
  2012-07-18 22:56 ` Mike Frysinger
@ 2012-08-06  7:37   ` Michal Simek
  2012-08-06 20:59     ` Simon Glass
  0 siblings, 1 reply; 5+ messages in thread
From: Michal Simek @ 2012-08-06  7:37 UTC (permalink / raw)
  To: u-boot

On 07/19/2012 12:56 AM, Mike Frysinger wrote:
> Acked-by: Mike Frysinger<vapier@gentoo.org>

Simon: Are you OK with this patch?

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h
  2012-08-06  7:37   ` Michal Simek
@ 2012-08-06 20:59     ` Simon Glass
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Glass @ 2012-08-06 20:59 UTC (permalink / raw)
  To: u-boot

Him

On Mon, Aug 6, 2012 at 12:37 AM, Michal Simek <monstr@monstr.eu> wrote:
> On 07/19/2012 12:56 AM, Mike Frysinger wrote:
>>
>> Acked-by: Mike Frysinger<vapier@gentoo.org>
>
>
> Simon: Are you OK with this patch?

Yes, thanks.

Acked-by: Simon Glass <sjg@chromium.org>

>
> Thanks,
> Michal
>
>
> --
> Michal Simek, Ing. (M.Eng)
> w: www.monstr.eu p: +42-0-721842854
> Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
> Microblaze U-BOOT custodian

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h
  2012-07-11 12:26 [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h Michal Simek
  2012-07-18 22:56 ` Mike Frysinger
@ 2012-08-10 19:56 ` Anatolij Gustschin
  1 sibling, 0 replies; 5+ messages in thread
From: Anatolij Gustschin @ 2012-08-10 19:56 UTC (permalink / raw)
  To: u-boot

Hi,

On Wed, 11 Jul 2012 14:26:38 +0200
Michal Simek <monstr@monstr.eu> wrote:

> Include arch specific gpio.h instead of asm-generic/gpio.h
> because several architectures (Microblaze, Blackfin, Nios2, OpenRISC)
> define gpio functions in header file.
> asm-generic/gpio.h can be included in arch specific gpio.h
> (For example: ARM)
> 
> Signed-off-by: Michal Simek <monstr@monstr.eu>
> CC: Simon Glass <sjg@chromium.org>
> 
> ---
> v2: Use only arch specific gpio.h
>     Remove commentary
> ---
>  lib/fdtdec.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

applied to my staging branch, thanks!

Anatolij

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-08-10 19:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-11 12:26 [U-Boot] [PATCH v2] fdt: Include arch specific gpio.h instead of asm-generic/gpio.h Michal Simek
2012-07-18 22:56 ` Mike Frysinger
2012-08-06  7:37   ` Michal Simek
2012-08-06 20:59     ` Simon Glass
2012-08-10 19:56 ` Anatolij Gustschin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox