On Sun, Apr 18, 2021 at 9:12 AM G S Niteesh Babu <niteesh.gs at 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 at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel