Messages in this thread | | | Date | Thu, 26 Nov 2015 08:19:26 +0000 | From | Salil Mehta <> | Subject | Re: [PATCH V3 net-next 3/5] net:hns: Add Hip06 "TSO(TCP Segment Offload)" support HNS Driver |
| |
On 11/22/2015 8:49 AM, Yuval Mintz wrote: >> +static void hns_ae_set_tso_stats(struct hnae_handle *handle, int >> +enable) { >> + struct hns_ppe_cb *ppe_cb = hns_get_ppe_cb(handle); >> + >> + hns_ppe_set_tso_enable(ppe_cb, enable); } > Style issues? I could not see the code style issue menioned above in earlier submitted patch. Looks like I am misssing something here!
+static void hns_ae_set_tso_stats(struct hnae_handle *handle, int enable) +{ + struct hns_ppe_cb *ppe_cb = hns_get_ppe_cb(handle); + + hns_ppe_set_tso_enable(ppe_cb, enable); +} +
> >> +void hns_ppe_set_tso_enable(struct hns_ppe_cb *ppe_cb, u32 value) { >> + dsaf_set_dev_bit(ppe_cb, PPEV2_CFG_TSO_EN_REG, 0, !!value); } >> + > Likewise Same in above case. Hope I am not missing anything.
#include "hns_dsaf_ppe.h" +void hns_ppe_set_tso_enable(struct hns_ppe_cb *ppe_cb, u32 value) +{ + dsaf_set_dev_bit(ppe_cb, PPEV2_CFG_TSO_EN_REG, 0, !!value); +} +
| |