2010-06-03 03:07:20

by Grant Likely

[permalink] [raw]
Subject: [git pull] (updated) OF-related build breakage fixes

Updated pull request. Added more fixes from Anatolij of the same sort.

g.

The following changes since commit aef4b9aaae1decc775778903922bd0075cce7a88:
Linus Torvalds (1):
Merge branch 'next' of git://git.kernel.org/.../benh/powerpc

are available in the git repository at:

git://git.secretlab.ca/git/linux-2.6 merge

Alexandre Bounine (1):
of/powerpc: fix 85xx RapidIO device node pointer

Anatolij Gustschin (12):
of/spi: mpc512x_psc_spi.c: Fix build failures
of/mtd/nand: mpc5121_nfc.c: Fix build failures
of/dma: mpc512x_dma.c: Fix build failures
of/pcmcia: m8xx_pcmcia.c: Fix build failures
of/video: fix build breakage in FB drivers
of/mtd: nand: fix build breakage in drivers
of/dma: fix build breakage in ppc4xx adma driver
of/crypto: crypto4xx_core.c: fix build breakage
of/usb: fsl_qe_udc.c: fix build breakage
of/net: fs_enet/mii-bitbang.c: fix build breakage
of/edac: fix build breakage in drivers
of/watchdog: gef_wdt.c: fix build breakage

Grant Likely (3):
of/usb: fix build error due to of_node pointer move
of/spi: Fix build failure on spi_ppc4xx.c
of/rtc: rtc-mpc5121.c: Fix build failures

Michael Guntsche (1):
watchdog: Fix build failure with OF changes

arch/powerpc/sysdev/fsl_rio.c | 2 +-
drivers/crypto/amcc/crypto4xx_core.c | 6 +++---
drivers/dma/mpc512x_dma.c | 10 +++++-----
drivers/dma/ppc4xx/adma.c | 4 ++--
drivers/edac/mpc85xx_edac.c | 12 ++++++------
drivers/edac/ppc4xx_edac.c | 6 +++---
drivers/mtd/nand/fsl_upm.c | 17 +++++++++--------
drivers/mtd/nand/mpc5121_nfc.c | 8 ++++----
drivers/mtd/nand/socrates_nand.c | 4 ++--
drivers/net/fs_enet/mii-bitbang.c | 4 ++--
drivers/pcmcia/m8xx_pcmcia.c | 4 ++--
drivers/rtc/rtc-mpc5121.c | 14 ++++++++------
drivers/spi/mpc512x_psc_spi.c | 12 ++++++------
drivers/spi/spi_ppc4xx.c | 2 +-
drivers/usb/gadget/fsl_qe_udc.c | 4 ++--
drivers/usb/host/ehci-xilinx-of.c | 2 +-
drivers/video/bw2.c | 2 +-
drivers/video/cg14.c | 2 +-
drivers/video/cg3.c | 2 +-
drivers/video/leo.c | 2 +-
drivers/video/mb862xx/mb862xxfb.c | 2 +-
drivers/video/p9100.c | 2 +-
drivers/video/tcx.c | 2 +-
drivers/watchdog/gef_wdt.c | 2 +-
drivers/watchdog/mpc8xxx_wdt.c | 2 +-
25 files changed, 66 insertions(+), 63 deletions(-)


On Wed, Jun 2, 2010 at 2:24 PM, Grant Likely <[email protected]> wrote:
> Hi Linus,
>
> Here's a set of build fixes related to the of_node and of_match_table
> device changes. ?All trivial fixes.
>
> Thanks,
> g.
>
> The following changes since commit aef4b9aaae1decc775778903922bd0075cce7a88:
> ?Linus Torvalds (1):
> ? ? ? ?Merge branch 'next' of git://git.kernel.org/.../benh/powerpc
>
> are available in the git repository at:
>
> ?git://git.secretlab.ca/git/linux-2.6 merge
>
> Alexandre Bounine (1):
> ? ? ?of/powerpc: fix 85xx RapidIO device node pointer
>
> Anatolij Gustschin (3):
> ? ? ?of/spi: mpc512x_psc_spi.c: Fix build failures
> ? ? ?of/mtd/nand: mpc5121_nfc.c: Fix build failures
> ? ? ?of/dma: mpc512x_dma.c: Fix build failures
>
> Grant Likely (3):
> ? ? ?of/usb: fix build error due to of_node pointer move
> ? ? ?of/spi: Fix build failure on spi_ppc4xx.c
> ? ? ?of/rtc: rtc-mpc5121.c: Fix build failures
>
> Michael Guntsche (1):
> ? ? ?watchdog: Fix build failure with OF changes
>
> ?arch/powerpc/sysdev/fsl_rio.c ? ? | ? ?2 +-
> ?drivers/dma/mpc512x_dma.c ? ? ? ? | ? 10 +++++-----
> ?drivers/mtd/nand/mpc5121_nfc.c ? ?| ? ?8 ++++----
> ?drivers/rtc/rtc-mpc5121.c ? ? ? ? | ? 14 ++++++++------
> ?drivers/spi/mpc512x_psc_spi.c ? ? | ? 12 ++++++------
> ?drivers/spi/spi_ppc4xx.c ? ? ? ? ?| ? ?2 +-
> ?drivers/usb/host/ehci-xilinx-of.c | ? ?2 +-
> ?drivers/watchdog/mpc8xxx_wdt.c ? ?| ? ?2 +-
> ?8 files changed, 27 insertions(+), 25 deletions(-)
>
> --
> Grant Likely, B.Sc., P.Eng.
> Secret Lab Technologies Ltd.
>



--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.


2010-06-03 04:01:50

by Wolfram Sang

[permalink] [raw]
Subject: Re: [git pull] (updated) OF-related build breakage fixes

On Wed, Jun 02, 2010 at 09:06:54PM -0600, Grant Likely wrote:
> Updated pull request. Added more fixes from Anatolij of the same sort.

Just a second. I think I found two more (using coccinelle)...

--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |


Attachments:
(No filename) (354.00 B)
signature.asc (197.00 B)
Digital signature
Download all attachments

2010-06-03 04:06:18

by Wolfram Sang

[permalink] [raw]
Subject: [PATCH 2/2] of/crypto: n2_core: fix build breakage

Signed-off-by: Wolfram Sang <[email protected]>
Cc: Grant Likely <[email protected]>
---
drivers/crypto/n2_core.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/crypto/n2_core.c b/drivers/crypto/n2_core.c
index 8566be8..3f813d3 100644
--- a/drivers/crypto/n2_core.c
+++ b/drivers/crypto/n2_core.c
@@ -1398,7 +1398,7 @@ static int find_devino_index(struct of_device *dev, struct spu_mdesc_info *ip,

intr = ip->ino_table[i].intr;

- dev_intrs = of_get_property(dev->node, "interrupts", NULL);
+ dev_intrs = of_get_property(dev->dev.of_node, "interrupts", NULL);
if (!dev_intrs)
return -ENODEV;

@@ -1574,7 +1574,7 @@ static int spu_mdesc_walk_arcs(struct mdesc_handle *mdesc,
id = mdesc_get_property(mdesc, tgt, "id", NULL);
if (table[*id] != NULL) {
dev_err(&dev->dev, "%s: SPU cpu slot already set.\n",
- dev->node->full_name);
+ dev->dev.of_node->full_name);
return -EINVAL;
}
cpu_set(*id, p->sharing);
@@ -1595,7 +1595,7 @@ static int handle_exec_unit(struct spu_mdesc_info *ip, struct list_head *list,
p = kzalloc(sizeof(struct spu_queue), GFP_KERNEL);
if (!p) {
dev_err(&dev->dev, "%s: Could not allocate SPU queue.\n",
- dev->node->full_name);
+ dev->dev.of_node->full_name);
return -ENOMEM;
}

@@ -1684,7 +1684,7 @@ static int __devinit grab_mdesc_irq_props(struct mdesc_handle *mdesc,
const unsigned int *reg;
u64 node;

- reg = of_get_property(dev->node, "reg", NULL);
+ reg = of_get_property(dev->dev.of_node, "reg", NULL);
if (!reg)
return -ENODEV;

@@ -1836,7 +1836,7 @@ static int __devinit n2_crypto_probe(struct of_device *dev,

n2_spu_driver_version();

- full_name = dev->node->full_name;
+ full_name = dev->dev.of_node->full_name;
pr_info("Found N2CP at %s\n", full_name);

np = alloc_n2cp();
@@ -1948,7 +1948,7 @@ static int __devinit n2_mau_probe(struct of_device *dev,

n2_spu_driver_version();

- full_name = dev->node->full_name;
+ full_name = dev->dev.of_node->full_name;
pr_info("Found NCP at %s\n", full_name);

mp = alloc_ncp();
--
1.7.0

2010-06-03 04:06:14

by Wolfram Sang

[permalink] [raw]
Subject: [PATCH 1/2] of/powerpc: fix fsl_msi device node pointer

Signed-off-by: Wolfram Sang <[email protected]>
Cc: Grant Likely <[email protected]>
---
arch/powerpc/sysdev/fsl_msi.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c
index 0f5bee9..962c2d8 100644
--- a/arch/powerpc/sysdev/fsl_msi.c
+++ b/arch/powerpc/sysdev/fsl_msi.c
@@ -347,7 +347,7 @@ static int __devinit fsl_of_msi_probe(struct of_device *dev,
goto error_out;
}
offset = 0;
- p = of_get_property(dev->node, "msi-available-ranges", &len);
+ p = of_get_property(dev->dev.of_node, "msi-available-ranges", &len);
if (p)
offset = *p / IRQS_PER_MSI_REG;

--
1.7.0

2010-06-03 04:25:27

by Grant Likely

[permalink] [raw]
Subject: Re: [PATCH 2/2] of/crypto: n2_core: fix build breakage

On Wed, Jun 2, 2010 at 10:03 PM, Wolfram Sang <[email protected]> wrote:
> Signed-off-by: Wolfram Sang <[email protected]>
> Cc: Grant Likely <[email protected]>
> ---
> ?drivers/crypto/n2_core.c | ? 12 ++++++------
> ?1 files changed, 6 insertions(+), 6 deletions(-)

Anatolij already posted a patch for this; and David let me know that
it has already been fixed in the crypto tree, so I dropped it before
pushing my tree out.

g.

2010-06-03 04:58:24

by Grant Likely

[permalink] [raw]
Subject: Re: [PATCH 1/2] of/powerpc: fix fsl_msi device node pointer

On Wed, Jun 2, 2010 at 10:03 PM, Wolfram Sang <[email protected]> wrote:
> Signed-off-by: Wolfram Sang <[email protected]>
> Cc: Grant Likely <[email protected]>
> ---
> ?arch/powerpc/sysdev/fsl_msi.c | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)

Applied, thanks.

Linus, I've also added this one to my tree. Wolfram did an coccinelle
patch and hasn't found any more instances of this breakage.

g.

>
> diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c
> index 0f5bee9..962c2d8 100644
> --- a/arch/powerpc/sysdev/fsl_msi.c
> +++ b/arch/powerpc/sysdev/fsl_msi.c
> @@ -347,7 +347,7 @@ static int __devinit fsl_of_msi_probe(struct of_device *dev,
> ? ? ? ? ? ? ? ?goto error_out;
> ? ? ? ?}
> ? ? ? ?offset = 0;
> - ? ? ? p = of_get_property(dev->node, "msi-available-ranges", &len);
> + ? ? ? p = of_get_property(dev->dev.of_node, "msi-available-ranges", &len);
> ? ? ? ?if (p)
> ? ? ? ? ? ? ? ?offset = *p / IRQS_PER_MSI_REG;
>
> --
> 1.7.0
>
>



--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

2010-06-03 09:48:56

by David Miller

[permalink] [raw]
Subject: Re: [PATCH 2/2] of/crypto: n2_core: fix build breakage

From: Wolfram Sang <[email protected]>
Date: Thu, 3 Jun 2010 06:03:47 +0200

> Signed-off-by: Wolfram Sang <[email protected]>
> Cc: Grant Likely <[email protected]>

Herbert you really need to push your pending crypto tree stuff to
Linus, this is the second time this build fix has been posted just
today alone and it's in your tree since last week :-)

2010-06-03 10:03:30

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH 2/2] of/crypto: n2_core: fix build breakage

On Thu, Jun 03, 2010 at 02:49:02AM -0700, David Miller wrote:
> From: Wolfram Sang <[email protected]>
> Date: Thu, 3 Jun 2010 06:03:47 +0200
>
> > Signed-off-by: Wolfram Sang <[email protected]>
> > Cc: Grant Likely <[email protected]>
>
> Herbert you really need to push your pending crypto tree stuff to
> Linus, this is the second time this build fix has been posted just
> today alone and it's in your tree since last week :-)

Sorry, I hadn't realised that the breakage was in the main tree
as opposed to linux-next.

I'll push it out now.

Thanks,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

2010-07-01 09:06:38

by Dan Williams

[permalink] [raw]
Subject: Re: [git pull] (updated) OF-related build breakage fixes

On Wed, Jun 2, 2010 at 8:06 PM, Grant Likely <[email protected]> wrote:
> Updated pull request. ?Added more fixes from Anatolij of the same sort.

Here is one more (attached) that seems to have escaped.

--
Dan


Attachments:
fix-ppc4xx-adma-build.patch (1.06 kB)

2010-07-01 13:16:30

by Anatolij Gustschin

[permalink] [raw]
Subject: Re: [git pull] (updated) OF-related build breakage fixes

On Thu, 1 Jul 2010 02:06:35 -0700
Dan Williams <[email protected]> wrote:

> of/dma: fix remaining ppc4xx build breakage
>
> From: Dan Williams <[email protected]>
>
> Commit 05c02542 "of/dma: fix build breakage in ppc4xx adma driver"
> missed one conversion site.
>
> Cc: Anatolij Gustschin <[email protected]>
> Cc: Grant Likely <[email protected]>
> Signed-off-by: Dan Williams <[email protected]>
> ---
>
> drivers/dma/ppc4xx/adma.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)

Acked-by: Anatolij Gustschin <[email protected]>

Thanks for fixing!

>
>
> diff --git a/drivers/dma/ppc4xx/adma.c b/drivers/dma/ppc4xx/adma.c
> index 5a22ca6..7c37479 100644
> --- a/drivers/dma/ppc4xx/adma.c
> +++ b/drivers/dma/ppc4xx/adma.c
> @@ -4257,10 +4257,12 @@ static int ppc440spe_adma_setup_irqs(struct ppc440spe_adma_device *adev,
> struct ppc440spe_adma_chan *chan,
> int *initcode)
> {
> + struct of_device *ofdev;
> struct device_node *np;
> int ret;
>
> - np = container_of(adev->dev, struct of_device, dev)->node;
> + ofdev = container_of(adev->dev, struct of_device, dev);
> + np = ofdev->dev.of_node;
> if (adev->id != PPC440SPE_XOR_ID) {
> adev->err_irq = irq_of_parse_and_map(np, 1);
> if (adev->err_irq == NO_IRQ) {

2010-07-02 21:53:39

by Grant Likely

[permalink] [raw]
Subject: Re: [git pull] (updated) OF-related build breakage fixes

On Thu, Jul 1, 2010 at 7:16 AM, Anatolij Gustschin <[email protected]> wrote:
> On Thu, 1 Jul 2010 02:06:35 -0700
> Dan Williams <[email protected]> wrote:
>
>> of/dma: fix remaining ppc4xx build breakage
>>
>> From: Dan Williams <[email protected]>
>>
>> Commit 05c02542 "of/dma: fix build breakage in ppc4xx adma driver"
>> missed one conversion site.

Thanks Dan. I've picked this up and will push it out to Linus

g.