* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-22 15:19 asm-ppc header issues when building ARCH=powerpc Kumar Gala
@ 2007-08-22 16:01 ` Scott Wood
2007-08-22 19:30 ` Josh Boyer
` (2 subsequent siblings)
3 siblings, 0 replies; 17+ messages in thread
From: Scott Wood @ 2007-08-22 16:01 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
On Wed, Aug 22, 2007 at 10:19:21AM -0500, Kumar Gala wrote:
> ./drivers/mtd/maps/tqm834x.c:#include <asm/ppcboot.h>
> ./drivers/mtd/maps/pq2fads.c:#include <asm/ppcboot.h>
These use the bd_t, and thus will only work on arch/ppc. On
arch/powerpc, we can use the device tree to specify the mapping.
-Scott
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-22 15:19 asm-ppc header issues when building ARCH=powerpc Kumar Gala
2007-08-22 16:01 ` Scott Wood
@ 2007-08-22 19:30 ` Josh Boyer
2007-08-23 2:49 ` David Gibson
2007-08-23 2:47 ` David Gibson
2007-08-23 3:33 ` Kumar Gala
3 siblings, 1 reply; 17+ messages in thread
From: Josh Boyer @ 2007-08-22 19:30 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
On Wed, 22 Aug 2007 10:19:21 -0500
Kumar Gala <galak@kernel.crashing.org> wrote:
> Guys,
>
> I was wondering if I could get your help with looking at the
> following lists and determining if we have an issue or not related
> the following files:
>
> Getting some classification on these would be good. Possibly
> classifications, doesn't build in ARCH=powerpc, remove include, real
> issue, etc.
Sure.
> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ocp.h>
> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ibm4xx.h>
These one depends on IBM_OCP in Kconfig. We don't select/enable that on
any existing arch/powerpc 4xx stuff so it won't be built anyway.
> ./drivers/mtd/maps/ebony.c:#include <asm/ibm44x.h>
This one depends on !PPC_MERGE in Kconfig. It can be ignored.
> ./drivers/net/ibm_emac/ibm_emac_core.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_core.h:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_tah.h:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_phy.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_mal.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_zmii.h:#include <asm/ocp.h>
These are guarded by !PPC_MERGE in Kconfig, so they won't be built in
arch/powerpc. And we all know we're waiting for the ibm_newemac driver
to show up for arch/powerpc... ;)
> ./drivers/mtd/maps/walnut.c:#include <asm/ibm4xx.h>
> ./drivers/mtd/maps/ocotea.c:#include <asm/ibm44x.h>
> ./drivers/mtd/nand/ndfc.c:#include <asm/ibm44x.h>
ndfc.c doesn't even need asm/ibm44x.h in arch/ppc. It could be removed
safely. However, the build will fail on arch/powerpc because of the
call to ioremap64. So we should guard it for now, until I get the
flash stuff working on Bamboo and come up with something better.
The patch below fixes these, similar to how drivers/mtd/maps/ebony.c is
guarded.
josh
[POWERPC] Don't build arch/ppc dependent drivers in arch/powerpc
These drivers are specific to 4xx support in arch/ppc at the moment. Make
sure they don't get built on arch/powerpc.
Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
---
drivers/mtd/maps/Kconfig | 4 ++--
drivers/mtd/nand/Kconfig | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
--- linux-2.6.orig/drivers/mtd/maps/Kconfig
+++ linux-2.6/drivers/mtd/maps/Kconfig
@@ -354,7 +354,7 @@ config MTD_CFI_FLAGADM
config MTD_WALNUT
tristate "Flash device mapped on IBM 405GP Walnut"
- depends on MTD_JEDECPROBE && WALNUT
+ depends on MTD_JEDECPROBE && WALNUT && !PPC_MERGE
help
This enables access routines for the flash chips on the IBM 405GP
Walnut board. If you have one of these boards and would like to
@@ -370,7 +370,7 @@ config MTD_EBONY
config MTD_OCOTEA
tristate "Flash devices mapped on IBM 440GX Ocotea"
- depends on MTD_CFI && OCOTEA
+ depends on MTD_CFI && OCOTEA && !PPC_MERGE
help
This enables access routines for the flash chips on the IBM 440GX
Ocotea board. If you have one of these boards and would like to
--- linux-2.6.orig/drivers/mtd/nand/Kconfig
+++ linux-2.6/drivers/mtd/nand/Kconfig
@@ -134,7 +134,7 @@ config MTD_NAND_S3C2410_HWECC
config MTD_NAND_NDFC
tristate "NDFC NanD Flash Controller"
- depends on 44x
+ depends on 44x && !PPC_MERGE
select MTD_NAND_ECC_SMC
help
NDFC Nand Flash Controllers are integrated in EP44x SoCs
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-22 19:30 ` Josh Boyer
@ 2007-08-23 2:49 ` David Gibson
2007-08-23 3:09 ` Kumar Gala
0 siblings, 1 reply; 17+ messages in thread
From: David Gibson @ 2007-08-23 2:49 UTC (permalink / raw)
To: Josh Boyer; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Wed, Aug 22, 2007 at 02:30:47PM -0500, Josh Boyer wrote:
> On Wed, 22 Aug 2007 10:19:21 -0500
> Kumar Gala <galak@kernel.crashing.org> wrote:
>
> > Guys,
> >
> > I was wondering if I could get your help with looking at the
> > following lists and determining if we have an issue or not related
> > the following files:
> >
> > Getting some classification on these would be good. Possibly
> > classifications, doesn't build in ARCH=powerpc, remove include, real
> > issue, etc.
>
> Sure.
>
> > ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ocp.h>
> > ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ibm4xx.h>
>
> These one depends on IBM_OCP in Kconfig. We don't select/enable that on
> any existing arch/powerpc 4xx stuff so it won't be built anyway.
Nor will we ever enable IBM_OCP in arch/powerpc: the device tree
entirely obsoletes the OCP crap.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 2:49 ` David Gibson
@ 2007-08-23 3:09 ` Kumar Gala
0 siblings, 0 replies; 17+ messages in thread
From: Kumar Gala @ 2007-08-23 3:09 UTC (permalink / raw)
To: David Gibson; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Aug 22, 2007, at 9:49 PM, David Gibson wrote:
> On Wed, Aug 22, 2007 at 02:30:47PM -0500, Josh Boyer wrote:
>> On Wed, 22 Aug 2007 10:19:21 -0500
>> Kumar Gala <galak@kernel.crashing.org> wrote:
>>
>>> Guys,
>>>
>>> I was wondering if I could get your help with looking at the
>>> following lists and determining if we have an issue or not related
>>> the following files:
>>>
>>> Getting some classification on these would be good. Possibly
>>> classifications, doesn't build in ARCH=powerpc, remove include, real
>>> issue, etc.
>>
>> Sure.
>>
>>> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ocp.h>
>>> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ibm4xx.h>
>>
>> These one depends on IBM_OCP in Kconfig. We don't select/enable
>> that on
>> any existing arch/powerpc 4xx stuff so it won't be built anyway.
>
> Nor will we ever enable IBM_OCP in arch/powerpc: the device tree
> entirely obsoletes the OCP crap.
Agreed. We shouldn't taint arch/powerpc with OCP :)
- k
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-22 15:19 asm-ppc header issues when building ARCH=powerpc Kumar Gala
2007-08-22 16:01 ` Scott Wood
2007-08-22 19:30 ` Josh Boyer
@ 2007-08-23 2:47 ` David Gibson
2007-08-23 3:22 ` Kumar Gala
2007-08-23 17:33 ` Scott Wood
2007-08-23 3:33 ` Kumar Gala
3 siblings, 2 replies; 17+ messages in thread
From: David Gibson @ 2007-08-23 2:47 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Wed, Aug 22, 2007 at 10:19:21AM -0500, Kumar Gala wrote:
> Guys,
>
> I was wondering if I could get your help with looking at the
> following lists and determining if we have an issue or not related
> the following files:
>
> Getting some classification on these would be good. Possibly
> classifications, doesn't build in ARCH=powerpc, remove include, real
> issue, etc.
>
> - k
>
> ./include/asm-powerpc/irq.h:#include <asm/mpc83xx.h> - protected by !
> CONFIG_PPC_MERGE
irq.h seems to have a big slab of CONFIG_PPC_MERGE dependent stuff.
Looks like a good candidate for splitting into asm-ppc/irq.h and
asm-powerpc/irq.h
> ./drivers/ide/ppc/mpc8xx.c:#include <asm/residual.h>
Since mpc8xx is certainly not PReP, I don't think there's any way it
ought to be including residual.h
> ./drivers/mtd/maps/tqm834x.c:#include <asm/ppcboot.h>
> ./drivers/mtd/maps/pq2fads.c:#include <asm/ppcboot.h>
Although these both have an extern of type bd_t (defined in
ppcboot.h), afaict they don't actually use it, so these should be
removable. Longer term, all these ugly hardcoded map files should be
relegated to arch/ppc only, replaced with physmap_of and suitable
information in the device tree for arch/powerpc. However, being able
to build them on arch/powerpc may be useful during transition.
> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ibm4xx.h>
This driver will need significant reworking to port to arch/powerpc
(similar to the treatment ibm_emac has received). Such rework will
remove the dependency on ibm4xx.h and ocp.h
> ./drivers/mtd/maps/walnut.c:#include <asm/ibm4xx.h>
Suspect it's not needed. But in any case, this too should be replaced
by physmap_of for arch/powerpc.
> ./include/asm-powerpc/irq.h:#include <asm/ibm4xx.h> - protected by !
> CONFIG_PPC_MERGE
>
> ./drivers/mtd/maps/ebony.c:#include <asm/ibm44x.h>
Ebony flash now works with physmap_of, so this is only used on
arch/ppc.
> ./drivers/mtd/maps/ocotea.c:#include <asm/ibm44x.h>
As for drivers/mtd/maps/walnut.c.
> ./drivers/mtd/nand/ndfc.c:#include <asm/ibm44x.h>
This probably also wants rework to make a device-tree-aware
arch/powerpc version.
> ./include/asm-powerpc/irq.h:#include <asm/ibm44x.h> - protected by !
> CONFIG_PPC_MERGE
>
> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ocp.h>
See above.
> ./drivers/net/ucc_geth_mii.c:#include <asm/ocp.h> - just bogus, needs
> removal
> ./drivers/net/ibm_emac/ibm_emac_core.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_core.h:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_tah.h:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_phy.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_mal.c:#include <asm/ocp.h>
> ./drivers/net/ibm_emac/ibm_emac_zmii.h:#include <asm/ocp.h>
ibm_emac is arch/ppc only. BenH and my 'ibm_newemac' patch adds a
new, device-tree aware arch/powerpc port of this driver.
>
> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h>
I think this is just unnecessary, maybe also for a bunch of the ones
below.
> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h>
> ./drivers/char/serial167.c:#include <asm/bootinfo.h>
> ./drivers/serial/dz.c:#include <asm/bootinfo.h>
> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h>
> ./drivers/net/macsonic.c:#include <asm/bootinfo.h>
> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h>
> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h>
> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h>
> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h>
> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h>
> ./drivers/video/macfb.c:#include <asm/bootinfo.h>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 2:47 ` David Gibson
@ 2007-08-23 3:22 ` Kumar Gala
2007-08-23 17:33 ` Scott Wood
1 sibling, 0 replies; 17+ messages in thread
From: Kumar Gala @ 2007-08-23 3:22 UTC (permalink / raw)
To: David Gibson; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Aug 22, 2007, at 9:47 PM, David Gibson wrote:
> On Wed, Aug 22, 2007 at 10:19:21AM -0500, Kumar Gala wrote:
>> Guys,
>>
>> I was wondering if I could get your help with looking at the
>> following lists and determining if we have an issue or not related
>> the following files:
>>
>> Getting some classification on these would be good. Possibly
>> classifications, doesn't build in ARCH=powerpc, remove include, real
>> issue, etc.
>>
>> - k
>>
>> ./include/asm-powerpc/irq.h:#include <asm/mpc83xx.h> - protected by !
>> CONFIG_PPC_MERGE
>
> irq.h seems to have a big slab of CONFIG_PPC_MERGE dependent stuff.
> Looks like a good candidate for splitting into asm-ppc/irq.h and
> asm-powerpc/irq.h
Yeah was going to look at a bit of that since the list of
CONFIG_PPC_MERGE headers is pretty short:
dcr.h, i8259.h, ipic.h, irq.h
>> ./drivers/ide/ppc/mpc8xx.c:#include <asm/residual.h>
>
> Since mpc8xx is certainly not PReP, I don't think there's any way it
> ought to be including residual.h
Yeah, not sure what's up with that. It needs a !MERGE in Kconfig for
now.
>> ./drivers/mtd/maps/tqm834x.c:#include <asm/ppcboot.h>
>> ./drivers/mtd/maps/pq2fads.c:#include <asm/ppcboot.h>
>
> Although these both have an extern of type bd_t (defined in
> ppcboot.h), afaict they don't actually use it, so these should be
> removable. Longer term, all these ugly hardcoded map files should be
> relegated to arch/ppc only, replaced with physmap_of and suitable
> information in the device tree for arch/powerpc. However, being able
> to build them on arch/powerpc may be useful during transition.
Yeah was going to kill tqm834x.c since there is no way to Kconfig it
into existence, and !MERGE pq2fads.c.
>> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ibm4xx.h>
>
> This driver will need significant reworking to port to arch/powerpc
> (similar to the treatment ibm_emac has received). Such rework will
> remove the dependency on ibm4xx.h and ocp.h
>
>> ./drivers/mtd/maps/walnut.c:#include <asm/ibm4xx.h>
>
> Suspect it's not needed. But in any case, this too should be replaced
> by physmap_of for arch/powerpc.
>
>> ./include/asm-powerpc/irq.h:#include <asm/ibm4xx.h> - protected by !
>> CONFIG_PPC_MERGE
>>
>> ./drivers/mtd/maps/ebony.c:#include <asm/ibm44x.h>
>
> Ebony flash now works with physmap_of, so this is only used on
> arch/ppc.
>
>> ./drivers/mtd/maps/ocotea.c:#include <asm/ibm44x.h>
>
> As for drivers/mtd/maps/walnut.c.
>
>> ./drivers/mtd/nand/ndfc.c:#include <asm/ibm44x.h>
>
> This probably also wants rework to make a device-tree-aware
> arch/powerpc version.
>
>> ./include/asm-powerpc/irq.h:#include <asm/ibm44x.h> - protected by !
>> CONFIG_PPC_MERGE
>>
>> ./drivers/i2c/busses/i2c-ibm_iic.c:#include <asm/ocp.h>
>
> See above.
>
>> ./drivers/net/ucc_geth_mii.c:#include <asm/ocp.h> - just bogus, needs
>> removal
>> ./drivers/net/ibm_emac/ibm_emac_core.c:#include <asm/ocp.h>
>> ./drivers/net/ibm_emac/ibm_emac_core.h:#include <asm/ocp.h>
>> ./drivers/net/ibm_emac/ibm_emac_tah.h:#include <asm/ocp.h>
>> ./drivers/net/ibm_emac/ibm_emac_phy.c:#include <asm/ocp.h>
>> ./drivers/net/ibm_emac/ibm_emac_mal.c:#include <asm/ocp.h>
>> ./drivers/net/ibm_emac/ibm_emac_zmii.h:#include <asm/ocp.h>
>
> ibm_emac is arch/ppc only. BenH and my 'ibm_newemac' patch adds a
> new, device-tree aware arch/powerpc port of this driver.
>
>>
>> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h>
>
> I think this is just unnecessary, maybe also for a bunch of the ones
> below.
most of the below are 68k or mips uses
>> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h>
>> ./drivers/char/serial167.c:#include <asm/bootinfo.h>
>> ./drivers/serial/dz.c:#include <asm/bootinfo.h>
>> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h>
>> ./drivers/net/macsonic.c:#include <asm/bootinfo.h>
>> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h>
>> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h>
>> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h>
>> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h>
>> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h>
>> ./drivers/video/macfb.c:#include <asm/bootinfo.h>
- k
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 2:47 ` David Gibson
2007-08-23 3:22 ` Kumar Gala
@ 2007-08-23 17:33 ` Scott Wood
2007-08-23 18:00 ` Kumar Gala
1 sibling, 1 reply; 17+ messages in thread
From: Scott Wood @ 2007-08-23 17:33 UTC (permalink / raw)
To: Kumar Gala, linuxppc-dev@ozlabs.org list, Paul Mackerras
On Thu, Aug 23, 2007 at 12:47:57PM +1000, David Gibson wrote:
> > ./drivers/mtd/maps/tqm834x.c:#include <asm/ppcboot.h>
> > ./drivers/mtd/maps/pq2fads.c:#include <asm/ppcboot.h>
>
> Although these both have an extern of type bd_t (defined in
> ppcboot.h), afaict they don't actually use it, so these should be
> removable.
They look like they're using it to me... See the bd->bi_flashstart and
bd->bi_flashsize references in init_pq2fads_mtd.
-Scott
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 17:33 ` Scott Wood
@ 2007-08-23 18:00 ` Kumar Gala
0 siblings, 0 replies; 17+ messages in thread
From: Kumar Gala @ 2007-08-23 18:00 UTC (permalink / raw)
To: Scott Wood; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Aug 23, 2007, at 12:33 PM, Scott Wood wrote:
> On Thu, Aug 23, 2007 at 12:47:57PM +1000, David Gibson wrote:
>>> ./drivers/mtd/maps/tqm834x.c:#include <asm/ppcboot.h>
>>> ./drivers/mtd/maps/pq2fads.c:#include <asm/ppcboot.h>
>>
>> Although these both have an extern of type bd_t (defined in
>> ppcboot.h), afaict they don't actually use it, so these should be
>> removable.
>
> They look like they're using it to me... See the bd->bi_flashstart
> and
> bd->bi_flashsize references in init_pq2fads_mtd.
Both of these are dead in one way or another. pq2fads.c isn't in the
makefile and you can't Kconfig tqm834x.c into existence. Thus my
patch to remove them.
- k
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-22 15:19 asm-ppc header issues when building ARCH=powerpc Kumar Gala
` (2 preceding siblings ...)
2007-08-23 2:47 ` David Gibson
@ 2007-08-23 3:33 ` Kumar Gala
2007-08-23 3:33 ` David Gibson
3 siblings, 1 reply; 17+ messages in thread
From: Kumar Gala @ 2007-08-23 3:33 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
On Aug 22, 2007, at 10:19 AM, Kumar Gala wrote:
> Guys,
>
> I was wondering if I could get your help with looking at the
> following lists and determining if we have an issue or not related
> the following files:
>
> Getting some classification on these would be good. Possibly
> classifications, doesn't build in ARCH=powerpc, remove include, real
> issue, etc.
>
> - k
My analysis of <asm/bootinfo.h> usage:
./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h> remove
./drivers/char/vme_scc.c:#include <asm/bootinfo.h> 68k only
./drivers/char/serial167.c:#include <asm/bootinfo.h> 68k only
./drivers/serial/dz.c:#include <asm/bootinfo.h> decstation
./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h> decstation
./drivers/net/macsonic.c:#include <asm/bootinfo.h> 68k
./drivers/net/jazzsonic.c:#include <asm/bootinfo.h> mips
./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h> mips
./drivers/video/maxinefb.c:#include <asm/bootinfo.h> mips
./drivers/video/logo/logo.c:#include <asm/bootinfo.h> mips
./drivers/video/macfb.c:#include <asm/bootinfo.h> 68k
./drivers/video/valkyriefb.c:#include <asm/bootinfo.h> 68k
- k
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 3:33 ` Kumar Gala
@ 2007-08-23 3:33 ` David Gibson
2007-08-23 4:16 ` Kumar Gala
0 siblings, 1 reply; 17+ messages in thread
From: David Gibson @ 2007-08-23 3:33 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Wed, Aug 22, 2007 at 10:33:55PM -0500, Kumar Gala wrote:
>
> On Aug 22, 2007, at 10:19 AM, Kumar Gala wrote:
>
> >Guys,
> >
> >I was wondering if I could get your help with looking at the
> >following lists and determining if we have an issue or not related
> >the following files:
> >
> >Getting some classification on these would be good. Possibly
> >classifications, doesn't build in ARCH=powerpc, remove include, real
> >issue, etc.
> >
> >- k
>
> My analysis of <asm/bootinfo.h> usage:
>
> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h> remove
> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h> 68k only
> ./drivers/char/serial167.c:#include <asm/bootinfo.h> 68k only
> ./drivers/serial/dz.c:#include <asm/bootinfo.h> decstation
> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h> decstation
> ./drivers/net/macsonic.c:#include <asm/bootinfo.h> 68k
> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h> mips
> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h> mips
> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h> mips
> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h> mips
> ./drivers/video/macfb.c:#include <asm/bootinfo.h> 68k
> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h> 68k
Uh.. I'm pretty sure valkyriefb.c is for (old) PowerMacs, not 68k.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 3:33 ` David Gibson
@ 2007-08-23 4:16 ` Kumar Gala
2007-08-23 9:47 ` Geert Uytterhoeven
2007-08-23 18:56 ` Brad Boyer
0 siblings, 2 replies; 17+ messages in thread
From: Kumar Gala @ 2007-08-23 4:16 UTC (permalink / raw)
To: David Gibson; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras
On Aug 22, 2007, at 10:33 PM, David Gibson wrote:
> On Wed, Aug 22, 2007 at 10:33:55PM -0500, Kumar Gala wrote:
>>
>> On Aug 22, 2007, at 10:19 AM, Kumar Gala wrote:
>>
>>> Guys,
>>>
>>> I was wondering if I could get your help with looking at the
>>> following lists and determining if we have an issue or not related
>>> the following files:
>>>
>>> Getting some classification on these would be good. Possibly
>>> classifications, doesn't build in ARCH=powerpc, remove include, real
>>> issue, etc.
>>>
>>> - k
>>
>> My analysis of <asm/bootinfo.h> usage:
>>
>> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h> remove
>> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h> 68k only
>> ./drivers/char/serial167.c:#include <asm/bootinfo.h> 68k only
>> ./drivers/serial/dz.c:#include <asm/bootinfo.h> decstation
>> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h> decstation
>> ./drivers/net/macsonic.c:#include <asm/bootinfo.h> 68k
>> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h> mips
>> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h> mips
>> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h> mips
>> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h> mips
>> ./drivers/video/macfb.c:#include <asm/bootinfo.h> 68k
>> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h> 68k
>
> Uh.. I'm pretty sure valkyriefb.c is for (old) PowerMacs, not 68k.
It appears to be both. If you look at the include its protected by a
#ifdef CONFIG_MAC which we is only defined on m68k.
- k
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 4:16 ` Kumar Gala
@ 2007-08-23 9:47 ` Geert Uytterhoeven
2007-08-23 18:56 ` Brad Boyer
1 sibling, 0 replies; 17+ messages in thread
From: Geert Uytterhoeven @ 2007-08-23 9:47 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
[-- Attachment #1: Type: TEXT/PLAIN, Size: 2269 bytes --]
On Wed, 22 Aug 2007, Kumar Gala wrote:
> On Aug 22, 2007, at 10:33 PM, David Gibson wrote:
> > On Wed, Aug 22, 2007 at 10:33:55PM -0500, Kumar Gala wrote:
> >> On Aug 22, 2007, at 10:19 AM, Kumar Gala wrote:
> >>> I was wondering if I could get your help with looking at the
> >>> following lists and determining if we have an issue or not related
> >>> the following files:
> >>>
> >>> Getting some classification on these would be good. Possibly
> >>> classifications, doesn't build in ARCH=powerpc, remove include, real
> >>> issue, etc.
> >>>
> >> My analysis of <asm/bootinfo.h> usage:
> >>
> >> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h> remove
> >> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h> 68k only
> >> ./drivers/char/serial167.c:#include <asm/bootinfo.h> 68k only
> >> ./drivers/serial/dz.c:#include <asm/bootinfo.h> decstation
> >> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h> decstation
> >> ./drivers/net/macsonic.c:#include <asm/bootinfo.h> 68k
> >> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/macfb.c:#include <asm/bootinfo.h> 68k
> >> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h> 68k
> >
> > Uh.. I'm pretty sure valkyriefb.c is for (old) PowerMacs, not 68k.
>
> It appears to be both. If you look at the include its protected by a
> #ifdef CONFIG_MAC which we is only defined on m68k.
Indeed, drivers/video/Kconfig says it depends on MAC || (PPC_PMAC && PPC32)
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 4:16 ` Kumar Gala
2007-08-23 9:47 ` Geert Uytterhoeven
@ 2007-08-23 18:56 ` Brad Boyer
2007-08-24 7:10 ` Geert Uytterhoeven
1 sibling, 1 reply; 17+ messages in thread
From: Brad Boyer @ 2007-08-23 18:56 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
On Wed, Aug 22, 2007 at 11:16:03PM -0500, Kumar Gala wrote:
> On Aug 22, 2007, at 10:33 PM, David Gibson wrote:
> >> My analysis of <asm/bootinfo.h> usage:
> >>
> >> ./drivers/macintosh/adb-iop.c:#include <asm/bootinfo.h> remove
> >> ./drivers/char/vme_scc.c:#include <asm/bootinfo.h> 68k only
> >> ./drivers/char/serial167.c:#include <asm/bootinfo.h> 68k only
> >> ./drivers/serial/dz.c:#include <asm/bootinfo.h> decstation
> >> ./drivers/mtd/devices/ms02-nv.c:#include <asm/bootinfo.h> decstation
> >> ./drivers/net/macsonic.c:#include <asm/bootinfo.h> 68k
> >> ./drivers/net/jazzsonic.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/pmag-aa-fb.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/maxinefb.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/logo/logo.c:#include <asm/bootinfo.h> mips
> >> ./drivers/video/macfb.c:#include <asm/bootinfo.h> 68k
> >> ./drivers/video/valkyriefb.c:#include <asm/bootinfo.h> 68k
> >
> > Uh.. I'm pretty sure valkyriefb.c is for (old) PowerMacs, not 68k.
>
> It appears to be both. If you look at the include its protected by a
> #ifdef CONFIG_MAC which we is only defined on m68k.
According to drivers/video/macfb.c, both the Quadra 630 and Performa 588
have the Valkyrie chip for their video. I don't remember which ppc based
models have it, but it's definitely on both architectures.
Just as an extra note, the file drivers/macintosh/adb-iop.c is m68k only,
so you should probably leave that alone as well. It probably doesn't need
that header, but the change should really come from the 68k side of things.
Brad Boyer
flar@allandria.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-23 18:56 ` Brad Boyer
@ 2007-08-24 7:10 ` Geert Uytterhoeven
2007-08-24 7:24 ` Kumar Gala
0 siblings, 1 reply; 17+ messages in thread
From: Geert Uytterhoeven @ 2007-08-24 7:10 UTC (permalink / raw)
To: Brad Boyer; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
[-- Attachment #1: Type: TEXT/PLAIN, Size: 904 bytes --]
On Thu, 23 Aug 2007, Brad Boyer wrote:
> Just as an extra note, the file drivers/macintosh/adb-iop.c is m68k only,
> so you should probably leave that alone as well. It probably doesn't need
> that header, but the change should really come from the 68k side of things.
Thanks, it's indeed not needed.
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-24 7:10 ` Geert Uytterhoeven
@ 2007-08-24 7:24 ` Kumar Gala
2007-08-24 7:24 ` Geert Uytterhoeven
0 siblings, 1 reply; 17+ messages in thread
From: Kumar Gala @ 2007-08-24 7:24 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
On Aug 24, 2007, at 2:10 AM, Geert Uytterhoeven wrote:
> On Thu, 23 Aug 2007, Brad Boyer wrote:
>> Just as an extra note, the file drivers/macintosh/adb-iop.c is
>> m68k only,
>> so you should probably leave that alone as well. It probably
>> doesn't need
>> that header, but the change should really come from the 68k side
>> of things.
>
> Thanks, it's indeed not needed.
If its ok that the removal comes from my patchset that would be
great. I'm tired of re-spinning these patches at this point ;)
- k
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: asm-ppc header issues when building ARCH=powerpc
2007-08-24 7:24 ` Kumar Gala
@ 2007-08-24 7:24 ` Geert Uytterhoeven
0 siblings, 0 replies; 17+ messages in thread
From: Geert Uytterhoeven @ 2007-08-24 7:24 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev@ozlabs.org list, Paul Mackerras, David Gibson
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1247 bytes --]
On Fri, 24 Aug 2007, Kumar Gala wrote:
> On Aug 24, 2007, at 2:10 AM, Geert Uytterhoeven wrote:
> > On Thu, 23 Aug 2007, Brad Boyer wrote:
> > > Just as an extra note, the file drivers/macintosh/adb-iop.c is m68k only,
> > > so you should probably leave that alone as well. It probably doesn't need
> > > that header, but the change should really come from the 68k side of
> > > things.
> >
> > Thanks, it's indeed not needed.
>
> If its ok that the removal comes from my patchset that would be great. I'm
> tired of re-spinning these patches at this point ;)
Sure, less work for me! ;-)
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply [flat|nested] 17+ messages in thread