On Mon, 09 Dec 2024 23:05:19 +0100, Hans de Goede wrote:
> Not all devices have an ACPI companion fwnode, so adev might be NULL. This
> can e.g. (theoretically) happen when a user manually binds one of
> the int3472 drivers to another i2c/platform device through sysfs.
>
> Add a check for adev not being set and return -ENODEV in that case to
> avoid a possible NULL pointer deref in skl_int3472_get_acpi_buffer().
>
> [...]
Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.
The list of commits applied:
[1/4] platform/x86: int3472: Check for adev == NULL
commit: cd2fd6eab480dfc247b737cf7a3d6b009c4d0f1c
[2/4] platform/x86: int3472: Make "pin number mismatch" message a debug message
commit: 1dd0cb9cabf37fbe20f0a66e4c3972cb21240aed
[3/4] platform/x86: int3472: Fix skl_int3472_handle_gpio_resources() return value
commit: 6718d42b6eb28228a554db6c8973693ad5320006
[4/4] platform/x86: int3472: Debug log the sensor name
commit: cc115abc1f9b55092c11c183ebff9ad921251609
--
i.
