Re: [PATCH v2 2/2] as3645a: Switch to fwnode property API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jacek,

On Wed, Feb 27, 2019 at 08:19:14PM +0100, Jacek Anaszewski wrote:
> > @@ -568,15 +573,21 @@ static int as3645a_parse_node(struct as3645a *flash,
> >   		goto out_err;
> >   	}
> > -	rval = of_property_read_string(flash->indicator_node, "label", &name);
> > -	if (!rval)
> > +	rval = fwnode_property_read_string(flash->indicator_node, "label",
> > +					   &name);
> > +	if (!rval) {
> >   		strlcpy(names->indicator, name, sizeof(names->indicator));
> > -	else
> > +	} else if (is_of_node(fwnode)) {
> >   		snprintf(names->indicator, sizeof(names->indicator),
> > -			 "%pOFn:indicator", node);
> > +			 "%pOFn:indicator", to_of_node(fwnode));
> > +	} else {
> > +		dev_err(&flash->client->dev, "flash node has no label!\n");
> > +		return -EINVAL;
> > +	}
> 
> s/flash node/indicator node/
> 
> If you agree, I can amend it myself.

Yes, please do. Thanks!!

-- 
Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux