> > +static int qoriq_backplane_config_init(struct phy_device *bpphy) > > +{ > [] > > + for (i = 0; i < bp_phy->num_lanes; i++) { > [] > > + ret = of_address_to_resource(lane_node, 0, &res); > > + if (ret) { > > + bpdev_err(bpphy, > > + "could not obtain lane memory map for index=%d, ret = %d\n", > > + i, ret); > > + return ret; > > This could use the new vsprintf %pe extension: Hi Joe Probably a FAQ. But is there plans to extend vsprintf to take an int errno value, rather than having to do this ugly ERR_PTR(ret) every time? Format string %de ? Andrew