On 8/31/23 07:04, Guenter Roeck wrote: > On Fri, Jul 14, 2023 at 11:46:54AM -0600, Rob Herring wrote: >> The DT of_device.h and of_platform.h date back to the separate >> of_platform_bus_type before it as merged into the regular platform bus. >> As part of that merge prepping Arm DT support 13 years ago, they >> "temporarily" include each other. They also include platform_device.h >> and of.h. As a result, there's a pretty much random mix of those include >> files used throughout the tree. In order to detangle these headers and >> replace the implicit includes with struct declarations, users need to >> explicitly include the correct includes. >> >> Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > > This patch results in the following build error. > > Building powerpc:ppc32_allmodconfig ... failed > -------------- > Error log: > drivers/macintosh/ams/ams-input.c: In function 'ams_input_enable': > drivers/macintosh/ams/ams-input.c:68:45: error: invalid use of undefined type 'struct platform_device' > 68 | input->dev.parent = &ams_info.of_dev->dev; > | ^~ > drivers/macintosh/ams/ams-input.c: In function 'ams_input_init': > drivers/macintosh/ams/ams-input.c:146:51: error: invalid use of undefined type 'struct platform_device' > 146 | return device_create_file(&ams_info.of_dev->dev, &dev_attr_joystick); > | ^~ > drivers/macintosh/ams/ams-input.c: In function 'ams_input_exit': > drivers/macintosh/ams/ams-input.c:151:44: error: invalid use of undefined type 'struct platform_device' > 151 | device_remove_file(&ams_info.of_dev->dev, &dev_attr_joystick); > | ^~ > drivers/macintosh/ams/ams-input.c: In function 'ams_input_init': > drivers/macintosh/ams/ams-input.c:147:1: error: control reaches end of non-void function > > Bisect log attached. Hi Guenter, I posted a patch for this 2 days ago and Michael Ellerman just did a pull request to Linus with the fix. -- ~Randy
