All of lore.kernel.org
 help / color / mirror / Atom feed
* linux 2.6.21-rc1 : FB Kconfig error
@ 2007-02-21  8:46 Toralf Förster
  2007-02-21 10:03 ` Geert Uytterhoeven
  0 siblings, 1 reply; 14+ messages in thread
From: Toralf Förster @ 2007-02-21  8:46 UTC (permalink / raw)
  To: linux-fbdev-devel


[-- Attachment #1.1: Type: text/plain, Size: 251 bytes --]

With latest sources I get :

...
 make mrproper ...
 make rndconfig ...drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'
 make oldconfig ...

-- 
MfG/Sincerely

Toralf Förster

[-- Attachment #1.2: Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 182 bytes --]

_______________________________________________
Linux-fbdev-devel mailing list
Linux-fbdev-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-21  8:46 linux 2.6.21-rc1 : FB Kconfig error Toralf Förster
@ 2007-02-21 10:03 ` Geert Uytterhoeven
  2007-02-21 12:12   ` Roman Zippel
  0 siblings, 1 reply; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-21 10:03 UTC (permalink / raw)
  To: Toralf Förster; +Cc: Roman Zippel, Linux Frame Buffer Device Development

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: TEXT/PLAIN; charset=UTF-8, Size: 735 bytes --]

	Hi Toralf,

On Wed, 21 Feb 2007, Toralf [iso-8859-1] Förster wrote:
> With latest sources I get :
> 
> ...
>  make mrproper ...
>  make rndconfig ...drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'
>  make oldconfig ...

This looks actually like a bug in kconf, since it doesn't obey the
`depends on PPC_PS3' one line above...

Roman, would it be hard to fix this?  Thx!

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

[-- Attachment #2: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #3: Type: text/plain, Size: 182 bytes --]

_______________________________________________
Linux-fbdev-devel mailing list
Linux-fbdev-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-21 10:03 ` Geert Uytterhoeven
@ 2007-02-21 12:12   ` Roman Zippel
  2007-02-21 12:27     ` Geert Uytterhoeven
  0 siblings, 1 reply; 14+ messages in thread
From: Roman Zippel @ 2007-02-21 12:12 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Toralf F�rster, Linux Frame Buffer Device Development

Hi,

On Wed, 21 Feb 2007, Geert Uytterhoeven wrote:

> >  make mrproper ...
> >  make rndconfig ...drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'
> >  make oldconfig ...
> 
> This looks actually like a bug in kconf, since it doesn't obey the
> `depends on PPC_PS3' one line above...

It has little to do with the dependencies, as select doesn't really care 
about them.
In this case it looks like someone went a little overboard with the select 
usage. It looks like most of the PS3 options could be hidden behind an 
advanced option, then you can also use normal dependencies.

bye, Roman

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-21 12:12   ` Roman Zippel
@ 2007-02-21 12:27     ` Geert Uytterhoeven
  2007-02-22  0:01       ` Roman Zippel
  0 siblings, 1 reply; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-21 12:27 UTC (permalink / raw)
  To: Roman Zippel, Linux Frame Buffer Device Development; +Cc: Toralf F�rster

On Wed, 21 Feb 2007, Roman Zippel wrote:
> On Wed, 21 Feb 2007, Geert Uytterhoeven wrote:
> > >  make mrproper ...
> > >  make rndconfig ...drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'
> > >  make oldconfig ...
> > 
> > This looks actually like a bug in kconf, since it doesn't obey the
> > `depends on PPC_PS3' one line above...
> 
> It has little to do with the dependencies, as select doesn't really care 
> about them.

Well, that's the problem :-)

Why doesn't it obey dependencies? On one side we want to put drivers of a
specific type under driver/*, on the other side we want to put
architecture-specific things under arch/*. But as soon as you want to select an
architecture-specific thing, this breaks (well, generates warnings).

> In this case it looks like someone went a little overboard with the select 
> usage. It looks like most of the PS3 options could be hidden behind an 
> advanced option, then you can also use normal dependencies.

We wanted to have PS3FB and PS3SND (not yet submitted) to select PS3AV, so it
gets enabled automatically. Else we need dependencies on things the user
shouldn't really need to care about.

What other solution do you propose?

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-21 12:27     ` Geert Uytterhoeven
@ 2007-02-22  0:01       ` Roman Zippel
  2007-02-22  8:53         ` Geert Uytterhoeven
  2007-02-22 17:55         ` [Linux-fbdev-devel] " Geert Uytterhoeven
  0 siblings, 2 replies; 14+ messages in thread
From: Roman Zippel @ 2007-02-22  0:01 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Toralf F�rster, Linux Frame Buffer Device Development

Hi,

On Wed, 21 Feb 2007, Geert Uytterhoeven wrote:

> Why doesn't it obey dependencies?

It was initially created to any override dependencies.

> > In this case it looks like someone went a little overboard with the select 
> > usage. It looks like most of the PS3 options could be hidden behind an 
> > advanced option, then you can also use normal dependencies.
> 
> We wanted to have PS3FB and PS3SND (not yet submitted) to select PS3AV, so it
> gets enabled automatically. Else we need dependencies on things the user
> shouldn't really need to care about.
> 
> What other solution do you propose?

As I said most PS3 could be hidden behind a PS3_ADVANCED option and use 
normal dependencies for the rest. Normal users don't care about all 
the options and advanced users should know what they're doing.

bye, Roman

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-22  0:01       ` Roman Zippel
@ 2007-02-22  8:53         ` Geert Uytterhoeven
  2007-02-22 17:55         ` [Linux-fbdev-devel] " Geert Uytterhoeven
  1 sibling, 0 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-22  8:53 UTC (permalink / raw)
  To: Roman Zippel; +Cc: Toralf F�rster, Linux Frame Buffer Device Development

On Thu, 22 Feb 2007, Roman Zippel wrote:
> On Wed, 21 Feb 2007, Geert Uytterhoeven wrote:
> > We wanted to have PS3FB and PS3SND (not yet submitted) to select PS3AV, so it
> > gets enabled automatically. Else we need dependencies on things the user
> > shouldn't really need to care about.
> > 
> > What other solution do you propose?
> 
> As I said most PS3 could be hidden behind a PS3_ADVANCED option and use 
> normal dependencies for the rest. Normal users don't care about all 
> the options and advanced users should know what they're doing.

Ah, OK, now I see what you mean. Thanks!

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: [Linux-fbdev-devel] linux 2.6.21-rc1 : FB Kconfig error
  2007-02-22  0:01       ` Roman Zippel
  2007-02-22  8:53         ` Geert Uytterhoeven
@ 2007-02-22 17:55         ` Geert Uytterhoeven
  2007-02-23 22:44             ` [Linux-fbdev-devel] " Roman Zippel
  1 sibling, 1 reply; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-22 17:55 UTC (permalink / raw)
  To: Roman Zippel
  Cc: Linux/PPC Development, Linux Frame Buffer Device Development,
	Toralf F�rster

On Thu, 22 Feb 2007, Roman Zippel wrote:
> On Wed, 21 Feb 2007, Geert Uytterhoeven wrote:
> > Why doesn't it obey dependencies?
> 
> It was initially created to any override dependencies.
> 
> > > In this case it looks like someone went a little overboard with the select 
> > > usage. It looks like most of the PS3 options could be hidden behind an 
> > > advanced option, then you can also use normal dependencies.
> > 
> > We wanted to have PS3FB and PS3SND (not yet submitted) to select PS3AV, so it
> > gets enabled automatically. Else we need dependencies on things the user
> > shouldn't really need to care about.
> > 
> > What other solution do you propose?
> 
> As I said most PS3 could be hidden behind a PS3_ADVANCED option and use 
> normal dependencies for the rest. Normal users don't care about all 
> the options and advanced users should know what they're doing.

Like this?

Subject: ps3: Introduce CONFIG_PS3_ADVANCED

From: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>

ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
control questions about PS3 subsystems that may not be obvious for the casual
user.

This gets rid of the following warning:
| drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Cc: Roman Zippel <zippel@linux-m68k.org>
---
 arch/powerpc/platforms/ps3/Kconfig |   29 ++++++++++++++++++++++-------
 drivers/video/Kconfig              |    3 +--
 2 files changed, 23 insertions(+), 9 deletions(-)

--- ps3-linux-2.6.21-rc1.orig/arch/powerpc/platforms/ps3/Kconfig
+++ ps3-linux-2.6.21-rc1/arch/powerpc/platforms/ps3/Kconfig
@@ -1,9 +1,25 @@
 menu "PS3 Platform Options"
 	depends on PPC_PS3
 
+config PS3_ADVANCED
+	depends on PPC_PS3
+	bool "PS3 Advanced configuration options"
+	default n
+	help
+	  This gives you access to some advanced options for the PS3. The
+	  defaults should be fine for most users, but these options may make
+	  it possible to better control the kernel configuration if you know
+	  what you are doing.
+
+	  Note that the answer to this question won't directly affect the
+	  kernel: saying N will just cause the configurator to skip all
+	  the questions about these options.
+
+	  Most users should say N to this question.
+
 config PS3_HTAB_SIZE
 	depends on PPC_PS3
-	int "PS3 Platform pagetable size"
+	int "PS3 Platform pagetable size" if PS3_ADVANCED
 	range 18 20
 	default 20
 	help
@@ -42,7 +58,7 @@ config PS3_USE_LPAR_ADDR
 
 config PS3_VUART
 	depends on PPC_PS3
-	bool "PS3 Virtual UART support"
+	bool "PS3 Virtual UART support" if PS3_ADVANCED
 	default y
 	help
 	  Include support for the PS3 Virtual UART.
@@ -52,9 +68,8 @@ config PS3_VUART
 	  general, all users will say Y.
 
 config PS3_PS3AV
-	tristate "PS3 AV settings driver"
-	depends on PPC_PS3
-	select PS3_VUART
+	tristate "PS3 AV settings driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 AV Settings driver.
@@ -63,8 +78,8 @@ config PS3_PS3AV
 	  general, all users will say Y or M.
 
 config PS3_SYS_MANAGER
-	bool "PS3 System Manager driver"
-	select PS3_VUART
+	bool "PS3 System Manager driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 System Manager.
--- ps3-linux-2.6.21-rc1.orig/drivers/video/Kconfig
+++ ps3-linux-2.6.21-rc1/drivers/video/Kconfig
@@ -1618,8 +1618,7 @@ config FB_IBM_GXT4500
 
 config FB_PS3
 	bool "PS3 GPU framebuffer driver"
-	depends on FB && PPC_PS3
-	select PS3_PS3AV
+	depends on FB && PS3_PS3AV
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
 	select FB_CFB_IMAGEBLIT

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

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

* Re: linux 2.6.21-rc1 : FB Kconfig error
  2007-02-22 17:55         ` [Linux-fbdev-devel] " Geert Uytterhoeven
@ 2007-02-23 22:44             ` Roman Zippel
  0 siblings, 0 replies; 14+ messages in thread
From: Roman Zippel @ 2007-02-23 22:44 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linux/PPC Development, Linux Frame Buffer Device Development,
	Geoff Levand, Toralf F�rster

Hi,

On Thu, 22 Feb 2007, Geert Uytterhoeven wrote:

> Like this?
> 
> Subject: ps3: Introduce CONFIG_PS3_ADVANCED

Looks good.

> --- ps3-linux-2.6.21-rc1.orig/arch/powerpc/platforms/ps3/Kconfig
> +++ ps3-linux-2.6.21-rc1/arch/powerpc/platforms/ps3/Kconfig
> @@ -1,9 +1,25 @@
>  menu "PS3 Platform Options"
>  	depends on PPC_PS3
>  
> +config PS3_ADVANCED
> +	depends on PPC_PS3
> +	bool "PS3 Advanced configuration options"
> +	default n

except that this default is not really necessary.

bye, Roman

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: [Linux-fbdev-devel] linux 2.6.21-rc1 : FB Kconfig error
@ 2007-02-23 22:44             ` Roman Zippel
  0 siblings, 0 replies; 14+ messages in thread
From: Roman Zippel @ 2007-02-23 22:44 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linux/PPC Development, Linux Frame Buffer Device Development,
	Toralf F�rster

Hi,

On Thu, 22 Feb 2007, Geert Uytterhoeven wrote:

> Like this?
> 
> Subject: ps3: Introduce CONFIG_PS3_ADVANCED

Looks good.

> --- ps3-linux-2.6.21-rc1.orig/arch/powerpc/platforms/ps3/Kconfig
> +++ ps3-linux-2.6.21-rc1/arch/powerpc/platforms/ps3/Kconfig
> @@ -1,9 +1,25 @@
>  menu "PS3 Platform Options"
>  	depends on PPC_PS3
>  
> +config PS3_ADVANCED
> +	depends on PPC_PS3
> +	bool "PS3 Advanced configuration options"
> +	default n

except that this default is not really necessary.

bye, Roman

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

* [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED
  2007-02-23 22:44             ` [Linux-fbdev-devel] " Roman Zippel
@ 2007-02-26 15:48               ` Geert Uytterhoeven
  -1 siblings, 0 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-26 15:48 UTC (permalink / raw)
  To: Linus Torvalds, Andrew Morton
  Cc: Geoff Levand, Paul Mackerras, Roman Zippel,
	Linux Frame Buffer Device Development, Linux/PPC Development,
	Linux Kernel Development, Toralf F�rster

ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
control questions about PS3 subsystems that may not be obvious for the casual
user.

This gets rid of the following warning on non-powerpc platforms:
| drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Cc: Geoff Levand <geoffrey.levand@am.sony.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Roman Zippel <zippel@linux-m68k.org>
---
 arch/powerpc/platforms/ps3/Kconfig |   28 +++++++++++++++++++++-------
 drivers/video/Kconfig              |    3 +--
 2 files changed, 22 insertions(+), 9 deletions(-)

--- ps3-linux-2.6.21-rc1.orig/arch/powerpc/platforms/ps3/Kconfig
+++ ps3-linux-2.6.21-rc1/arch/powerpc/platforms/ps3/Kconfig
@@ -1,9 +1,24 @@
 menu "PS3 Platform Options"
 	depends on PPC_PS3
 
+config PS3_ADVANCED
+	depends on PPC_PS3
+	bool "PS3 Advanced configuration options"
+	help
+	  This gives you access to some advanced options for the PS3. The
+	  defaults should be fine for most users, but these options may make
+	  it possible to better control the kernel configuration if you know
+	  what you are doing.
+
+	  Note that the answer to this question won't directly affect the
+	  kernel: saying N will just cause the configurator to skip all
+	  the questions about these options.
+
+	  Most users should say N to this question.
+
 config PS3_HTAB_SIZE
 	depends on PPC_PS3
-	int "PS3 Platform pagetable size"
+	int "PS3 Platform pagetable size" if PS3_ADVANCED
 	range 18 20
 	default 20
 	help
@@ -42,7 +57,7 @@ config PS3_USE_LPAR_ADDR
 
 config PS3_VUART
 	depends on PPC_PS3
-	bool "PS3 Virtual UART support"
+	bool "PS3 Virtual UART support" if PS3_ADVANCED
 	default y
 	help
 	  Include support for the PS3 Virtual UART.
@@ -52,9 +67,8 @@ config PS3_VUART
 	  general, all users will say Y.
 
 config PS3_PS3AV
-	tristate "PS3 AV settings driver"
-	depends on PPC_PS3
-	select PS3_VUART
+	tristate "PS3 AV settings driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 AV Settings driver.
@@ -63,8 +77,8 @@ config PS3_PS3AV
 	  general, all users will say Y or M.
 
 config PS3_SYS_MANAGER
-	bool "PS3 System Manager driver"
-	select PS3_VUART
+	bool "PS3 System Manager driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 System Manager.
--- ps3-linux-2.6.21-rc1.orig/drivers/video/Kconfig
+++ ps3-linux-2.6.21-rc1/drivers/video/Kconfig
@@ -1618,8 +1618,7 @@ config FB_IBM_GXT4500
 
 config FB_PS3
 	bool "PS3 GPU framebuffer driver"
-	depends on FB && PPC_PS3
-	select PS3_PS3AV
+	depends on FB && PS3_PS3AV
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
 	select FB_CFB_IMAGEBLIT

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

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

* [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED
@ 2007-02-26 15:48               ` Geert Uytterhoeven
  0 siblings, 0 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2007-02-26 15:48 UTC (permalink / raw)
  To: Linus Torvalds, Andrew Morton
  Cc: Linux Frame Buffer Device Development, Toralf F�rster,
	Linux Kernel Development, Linux/PPC Development, Paul Mackerras

ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
control questions about PS3 subsystems that may not be obvious for the casual
user.

This gets rid of the following warning on non-powerpc platforms:
| drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Cc: Geoff Levand <geoffrey.levand@am.sony.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Roman Zippel <zippel@linux-m68k.org>
---
 arch/powerpc/platforms/ps3/Kconfig |   28 +++++++++++++++++++++-------
 drivers/video/Kconfig              |    3 +--
 2 files changed, 22 insertions(+), 9 deletions(-)

--- ps3-linux-2.6.21-rc1.orig/arch/powerpc/platforms/ps3/Kconfig
+++ ps3-linux-2.6.21-rc1/arch/powerpc/platforms/ps3/Kconfig
@@ -1,9 +1,24 @@
 menu "PS3 Platform Options"
 	depends on PPC_PS3
 
+config PS3_ADVANCED
+	depends on PPC_PS3
+	bool "PS3 Advanced configuration options"
+	help
+	  This gives you access to some advanced options for the PS3. The
+	  defaults should be fine for most users, but these options may make
+	  it possible to better control the kernel configuration if you know
+	  what you are doing.
+
+	  Note that the answer to this question won't directly affect the
+	  kernel: saying N will just cause the configurator to skip all
+	  the questions about these options.
+
+	  Most users should say N to this question.
+
 config PS3_HTAB_SIZE
 	depends on PPC_PS3
-	int "PS3 Platform pagetable size"
+	int "PS3 Platform pagetable size" if PS3_ADVANCED
 	range 18 20
 	default 20
 	help
@@ -42,7 +57,7 @@ config PS3_USE_LPAR_ADDR
 
 config PS3_VUART
 	depends on PPC_PS3
-	bool "PS3 Virtual UART support"
+	bool "PS3 Virtual UART support" if PS3_ADVANCED
 	default y
 	help
 	  Include support for the PS3 Virtual UART.
@@ -52,9 +67,8 @@ config PS3_VUART
 	  general, all users will say Y.
 
 config PS3_PS3AV
-	tristate "PS3 AV settings driver"
-	depends on PPC_PS3
-	select PS3_VUART
+	tristate "PS3 AV settings driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 AV Settings driver.
@@ -63,8 +77,8 @@ config PS3_PS3AV
 	  general, all users will say Y or M.
 
 config PS3_SYS_MANAGER
-	bool "PS3 System Manager driver"
-	select PS3_VUART
+	bool "PS3 System Manager driver" if PS3_ADVANCED
+	depends on PS3_VUART
 	default y
 	help
 	  Include support for the PS3 System Manager.
--- ps3-linux-2.6.21-rc1.orig/drivers/video/Kconfig
+++ ps3-linux-2.6.21-rc1/drivers/video/Kconfig
@@ -1618,8 +1618,7 @@ config FB_IBM_GXT4500
 
 config FB_PS3
 	bool "PS3 GPU framebuffer driver"
-	depends on FB && PPC_PS3
-	select PS3_PS3AV
+	depends on FB && PS3_PS3AV
 	select FB_CFB_FILLRECT
 	select FB_CFB_COPYAREA
 	select FB_CFB_IMAGEBLIT

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium

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

* Re: [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED
  2007-02-26 15:48               ` Geert Uytterhoeven
  (?)
@ 2007-02-26 16:35                 ` Geoff Levand
  -1 siblings, 0 replies; 14+ messages in thread
From: Geoff Levand @ 2007-02-26 16:35 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Geert Uytterhoeven, Andrew Morton, Paul Mackerras, Roman Zippel,
	Linux Frame Buffer Device Development, Linux/PPC Development,
	Linux Kernel Development, �

Geert Uytterhoeven wrote:
> ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
> control questions about PS3 subsystems that may not be obvious for the casual
> user.
> 
> This gets rid of the following warning on non-powerpc platforms:
> | drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Please apply, fixes build warning in 2.6.21-rc1.

Acked-by: Geoff Levand <geoffrey.levand@am.sony.com>


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

* Re: [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED
@ 2007-02-26 16:35                 ` Geoff Levand
  0 siblings, 0 replies; 14+ messages in thread
From: Geoff Levand @ 2007-02-26 16:35 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Linux Frame Buffer Device Development, Roman Zippel, �,
	Linux Kernel Development, Linux/PPC Development, Paul Mackerras,
	Geert Uytterhoeven, Andrew Morton

Geert Uytterhoeven wrote:
> ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
> control questions about PS3 subsystems that may not be obvious for the casual
> user.
> 
> This gets rid of the following warning on non-powerpc platforms:
> | drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Please apply, fixes build warning in 2.6.21-rc1.

Acked-by: Geoff Levand <geoffrey.levand@am.sony.com>


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

* Re: [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED
@ 2007-02-26 16:35                 ` Geoff Levand
  0 siblings, 0 replies; 14+ messages in thread
From: Geoff Levand @ 2007-02-26 16:35 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Linux Frame Buffer Device Development, �,
	Linux Kernel Development, Linux/PPC Development, Paul Mackerras,
	Geert Uytterhoeven, Andrew Morton

Geert Uytterhoeven wrote:
> ps3: Introduce CONFIG_PS3_ADVANCED, as suggested by Roman Zippel, and use it to
> control questions about PS3 subsystems that may not be obvious for the casual
> user.
> 
> This gets rid of the following warning on non-powerpc platforms:
> | drivers/video/Kconfig:1604:warning: 'select' used by config symbol 'FB_PS3' refer to undefined symbol 'PS3_PS3AV'

Please apply, fixes build warning in 2.6.21-rc1.

Acked-by: Geoff Levand <geoffrey.levand@am.sony.com>

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

end of thread, other threads:[~2007-02-26 16:35 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-21  8:46 linux 2.6.21-rc1 : FB Kconfig error Toralf Förster
2007-02-21 10:03 ` Geert Uytterhoeven
2007-02-21 12:12   ` Roman Zippel
2007-02-21 12:27     ` Geert Uytterhoeven
2007-02-22  0:01       ` Roman Zippel
2007-02-22  8:53         ` Geert Uytterhoeven
2007-02-22 17:55         ` [Linux-fbdev-devel] " Geert Uytterhoeven
2007-02-23 22:44           ` Roman Zippel
2007-02-23 22:44             ` [Linux-fbdev-devel] " Roman Zippel
2007-02-26 15:48             ` [PATCH] ps3: Introduce CONFIG_PS3_ADVANCED Geert Uytterhoeven
2007-02-26 15:48               ` Geert Uytterhoeven
2007-02-26 16:35               ` Geoff Levand
2007-02-26 16:35                 ` Geoff Levand
2007-02-26 16:35                 ` Geoff Levand

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.