> -----Original Message----- > From: ccross@xxxxxxxxxx [mailto:ccross@xxxxxxxxxx] On Behalf Of > Colin Cross > Sent: Wednesday, January 26, 2011 1:23 PM > To: Santosh Shilimkar > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux- > omap@xxxxxxxxxxxxxxx; catalin.marinas@xxxxxxx; > linux@xxxxxxxxxxxxxxxx; linus.ml.walleij@xxxxxxxxx; Russell King > Subject: Re: [PATCH 1/5] ARM: gic: Add hooks for architecture > specific extensions > > On Tue, Jan 25, 2011 at 11:31 PM, Santosh Shilimkar > <santosh.shilimkar@xxxxxx> wrote: > >> -----Original Message----- > >> From: ccross@xxxxxxxxxx [mailto:ccross@xxxxxxxxxx] On Behalf Of > >> Colin Cross > >> Sent: Wednesday, January 26, 2011 12:54 PM > >> To: Santosh Shilimkar > >> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux- > >> omap@xxxxxxxxxxxxxxx; catalin.marinas@xxxxxxx; > >> linux@xxxxxxxxxxxxxxxx; linus.ml.walleij@xxxxxxxxx; Russell King > >> Subject: Re: [PATCH 1/5] ARM: gic: Add hooks for architecture > >> specific extensions > >> > > [....] > > > >> >> > >> > Great. > >> > Can I fold below changes in my patch and add you ack and > tested- > >> by? > >> > >> Sure > > > > After reading your initial comment, you mentioned you need to have > > 'gic_set_type' as well. Is this still true. If yes then we need to > > have arch_extn call for that as well. > > You are right, I missed adding the extension for gic_set_type. My > testing doesn't cover that case right now, because I don't have any > drivers updated to linux-next that use a wake source that is > compatible with Tegra's lowest power suspend mode, and that is the > only time the extension to gic_set_type is necessary. > Ok. So I will go ahead and add an extension for the same so that we have most of the usecases covered. Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html