Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1578218

Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif()

From Joe Perches <joe@perches.com>
Newsgroups linux.kernel
Subject Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif()
Date 2017-02-10 08:40 +0100
Message-ID <t9dYd-q5-7@gated-at.bofh.it> (permalink)
References (1 earlier) <t6sh4-6ho-25@gated-at.bofh.it> <t8a6m-VT-19@gated-at.bofh.it> <t8ffH-49n-11@gated-at.bofh.it> <t9dvc-fW-39@gated-at.bofh.it> <t9dES-jp-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

On Thu, 2017-02-09 at 23:14 -0800, Adrian Chadd wrote:

> If there
> were accessors for the skb data / len fields (like we do for mbufs)
> then porting the code would've involved about 5,000 less changed
> lines.

What generic mechanisms would you suggest to make
porting easier between bsd and linux and what in
your opinion are the best naming schemes to make
these functions easiest to read and implement
without resorting to excessive identifier lengths?

If you have some, please provide examples.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Amadeusz Sławiński          <amadeusz.slawinski@tieto.com> - 2017-02-02 11:30 +0100
  [PATCH 3/3] ath10k: fix comment Amadeusz Sławiński          <amadeusz.slawinski@tieto.com> - 2017-02-02 11:30 +0100
  Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Adrian Chadd <adrian@freebsd.org> - 2017-02-02 17:20 +0100
    Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() "Valo, Kalle" <kvalo@qca.qualcomm.com> - 2017-02-07 10:20 +0100
      Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Ben Greear <greearb@candelatech.com> - 2017-02-07 15:50 +0100
        Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() "Valo, Kalle" <kvalo@qca.qualcomm.com> - 2017-02-10 08:10 +0100
          Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Adrian Chadd <adrian@freebsd.org> - 2017-02-10 08:20 +0100
            Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Joe Perches <joe@perches.com> - 2017-02-10 08:40 +0100
              Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Adrian Chadd <adrian@freebsd.org> - 2017-02-10 16:40 +0100
          Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() Ben Greear <greearb@candelatech.com> - 2017-02-10 18:40 +0100

csiph-web