diff options
| author | Erick Archer <[email protected]> | 2024-06-02 19:23:54 +0200 |
|---|---|---|
| committer | Jonathan Cameron <[email protected]> | 2024-06-13 19:19:23 +0100 |
| commit | f91f9ece67972885fd41d06dfd034dcce057ef20 (patch) | |
| tree | d7fd7ce94bb2f3e46cbe54f32bbedf6827847700 | |
| parent | f451fbd73b0d54ef53bc2c80e4d969df1c7c7d96 (diff) | |
iio: event: use sizeof(*pointer) instead of sizeof(type)
It is preferred to use sizeof(*pointer) instead of sizeof(type)
due to the type of the variable can change and one needs not
change the former (unlike the latter). At the same time refactor
the NULL comparison.
This patch has no effect on runtime behavior.
Signed-off-by: Erick Archer <[email protected]>
Link: https://lore.kernel.org/r/AS8PR02MB7237D024459C314CECE72EAF8BFE2@AS8PR02MB7237.eurprd02.prod.outlook.com
Signed-off-by: Jonathan Cameron <[email protected]>
| -rw-r--r-- | drivers/iio/industrialio-event.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/iio/industrialio-event.c b/drivers/iio/industrialio-event.c index a64f8fbac597..db06501b0e61 100644 --- a/drivers/iio/industrialio-event.c +++ b/drivers/iio/industrialio-event.c @@ -581,8 +581,8 @@ int iio_device_register_eventset(struct iio_dev *indio_dev) iio_check_for_dynamic_events(indio_dev))) return 0; - ev_int = kzalloc(sizeof(struct iio_event_interface), GFP_KERNEL); - if (ev_int == NULL) + ev_int = kzalloc(sizeof(*ev_int), GFP_KERNEL); + if (!ev_int) return -ENOMEM; iio_dev_opaque->event_interface = ev_int; |