From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp-out-052.synserver.de ([212.40.185.52]:1687 "EHLO smtp-out-052.synserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751469Ab3AJJyb (ORCPT ); Thu, 10 Jan 2013 04:54:31 -0500 Message-ID: <50EE8FFD.2000000@metafoo.de> Date: Thu, 10 Jan 2013 10:55:09 +0100 From: Lars-Peter Clausen MIME-Version: 1.0 To: Julia Lawall CC: Grant Likely , spi-devel-general@lists.sourceforge.net, linux-kernel@vger.kernel.org, Jonathan Cameron , linux-iio@vger.kernel.org Subject: Re: [PATCH 1/3] spi: Add helper functions for setting up transfers References: <1357752671-30222-1-git-send-email-lars@metafoo.de> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 01/10/2013 09:53 AM, Julia Lawall wrote: >> +@r1@ >> +identifier fn; >> +identifier xfers; >> +@@ >> +fn(...) >> +{ >> + ... >> +( >> + struct spi_transfer xfers[...]; >> +| >> + struct spi_transfer xfers[]; >> +) >> + ... >> +} > > Can it happen that there would be more than one spi_transfer or spi_message > variable per function? This semantic patch will only treat the case where > there is only one, because the ... before an after the variable declaration > won't match another declaration of the same form. > > julia I guess it could happen, but I would consider it to be very rare. There are a few examples of multiple transfers in the kernel. But most of them look like struct spi_message msg; struct spi_transfer xfer_foo; struct spi_transfer xfer_bar; ... spi_message_add_tail(&xfer_foo, &msg); spi_message_add_tail(&xfer_bar, &msg); So the transformation can't be applied here anyway. Do you have an idea how to change the rule to work with multiple transfers/messages per function? If it would make the cocci file more complex I wouldn't bother to take care of it, since it basically has no practical use. - Lars