https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=238037

Austin Shafer <asha...@badland.io> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #204527|0                           |1
        is obsolete|                            |

--- Comment #5 from Austin Shafer <asha...@badland.io> ---
Created attachment 205447
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=205447&action=edit
ig4 suspend/resume patch

Thanks for the feedback! I'm glad I could help.

You guys are definitely right, I updated my patch with the write barrier
changes, and made sure to use IG4_REG_RESETS_SKL for skylake.

As for if the barriers are needed, I never really tested but assumed it was
required because the `reg_write` wrapper in ig4_iic.c uses it. I added the
barriers just to be extra sure that the reset went through to avoid any
nastiness on suspend or resume.

Please let me know if you have any other advice for things I should change.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

Reply via email to