From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755885AbcCBTHr (ORCPT ); Wed, 2 Mar 2016 14:07:47 -0500 Received: from mout.kundenserver.de ([212.227.126.130]:63265 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752460AbcCBTHq (ORCPT ); Wed, 2 Mar 2016 14:07:46 -0500 From: Arnd Bergmann To: Karsten Keil Cc: linux-arm-kernel@lists.infradead.org, Arnd Bergmann , "David S. Miller" , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/2] isdn: icn: remove a #warning Date: Wed, 2 Mar 2016 20:06:45 +0100 Message-Id: <1456945629-1793533-1-git-send-email-arnd@arndb.de> X-Mailer: git-send-email 2.7.0 X-Provags-ID: V03:K0:FEFMFlIU1XQBon9mmzdJznu0uMmCTkpJn5uMTkzkDaYGNuHKvL6 mBWU3D9RkOsANTRVw8ZHFF+yqKpJy07qvsDpGy2z581M4ns8D65opETfEJjPUIO0db8wn8d iuwYbwPmfkryN2/Om79G7YEHHCDC58lgHjmN/HhcTR0vzRkuV4DEiVnt3sspNUVUPhWKenV e8qpKG+x2k0XuMiIqBkoA== X-UI-Out-Filterresults: notjunk:1;V01:K0:4c8lTejD4cU=:BPlPfJEprMwNy7iWf1mbR+ iyTt/U+PG6o6r28hvIvL2bAMhm/LSnIpNF7GskBb5/8j7ms00Cb+bt/8ZRE0miZkpEm3qbFNb IK6lYsEUn04MRg1AcY1lDfJLfZA9sGsutLJ8rkPtNkUlu75CMg6oZFqK1YG5wZcFIgOCEC6cZ GPKUd6vjtEH7sroZs5t6Fs/NuW1snDf2dsOFJjmqSy1RSfYqTaWCpydTOBqXLReZ8+ZJr+yOt htvcsqE9o2k32xoklO4rXi/ilfR6et9yKS/EYGH5bCbxIcT+zT+zOgiorrouK/eh5sOjBnz7x c5JPsgHZ7v5N1pdmrT38CeCl4jOjPofZWJHcIIuFdIjEyP13Q3bketXv2Bo2N1c6pIUdzM5PW 7TbZZUpufqt90Qs35SEotsjLWdMYuZKZMOVMXGVKmVFBoilyVSeXRgLPJ6FpMXIqRnlM5bt45 XlcF3PlPkSxlNqWODooBgNCdsanQ4YF1ELP0kJA4ccYWZJtsuOP2Mwbw4Mbr4CpQ5Z+InmEKg KN/uhSbYXsRGU85sCeqX5aUUJ1iRNaftp/ot+k0+VA9D5klnTXYPd2U9xr6FUTSJ5JjeyLfM+ v5m1ruHY1jOVel7dmSWqGFiDNnzkrLdsGs3GOyQ6Ks/YG3pvGmq8ZzTANtr0ZFECldaEzpNlE 3qGHssSoSYGrnbIGbjEdMwIKQ/ae2VjgLqMAHV2W2w7nL+Q0gnGYua92euWHAlzU3QKs= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The icn driver currently produces an unconditional #warning whenever we build it, introduced by Karsten Keil back in 2003: #warning TODO test headroom or use skb->nb to flag ACK Karsten's original commit (from BitKeeper) contains this description: - here are lot of bugs left, so ISDN is not stable yet but I think it's really time to fix it, even if it need some cycles to get it right (normally I'm only send patches if it works 100% for me). - I add some additional #warnings to address places which need fixing (I hope that some of the other ISDN developer jump in) Apparently this has not happened, and it is unlikely that it ever will, given that the driver doesn't seem to work. No substantial bug fixes other than janitorial cleanups have happened in the driver since then, and I see no indication that anyone who patched it had the hardware. We should probably either remove the driver, or remove all of i4l, but for now, this shuts up the distracting #warning by turning it into a comment. Signed-off-by: Arnd Bergmann Link: http://git.meleeweb.net/linux.git/commit/?id=b0deac0886b0056765afd149e9834373b38e096b --- drivers/isdn/icn/icn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/isdn/icn/icn.c b/drivers/isdn/icn/icn.c index 358a574d9e8b..46d957c34be1 100644 --- a/drivers/isdn/icn/icn.c +++ b/drivers/isdn/icn/icn.c @@ -718,7 +718,7 @@ icn_sendbuf(int channel, int ack, struct sk_buff *skb, icn_card *card) return 0; if (card->sndcount[channel] > ICN_MAX_SQUEUE) return 0; -#warning TODO test headroom or use skb->nb to flag ACK + /* TODO test headroom or use skb->nb to flag ACK */ nskb = skb_clone(skb, GFP_ATOMIC); if (nskb) { /* Push ACK flag as one -- 2.7.0