On Wed 08 May 2019 at 17:53, Corentin Labbe <[email protected]> wrote: >> -/* >> - * FIXME: need to provide subdriver defaults or separate dt parsing from >> - * allocation. >> - */ >> static int gnss_serial_parse_dt(struct serdev_device *serdev) >> { >> struct gnss_serial *gserial = serdev_device_get_drvdata(serdev); >> struct device_node *node = serdev->dev.of_node; >> - u32 speed = 4800; >> + uint speed; >> >> - of_property_read_u32(node, "current-speed", &speed); >> + of_property_read_u32(node, "default-speed", &speed); > > Hello > > of_property_read_u32 use u32, so no reason to use uint instead. > > Regards
Hello Corentin, Thank you for your review. A v2 has been sent: https://lore.kernel.org/lkml/[email protected]/ Cheers, Loys

