Re: [Milkymist-devel] [PATCH] Warnings on navre

2013-01-08 Thread Florent Kermarrec
Hi, for the default changes, you're right, it's only to avoid a default missing warning with quartus. Florent ___ http://lists.milkymist.org/listinfo.cgi/devel-milkymist.org IRC: #milkymist@Freenode

Re: [Milkymist-devel] [PATCH] Warnings on navre

2013-01-08 Thread Paul
rs' list Subject: Re: [Milkymist-devel] [PATCH] Warnings on navre Hi, thanks for the patch! > -parameter IO_SEL_EXT = 2'd0; > -parameter IO_SEL_STACK = 2'd1; > -parameter IO_SEL_SREG= 2'd2; > +localparam IO_SEL_EXT= 2'd0; > +lo

Re: [Milkymist-devel] [PATCH] Warnings on navre

2013-01-08 Thread Sebastien Bourdeauducq
Hi, thanks for the patch! > -parameter IO_SEL_EXT = 2'd0; > -parameter IO_SEL_STACK = 2'd1; > -parameter IO_SEL_SREG= 2'd2; > +localparam IO_SEL_EXT= 2'd0; > +localparam IO_SEL_STACK = 2'd1; > +localparam IO_SEL_SREG = 2'd2; Applied. > case(Rd) > > -

[Milkymist-devel] [PATCH] Warnings on navre

2013-01-05 Thread Florent Kermarrec
Hello, here is a small patch for navre that fixes some warnings on Quartus (and also ISE): - "Parameter Declaration in module "softusb_navre" behaves as a Local Parameter Declaration because the module has a Module Parameter Port List" appears on each parameter - "Warning (10270): Verilog HDL Ca