Hi Chris, On Fri, 8 May 2020 at 15:13, Emil Renner Berthing <emil.renner.berthing@xxxxxxxxx> wrote: > If you do something like this you won't need the temporary num_cs variable: > > if (of_property_read_u32(np, "num-cs", &master->num_chipselect)) > master->num_chipselect = 1; Sorry, that should be of_property_read_u16, since num_chipselect is a u16. /Emil