[flashrom] [SPAM?] Re: [PATCH 08/12] simplify ich_set_bbar
Stefan Tauner
stefan.tauner at student.tuwien.ac.at
Mon Jun 13 23:23:33 CEST 2011
On Mon, 13 Jun 2011 23:12:45 +0200
Stefan Tauner <stefan.tauner at student.tuwien.ac.at> wrote:
> On Mon, 13 Jun 2011 22:09:35 +0200
> Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> wrote:
>
> > Since you're already changing the function prototype of ich_set_bbar,
> > can you make it static?
> >
> > And another point about the coding style:
> > Can you use int bbar_offs instead of void *bbar_addr?
> > That way, it is easier to check directly which physmap region the
> > accesses correspond to.
>
> like so?
and:
- ichspi_bbar = mmio_readl(ich_spibar + bbar_off);
+ ichspi_bbar = mmio_readl(ich_spibar + bbar_off) & BBAR_MASK;
at the end else we would use the reserved bits too
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
More information about the flashrom
mailing list