Changeset 33906


Ignore:
Timestamp:
2012-10-24T15:05:04+02:00 (5 years ago)
Author:
florian
Message:

[kernel] swconfig: get rid of the NLA_PUT_* macros

Upstream kernel 3.6 has removed this usage, use the plain functions directly as
intended.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/target/linux/generic/files/drivers/net/phy/swconfig.c

    r30952 r33906  
    382382                return -1; 
    383383 
    384         NLA_PUT_U32(msg, SWITCH_ATTR_OP_ID, id); 
    385         NLA_PUT_U32(msg, SWITCH_ATTR_OP_TYPE, op->type); 
    386         NLA_PUT_STRING(msg, SWITCH_ATTR_OP_NAME, op->name); 
     384        if (nla_put_u32(msg, SWITCH_ATTR_OP_ID, id)) 
     385                goto nla_put_failure; 
     386        if (nla_put_u32(msg, SWITCH_ATTR_OP_TYPE, op->type)) 
     387                goto nla_put_failure; 
     388        if (nla_put_string(msg, SWITCH_ATTR_OP_NAME, op->name)) 
     389                goto nla_put_failure; 
    387390        if (op->description) 
    388                 NLA_PUT_STRING(msg, SWITCH_ATTR_OP_DESCRIPTION, 
    389                         op->description); 
     391                if (nla_put_string(msg, SWITCH_ATTR_OP_DESCRIPTION, 
     392                        op->description)) 
     393                        goto nla_put_failure; 
    390394 
    391395        return genlmsg_end(msg, hdr); 
     
    712716                goto error; 
    713717 
    714         NLA_PUT_U32(cb->msg, SWITCH_PORT_ID, port->id); 
    715         if (port->flags & (1 << SWITCH_PORT_FLAG_TAGGED)) 
    716                 NLA_PUT_FLAG(cb->msg, SWITCH_PORT_FLAG_TAGGED); 
     718        if (nla_put_u32(cb->msg, SWITCH_PORT_ID, port->id)) 
     719                goto nla_put_failure; 
     720        if (port->flags & (1 << SWITCH_PORT_FLAG_TAGGED)) { 
     721                if (nla_put_flag(cb->msg, SWITCH_PORT_FLAG_TAGGED)) 
     722                        goto nla_put_failure; 
     723        } 
    717724 
    718725        nla_nest_end(cb->msg, p); 
     
    799806        switch(attr->type) { 
    800807        case SWITCH_TYPE_INT: 
    801                 NLA_PUT_U32(msg, SWITCH_ATTR_OP_VALUE_INT, val.value.i); 
     808                if (nla_put_u32(msg, SWITCH_ATTR_OP_VALUE_INT, val.value.i)) 
     809                        goto nla_put_failure; 
    802810                break; 
    803811        case SWITCH_TYPE_STRING: 
    804                 NLA_PUT_STRING(msg, SWITCH_ATTR_OP_VALUE_STR, val.value.s); 
     812                if (nla_put_string(msg, SWITCH_ATTR_OP_VALUE_STR, val.value.s)) 
     813                        goto nla_put_failure; 
    805814                break; 
    806815        case SWITCH_TYPE_PORTS: 
     
    843852                return -1; 
    844853 
    845         NLA_PUT_U32(msg, SWITCH_ATTR_ID, dev->id); 
    846         NLA_PUT_STRING(msg, SWITCH_ATTR_DEV_NAME, dev->devname); 
    847         NLA_PUT_STRING(msg, SWITCH_ATTR_ALIAS, dev->alias); 
    848         NLA_PUT_STRING(msg, SWITCH_ATTR_NAME, dev->name); 
    849         NLA_PUT_U32(msg, SWITCH_ATTR_VLANS, dev->vlans); 
    850         NLA_PUT_U32(msg, SWITCH_ATTR_PORTS, dev->ports); 
    851         NLA_PUT_U32(msg, SWITCH_ATTR_CPU_PORT, dev->cpu_port); 
     854        if (nla_put_u32(msg, SWITCH_ATTR_ID, dev->id)) 
     855                goto nla_put_failure; 
     856        if (nla_put_string(msg, SWITCH_ATTR_DEV_NAME, dev->devname)) 
     857                goto nla_put_failure; 
     858        if (nla_put_string(msg, SWITCH_ATTR_ALIAS, dev->alias)) 
     859                goto nla_put_failure; 
     860        if (nla_put_string(msg, SWITCH_ATTR_NAME, dev->name)) 
     861                goto nla_put_failure; 
     862        if (nla_put_u32(msg, SWITCH_ATTR_VLANS, dev->vlans)) 
     863                goto nla_put_failure; 
     864        if (nla_put_u32(msg, SWITCH_ATTR_PORTS, dev->ports)) 
     865                goto nla_put_failure; 
     866        if (nla_put_u32(msg, SWITCH_ATTR_CPU_PORT, dev->cpu_port)) 
     867                goto nla_put_failure; 
    852868 
    853869        return genlmsg_end(msg, hdr); 
Note: See TracChangeset for help on using the changeset viewer.