On Sun, Apr 18, 2021 at 9:12 AM G S Niteesh Babu <niteesh...@gmail.com> wrote: > > Adds "rtems,path" as an additional bus path for the i2c driver. > Previously the bus path was provided in "rtems,i2c-path" property > only. > --- > rtemsbsd/sys/dev/iicbus/rtems-i2c.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/rtemsbsd/sys/dev/iicbus/rtems-i2c.c > b/rtemsbsd/sys/dev/iicbus/rtems-i2c.c > index b965f248..7ac7a7c4 100644 > --- a/rtemsbsd/sys/dev/iicbus/rtems-i2c.c > +++ b/rtemsbsd/sys/dev/iicbus/rtems-i2c.c > @@ -87,9 +87,12 @@ rtems_i2c_attach(device_t dev) > > len = OF_getprop_alloc(node, "rtems,i2c-path", &sc->path); > if (len == -1){ > - device_printf(sc->dev, "Path not found in Device Tree"); > - OF_prop_free(sc->path); > - return (ENXIO); > + len = OF_getprop_alloc(node, "rtems,path", &sc->path); > + if (len == -1) { > + device_printf(sc->dev, "Path not found in Device > Tree"); > + OF_prop_free(sc->path); If OF_get_prop_alloc failed, is sc->path valid here? Or do you free(NULL) maybe.
> + return (ENXIO); > + } > } > > if ((sc->sc_iicbus = device_add_child(sc->dev, "iicbus", -1)) == > NULL) { > -- > 2.17.1 > > _______________________________________________ > devel mailing list > devel@rtems.org > http://lists.rtems.org/mailman/listinfo/devel _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel