* Re: [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled
[not found] ` <c09aa50a0911091218i681449e0r5cb96b9db3e0def6@mail.gmail.com>
@ 2009-11-09 20:39 ` Guennadi Liakhovetski
2009-11-10 1:36 ` Ian Molton
0 siblings, 1 reply; 4+ messages in thread
From: Guennadi Liakhovetski @ 2009-11-09 20:39 UTC (permalink / raw)
To: Ian Molton; +Cc: Linux Media Mailing List
(re-adding accidentally dropped ML)
On Mon, 9 Nov 2009, Ian Molton wrote:
> Well, I presume we want to know when the card gets removed :)
Sure, that's why we shouldn't mask those interrupts:-) If they do get
masked and missed, I do not know, if the interrupt remains pending in this
case, because they never get detected then:)
>
> 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> > Hi Ian
> >
> > Why did you drop all CCs?
> >
> > On Mon, 9 Nov 2009, Ian Molton wrote:
> >
> >> I havent looked at the consequences for the driver if a insert IRQ
> >> occurs during IO, however it seems logical that we should not
> >> permanently mask the IRQ.
> >>
> >> I presume that the IRQ remains pending?
> >
> > Don't know, never checked. Is this important to know?
> >
> > Thanks
> > Guennadi
> >
> >>
> >> 2009/11/6 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> >> > On SuperH platforms the SDHI controller does not produce any command IRQs
> >> > after a completed IO. This leads to card-detect interrupts staying
> >> > disabled. Do not disable card-detect interrupts on DATA IRQs.
> >> >
> >> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> >> > ---
> >> >
> >> > Marked as RFC because I'm not really sure this is a correct approach to
> >> > fix this problem, and whether this will have negative effect on other
> >> > tmio_mmc MFD users.
> >> >
> >> > diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
> >> > index c676767..0b31d44 100644
> >> > --- a/drivers/mmc/host/tmio_mmc.h
> >> > +++ b/drivers/mmc/host/tmio_mmc.h
> >> > @@ -55,10 +55,8 @@
> >> > /* Define some IRQ masks */
> >> > /* This is the mask used at reset by the chip */
> >> > #define TMIO_MASK_ALL 0x837f031d
> >> > -#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND | \
> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> > -#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND | \
> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> > +#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND)
> >> > +#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND)
> >> > #define TMIO_MASK_CMD (TMIO_STAT_CMDRESPEND | TMIO_STAT_CMDTIMEOUT | \
> >> > TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> > #define TMIO_MASK_IRQ (TMIO_MASK_READOP | TMIO_MASK_WRITEOP | TMIO_MASK_CMD)
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled
2009-11-09 20:39 ` [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled Guennadi Liakhovetski
@ 2009-11-10 1:36 ` Ian Molton
2009-11-28 19:06 ` Guennadi Liakhovetski
0 siblings, 1 reply; 4+ messages in thread
From: Ian Molton @ 2009-11-10 1:36 UTC (permalink / raw)
To: Guennadi Liakhovetski; +Cc: Linux Media Mailing List
Well if they are only masked they shouldnt stop being asserted. But we
should unmask them again.
Im not really sure we should mask them anyway, with the card possibly
being gone... Will need to look into it further.
2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> (re-adding accidentally dropped ML)
>
> On Mon, 9 Nov 2009, Ian Molton wrote:
>
>> Well, I presume we want to know when the card gets removed :)
>
> Sure, that's why we shouldn't mask those interrupts:-) If they do get
> masked and missed, I do not know, if the interrupt remains pending in this
> case, because they never get detected then:)
>
>>
>> 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
>> > Hi Ian
>> >
>> > Why did you drop all CCs?
>> >
>> > On Mon, 9 Nov 2009, Ian Molton wrote:
>> >
>> >> I havent looked at the consequences for the driver if a insert IRQ
>> >> occurs during IO, however it seems logical that we should not
>> >> permanently mask the IRQ.
>> >>
>> >> I presume that the IRQ remains pending?
>> >
>> > Don't know, never checked. Is this important to know?
>> >
>> > Thanks
>> > Guennadi
>> >
>> >>
>> >> 2009/11/6 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
>> >> > On SuperH platforms the SDHI controller does not produce any command IRQs
>> >> > after a completed IO. This leads to card-detect interrupts staying
>> >> > disabled. Do not disable card-detect interrupts on DATA IRQs.
>> >> >
>> >> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
>> >> > ---
>> >> >
>> >> > Marked as RFC because I'm not really sure this is a correct approach to
>> >> > fix this problem, and whether this will have negative effect on other
>> >> > tmio_mmc MFD users.
>> >> >
>> >> > diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
>> >> > index c676767..0b31d44 100644
>> >> > --- a/drivers/mmc/host/tmio_mmc.h
>> >> > +++ b/drivers/mmc/host/tmio_mmc.h
>> >> > @@ -55,10 +55,8 @@
>> >> > /* Define some IRQ masks */
>> >> > /* This is the mask used at reset by the chip */
>> >> > #define TMIO_MASK_ALL 0x837f031d
>> >> > -#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND | \
>> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
>> >> > -#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND | \
>> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
>> >> > +#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND)
>> >> > +#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND)
>> >> > #define TMIO_MASK_CMD (TMIO_STAT_CMDRESPEND | TMIO_STAT_CMDTIMEOUT | \
>> >> > TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
>> >> > #define TMIO_MASK_IRQ (TMIO_MASK_READOP | TMIO_MASK_WRITEOP | TMIO_MASK_CMD)
>
> Thanks
> Guennadi
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
> http://www.open-technology.de/
>
--
Ian Molton
Linux, Automotive, and other hacking:
http://www.mnementh.co.uk/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled
2009-11-10 1:36 ` Ian Molton
@ 2009-11-28 19:06 ` Guennadi Liakhovetski
2009-12-16 21:24 ` Guennadi Liakhovetski
0 siblings, 1 reply; 4+ messages in thread
From: Guennadi Liakhovetski @ 2009-11-28 19:06 UTC (permalink / raw)
To: Ian Molton; +Cc: Linux Media Mailing List
On Tue, 10 Nov 2009, Ian Molton wrote:
> Well if they are only masked they shouldnt stop being asserted. But we
> should unmask them again.
>
> Im not really sure we should mask them anyway, with the card possibly
> being gone... Will need to look into it further.
Hi Ingo
What's the status of this patch? Are you going to queue it for 2.6.33?
Thanks
Guennadi
>
> 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> > (re-adding accidentally dropped ML)
> >
> > On Mon, 9 Nov 2009, Ian Molton wrote:
> >
> >> Well, I presume we want to know when the card gets removed :)
> >
> > Sure, that's why we shouldn't mask those interrupts:-) If they do get
> > masked and missed, I do not know, if the interrupt remains pending in this
> > case, because they never get detected then:)
> >
> >>
> >> 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> >> > Hi Ian
> >> >
> >> > Why did you drop all CCs?
> >> >
> >> > On Mon, 9 Nov 2009, Ian Molton wrote:
> >> >
> >> >> I havent looked at the consequences for the driver if a insert IRQ
> >> >> occurs during IO, however it seems logical that we should not
> >> >> permanently mask the IRQ.
> >> >>
> >> >> I presume that the IRQ remains pending?
> >> >
> >> > Don't know, never checked. Is this important to know?
> >> >
> >> > Thanks
> >> > Guennadi
> >> >
> >> >>
> >> >> 2009/11/6 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> >> >> > On SuperH platforms the SDHI controller does not produce any command IRQs
> >> >> > after a completed IO. This leads to card-detect interrupts staying
> >> >> > disabled. Do not disable card-detect interrupts on DATA IRQs.
> >> >> >
> >> >> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> >> >> > ---
> >> >> >
> >> >> > Marked as RFC because I'm not really sure this is a correct approach to
> >> >> > fix this problem, and whether this will have negative effect on other
> >> >> > tmio_mmc MFD users.
> >> >> >
> >> >> > diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
> >> >> > index c676767..0b31d44 100644
> >> >> > --- a/drivers/mmc/host/tmio_mmc.h
> >> >> > +++ b/drivers/mmc/host/tmio_mmc.h
> >> >> > @@ -55,10 +55,8 @@
> >> >> > /* Define some IRQ masks */
> >> >> > /* This is the mask used at reset by the chip */
> >> >> > #define TMIO_MASK_ALL 0x837f031d
> >> >> > -#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND | \
> >> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> >> > -#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND | \
> >> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> >> > +#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND)
> >> >> > +#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND)
> >> >> > #define TMIO_MASK_CMD (TMIO_STAT_CMDRESPEND | TMIO_STAT_CMDTIMEOUT | \
> >> >> > TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> >> >> > #define TMIO_MASK_IRQ (TMIO_MASK_READOP | TMIO_MASK_WRITEOP | TMIO_MASK_CMD)
> >
> > Thanks
> > Guennadi
> > ---
> > Guennadi Liakhovetski, Ph.D.
> > Freelance Open-Source Software Developer
> > http://www.open-technology.de/
> >
>
>
>
> --
> Ian Molton
> Linux, Automotive, and other hacking:
> http://www.mnementh.co.uk/
>
---
Guennadi Liakhovetski
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled
2009-11-28 19:06 ` Guennadi Liakhovetski
@ 2009-12-16 21:24 ` Guennadi Liakhovetski
0 siblings, 0 replies; 4+ messages in thread
From: Guennadi Liakhovetski @ 2009-12-16 21:24 UTC (permalink / raw)
To: Ian Molton; +Cc: Linux Media Mailing List
On Sat, 28 Nov 2009, Guennadi Liakhovetski wrote:
> On Tue, 10 Nov 2009, Ian Molton wrote:
>
> > Well if they are only masked they shouldnt stop being asserted. But we
> > should unmask them again.
> >
> > Im not really sure we should mask them anyway, with the card possibly
> > being gone... Will need to look into it further.
>
> Hi Ian
>
> What's the status of this patch? Are you going to queue it for 2.6.33?
Ian, ping? I still don't see this patch in any (Linus' / next) upstream
tree and the merge window will be closing soon. Any update?
Thanks
Guennadi
>
> Thanks
> Guennadi
>
> >
> > 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> > > (re-adding accidentally dropped ML)
> > >
> > > On Mon, 9 Nov 2009, Ian Molton wrote:
> > >
> > >> Well, I presume we want to know when the card gets removed :)
> > >
> > > Sure, that's why we shouldn't mask those interrupts:-) If they do get
> > > masked and missed, I do not know, if the interrupt remains pending in this
> > > case, because they never get detected then:)
> > >
> > >>
> > >> 2009/11/9 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> > >> > Hi Ian
> > >> >
> > >> > Why did you drop all CCs?
> > >> >
> > >> > On Mon, 9 Nov 2009, Ian Molton wrote:
> > >> >
> > >> >> I havent looked at the consequences for the driver if a insert IRQ
> > >> >> occurs during IO, however it seems logical that we should not
> > >> >> permanently mask the IRQ.
> > >> >>
> > >> >> I presume that the IRQ remains pending?
> > >> >
> > >> > Don't know, never checked. Is this important to know?
> > >> >
> > >> > Thanks
> > >> > Guennadi
> > >> >
> > >> >>
> > >> >> 2009/11/6 Guennadi Liakhovetski <g.liakhovetski@gmx.de>:
> > >> >> > On SuperH platforms the SDHI controller does not produce any command IRQs
> > >> >> > after a completed IO. This leads to card-detect interrupts staying
> > >> >> > disabled. Do not disable card-detect interrupts on DATA IRQs.
> > >> >> >
> > >> >> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> > >> >> > ---
> > >> >> >
> > >> >> > Marked as RFC because I'm not really sure this is a correct approach to
> > >> >> > fix this problem, and whether this will have negative effect on other
> > >> >> > tmio_mmc MFD users.
> > >> >> >
> > >> >> > diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
> > >> >> > index c676767..0b31d44 100644
> > >> >> > --- a/drivers/mmc/host/tmio_mmc.h
> > >> >> > +++ b/drivers/mmc/host/tmio_mmc.h
> > >> >> > @@ -55,10 +55,8 @@
> > >> >> > /* Define some IRQ masks */
> > >> >> > /* This is the mask used at reset by the chip */
> > >> >> > #define TMIO_MASK_ALL 0x837f031d
> > >> >> > -#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND | \
> > >> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> > >> >> > -#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND | \
> > >> >> > - TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> > >> >> > +#define TMIO_MASK_READOP (TMIO_STAT_RXRDY | TMIO_STAT_DATAEND)
> > >> >> > +#define TMIO_MASK_WRITEOP (TMIO_STAT_TXRQ | TMIO_STAT_DATAEND)
> > >> >> > #define TMIO_MASK_CMD (TMIO_STAT_CMDRESPEND | TMIO_STAT_CMDTIMEOUT | \
> > >> >> > TMIO_STAT_CARD_REMOVE | TMIO_STAT_CARD_INSERT)
> > >> >> > #define TMIO_MASK_IRQ (TMIO_MASK_READOP | TMIO_MASK_WRITEOP | TMIO_MASK_CMD)
> > >
> > > Thanks
> > > Guennadi
> > > ---
> > > Guennadi Liakhovetski, Ph.D.
> > > Freelance Open-Source Software Developer
> > > http://www.open-technology.de/
> > >
> >
> >
> >
> > --
> > Ian Molton
> > Linux, Automotive, and other hacking:
> > http://www.mnementh.co.uk/
> >
>
> ---
> Guennadi Liakhovetski
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-12-16 21:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <Pine.LNX.4.64.0911061127240.4389@axis700.grange>
[not found] ` <c09aa50a0911090242l35d0dfb2vec0cdeff8b86d33e@mail.gmail.com>
[not found] ` <Pine.LNX.4.64.0911091530030.4289@axis700.grange>
[not found] ` <c09aa50a0911091218i681449e0r5cb96b9db3e0def6@mail.gmail.com>
2009-11-09 20:39 ` [PATCH/RFC] tmio_mmc: keep card-detect interrupts enabled Guennadi Liakhovetski
2009-11-10 1:36 ` Ian Molton
2009-11-28 19:06 ` Guennadi Liakhovetski
2009-12-16 21:24 ` Guennadi Liakhovetski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox