Hi Jacopo, On Wed, Dec 14, 2016 at 12:54 PM, jacopo@xxxxxxxxxx <jacopo@xxxxxxxxxx> wrote: >>> +struct max11100_state { >>> + const struct max11100_chip_desc *desc; >>> + struct spi_device *spi; >>> + int vref_uv; >> >> unsi > > Was this a suggestion to turn "vref_uv" into unsigned? It was a suggestion I started writing, and aborted improperly when noticing the below ;-) > As you can see in probe function it can get assigned to -EINVAL when a dummy > regulator is returned from regulator-core. > I had to make it signed for this reason Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html