Messages in this thread | | | From | Jian-Hong Pan <> | Date | Thu, 17 Jan 2019 00:35:13 +0800 | Subject | Re: [RFC PATCH 5/5] net: lorawan: Split skb definitions into another header |
| |
Andreas Färber <afaerber@suse.de> 於 2019年1月16日 週三 下午10:50寫道: > > Am 16.01.19 um 15:36 schrieb Jiri Pirko: > > Wed, Jan 16, 2019 at 03:25:02PM CET, starnight@g.ncu.edu.tw wrote: > >> Split LoRaWAN related skb definitions from lora/lorawan_netdev.h into > >> another header lora/lorawan_skb.h. > > > > What is the motivation for this change? > > I suggested it because skbs could be used with either LoRa or LoRaWAN > netdevs. I have a lora/skb.h. > > In general the lorawan_foo.h looks ugly to me, so I thought I suggested > to avoid that by using [lora/]lorawan/skb.h?
Seems we have some choices: 1. Split the lorawan skb stuff from lora/lorawan_netdev.h and merge into include/linux/lora/skb.h 2. Split the lorawan skb stuff from lora/lorawan_netdev.h to include/linux/lora/lorawan_skb.h 3. Split the lorawan skb stuff from lora/lorawan_netdev.h to include/linux/lora/lorawan/skb.h 4. Split the lorawan skb stuff from lora/lorawan_netdev.h to include/linux/lorawan/skb.h
#1, #2 and #3 are good to me. So, the intersection is choice #3.
Regards, Jian-Hong Pan
| |