All of lore.kernel.org
 help / color / mirror / Atom feed
* ARC:  ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
@ 2014-06-04  9:33 Paul Bolle
  2014-06-09  9:05 ` Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Paul Bolle @ 2014-06-04  9:33 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: linux-kernel

Hi Vineet,

Linux-next included commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of
legacy BVCI latency unit support") in next-20140604. It removes the
Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that symbol's entry is the
only place were the symbols ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 are
currently used. So it follows that the entries for ARC_BOARD_ANGEL4 and
ARC_BOARD_ML509 could be removed too.

If that's correct: is the trivial patch to remove the Kconfig entries
for ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 queued somewhere?


Paul Bolle


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

* Re: ARC:  ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-06-04  9:33 ARC: ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 Paul Bolle
@ 2014-06-09  9:05 ` Vineet Gupta
  2014-09-12  9:00   ` Paul Bolle
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2014-06-09  9:05 UTC (permalink / raw)
  To: Paul Bolle; +Cc: linux-kernel

Hi Paul,

On Wednesday 04 June 2014 03:04 PM, Paul Bolle wrote:
> Hi Vineet,
>
> Linux-next included commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of
> legacy BVCI latency unit support") in next-20140604. It removes the
> Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that symbol's entry is the
> only place were the symbols ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 are
> currently used. So it follows that the entries for ARC_BOARD_ANGEL4 and
> ARC_BOARD_ML509 could be removed too.

Thx for noticing this.

I'm planning to use these symbols to set up default builtin DT blobs from Kconfig
itself (rather than current defconfigs). If that doesn't workout - sure I will
remove the symbols.

Thx,
-Vineet

>
> If that's correct: is the trivial patch to remove the Kconfig entries
> for ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 queued somewhere?
>
>
> Paul Bolle
>
>


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

* Re: ARC:  ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-06-09  9:05 ` Vineet Gupta
@ 2014-09-12  9:00   ` Paul Bolle
  2014-09-22  5:16     ` Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Paul Bolle @ 2014-09-12  9:00 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: linux-kernel

Hi Vineet,

On Mon, 2014-06-09 at 09:05 +0000, Vineet Gupta wrote:
> On Wednesday 04 June 2014 03:04 PM, Paul Bolle wrote:
> > Linux-next included commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of
> > legacy BVCI latency unit support") in next-20140604. It removes the
> > Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that symbol's entry is the
> > only place were the symbols ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 are
> > currently used. So it follows that the entries for ARC_BOARD_ANGEL4 and
> > ARC_BOARD_ML509 could be removed too.
> 
> Thx for noticing this.
> 
> I'm planning to use these symbols to set up default builtin DT blobs from Kconfig
> itself (rather than current defconfigs). If that doesn't workout - sure I will
> remove the symbols.

No news here as of v3.17-rc4 and next-20140912. Are you still planning
to use these symbols or should I submit the trivial patch to drop them?

Thanks,


Paul Bolle


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

* Re: ARC:  ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-09-12  9:00   ` Paul Bolle
@ 2014-09-22  5:16     ` Vineet Gupta
  2014-09-23  9:39       ` [PATCH] ARC: [arcfpga] Get rid of " Paul Bolle
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2014-09-22  5:16 UTC (permalink / raw)
  To: Paul Bolle; +Cc: linux-kernel

On Friday 12 September 2014 02:30 PM, Paul Bolle wrote:
> Hi Vineet,
>
> On Mon, 2014-06-09 at 09:05 +0000, Vineet Gupta wrote:
>> On Wednesday 04 June 2014 03:04 PM, Paul Bolle wrote:
>>> Linux-next included commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of
>>> legacy BVCI latency unit support") in next-20140604. It removes the
>>> Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that symbol's entry is the
>>> only place were the symbols ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 are
>>> currently used. So it follows that the entries for ARC_BOARD_ANGEL4 and
>>> ARC_BOARD_ML509 could be removed too.
>> Thx for noticing this.
>>
>> I'm planning to use these symbols to set up default builtin DT blobs from Kconfig
>> itself (rather than current defconfigs). If that doesn't workout - sure I will
>> remove the symbols.
> No news here as of v3.17-rc4 and next-20140912. Are you still planning
> to use these symbols or should I submit the trivial patch to drop them?

Please send over patch - I'll queue it up for 3.18

Thx,
-Vineet

>
> Thanks,
>
>
> Paul Bolle
>
>


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

* [PATCH] ARC: [arcfpga] Get rid of ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-09-22  5:16     ` Vineet Gupta
@ 2014-09-23  9:39       ` Paul Bolle
  2014-09-25  7:22         ` Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Paul Bolle @ 2014-09-23  9:39 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: linux-kernel

Commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of legacy BVCI latency unit
support") removed the Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that
symbol's entry was the only place were the symbols ARC_BOARD_ANGEL4 and
ARC_BOARD_ML509 were used. So ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 can
be removed too.

Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
---
Done on top of next-20140923. Tested by grepping the tree only.

There are still references in defconfig files. Should those be removed
too?

 arch/arc/plat-arcfpga/Kconfig | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/arch/arc/plat-arcfpga/Kconfig b/arch/arc/plat-arcfpga/Kconfig
index 4965f9f4ffdc..217593a70751 100644
--- a/arch/arc/plat-arcfpga/Kconfig
+++ b/arch/arc/plat-arcfpga/Kconfig
@@ -18,17 +18,6 @@ menuconfig ARC_PLAT_FPGA_LEGACY
 
 if ARC_PLAT_FPGA_LEGACY
 
-config ARC_BOARD_ANGEL4
-	bool "ARC Angel4"
-	default y
-	help
-	  ARC Angel4 FPGA Ref Platform (Xilinx Virtex Based)
-
-config ARC_BOARD_ML509
-	bool "ML509"
-	help
-	  ARC ML509 FPGA Ref Platform (Xilinx Virtex-5 Based)
-
 config ISS_SMP_EXTN
 	bool "ARC SMP Extensions (ISS Models only)"
 	default n
-- 
1.9.3


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

* Re: [PATCH] ARC: [arcfpga] Get rid of ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-09-23  9:39       ` [PATCH] ARC: [arcfpga] Get rid of " Paul Bolle
@ 2014-09-25  7:22         ` Vineet Gupta
  2014-09-25  7:58           ` Vineet Gupta
  0 siblings, 1 reply; 7+ messages in thread
From: Vineet Gupta @ 2014-09-25  7:22 UTC (permalink / raw)
  To: Paul Bolle; +Cc: linux-kernel

On Tuesday 23 September 2014 03:09 PM, Paul Bolle wrote:

Commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of legacy BVCI latency unit
support") removed the Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that
symbol's entry was the only place were the symbols ARC_BOARD_ANGEL4 and
ARC_BOARD_ML509 were used. So ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 can
be removed too.

Signed-off-by: Paul Bolle <pebolle@tiscali.nl><mailto:pebolle@tiscali.nl>
---
Done on top of next-20140923. Tested by grepping the tree only.

Applied to ARC for-next.

Thx Paul,
-Vineet

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

* Re: [PATCH] ARC: [arcfpga] Get rid of ARC_BOARD_ANGEL4 and ARC_BOARD_ML509
  2014-09-25  7:22         ` Vineet Gupta
@ 2014-09-25  7:58           ` Vineet Gupta
  0 siblings, 0 replies; 7+ messages in thread
From: Vineet Gupta @ 2014-09-25  7:58 UTC (permalink / raw)
  To: Paul Bolle; +Cc: linux-kernel

On Thursday 25 September 2014 12:52 PM, Vineet Gupta wrote:
On Tuesday 23 September 2014 03:09 PM, Paul Bolle wrote:

Commit c00bfd974fb0 ("ARC: [arcfpga] Get rid of legacy BVCI latency unit
support") removed the Kconfig symbol ARC_HAS_BVCI_LAT_UNIT. And that
symbol's entry was the only place were the symbols ARC_BOARD_ANGEL4 and
ARC_BOARD_ML509 were used. So ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 can
be removed too.

Signed-off-by: Paul Bolle <pebolle@tiscali.nl><mailto:pebolle@tiscali.nl>
---
Done on top of next-20140923. Tested by grepping the tree only.

Applied to ARC for-next.

Thx Paul,
-Vineet

I did a grep for ARC_BOARD in arch/arc and it seems we have a few stragglers in def configs.
I've pruned them and squashed in same commit.

-Vineet

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

end of thread, other threads:[~2014-09-25  7:58 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-04  9:33 ARC: ARC_BOARD_ANGEL4 and ARC_BOARD_ML509 Paul Bolle
2014-06-09  9:05 ` Vineet Gupta
2014-09-12  9:00   ` Paul Bolle
2014-09-22  5:16     ` Vineet Gupta
2014-09-23  9:39       ` [PATCH] ARC: [arcfpga] Get rid of " Paul Bolle
2014-09-25  7:22         ` Vineet Gupta
2014-09-25  7:58           ` Vineet Gupta

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.