Re: [PATCH 0/7] platform/surface: aggregator: Extend user-space interface for events

From: Maximilian Luz
Date: Fri Jun 04 2021 - 08:58:23 EST


On 6/4/21 1:40 PM, Hans de Goede wrote:
Hi Maxime,

On 6/4/21 1:45 AM, Maximilian Luz wrote:
Extend the user-space debug interface so that it can be used to receive
SSAM events in user-space.


[...]


Overall this series looks good to me. I've found one small issue with
PATCH 4/7 (see my reply to that patch) and as the kernel test robot
pointed out there is an used "struct ssam_nf_head *nf_head;" in
PATCH 2/7.

With these 2 small issues fixed you can add my:

Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>

to v2 of the series.

Thank you for your review.

Based on the issue reported by the kernel test robot I've been
restructuring PATCH 2/7 to remove some code-duplication. I'll add your
R-b to all except that one.

Thanks,
Max