From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754323AbdBGOqv (ORCPT ); Tue, 7 Feb 2017 09:46:51 -0500 Received: from mail2.candelatech.com ([208.74.158.173]:60240 "EHLO mail2.candelatech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753497AbdBGOqu (ORCPT ); Tue, 7 Feb 2017 09:46:50 -0500 Message-ID: <5899DDD7.7000605@candelatech.com> Date: Tue, 07 Feb 2017 06:46:47 -0800 From: Ben Greear User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: "Valo, Kalle" , Adrian Chadd CC: "netdev@vger.kernel.org" , "linux-wireless@vger.kernel.org" , =?UTF-8?B?QW1hZGV1c3ogU8WCYXdpxYRza2k=?= , "ath10k@lists.infradead.org" , Linux Kernel Mailing List Subject: Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif() References: <1486030773-30600-1-git-send-email-amadeusz.slawinski@tieto.com> <87y3xi4b28.fsf@qca.qualcomm.com> In-Reply-To: <87y3xi4b28.fsf@qca.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/07/2017 01:14 AM, Valo, Kalle wrote: > Adrian Chadd writes: > >> Removing this method makes the diff to FreeBSD larger, as "vif" in >> FreeBSD is a different pointer. >> >> (Yes, I have ath10k on freebsd working and I'd like to find a way to >> reduce the diff moving forward.) > > I don't like this "(void *) vif->drv_priv" style that much either but > apparently it's commonly used in Linux wireless code and already parts > of ath10k. So this patch just unifies the coding style. Surely the code compiles to the same thing, so why add a patch that makes it more difficult for Adrian and makes the code no easier to read for the rest of us? Thanks, Ben -- Ben Greear Candela Technologies Inc http://www.candelatech.com