[Libre-soc-dev] daily kan-ban update 06AUG2020

Cole Poirier colepoirier at gmail.com
Thu Aug 6 19:10:50 BST 2020


On Thu, Aug 6, 2020 at 11:02 AM Luke Kenneth Casson Leighton
<lkcl at lkcl.net> wrote:
>
> cole raise a bugreport about this one, we need to move the discussion there.

Done

> +        self.addr = [
> +            Signal(1, reset_less=True, name=f"reg_stage_addr{i}") for i in rang
> e(64)],
>
> is not below 80 chars - *please get into the habit of limiting the
> editor window to 80 chars at all times*

Doing that *right* now, sorry about this. Shows the perils of not
following your direction
to use only 80 char xterms.

> and should be simply "self.addr = SIgnal(64)"
>
> there is no need to create a 64-long array of 1 bit signals.

Thanks that's very helpful, I was working off of the bpermd proof and
my understanding
of the difference between Signal(64) and [Signal(1) for i in
range(64)], will move this
discussion to bug 450.



More information about the Libre-soc-dev mailing list