All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
@ 2013-12-12 17:56 Ben Dooks
  2013-12-12 18:04 ` Sergei Shtylyov
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Ben Dooks @ 2013-12-12 17:56 UTC (permalink / raw)
  To: linux-sh

The PCI sub-system is not enabled by default on ARM and on certain
Renesas devices the build does not select it. This means that there
are configurations that do not allow the AHB-PCI bridge used for the
USB sub-systems to be built.

For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
where there may be many Reneasas devices selected.

Cc: Simon Horman <horms@verge.net.au>
Cc: Magnus Damm <magnus.damm@gmail.com>
CC: Linux SH List <linux-sh@vger.kernel.org>
Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 arch/arm/mach-shmobile/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index 3e57d45..010d2a6 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -12,6 +12,7 @@ config ARCH_SHMOBILE_MULTI
 	select HAVE_SMP
 	select ARM_GIC
 	select MIGHT_HAVE_CACHE_L2X0
+	select MIGHT_HAVE_PCI
 	select NO_IOPORT
 	select PINCTRL
 	select ARCH_REQUIRE_GPIOLIB
@@ -101,6 +102,7 @@ config ARCH_R8A7779
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select USB_ARCH_HAS_EHCI
 	select USB_ARCH_HAS_OHCI
@@ -111,6 +113,7 @@ config ARCH_R8A7790
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select RENESAS_IRQC
 
@@ -127,6 +130,7 @@ config ARCH_EMEV2
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select USE_OF
 	select AUTO_ZRELADDR
 
-- 
1.8.5.1


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

* Re: [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
@ 2013-12-12 18:04 ` Sergei Shtylyov
  2013-12-12 18:10 ` Ben Dooks
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Sergei Shtylyov @ 2013-12-12 18:04 UTC (permalink / raw)
  To: linux-sh

Hello.

On 12/12/2013 08:56 PM, Ben Dooks wrote:

> The PCI sub-system is not enabled by default on ARM and on certain
> Renesas devices the build does not select it. This means that there
> are configurations that do not allow the AHB-PCI bridge used for the
> USB sub-systems to be built.

> For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
> PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
> where there may be many Reneasas devices selected.

> Cc: Simon Horman <horms@verge.net.au>
> Cc: Magnus Damm <magnus.damm@gmail.com>
> CC: Linux SH List <linux-sh@vger.kernel.org>
> Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
> ---
>   arch/arm/mach-shmobile/Kconfig | 4 ++++
>   1 file changed, 4 insertions(+)

> diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> index 3e57d45..010d2a6 100644
> --- a/arch/arm/mach-shmobile/Kconfig
> +++ b/arch/arm/mach-shmobile/Kconfig
[...]
> @@ -101,6 +102,7 @@ config ARCH_R8A7779
>   	select ARCH_WANT_OPTIONAL_GPIOLIB
>   	select ARM_GIC
>   	select CPU_V7
> +	select MIGHT_HAVE_PCI
>   	select SH_CLK_CPG
>   	select USB_ARCH_HAS_EHCI
>   	select USB_ARCH_HAS_OHCI

    Er, this is R8A7779 section. This SoC doesn't have PCI-AHB bridge.

> @@ -111,6 +113,7 @@ config ARCH_R8A7790
>   	select ARCH_WANT_OPTIONAL_GPIOLIB
>   	select ARM_GIC
>   	select CPU_V7
> +	select MIGHT_HAVE_PCI
>   	select SH_CLK_CPG
>   	select RENESAS_IRQC
>

    R8A7791 section is contrarywise missing. BTW, there's stray letter 'c' 
before that section.

WBR, Sergei


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

* Re: [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
  2013-12-12 18:04 ` Sergei Shtylyov
@ 2013-12-12 18:10 ` Ben Dooks
  2013-12-12 18:11 ` Ben Dooks
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Ben Dooks @ 2013-12-12 18:10 UTC (permalink / raw)
  To: linux-sh

On 12/12/13 19:04, Sergei Shtylyov wrote:
> Hello.
>
> On 12/12/2013 08:56 PM, Ben Dooks wrote:
>
>> The PCI sub-system is not enabled by default on ARM and on certain
>> Renesas devices the build does not select it. This means that there
>> are configurations that do not allow the AHB-PCI bridge used for the
>> USB sub-systems to be built.
>
>> For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
>> PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
>> where there may be many Reneasas devices selected.
>
>> Cc: Simon Horman <horms@verge.net.au>
>> Cc: Magnus Damm <magnus.damm@gmail.com>
>> CC: Linux SH List <linux-sh@vger.kernel.org>
>> Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
>> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
>> ---
>>   arch/arm/mach-shmobile/Kconfig | 4 ++++
>>   1 file changed, 4 insertions(+)
>
>> diff --git a/arch/arm/mach-shmobile/Kconfig
>> b/arch/arm/mach-shmobile/Kconfig
>> index 3e57d45..010d2a6 100644
>> --- a/arch/arm/mach-shmobile/Kconfig
>> +++ b/arch/arm/mach-shmobile/Kconfig
> [...]
>> @@ -101,6 +102,7 @@ config ARCH_R8A7779
>>       select ARCH_WANT_OPTIONAL_GPIOLIB
>>       select ARM_GIC
>>       select CPU_V7
>> +    select MIGHT_HAVE_PCI
>>       select SH_CLK_CPG
>>       select USB_ARCH_HAS_EHCI
>>       select USB_ARCH_HAS_OHCI
>
>     Er, this is R8A7779 section. This SoC doesn't have PCI-AHB bridge.
>
>> @@ -111,6 +113,7 @@ config ARCH_R8A7790
>>       select ARCH_WANT_OPTIONAL_GPIOLIB
>>       select ARM_GIC
>>       select CPU_V7
>> +    select MIGHT_HAVE_PCI
>>       select SH_CLK_CPG
>>       select RENESAS_IRQC
>>
>
>     R8A7791 section is contrarywise missing. BTW, there's stray letter
> 'c' before that section.

Thanks. I've fixed the R8A7791. I cannot see the stray 'c'.

> WBR, Sergei
>


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

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

* [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
  2013-12-12 18:04 ` Sergei Shtylyov
  2013-12-12 18:10 ` Ben Dooks
@ 2013-12-12 18:11 ` Ben Dooks
  2013-12-12 18:12 ` Ben Dooks
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Ben Dooks @ 2013-12-12 18:11 UTC (permalink / raw)
  To: linux-sh

The PCI sub-system is not enabled by default on ARM and on certain
Renesas devices the build does not select it. This means that there
are configurations that do not allow the AHB-PCI bridge used for the
USB sub-systems to be built.

For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
where there may be many Reneasas devices selected.

Cc: Simon Horman <horms@verge.net.au>
Cc: Magnus Damm <magnus.damm@gmail.com>
CC: Linux SH List <linux-sh@vger.kernel.org>
Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 arch/arm/mach-shmobile/Kconfig | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index 3e57d45..4dcb216 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -12,6 +12,7 @@ config ARCH_SHMOBILE_MULTI
 	select HAVE_SMP
 	select ARM_GIC
 	select MIGHT_HAVE_CACHE_L2X0
+	select MIGHT_HAVE_PCI
 	select NO_IOPORT
 	select PINCTRL
 	select ARCH_REQUIRE_GPIOLIB
@@ -101,6 +102,7 @@ config ARCH_R8A7779
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select USB_ARCH_HAS_EHCI
 	select USB_ARCH_HAS_OHCI
@@ -111,6 +113,7 @@ config ARCH_R8A7790
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select RENESAS_IRQC
 
@@ -119,6 +122,7 @@ config ARCH_R8A7791
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select RENESAS_IRQC
 
@@ -127,6 +131,7 @@ config ARCH_EMEV2
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select USE_OF
 	select AUTO_ZRELADDR
 
-- 
1.8.5.1


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

* Re: [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
                   ` (2 preceding siblings ...)
  2013-12-12 18:11 ` Ben Dooks
@ 2013-12-12 18:12 ` Ben Dooks
  2013-12-12 18:14 ` Ben Dooks
  2013-12-14  1:46 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Ben Dooks @ 2013-12-12 18:12 UTC (permalink / raw)
  To: linux-sh

On 12/12/13 18:11, Ben Dooks wrote:
> The PCI sub-system is not enabled by default on ARM and on certain
> Renesas devices the build does not select it. This means that there
> are configurations that do not allow the AHB-PCI bridge used for the
> USB sub-systems to be built.
>
> For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
> PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
> where there may be many Reneasas devices selected.
>
> Cc: Simon Horman <horms@verge.net.au>
> Cc: Magnus Damm <magnus.damm@gmail.com>
> CC: Linux SH List <linux-sh@vger.kernel.org>
> Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>

Argh, sorry did not remove the extra Kconfig entry.


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

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

* [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
                   ` (3 preceding siblings ...)
  2013-12-12 18:12 ` Ben Dooks
@ 2013-12-12 18:14 ` Ben Dooks
  2013-12-14  1:46 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Ben Dooks @ 2013-12-12 18:14 UTC (permalink / raw)
  To: linux-sh

The PCI sub-system is not enabled by default on ARM and on certain
Renesas devices the build does not select it. This means that there
are configurations that do not allow the AHB-PCI bridge used for the
USB sub-systems to be built.

For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
where there may be many Reneasas devices selected.

Cc: Simon Horman <horms@verge.net.au>
Cc: Magnus Damm <magnus.damm@gmail.com>
CC: Linux SH List <linux-sh@vger.kernel.org>
Reviewed-by: Ian Molton <ian.molton@codethink.co.uk>
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 arch/arm/mach-shmobile/Kconfig | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
index 3e57d45..f01e878 100644
--- a/arch/arm/mach-shmobile/Kconfig
+++ b/arch/arm/mach-shmobile/Kconfig
@@ -12,6 +12,7 @@ config ARCH_SHMOBILE_MULTI
 	select HAVE_SMP
 	select ARM_GIC
 	select MIGHT_HAVE_CACHE_L2X0
+	select MIGHT_HAVE_PCI
 	select NO_IOPORT
 	select PINCTRL
 	select ARCH_REQUIRE_GPIOLIB
@@ -111,6 +112,7 @@ config ARCH_R8A7790
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select RENESAS_IRQC
 
@@ -119,6 +121,7 @@ config ARCH_R8A7791
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select SH_CLK_CPG
 	select RENESAS_IRQC
 
@@ -127,6 +130,7 @@ config ARCH_EMEV2
 	select ARCH_WANT_OPTIONAL_GPIOLIB
 	select ARM_GIC
 	select CPU_V7
+	select MIGHT_HAVE_PCI
 	select USE_OF
 	select AUTO_ZRELADDR
 
-- 
1.8.5.1


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

* Re: [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code
  2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
                   ` (4 preceding siblings ...)
  2013-12-12 18:14 ` Ben Dooks
@ 2013-12-14  1:46 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2013-12-14  1:46 UTC (permalink / raw)
  To: linux-sh

On Thu, Dec 12, 2013 at 06:14:21PM +0000, Ben Dooks wrote:
> The PCI sub-system is not enabled by default on ARM and on certain
> Renesas devices the build does not select it. This means that there
> are configurations that do not allow the AHB-PCI bridge used for the
> USB sub-systems to be built.
> 
> For the R8A7790, R8A7791 and EMEV-2 select MIGHT_HAVE_PCI to allow the
> PCI drivers to be built. Also select MIGHT_HAVE_PCI for the multi-config
> where there may be many Reneasas devices selected.

Thanks, I will queue this up.

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

end of thread, other threads:[~2013-12-14  1:46 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-12 17:56 [PATCH] ARM: shmobile: Add select MIGHT_HAVE_PCI for PCI-AHB bridge code Ben Dooks
2013-12-12 18:04 ` Sergei Shtylyov
2013-12-12 18:10 ` Ben Dooks
2013-12-12 18:11 ` Ben Dooks
2013-12-12 18:12 ` Ben Dooks
2013-12-12 18:14 ` Ben Dooks
2013-12-14  1:46 ` Simon Horman

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.