linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Add PCI config dependency for PTI.
@ 2011-07-07 19:24 james_p_freyensee
  2011-07-07 19:31 ` Randy Dunlap
  0 siblings, 1 reply; 8+ messages in thread
From: james_p_freyensee @ 2011-07-07 19:24 UTC (permalink / raw)
  To: linux-next
  Cc: linux-kernel, randy.dunlap, sfr, christophe.guerard,
	pranav.k.sanghadia

From: J Freyensee <james_p_freyensee@linux.intel.com>

Intel-Atom implementation of PTI will not work with PCI
configured off, so the pti driver should not be compiled
in the event someone does not enable PCI in the linux kernel.

Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
 drivers/misc/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index 4e349cd..7fd32a6 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -146,6 +146,7 @@ config PHANTOM
 
 config INTEL_MID_PTI
 	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
+	depends on PCI
 	default n
 	help
 	  The PTI (Parallel Trace Interface) driver directs
-- 
1.7.0.4

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

* Re: [PATCH] Add PCI config dependency for PTI.
  2011-07-07 19:24 james_p_freyensee
@ 2011-07-07 19:31 ` Randy Dunlap
  2011-07-07 19:59   ` J Freyensee
  0 siblings, 1 reply; 8+ messages in thread
From: Randy Dunlap @ 2011-07-07 19:31 UTC (permalink / raw)
  To: james_p_freyensee
  Cc: linux-next, linux-kernel, randy.dunlap, sfr, christophe.guerard,
	pranav.k.sanghadia

On Thu,  7 Jul 2011 12:24:13 -0700 james_p_freyensee@linux.intel.com wrote:

> From: J Freyensee <james_p_freyensee@linux.intel.com>
> 
> Intel-Atom implementation of PTI will not work with PCI
> configured off, so the pti driver should not be compiled
> in the event someone does not enable PCI in the linux kernel.
> 
> Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>
> Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

now you can say that above. ^^^

Also
Reported-by: Randy Dunlap <randy.dunlap@oracle.com>

Thanks.

> Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
>  drivers/misc/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 4e349cd..7fd32a6 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -146,6 +146,7 @@ config PHANTOM
>  
>  config INTEL_MID_PTI
>  	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
> +	depends on PCI
>  	default n
>  	help
>  	  The PTI (Parallel Trace Interface) driver directs
> -- 


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

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

* Re: [PATCH] Add PCI config dependency for PTI.
  2011-07-07 19:31 ` Randy Dunlap
@ 2011-07-07 19:59   ` J Freyensee
  2011-07-07 20:59     ` Randy Dunlap
  0 siblings, 1 reply; 8+ messages in thread
From: J Freyensee @ 2011-07-07 19:59 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: linux-next, linux-kernel, sfr, christophe.guerard,
	pranav.k.sanghadia

On 07/07/2011 12:31 PM, Randy Dunlap wrote:
> On Thu,  7 Jul 2011 12:24:13 -0700 james_p_freyensee@linux.intel.com wrote:
>
>> From: J Freyensee<james_p_freyensee@linux.intel.com>
>>
>> Intel-Atom implementation of PTI will not work with PCI
>> configured off, so the pti driver should not be compiled
>> in the event someone does not enable PCI in the linux kernel.
>>
>> Signed-off-by: J Freyensee<james_p_freyensee@linux.intel.com>
>> Acked-by: Randy Dunlap<randy.dunlap@oracle.com>
> now you can say that above. ^^^
So you want me to stick the 'Acked-by' and 'Reported-by' above the 
Signed-off-by line?

> Also
> Reported-by: Randy Dunlap<randy.dunlap@oracle.com>
>
> Thanks.
>
>> Acked-by: Stephen Rothwell<sfr@canb.auug.org.au>
>> ---
>>   drivers/misc/Kconfig |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
>> index 4e349cd..7fd32a6 100644
>> --- a/drivers/misc/Kconfig
>> +++ b/drivers/misc/Kconfig
>> @@ -146,6 +146,7 @@ config PHANTOM
>>
>>   config INTEL_MID_PTI
>>   	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
>> +	depends on PCI
>>   	default n
>>   	help
>>   	  The PTI (Parallel Trace Interface) driver directs
>> -- 
>
> ---
> ~Randy
> *** Remember to use Documentation/SubmitChecklist when testing your code ***

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

* Re: [PATCH] Add PCI config dependency for PTI.
  2011-07-07 19:59   ` J Freyensee
@ 2011-07-07 20:59     ` Randy Dunlap
  0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2011-07-07 20:59 UTC (permalink / raw)
  To: J Freyensee
  Cc: linux-next, linux-kernel, sfr, christophe.guerard,
	pranav.k.sanghadia

On 07/07/11 12:59, J Freyensee wrote:
> On 07/07/2011 12:31 PM, Randy Dunlap wrote:
>> On Thu,  7 Jul 2011 12:24:13 -0700 james_p_freyensee@linux.intel.com
>> wrote:
>>
>>> From: J Freyensee<james_p_freyensee@linux.intel.com>
>>>
>>> Intel-Atom implementation of PTI will not work with PCI
>>> configured off, so the pti driver should not be compiled
>>> in the event someone does not enable PCI in the linux kernel.
>>>
>>> Signed-off-by: J Freyensee<james_p_freyensee@linux.intel.com>
>>> Acked-by: Randy Dunlap<randy.dunlap@oracle.com>
>> now you can say that above. ^^^
> So you want me to stick the 'Acked-by' and 'Reported-by' above the
> Signed-off-by line?

You had Acked-by: prematurely.

Yes, now you can put both Acked-by: and Reported-by:
above the S-O-B: line.

>> Also
>> Reported-by: Randy Dunlap<randy.dunlap@oracle.com>
>>
>> Thanks.
>>
>>> Acked-by: Stephen Rothwell<sfr@canb.auug.org.au>
>>> ---
>>>   drivers/misc/Kconfig |    1 +
>>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
>>> index 4e349cd..7fd32a6 100644
>>> --- a/drivers/misc/Kconfig
>>> +++ b/drivers/misc/Kconfig
>>> @@ -146,6 +146,7 @@ config PHANTOM
>>>
>>>   config INTEL_MID_PTI
>>>       tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG
>>> standard"
>>> +    depends on PCI
>>>       default n
>>>       help
>>>         The PTI (Parallel Trace Interface) driver directs
>>> -- 
>>
>> ---
>> ~Randy
>> *** Remember to use Documentation/SubmitChecklist when testing your
>> code ***
> 


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

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

* [PATCH] Add PCI config dependency for PTI.
@ 2011-07-07 21:12 james_p_freyensee
  2011-07-07 21:17 ` Randy Dunlap
  0 siblings, 1 reply; 8+ messages in thread
From: james_p_freyensee @ 2011-07-07 21:12 UTC (permalink / raw)
  To: linux-next
  Cc: linux-kernel, randy.dunlap, sfr, christophe.guerard,
	pranav.k.sanghadia, james_p_freyensee

From: J Freyensee <james_p_freyensee@linux.intel.com>

Intel-Atom implementation of PTI will not work with PCI
configured off, so the pti driver should not be compiled
in the event someone does not enable PCI in the linux kernel.

Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>
---
 drivers/misc/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index 4e349cd..7fd32a6 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -146,6 +146,7 @@ config PHANTOM
 
 config INTEL_MID_PTI
 	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
+	depends on PCI
 	default n
 	help
 	  The PTI (Parallel Trace Interface) driver directs
-- 
1.7.0.4

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

* Re: [PATCH] Add PCI config dependency for PTI.
  2011-07-07 21:12 james_p_freyensee
@ 2011-07-07 21:17 ` Randy Dunlap
  0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2011-07-07 21:17 UTC (permalink / raw)
  To: james_p_freyensee
  Cc: linux-next, linux-kernel, sfr, christophe.guerard,
	pranav.k.sanghadia

On 07/07/11 14:12, james_p_freyensee@linux.intel.com wrote:
> From: J Freyensee <james_p_freyensee@linux.intel.com>
> 
> Intel-Atom implementation of PTI will not work with PCI
> configured off, so the pti driver should not be compiled
> in the event someone does not enable PCI in the linux kernel.
> 
> Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
> Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

Yes, that also.

Thanks.

> Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>
> ---
>  drivers/misc/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 4e349cd..7fd32a6 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -146,6 +146,7 @@ config PHANTOM
>  
>  config INTEL_MID_PTI
>  	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
> +	depends on PCI
>  	default n
>  	help
>  	  The PTI (Parallel Trace Interface) driver directs


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

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

* [PATCH] Add PCI config dependency for PTI.
@ 2011-07-07 22:02 james_p_freyensee
  2011-07-07 22:09 ` Randy Dunlap
  0 siblings, 1 reply; 8+ messages in thread
From: james_p_freyensee @ 2011-07-07 22:02 UTC (permalink / raw)
  To: linux-next
  Cc: linux-kernel, randy.dunlap, sfr, christophe.guerard,
	pranav.k.sanghadia, james_p_freyensee

From: J Freyensee <james_p_freyensee@linux.intel.com>

Intel-Atom implementation of PTI will not work with PCI
configured off, so the pti driver should not be compiled
in the event someone does not enable PCI in the linux kernel.

Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>
---
 drivers/misc/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
index 4e349cd..7fd32a6 100644
--- a/drivers/misc/Kconfig
+++ b/drivers/misc/Kconfig
@@ -146,6 +146,7 @@ config PHANTOM
 
 config INTEL_MID_PTI
 	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
+	depends on PCI
 	default n
 	help
 	  The PTI (Parallel Trace Interface) driver directs
-- 
1.7.0.4

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

* Re: [PATCH] Add PCI config dependency for PTI.
  2011-07-07 22:02 [PATCH] Add PCI config dependency for PTI james_p_freyensee
@ 2011-07-07 22:09 ` Randy Dunlap
  0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2011-07-07 22:09 UTC (permalink / raw)
  To: james_p_freyensee
  Cc: linux-next, linux-kernel, sfr, christophe.guerard,
	pranav.k.sanghadia

On 07/07/11 15:02, james_p_freyensee@linux.intel.com wrote:
> From: J Freyensee <james_p_freyensee@linux.intel.com>
> 
> Intel-Atom implementation of PTI will not work with PCI
> configured off, so the pti driver should not be compiled
> in the event someone does not enable PCI in the linux kernel.
> 
> Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
> Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
> Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com>

That's good.  Thanks.
Sorry this was so much trouble.

> ---
>  drivers/misc/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 4e349cd..7fd32a6 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -146,6 +146,7 @@ config PHANTOM
>  
>  config INTEL_MID_PTI
>  	tristate "Parallel Trace Interface for MIPI P1149.7 cJTAG standard"
> +	depends on PCI
>  	default n
>  	help
>  	  The PTI (Parallel Trace Interface) driver directs


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

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

end of thread, other threads:[~2011-07-07 22:09 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-07 22:02 [PATCH] Add PCI config dependency for PTI james_p_freyensee
2011-07-07 22:09 ` Randy Dunlap
  -- strict thread matches above, loose matches on Subject: below --
2011-07-07 21:12 james_p_freyensee
2011-07-07 21:17 ` Randy Dunlap
2011-07-07 19:24 james_p_freyensee
2011-07-07 19:31 ` Randy Dunlap
2011-07-07 19:59   ` J Freyensee
2011-07-07 20:59     ` Randy Dunlap

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).