Hello, On Tue, Dec 03, 2024 at 08:33:22AM +0800, Zijun Hu wrote: > This patch series is to constify the following API: > struct device *device_find_child(struct device *dev, void *data, > int (*match)(struct device *dev, void *data)); > To : > struct device *device_find_child(struct device *dev, const void *data, > device_match_t match); > typedef int (*device_match_t)(struct device *dev, const void *data); This series isn't bisectible. With only the first two patches applied I hit: CC drivers/pwm/core.o drivers/pwm/core.c: In function ‘pwm_unexport_child’: drivers/pwm/core.c:1292:55: error: passing argument 3 of ‘device_find_child’ from incompatible pointer type [-Wincompatible-pointer-types] 1292 | pwm_dev = device_find_child(pwmchip_dev, pwm, pwm_unexport_match); | ^~~~~~~~~~~~~~~~~~ | | | int (*)(struct device *, void *) In file included from include/linux/acpi.h:14, from drivers/pwm/core.c:11: include/linux/device.h:1085:49: note: expected ‘device_match_t’ {aka ‘int (*)(struct device *, const void *)’} but argument is of type ‘int (*)(struct device *, void *)’ 1085 | device_match_t match); | ~~~~~~~~~~~~~~~^~~~~ drivers/pwm/core.c: In function ‘pwm_class_get_state’: drivers/pwm/core.c:1386:55: error: passing argument 3 of ‘device_find_child’ from incompatible pointer type [-Wincompatible-pointer-types] 1386 | pwm_dev = device_find_child(pwmchip_dev, pwm, pwm_unexport_match); | ^~~~~~~~~~~~~~~~~~ | | | int (*)(struct device *, void *) include/linux/device.h:1085:49: note: expected ‘device_match_t’ {aka ‘int (*)(struct device *, const void *)’} but argument is of type ‘int (*)(struct device *, void *)’ 1085 | device_match_t match); | ~~~~~~~~~~~~~~~^~~~~ make[5]: *** [scripts/Makefile.build:194: drivers/pwm/core.o] Error 1 make[4]: *** [scripts/Makefile.build:440: drivers/pwm] Error 2 make[3]: *** [scripts/Makefile.build:440: drivers] Error 2 make[2]: *** [Makefile:1989: .] Error 2 make[1]: *** [Makefile:372: __build_one_by_one] Error 2 make: *** [Makefile:251: __sub-make] Error 2 Best regards Uwe
Attachment:
signature.asc
Description: PGP signature