From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753579AbcD0ROw (ORCPT ); Wed, 27 Apr 2016 13:14:52 -0400 Received: from mail-pf0-f177.google.com ([209.85.192.177]:34836 "EHLO mail-pf0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753018AbcD0ROt (ORCPT ); Wed, 27 Apr 2016 13:14:49 -0400 Date: Wed, 27 Apr 2016 10:14:45 -0700 From: Bjorn Andersson To: David Miller Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-arm-msm@vger.kernel.org, courtney.cavin@sonymobile.com, bjorn.andersson@sonymobile.com Subject: Re: [PATCH v2] net: Add Qualcomm IPC router Message-ID: <20160427171445.GO3202@tuxbot> References: <1461736085-24862-1-git-send-email-bjorn.andersson@linaro.org> <20160427.122242.1614940676503935894.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160427.122242.1614940676503935894.davem@davemloft.net> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed 27 Apr 09:22 PDT 2016, David Miller wrote: > From: Bjorn Andersson > Date: Tue, 26 Apr 2016 22:48:05 -0700 > > > + rc = qcom_smd_send(qdev->channel, skb->data, skb->len); > > I truly dislike adding networking protocols that depend upon some > piece of infrastructure that only some platforms can enable, it's even > worse when that set of platforms doesn't intersect with x86-64. > > When you do things like this, it's quite hard to make protocol wide > changes to APIs because build testing becomes an issue. > That's a very valid concern. > This code can now only be build tested on ARCH_QCOM architectures, and > that's a serious negative downside. For normal usage the QRTR_SMD doesn't make much sense to be selectable unless QCOM_SMD is compiled in, but I can fix up the QCOM_SMD exports and slap a COMPILE_TEST on it. Looking at it again, we already have the conditional for QRTR and the OF code in the driver went away a while back, so we're down to something like: depends on QCOM_SMD || COMPILE_TEST Regards, Bjorn