Bug #223

use bus_dma in i386/isa/clock.c

Added by corecode over 8 years ago. Updated 7 months ago.

Status:ClosedStart date:
Priority:LowDue date:
Assignee:tuxillo% Done:

0%

Category:Kernel
Target version:3.8.0

Description

we need to rewrite clock.c to use bus_dma to reduce warnings

History

#1 Updated by tuxillo about 5 years ago

Are you referring to sys/platform/pc32/isa/clock.c?
Can you confirm this is still needed?

#2 Updated by corecode about 5 years ago

yes, still needed

#3 Updated by tuxillo 8 months ago

  • Category set to Kernel
  • Status changed from New to Feedback
  • Assignee deleted (0)
  • Target version set to 3.8.0

Hi,

Can we have a bit more information what's exactly needed here please?

Cheers,
Antonio Huete

#4 Updated by tuxillo 7 months ago

  • Assignee set to tuxillo

Apparently related to build warnings. Grabbing.

#5 Updated by tuxillo 7 months ago

  • Status changed from Feedback to Closed

Hi,

There are no warnings at all. In fact it is built with -Werror:

[...]
cc -c -O -pipe -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wold-style-definition -Wold-style-declaration -Wno-inline -Wno-pointer-sign -std=c99 -Wno-unused-but-set-variable -g -nostdinc -I. -Iinclude -I/usr/src/sys -I/usr/src/sys/../include -I/usr/obj/usr/src/sys/GENERIC -I"/usr/src/sys/dev/acpica" -I"/usr/src/sys/contrib/dev/acpica-unix/include" -I/usr/src/sys/dev/netif/ath/hal -I/usr/src/sys/dev/netif/ath/hal/ath_hal -I/usr/src/sys/dev/drm/include -D_KERNEL -include opt_global.h -finline-limit=8000 --param inline-unit-growth=100 --param large-function-growth=1000 -fno-common -ffreestanding -mpreferred-stack-boundary=2 -fno-stack-protector -mno-mmx -mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -mno-ssse3 -mno-sse4.1 -mno-sse4.2 -mno-sse4 -mno-sse4a -mno-sse5 -mno-abm -mno-aes -mno-avx -mno-pclmul -mno-popcnt -mno-avx2 -mno-fsgsbase -mno-rdrnd -mno-f16c -mno-fma -mno-fma4 -mno-bmi -mno-bmi2 -mno-xop -mno-lwp -mno-lzcnt -mno-tbm -msoft-float /usr/src/sys/platform/pc32/isa/clock.c -Werror
[...]

Closing this one.

Cheers,
Antonio Huete

Also available in: Atom PDF