> @@ -123,8 +123,10 @@  static int __init meson_gx_socinfo_init(
>               return -ENODEV;
>
>       /* check if interface is enabled */
> -     if (!of_device_is_available(np))
> +     if (!of_device_is_available(np)) {
> +             of_node_put(np);
>               return -ENODEV;
> +     }
>
>       /* check if chip-id is available */
>       if (!of_property_read_bool(np, "amlogic,has-chip-id"))

How do you think about to adjust the exception handling in this function
implementation a bit more according to the Linux coding style?

Regards,
Markus

Reply via email to