diff options
| author |   Daniel Scally <djrscally@gmail.com> | 2022-02-16 22:53:02 +0000 |
|---|---|---|
| committer |   Hans de Goede <hdegoede@redhat.com> | 2022-02-21 14:43:47 +0100 |
| commit | 4b1be2fe63b80fd3af591268f1698a495e7f29be (patch) | |
| tree | 2032acccff0609499b079fcad8c107ce1d13da83 | |
| parent | platform/dcdbas: move EXPORT_SYMBOL after function (diff) | |
| download | linux-dev-4b1be2fe63b80fd3af591268f1698a495e7f29be.tar.xz linux-dev-4b1be2fe63b80fd3af591268f1698a495e7f29be.zip | |
platform/x86: int3472: Add terminator to gpiod_lookup_table
Without the terminator, if a con_id is passed to gpio_find() that
does not exist in the lookup table the function will not stop looping
correctly, and eventually cause an oops.
Fixes: 19d8d6e36b4b ("platform/x86: int3472: Pass tps68470_regulator_platform_data to the tps68470-regulator MFD-cell")
Signed-off-by: Daniel Scally <djrscally@gmail.com>
Link: https://lore.kernel.org/r/20220216225304.53911-5-djrscally@gmail.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
| -rw-r--r-- | drivers/platform/x86/intel/int3472/tps68470_board_data.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/platform/x86/intel/int3472/tps68470_board_data.c b/drivers/platform/x86/intel/int3472/tps68470_board_data.c index f93d437fd192..525f09a3b5ff 100644 --- a/drivers/platform/x86/intel/int3472/tps68470_board_data.c +++ b/drivers/platform/x86/intel/int3472/tps68470_board_data.c @@ -100,7 +100,8 @@ static struct gpiod_lookup_table surface_go_tps68470_gpios = { .dev_id = "i2c-INT347A:00", .table = { GPIO_LOOKUP("tps68470-gpio", 9, "reset", GPIO_ACTIVE_LOW), - GPIO_LOOKUP("tps68470-gpio", 7, "powerdown", GPIO_ACTIVE_LOW) + GPIO_LOOKUP("tps68470-gpio", 7, "powerdown", GPIO_ACTIVE_LOW), + { } } }; |