Lee Jones writes:
> Very well, Russell and yourself have convinced me. If you fixup the
> remainder of comments, I'm happy.
Cool.
Let me a couple of days to gather my wits, cross-check I have not forgotten a
comment, make some testing on the board and then post v4.
Cheers.
--
Robert
--
To
On Wed, Jan 21, 2015 at 08:40:44AM -0800, Joe Perches wrote:
> On Wed, 2015-01-21 at 16:11 +, Russell King - ARM Linux wrote:
> > On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
> > > (adding netdev)
> >
> > I wasn't actually reporting that as an issue; I was using it as an
> >
On Wed, 2015-01-21 at 16:11 +, Russell King - ARM Linux wrote:
> On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
> > (adding netdev)
>
> I wasn't actually reporting that as an issue; I was using it as an
> example. It's from a very old kernel (2.6.27.21) which I run on one
> of
On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
> (adding netdev)
I wasn't actually reporting that as an issue; I was using it as an
example. It's from a very old kernel (2.6.27.21) which I run on one
of my old x86 machines.
--
FTTC broadband for 0.8mile line: currently at
(adding netdev)
On Wed, 2015-01-21 at 09:44 +, Russell King - ARM Linux wrote:
> On Wed, Jan 21, 2015 at 08:46:29AM +0100, Robert Jarzmik wrote:
> > Russell King - ARM Linux writes:
> >
> > > What I'd suggest (and always have done) is:
> > >
> > > dev_err(>dev, "couldn't request main
On Wed, 21 Jan 2015, robert.jarz...@free.fr wrote:
> > - Mail original -
> > De: "Lee Jones"
> First of all, this is my web mail interface, so please be kind with
> my mail formatting ...
I have no idea what you want me to change or do differently?
Perhaps it might be more prudent for
> - Mail original -
> De: "Lee Jones"
First of all, this is my web mail interface, so please be kind with
my mail formatting ...
> Looking at one of the other patches in the series it appears the flag
> you're trying to capture is IORESOURCE_IRQ_LOWEDGE. When I grep for
> where this is
On Wed, Jan 21, 2015 at 12:35:51PM +, Lee Jones wrote:
> I think to set an edge trigger on an IRQ, you should instead do so via
> irq_set_irq_type(), or have a missed a line or two?
Setting the IRQ type in request_irq() is perfectly acceptable.
include/linux/interrupt.h:
/*
* These
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
> Lee Jones writes:
>
> > On Wed, 21 Jan 2015, Robert Jarzmik wrote:
> >> > > > platform_get_irq()?
> >> > > No. I need the flags.
> >> > Where are they used?
> >> A couple of lines below, using local "irqflags" variable :
> >>ret =
On Wed, Jan 21, 2015 at 08:16:45AM +, Lee Jones wrote:
> On Wed, 21 Jan 2015, Robert Jarzmik wrote:
> > > > > platform_get_irq()?
> > > > No. I need the flags.
> > > Where are they used?
> > A couple of lines below, using local "irqflags" variable :
> >ret = devm_request_irq(>dev,
On Wed, Jan 21, 2015 at 08:46:29AM +0100, Robert Jarzmik wrote:
> Russell King - ARM Linux writes:
>
> > What I'd suggest (and always have done) is:
> >
> > dev_err(>dev, "couldn't request main irq%d: %d\n",
> > irq, ret);
> I like it, it's even more compact, I'll use it for next
Lee Jones writes:
> On Wed, 21 Jan 2015, Robert Jarzmik wrote:
>> > > > platform_get_irq()?
>> > > No. I need the flags.
>> > Where are they used?
>> A couple of lines below, using local "irqflags" variable :
>>ret = devm_request_irq(>dev, cot->irq, lubbock_irq_handler,
>>
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
> > > > platform_get_irq()?
> > > No. I need the flags.
> > Where are they used?
> A couple of lines below, using local "irqflags" variable :
>ret = devm_request_irq(>dev, cot->irq, lubbock_irq_handler,
> irqflags,
Lee Jones lee.jo...@linaro.org writes:
Very well, Russell and yourself have convinced me. If you fixup the
remainder of comments, I'm happy.
Cool.
Let me a couple of days to gather my wits, cross-check I have not forgotten a
comment, make some testing on the board and then post v4.
Cheers.
On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
(adding netdev)
I wasn't actually reporting that as an issue; I was using it as an
example. It's from a very old kernel (2.6.27.21) which I run on one
of my old x86 machines.
--
FTTC broadband for 0.8mile line: currently at 10.5Mbps
On Wed, 2015-01-21 at 16:11 +, Russell King - ARM Linux wrote:
On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
(adding netdev)
I wasn't actually reporting that as an issue; I was using it as an
example. It's from a very old kernel (2.6.27.21) which I run on one
of my old
On Wed, Jan 21, 2015 at 08:40:44AM -0800, Joe Perches wrote:
On Wed, 2015-01-21 at 16:11 +, Russell King - ARM Linux wrote:
On Wed, Jan 21, 2015 at 08:05:21AM -0800, Joe Perches wrote:
(adding netdev)
I wasn't actually reporting that as an issue; I was using it as an
example.
(adding netdev)
On Wed, 2015-01-21 at 09:44 +, Russell King - ARM Linux wrote:
On Wed, Jan 21, 2015 at 08:46:29AM +0100, Robert Jarzmik wrote:
Russell King - ARM Linux li...@arm.linux.org.uk writes:
What I'd suggest (and always have done) is:
dev_err(pdev-dev, couldn't
On Wed, Jan 21, 2015 at 08:16:45AM +, Lee Jones wrote:
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
platform_get_irq()?
No. I need the flags.
Where are they used?
A couple of lines below, using local irqflags variable :
ret = devm_request_irq(pdev-dev, cot-irq,
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
platform_get_irq()?
No. I need the flags.
Where are they used?
A couple of lines below, using local irqflags variable :
ret = devm_request_irq(pdev-dev, cot-irq, lubbock_irq_handler,
irqflags,
On Wed, Jan 21, 2015 at 08:46:29AM +0100, Robert Jarzmik wrote:
Russell King - ARM Linux li...@arm.linux.org.uk writes:
What I'd suggest (and always have done) is:
dev_err(pdev-dev, couldn't request main irq%d: %d\n,
irq, ret);
I like it, it's even more compact, I'll
Lee Jones lee.jo...@linaro.org writes:
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
platform_get_irq()?
No. I need the flags.
Where are they used?
A couple of lines below, using local irqflags variable :
ret = devm_request_irq(pdev-dev, cot-irq, lubbock_irq_handler,
On Wed, Jan 21, 2015 at 12:35:51PM +, Lee Jones wrote:
I think to set an edge trigger on an IRQ, you should instead do so via
irq_set_irq_type(), or have a missed a line or two?
Setting the IRQ type in request_irq() is perfectly acceptable.
include/linux/interrupt.h:
/*
* These correspond
On Wed, 21 Jan 2015, robert.jarz...@free.fr wrote:
- Mail original -
De: Lee Jones lee.jo...@linaro.org
First of all, this is my web mail interface, so please be kind with
my mail formatting ...
I have no idea what you want me to change or do differently?
Perhaps it might be more
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
Lee Jones lee.jo...@linaro.org writes:
On Wed, 21 Jan 2015, Robert Jarzmik wrote:
platform_get_irq()?
No. I need the flags.
Where are they used?
A couple of lines below, using local irqflags variable :
ret =
- Mail original -
De: Lee Jones lee.jo...@linaro.org
First of all, this is my web mail interface, so please be kind with
my mail formatting ...
Looking at one of the other patches in the series it appears the flag
you're trying to capture is IORESOURCE_IRQ_LOWEDGE. When I grep for
Russell King - ARM Linux writes:
> What I'd suggest (and always have done) is:
>
> dev_err(>dev, "couldn't request main irq%d: %d\n",
> irq, ret);
I like it, it's even more compact, I'll use it for next patch version.
> but I guess printing the IRQ number no longer makes
On Tue, Jan 20, 2015 at 10:29:19AM +, Lee Jones wrote:
> On Mon, 19 Jan 2015, Robert Jarzmik wrote:
> > >> +if (ret) {
> > >> +dev_err(>dev, "Couldn't request main irq : ret =
> > >> %d\n",
> > >> +ret);
> > >
> > > I'm not keen on this type of
On Mon, 19 Jan 2015, Robert Jarzmik wrote:
> Lee Jones writes:
> > On Fri, 16 Jan 2015, Robert Jarzmik wrote:
[...]
> >> + res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
> >
> > platform_get_irq()?
> No. I need the flags.
Where are they used?
[...]
> >> + if (ret) {
> >> +
On Mon, 19 Jan 2015, Robert Jarzmik wrote:
Lee Jones lee.jo...@linaro.org writes:
On Fri, 16 Jan 2015, Robert Jarzmik wrote:
[...]
+ res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
platform_get_irq()?
No. I need the flags.
Where are they used?
[...]
+ if (ret) {
+
On Tue, Jan 20, 2015 at 10:29:19AM +, Lee Jones wrote:
On Mon, 19 Jan 2015, Robert Jarzmik wrote:
+if (ret) {
+dev_err(pdev-dev, Couldn't request main irq : ret =
%d\n,
+ret);
I'm not keen on this type of formatting. Besides
Russell King - ARM Linux li...@arm.linux.org.uk writes:
What I'd suggest (and always have done) is:
dev_err(pdev-dev, couldn't request main irq%d: %d\n,
irq, ret);
I like it, it's even more compact, I'll use it for next patch version.
but I guess printing the IRQ number
Lee Jones writes:
> On Fri, 16 Jan 2015, Robert Jarzmik wrote:
>
>> As a fix, move the gpio0 chained handler setup to a place where we have
>> the guarantee that pxa_gpio_probe() was called before, so that lubbock
>> handler becomes the true IRQ chained handler of GPIO0, demuxing the
>> lubbock
On Fri, 16 Jan 2015, Robert Jarzmik wrote:
> Lubbock () board is the IO motherboard of the Intel PXA25x Development
> Platform, which supports the Lubbock pxa25x soc board.
>
> Historically, this support was in arch/arm/mach-pxa/lubbock.c. When
> gpio-pxa was moved to drivers/pxa, it became a
On Fri, 16 Jan 2015, Robert Jarzmik wrote:
Lubbock () board is the IO motherboard of the Intel PXA25x Development
Platform, which supports the Lubbock pxa25x soc board.
Historically, this support was in arch/arm/mach-pxa/lubbock.c. When
gpio-pxa was moved to drivers/pxa, it became a driver,
Lee Jones lee.jo...@linaro.org writes:
On Fri, 16 Jan 2015, Robert Jarzmik wrote:
As a fix, move the gpio0 chained handler setup to a place where we have
the guarantee that pxa_gpio_probe() was called before, so that lubbock
handler becomes the true IRQ chained handler of GPIO0, demuxing the
36 matches
Mail list logo