On Tue, Feb 22, 2011 at 09:56:50AM +0100, Dirk Eibach wrote:> struct pca953x_platform_data *pdata;> struct device_node *node;> - const uint16_t *val;> + const u32 *val;> This should probably be signed? We compare it against zero later on. if (*val < 0) dev_warn(&client->dev, "invalid gpio-base in device tree\n");regards,dan carpenter