Re: [PATCH v4 4/4] SFH: Create HID report to Enable support of AMD sensor fusion Hub (SFH)

From: Singh, Sandeep
Date: Mon Mar 30 2020 - 00:24:10 EST


Hi Roger,

On 3/30/2020 12:10 AM, Roger Gammans wrote:
[CAUTION: External Email]

On Thu, Feb 27, 2020 at 05:10:57AM +0000, Sandeep Singh wrote:

From: Sandeep Singh <sandeep.singh@xxxxxxx>

+#include "amd_sfh_hid_report_descriptor.h"
+#include "amd_mp2_pcie.h"
I had to change the last line to

#include "../amd_mp2_pcie.h"

to make it compile out of tree. After that I got a clean compile.

Also I don't seem to be getting any data through
to monitor-sensor in user space when I do install it, but I am
gettting logs of log messages similar to this:

pcie_mp2_amd 0000:04:00.7: AMD-Vi: Event logged [IO_PAGE_FAULT domain=0x000c address=0x5267f000 flags=0x0020]
pcie_mp2_amd 0000:04:00.7: AMD-Vi: Event logged [IO_PAGE_FAULT domain=0x000c address=0x5267f000 flags=0x0020]

This is compiled against the debian 5.4.0 kernel, on a
HP ENVY x360 Convertible 15-ds0xxx, SKU 6TD07EA#ABU

I can run other tests or try other kernels if you think it might help,
let me know what you need.
Can you disable IOMMU from bios and give a try or if your laptop does not have iommu option you can pass boot parameter "iommu=soft"

--
Roger Gammans