[PATCH] bc47xx : export ssb_watchdog_timer_set

[PATCH] bc47xx : export ssb_watchdog_timer_set

[matthieu castet]

[-- Attachment #1: Type: text/plain, Size: 154 bytes --]

Hi,

this patch export ssb_watchdog_timer_set to allow to use it in a Linux 
watchdog driver.


Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>


[-- Attachment #2: export_ssb_watchdog.patch --]
[-- Type: text/x-diff, Size: 344 bytes --]

diff --git a/drivers/ssb/embedded.c b/drivers/ssb/embedded.c
index 7dc3a6b..a0e0d24 100644
--- a/drivers/ssb/embedded.c
+++ b/drivers/ssb/embedded.c
@@ -29,6 +29,7 @@ int ssb_watchdog_timer_set(struct ssb_bus *bus, u32 ticks)
 	}
 	return -ENODEV;
 }
+EXPORT_SYMBOL(ssb_watchdog_timer_set);
 
 u32 ssb_gpio_in(struct ssb_bus *bus, u32 mask)
 {

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[John W. Linville]

On Tue, May 19, 2009 at 12:10:07AM +0200, matthieu castet wrote:
> Hi,
>
> this patch export ssb_watchdog_timer_set to allow to use it in a Linux  
> watchdog driver.
>
>
> Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>

What is the merge path for ssb nowadays?  I used to take these patches
(and I'm still happy to do so), but maybe Ralf is (or should be)
taking them now?

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Michael Buesch]

On Tuesday 19 May 2009 00:10:07 matthieu castet wrote:
> Hi,
> 
> this patch export ssb_watchdog_timer_set to allow to use it in a Linux 
> watchdog driver.
> 
> 
> Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>

Well, you need to be careful. The watchdog is also used for system reboot.
Make sure to disable the watchdog driver when the bcm47xx system code wants
to use it.

Otherwise, ack. You can submit this to linville@tuxdriver.com.

-- 
Greetings, Michael.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Michael Buesch]

On Tuesday 19 May 2009 00:41:28 John W. Linville wrote:
> On Tue, May 19, 2009 at 12:10:07AM +0200, matthieu castet wrote:
> > Hi,
> >
> > this patch export ssb_watchdog_timer_set to allow to use it in a Linux  
> > watchdog driver.
> >
> >
> > Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>
> 
> What is the merge path for ssb nowadays?  I used to take these patches
> (and I'm still happy to do so), but maybe Ralf is (or should be)
> taking them now?

That depends on his speed. Last time I submitted a patch through his path,
it bitrotted for several months before it finally hit mainline.

-- 
Greetings, Michael.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Ralf Baechle]

On Tue, May 19, 2009 at 03:24:20PM +0200, Michael Buesch wrote:

> > What is the merge path for ssb nowadays?  I used to take these patches
> > (and I'm still happy to do so), but maybe Ralf is (or should be)
> > taking them now?
> 
> That depends on his speed. Last time I submitted a patch through his path,
> it bitrotted for several months before it finally hit mainline.

Maybe because I felt drivers/ssb/ was outside my jurisdiction - and unlike
what alot of people may seem to think I'm not a full time MIPS kernel
hacker.

I can deal with SSB patch if you so desire - but I have no experience with
SSB, so I'd have somebody to rubberstamp non-trivial SSB patches before I
queue them up.  I can keep them either in the usual MIPS trees on
linux-mips.org or I could create a separate linux-ssb tree, depending on
what seems to be sensible.  Also, reading the entry in the maintainers
file I wonder if netdev is really the list of a choice?

  Ralf

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Michael Buesch]

On Tuesday 19 May 2009 19:09:57 Ralf Baechle wrote:
> Maybe because I felt drivers/ssb/ was outside my jurisdiction - and unlike
> what alot of people may seem to think I'm not a full time MIPS kernel
> hacker.

Ok nice.

> I can deal with SSB patch if you so desire - but I have no experience with
> SSB, so I'd have somebody to rubberstamp non-trivial SSB patches before I
> queue them up.

 **Fwoo..
[stamp here]
      ..mppp**


Done. :)

> I can keep them either in the usual MIPS trees on 
> linux-mips.org or I could create a separate linux-ssb tree, depending on
> what seems to be sensible.  Also, reading the entry in the maintainers
> file I wonder if netdev is really the list of a choice?

Yes it is, because the bus is only used on networking devices.
(Ethernet cards, wireless cards, and network routers)
I don't think you need to create a separate tree. ssb is pretty mature. There
won't be that many patches.

-- 
Greetings, Michael.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[John W. Linville]

On Tue, May 19, 2009 at 07:29:20PM +0200, Michael Buesch wrote:
> On Tuesday 19 May 2009 19:09:57 Ralf Baechle wrote:
> > Maybe because I felt drivers/ssb/ was outside my jurisdiction - and unlike
> > what alot of people may seem to think I'm not a full time MIPS kernel
> > hacker.
> 
> Ok nice.
> 
> > I can deal with SSB patch if you so desire - but I have no experience with
> > SSB, so I'd have somebody to rubberstamp non-trivial SSB patches before I
> > queue them up.
> 
>  **Fwoo..
> [stamp here]
>       ..mppp**
> 
> 
> Done. :)
> 
> > I can keep them either in the usual MIPS trees on 
> > linux-mips.org or I could create a separate linux-ssb tree, depending on
> > what seems to be sensible.  Also, reading the entry in the maintainers
> > file I wonder if netdev is really the list of a choice?
> 
> Yes it is, because the bus is only used on networking devices.
> (Ethernet cards, wireless cards, and network routers)
> I don't think you need to create a separate tree. ssb is pretty mature. There
> won't be that many patches.

OK, now I'm confused again -- should I take SSB patches, or is Ralf
going to do it? :-)

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Ralf Baechle]

On Tue, May 19, 2009 at 01:35:03PM -0400, John W. Linville wrote:
> From: "John W. Linville" <linville@tuxdriver.com>
> Date: Tue, 19 May 2009 13:35:03 -0400
> To: Michael Buesch <mb@bu3sch.de>
> Cc: Ralf Baechle <ralf@linux-mips.org>,
> 	matthieu castet <castet.matthieu@free.fr>,
> 	linux-mips@linux-mips.org, netdev@vger.kernel.org
> Subject: Re: [PATCH] bc47xx : export ssb_watchdog_timer_set
> Content-Type: text/plain; charset=us-ascii
> 
> On Tue, May 19, 2009 at 07:29:20PM +0200, Michael Buesch wrote:
> > On Tuesday 19 May 2009 19:09:57 Ralf Baechle wrote:
> > > Maybe because I felt drivers/ssb/ was outside my jurisdiction - and unlike
> > > what alot of people may seem to think I'm not a full time MIPS kernel
> > > hacker.
> > 
> > Ok nice.
> > 
> > > I can deal with SSB patch if you so desire - but I have no experience with
> > > SSB, so I'd have somebody to rubberstamp non-trivial SSB patches before I
> > > queue them up.
> > 
> >  **Fwoo..
> > [stamp here]
> >       ..mppp**
> > 
> > 
> > Done. :)
> > 
> > > I can keep them either in the usual MIPS trees on 
> > > linux-mips.org or I could create a separate linux-ssb tree, depending on
> > > what seems to be sensible.  Also, reading the entry in the maintainers
> > > file I wonder if netdev is really the list of a choice?
> > 
> > Yes it is, because the bus is only used on networking devices.
> > (Ethernet cards, wireless cards, and network routers)
> > I don't think you need to create a separate tree. ssb is pretty mature. There
> > won't be that many patches.
> 
> OK, now I'm confused again -- should I take SSB patches, or is Ralf
> going to do it? :-)

My decoder ring says, I'll do it from now on :)

Cheers,

  Ralf

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[John W. Linville]

On Wed, May 20, 2009 at 08:15:13AM +0100, Ralf Baechle wrote:
> On Tue, May 19, 2009 at 01:35:03PM -0400, John W. Linville wrote:

> > OK, now I'm confused again -- should I take SSB patches, or is Ralf
> > going to do it? :-)
> 
> My decoder ring says, I'll do it from now on :)

ACK

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[Michael Buesch]

On Wednesday 20 May 2009 09:15:13 Ralf Baechle wrote:
> On Tue, May 19, 2009 at 01:35:03PM -0400, John W. Linville wrote:
> > From: "John W. Linville" <linville@tuxdriver.com>
> > Date: Tue, 19 May 2009 13:35:03 -0400
> > To: Michael Buesch <mb@bu3sch.de>
> > Cc: Ralf Baechle <ralf@linux-mips.org>,
> > 	matthieu castet <castet.matthieu@free.fr>,
> > 	linux-mips@linux-mips.org, netdev@vger.kernel.org
> > Subject: Re: [PATCH] bc47xx : export ssb_watchdog_timer_set
> > Content-Type: text/plain; charset=us-ascii
> > 
> > On Tue, May 19, 2009 at 07:29:20PM +0200, Michael Buesch wrote:
> > > On Tuesday 19 May 2009 19:09:57 Ralf Baechle wrote:
> > > > Maybe because I felt drivers/ssb/ was outside my jurisdiction - and unlike
> > > > what alot of people may seem to think I'm not a full time MIPS kernel
> > > > hacker.
> > > 
> > > Ok nice.
> > > 
> > > > I can deal with SSB patch if you so desire - but I have no experience with
> > > > SSB, so I'd have somebody to rubberstamp non-trivial SSB patches before I
> > > > queue them up.
> > > 
> > >  **Fwoo..
> > > [stamp here]
> > >       ..mppp**
> > > 
> > > 
> > > Done. :)
> > > 
> > > > I can keep them either in the usual MIPS trees on 
> > > > linux-mips.org or I could create a separate linux-ssb tree, depending on
> > > > what seems to be sensible.  Also, reading the entry in the maintainers
> > > > file I wonder if netdev is really the list of a choice?
> > > 
> > > Yes it is, because the bus is only used on networking devices.
> > > (Ethernet cards, wireless cards, and network routers)
> > > I don't think you need to create a separate tree. ssb is pretty mature. There
> > > won't be that many patches.
> > 
> > OK, now I'm confused again -- should I take SSB patches, or is Ralf
> > going to do it? :-)
> 
> My decoder ring says, I'll do it from now on :)


Ok, nice. :)
Matthieu, can you resend them to Ralf? He doesn't have them, yet.

-- 
Greetings, Michael.

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[matthieu castet]

Michael Buesch wrote:
> On Tuesday 19 May 2009 00:10:07 matthieu castet wrote:
>> Hi,
>>
>> this patch export ssb_watchdog_timer_set to allow to use it in a Linux 
>> watchdog driver.
>>
>>
>> Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>
> 
> Well, you need to be careful. The watchdog is also used for system reboot.
> Make sure to disable the watchdog driver when the bcm47xx system code wants
> to use it.
> 
It shouldn't be a problem : the system code always disable irq before 
using watchdog to reboot/halt (unless I miss other watchdog usage).

Matthieu

Re: [PATCH] bc47xx : export ssb_watchdog_timer_set

[matthieu castet]

Hi,

Michael Buesch wrote:
> On Wednesday 20 May 2009 09:15:13 Ralf Baechle wrote:
> 
> 
> Ok, nice. :)
> Matthieu, can you resend them to Ralf? He doesn't have them, yet.
> 
Off course.

BTW what's the procedure for submitting patch on mips ?
I already CC linux-mips, on the first patch submission.
Now that it is acked by maintainer, I should resend it to linux-mips + 
Ralf ?


Thanks

Matthieu