remove unused global vars

Don't use != 0 to check whether a pointer is non-NULL.

Constify struct cfattach.

ok visa@ a long time ago, ok patrick@

panic does not require a \n at the end. When one is provided, it looks wrong.

remove uneeded includes in md armv7 files

based on include-what-you-use suggestions

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Don't use != 0 to check whether a pointer is non-NULL.

Constify struct cfattach.

ok visa@ a long time ago, ok patrick@

panic does not require a \n at the end. When one is provided, it looks wrong.

remove uneeded includes in md armv7 files

based on include-what-you-use suggestions

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Constify struct cfattach.

ok visa@ a long time ago, ok patrick@

panic does not require a \n at the end. When one is provided, it looks wrong.

remove uneeded includes in md armv7 files

based on include-what-you-use suggestions

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

panic does not require a \n at the end. When one is provided, it looks wrong.

remove uneeded includes in md armv7 files

based on include-what-you-use suggestions

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

remove uneeded includes in md armv7 files

based on include-what-you-use suggestions

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Extend the interrupt API on arm64 and armv7 to be able to pass around
a struct cpu_info *. From a driver point of view the fdt_intr_establish_*
API now also exist same functions with a *_cpu suffix. Internally the
"old" functions now call their *_cpu counterparts, passing NULL as ci.
NULL will be interpreted as primary CPU in the interrupt controller code.

The internal framework for interrupt controllers has been changed so that
the establish methods provided by an interrupt controller function always
takes a struct cpu_info *.

Some drivers, like imxgpio(4) and rkgpio(4), only have a single interrupt
line for multiple pins. On those we simply disallow trying to establish
an interrupt on a non-primary CPU, returning NULL.

Since we do not have MP yet on armv7, all armv7 interrupt controllers do
return NULL if an attempt is made to establish an interrupt on a different
CPU. That said, so far there's no way this can happen. If we ever gain
MP support, this is a reminder that the interrupt controller drivers have
to be adjusted.

Prompted by dlg@
ok kettenis@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Use mask to extract IPL; should have been part of the earlier diff that
added IPI support on armv7.

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Whitespace removal

Pointed out by Jerome Pinot

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Remove unused include.

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@

Move Allwinner drivers that will soon be used by OpenBSD/arm64
to the new MI FDT driver directory.

"go ahead" kettenis@

Always allocate intrhand with M_WAITOK.

Requested by and ok kettenis@

We don't want drivers with numbers in their names.

Requested by deraadt@