From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754659Ab1LNH5j (ORCPT ); Wed, 14 Dec 2011 02:57:39 -0500 Received: from webbox1416.server-home.net ([77.236.96.61]:42717 "EHLO webbox1416.server-home.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753045Ab1LNH5g (ORCPT ); Wed, 14 Dec 2011 02:57:36 -0500 From: Alexander Stein To: Greg KH Cc: Joachim Foerster , Paul Gortmaker , Grant Likely , Greg Kroah-Hartman , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] USB: ISP1760: Add flags for interrupt polarity and edge/level trigger Date: Wed, 14 Dec 2011 08:57:33 +0100 Message-ID: <5171961.tdOaMPUdsK@ws-stein> User-Agent: KMail/4.7.3 (Linux/3.0.6-gentoo; KDE/4.7.3; x86_64; ; ) In-Reply-To: <20111213180010.GA29980@kroah.com> References: <1323768793-20633-1-git-send-email-alexander.stein@systec-electronic.com> <20111213180010.GA29980@kroah.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 13 December 2011 10:00:10 Greg KH wrote: > On Tue, Dec 13, 2011 at 10:33:13AM +0100, Alexander Stein wrote: > > Signed-off-by: Alexander Stein > > --- > > > > drivers/usb/host/isp1760-if.c | 4 ++++ > > include/linux/usb/isp1760.h | 2 ++ > > 2 files changed, 6 insertions(+), 0 deletions(-) > > > > diff --git a/drivers/usb/host/isp1760-if.c > > b/drivers/usb/host/isp1760-if.c index a7dc1e1..0ef9867 100644 > > --- a/drivers/usb/host/isp1760-if.c > > +++ b/drivers/usb/host/isp1760-if.c > > @@ -388,6 +388,10 @@ static int __devinit isp1760_plat_probe(struct > > platform_device *pdev)> > > devflags |= ISP1760_FLAG_DACK_POL_HIGH; > > > > if (priv->dreq_polarity_high) > > > > devflags |= ISP1760_FLAG_DREQ_POL_HIGH; > > > > + if (priv->interrupt_polarity_high) > > + devflags |= ISP1760_FLAG_INTR_POL_HIGH; > > + if (priv->interrupt_edge_triggered) > > + devflags |= ISP1760_FLAG_INTR_EDGE_TRIG; > > > > } > > > > hcd = isp1760_register(mem_res->start, mem_size, irq_res->start, > > > > diff --git a/include/linux/usb/isp1760.h b/include/linux/usb/isp1760.h > > index de7de53..c5edbc8 100644 > > --- a/include/linux/usb/isp1760.h > > +++ b/include/linux/usb/isp1760.h > > @@ -13,6 +13,8 @@ struct isp1760_platform_data { > > > > unsigned analog_oc:1; /* Analog overcurrent */ > > unsigned dack_polarity_high:1; /* DACK active high */ > > unsigned dreq_polarity_high:1; /* DREQ active high */ > > > > + unsigned interrupt_polarity_high:1; > > + unsigned interrupt_edge_triggered:1; > > So you added these, and you check them, but who sets them? What is the > point of this patch at this point in time? These platform data bits are set by board configurations, which is completly optional. So currently maybe noone sets/use it, but in future someone may need it. Also dreq_polarity_high is also there and I only found board files setting them to 0... So why not add the feature bits too? Regards, Alexander