CVS commit: src/sys/arch/sandpoint/sandpoint

2023-05-09 Thread Taylor R Campbell
Module Name:src Committed By: riastradh Date: Wed May 10 00:08:14 UTC 2023 Modified Files: src/sys/arch/sandpoint/sandpoint: flash_cfi.c Log Message: sandpoint: Use config_detach_children to reduce error branch bugs. To generate a diff of this commit: cvs rdiff -u -r1.4

CVS commit: src/sys/arch/sandpoint/sandpoint

2023-05-09 Thread Taylor R Campbell
Module Name:src Committed By: riastradh Date: Wed May 10 00:08:14 UTC 2023 Modified Files: src/sys/arch/sandpoint/sandpoint: flash_cfi.c Log Message: sandpoint: Use config_detach_children to reduce error branch bugs. To generate a diff of this commit: cvs rdiff -u -r1.4

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-30 Thread Frank Wille
David Young wrote: > You should be able to detach it so that you know that the driver can > return the hardware to initial conditions, and so that in a modular > universe you can unload the old driver software and load new. Ok, you're right. I forgot about the modules. I have reverted my changes.

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-24 Thread David Holland
On Tue, Jan 24, 2012 at 09:07:47PM +0100, Frank Wille wrote: > >> > > A detach function for the onboard flash probably makes no sense, > >> > > so remove it. > >> > > >> > Sure it makes sense. > >> > >> Hmm... how do you detach it? With drvctl(8)? Why should you do that? > > > > Power con

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-24 Thread Frank Wille
David Holland wrote: > On Tue, Jan 24, 2012 at 12:10:07PM +0100, Frank Wille wrote: >> > > A detach function for the onboard flash probably makes no sense, >> > > so remove it. >> > >> > Sure it makes sense. >> >> Hmm... how do you detach it? With drvctl(8)? Why should you do that? > > Power con

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-24 Thread David Holland
On Tue, Jan 24, 2012 at 12:10:07PM +0100, Frank Wille wrote: > > > A detach function for the onboard flash probably makes no sense, so > > > remove it. > > > > Sure it makes sense. > > Hmm... how do you detach it? With drvctl(8)? Why should you do that? Power consumption comes to mind...

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-24 Thread David Young
On Tue, Jan 24, 2012 at 12:10:07PM +0100, Frank Wille wrote: > On Mon, 23 Jan 2012 18:52:11 -0600 > David Young wrote: > > > > Log Message: > > > A detach function for the onboard flash probably makes no sense, so > > > remove it. > > > > Sure it makes sense. > > Hmm... how do you detach it? Wi

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-24 Thread Frank Wille
On Mon, 23 Jan 2012 18:52:11 -0600 David Young wrote: > > Log Message: > > A detach function for the onboard flash probably makes no sense, so > > remove it. > > Sure it makes sense. Hmm... how do you detach it? With drvctl(8)? Why should you do that? > Has it been tested, though? The detach

Re: CVS commit: src/sys/arch/sandpoint/sandpoint

2012-01-23 Thread David Young
On Mon, Jan 23, 2012 at 03:16:38PM +, Frank Wille wrote: > Module Name: src > Committed By: phx > Date: Mon Jan 23 15:16:38 UTC 2012 > > Modified Files: > src/sys/arch/sandpoint/sandpoint: flash_cfi.c > > Log Message: > A detach function for the onboard flash probably makes no