https://bugs.freedesktop.org/show_bug.cgi?id=86539

--- Comment #11 from Hans de Goede <[email protected]> ---
Hi Pierre,

Thanks for working on this. I'm not sure what the nbs suffix stands for, maybe
rename it to
bl_interface_nr ?

Chances are that nv50_backlight_init() may run twice at the same time if 2
cards are being probed simultaneously. So I would make bl_interface_nr an
atomic_t and use:

int nr = atomic_inc_return(&bl_interface_nr) - 1;

to increment it and get the _NEW_ value. Since this gives you the new value,
you should substract 1 as is done above.

Regards,

Hans

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to