From: "Benoît Thébaudeau" <benoit.thebaudeau@advansee.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] MX: Set a common gpio.h for all i.MX
Date: Sun, 19 Aug 2012 02:25:12 +0200 (CEST) [thread overview]
Message-ID: <1610423122.2545097.1345335912530.JavaMail.root@advansee.com> (raw)
In-Reply-To: <50301391.7010307@denx.de>
Hi Stefano,
> >> #define MXC_GPIO_PORT_TO_NUM(port, bit) (((port - 1) << 5) + (bit
> >> &
> >> 0x1f))
> >
> > Keeping this is also useless. GPIO_NUMBER() from the new
> > <asm/imx-common/gpio.h>
> > can be used instead everywhere needed.
>
> That is right - I drop it.
I don't know if you are aware of it, but just to let you know, I've seen the
following patch that will interfere:
http://patchwork.ozlabs.org/patch/165311/
http://git.denx.de/?p=u-boot/u-boot-staging.git;a=commitdiff;h=72739219a12bf02820d29a89cb2b7fdc4d0e840f
You may want to merge it to your imx tree and rebase after it for your patch.
> >>
> >> -/* GPIO registers */
> >> -struct gpio_regs {
> >> - u32 gpio_dr;
> >> - u32 gpio_dir;
> >> - u32 gpio_psr;
> >> -};
> >> +#include <asm/imx-common/gpio.h>
> >>
> >> #endif /* __ASM_ARCH_MX6_GPIO_H */
> >
> > Why do you keep all these old <asm/gpio.h>? The new
> > <asm/imx-common/gpio.h> can
> > be included instead everywhere needed.
>
> No. The GPIO is common for all SOCs in u-boot, not only i.MX. The
> common
> interface requires that a asm/gpio.h exists. See common/cmd_gpio.c.
Right.
Best regards,
Beno?t
next prev parent reply other threads:[~2012-08-19 0:25 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-18 15:26 [U-Boot] [PATCH] MX: Set a common gpio.h for all i.MX Stefano Babic
2012-08-18 19:25 ` Benoît Thébaudeau
2012-08-18 20:55 ` Matt Sealey
2012-08-18 22:01 ` Marek Vasut
2012-08-18 22:13 ` stefano babic
2012-08-19 0:25 ` Benoît Thébaudeau [this message]
2012-08-19 8:35 ` Stefano Babic
2012-08-18 22:59 ` Fabio Estevam
2012-08-18 23:01 ` Marek Vasut
2012-08-19 7:46 ` stefano babic
2012-08-19 8:40 ` [U-Boot] [PATCH v2] " Stefano Babic
2012-08-19 9:11 ` [U-Boot] [PATCH v3] " Stefano Babic
2012-08-19 13:30 ` Benoît Thébaudeau
2012-08-19 16:41 ` stefano babic
2012-08-20 7:33 ` [U-Boot] [PATCH v4] " Stefano Babic
2012-08-20 20:33 ` Benoît Thébaudeau
2012-08-20 21:20 ` Matt Sealey
2012-08-21 6:12 ` Stefano Babic
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=1610423122.2545097.1345335912530.JavaMail.root@advansee.com \
--to=benoit.thebaudeau@advansee.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.