[coreboot] patch: remove static variable from cs5536 smbus function

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Wed Sep 17 03:22:05 CEST 2008


On 17.09.2008 02:40, ron minnich wrote:
> You might like this one better.
>
> Just a tiny change, not code dup, but the opportunity is open for more
> complex cases.
>
> This global variable infrastructure is really nice.
>   

Cool thing. You're using the infrastructure in novel ways and that makes
me happy.

The assignment is not MP safe, but I seriously doubt someone will use
the CS5536 in a MP environment. Besides that, the global variable
infrastructure is not MP safe right now, so I'll have to audit all users
anyway once I make the infrastructure MP safe.
Read the sentence above as "no objection, code is fine". ;-)

Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list