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

[Roger Gammans]

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.

-- 
Roger Gammans