On Sun, May 12, 2019 at 11:59 PM Anson Huang <[email protected]> wrote:
> + soc_dev = soc_device_register(soc_dev_attr);
> + if (IS_ERR(soc_dev)) {
> + kfree(soc_dev_attr->revision);
> + return -ENODEV;
This should return PTR_ERR(soc_dev) instead.

