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);
+}
+




--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to