On Thu, 9 Sep 2021 17:25:02 -0500 Ian Pilcher <arequipeno@xxxxxxxxx> wrote: > @@ -166,6 +166,9 @@ struct gendisk { > #endif /* CONFIG_BLK_DEV_INTEGRITY */ > #if IS_ENABLED(CONFIG_CDROM) > struct cdrom_device_info *cdi; > +#endif > +#if IS_ENABLED(CONFIG_LEDS_TRIGGER_BLKDEV) > + struct ledtrig_blkdev_disk *ledtrig; > #endif So in this patch you are adding pointer to a structure, but the structure is introduced in a later patch (07/15). Please send these changes as one patch, it is easier to review. Marek