CVS log for src/sys/arch/evbarm/conf/GENERIC

Up to [cvs.NetBSD.org] / src / sys / arch / evbarm / conf

Request diff between arbitrary revisions

Default branch: MAIN
Current tag: netbsd-9-3-RELEASE

Revision 1.48.2.8 / (download) - annotate - [select for diffs], Mon May 18 18:12:25 2020 UTC (3 years, 11 months ago) by martin
Branch: netbsd-9
CVS Tags: netbsd-9-3-RELEASE, netbsd-9-2-RELEASE, netbsd-9-1-RELEASE
Changes since 1.48.2.7: +1 -2 lines
Diff to previous 1.48.2.7 (colored) to branchpoint 1.48 (colored) next main 1.49 (colored)

Pull up following revision(s) (requested by isaki in ticket #911):

	sys/dev/audio/audiodef.h: revision 1.13
	sys/dev/audio/audiodef.h: revision 1.14
	sys/arch/sparc/include/param.h: revision 1.74
	sys/arch/evbarm/conf/GENERIC64: revision 1.148
	sys/arch/sh3/include/param.h: revision 1.25
	sys/arch/evbarm/conf/GENERIC: revision 1.75
	sys/arch/m68k/include/param.h: revision 1.23
	sys/arch/amd64/conf/GENERIC: revision 1.564
	sys/arch/hppa/include/param.h: revision 1.27
	sys/dev/audio/audio.c: revision 1.68
	sys/arch/vax/include/param.h: revision 1.63
	sys/dev/audio/audio.c: revision 1.69

Reduce default AUDIO_BLK_MS from 40msec to 10msec on all platform except m68k
(m68k uses 40msec default as before).  And remove the option from GENERIC.

- It's not good idea to set such parameter in individual GENERICs.
- 4msec is (probably no problem for most modern real hardware but)
  too aggressive to be default.
- 10msec is too severe for antique machines but it's hard to draw a line.

Set AUDIO_BLK_MS 40 msec on other old(slow) architectures not only m68k.

Thanks tsutsui@ for comment about architecture choice.
And move it from audiodef.h to audio.c as suggested by joerg@.


Move machine dependent AUDIO_BLK_MS default value to <machine/param.h>.
If the port has __AUDIO_BLK_MS in <machine/param.h>, it will be used.

Otherwise the default value (currently 10 msec) defined in audio.c will
be used.  This mechanism is for very old ports which cannot satisfactorily
handle 10 msec block.  Currently hppa, m68k, sh3, sparc(!64) and vax are.

For port maintainers, if general models in your port cannot satisfactorily
handle 10 msec block, please consider to define your suitable longer period
(40 msec would be a good first choice).

But please don't be eager to make the default value shorter.
<machine/param.h> was discussed in source-changes-d.  It's better than
ifdef storm, or adding 60+ new header files in every arch/*/include/
directories for this.  Thanks mrg@, ad@, and everyone.
http://mail-index.netbsd.org/source-changes-d/2020/05/01/msg012572.html

This form allows you to request diff's between any two revisions of a file. You may select a symbolic revision name using the selection box or you may type in a numeric name using the type-in text box.