Hi Wolfram, On Sat, 24 Mar 2018 23:35:18 +0100 Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: > Hi Boris, > > > - rebase on v4.15-rc1 > > This code has changed a little meanwhile. Please check my for-next > branch. Some changes are identical, some similar. Actually it was rebased on top of v4.16-rc1, but I'll check if this patch applies correctly on top of your for-next branch. > > > - info.archdata = &dev_ad; > > Why did you drop this? Well, this line could have been moved in of_i2c_register_device(), but I realized dev_ad was zero initialized without any further modification, and leaving info->archdata to NULL has the exact same effect (see [1]). Regards, Boris [1]https://elixir.bootlin.com/linux/v4.16-rc6/source/drivers/i2c/i2c-core-base.c#L711 -- Boris Brezillon, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html