| Message ID | 1346110528-30858-2-git-send-email-thomas.petazzoni@free-electrons.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, Aug 28, 2012 at 01:35:28AM +0200, Thomas Petazzoni wrote: > Commit c085d965fb63ac3b4cc7379d45588c0b39e2bdb0 made the ARCH_MVEBU > platform select PLAT_ORION, which means that now all Orion drivers can > be enabled on ARCH_MVEBU. This works fine for most drivers, except for > orion_nand, because it includes <mach/hardware.h>, but mach-mvebu does > not have a mach/hardware.h header (it is considered as a deprecated > practice). > > It turns out that the <mach/hardware.h> include in orion_nand is not > necessary: the driver builds perfectly fine without it, so we simply > get rid of it. > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > Cc: David Woodhouse <dwmw2@infradead.org> > Cc: linux-mtd@lists.infradead.org > Cc: Artem Bityutskiy <dedekind1@gmail.com> > --- > drivers/mtd/nand/orion_nand.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/mtd/nand/orion_nand.c b/drivers/mtd/nand/orion_nand.c > index fc5a868..9ee436d 100644 > --- a/drivers/mtd/nand/orion_nand.c > +++ b/drivers/mtd/nand/orion_nand.c > @@ -21,7 +21,6 @@ > #include <linux/err.h> > #include <asm/io.h> > #include <asm/sizes.h> > -#include <mach/hardware.h> > #include <plat/orion_nand.h> > > static void orion_nand_cmd_ctrl(struct mtd_info *mtd, int cmd, unsigned int ctrl) > -- > 1.7.9.5 > Hi Thomas I compiled tested on Dove, Kirkwood, Orion5x and mv78xx0. Tested-by: Andrew Lunn <andrew@lunn.ch> One other driver you might want to look at is: drivers/watchdog/orion_wdt.c It uses <mach/bridge-regs.h> Andrew
On 8/28/12, Andrew Lunn <andrew@lunn.ch> wrote: > One other driver you might want to look at is: > > drivers/watchdog/orion_wdt.c > It uses <mach/bridge-regs.h> Andrew, Thomas, both current watchdog and timer are using the bridge-regs to clear the interrupt cause. On orion the cause register is located in bridge-regs. From the work on timer together with Gregory, I expect Armada 370/XP to have the watchdog cause bit also within timer registers. Sebastian
Le Tue, 28 Aug 2012 11:11:39 +0200, Andrew Lunn <andrew@lunn.ch> a écrit : > I compiled tested on Dove, Kirkwood, Orion5x and mv78xx0. > > Tested-by: Andrew Lunn <andrew@lunn.ch> Thanks! Will Jason take care of merging this on top of the previous address decoding patches? Note that if you want to keep build bisectability, it should be merged before my "arm: plat-orion: introduce PLAT_ORION_LEGACY hidden config option" patch. > One other driver you might want to look at is: > > drivers/watchdog/orion_wdt.c > > It uses <mach/bridge-regs.h> This driver is: config ORION_WATCHDOG tristate "Orion watchdog" depends on ARCH_ORION5X || ARCH_KIRKWOOD so it isn't selectable with ARCH_MVEBU, so it will not cause problems for now. Of course, at some point, we'll have a look at the watchdog driver but for now, it's fine. Best regards, Thomas
On Tue, Aug 28, 2012 at 01:35:28AM +0200, Thomas Petazzoni wrote: > Commit c085d965fb63ac3b4cc7379d45588c0b39e2bdb0 made the ARCH_MVEBU > platform select PLAT_ORION, which means that now all Orion drivers can > be enabled on ARCH_MVEBU. This works fine for most drivers, except for > orion_nand, because it includes <mach/hardware.h>, but mach-mvebu does > not have a mach/hardware.h header (it is considered as a deprecated > practice). > > It turns out that the <mach/hardware.h> include in orion_nand is not > necessary: the driver builds perfectly fine without it, so we simply > get rid of it. > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> > Cc: David Woodhouse <dwmw2@infradead.org> > Cc: linux-mtd@lists.infradead.org > Cc: Artem Bityutskiy <dedekind1@gmail.com> Applied to: git://git.infradead.org/users/jcooper/linux.git boards-for-v3.7-v2 thx, Jason.
On Tue, 2012-08-28 at 01:35 +0200, Thomas Petazzoni wrote: > Commit c085d965fb63ac3b4cc7379d45588c0b39e2bdb0 made the ARCH_MVEBU > platform select PLAT_ORION, which means that now all Orion drivers can > be enabled on ARCH_MVEBU. This works fine for most drivers, except for > orion_nand, because it includes <mach/hardware.h>, but mach-mvebu does > not have a mach/hardware.h header (it is considered as a deprecated > practice). > > It turns out that the <mach/hardware.h> include in orion_nand is not > necessary: the driver builds perfectly fine without it, so we simply > get rid of it. > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Pushed to l2-mtd.git, thanks!
diff --git a/drivers/mtd/nand/orion_nand.c b/drivers/mtd/nand/orion_nand.c index fc5a868..9ee436d 100644 --- a/drivers/mtd/nand/orion_nand.c +++ b/drivers/mtd/nand/orion_nand.c @@ -21,7 +21,6 @@ #include <linux/err.h> #include <asm/io.h> #include <asm/sizes.h> -#include <mach/hardware.h> #include <plat/orion_nand.h> static void orion_nand_cmd_ctrl(struct mtd_info *mtd, int cmd, unsigned int ctrl)
Commit c085d965fb63ac3b4cc7379d45588c0b39e2bdb0 made the ARCH_MVEBU platform select PLAT_ORION, which means that now all Orion drivers can be enabled on ARCH_MVEBU. This works fine for most drivers, except for orion_nand, because it includes <mach/hardware.h>, but mach-mvebu does not have a mach/hardware.h header (it is considered as a deprecated practice). It turns out that the <mach/hardware.h> include in orion_nand is not necessary: the driver builds perfectly fine without it, so we simply get rid of it. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Cc: David Woodhouse <dwmw2@infradead.org> Cc: linux-mtd@lists.infradead.org Cc: Artem Bityutskiy <dedekind1@gmail.com> --- drivers/mtd/nand/orion_nand.c | 1 - 1 file changed, 1 deletion(-)