Use GIC-specific decoding function in mv_common.c

[bsz_semihalf.com]

Summary

Add gic_decode_fdt function to fdt_pic_table, allowing to recognize GIC
interrupts on Armada38x. SOC_MV_ARMADA38X ifdef is required because A38X
is the only Marvell's platform in FreeBSD using GIC; lack of ifdef would
lead to linking errors on other platforms.

Obtained from: Semihalf
Sponsored by: Stormshield
Submitted by: Michal Stanek <[e-mail]>

[• ian]

I think you wouldn't need this if you enabled ARM_INTRNG for the platform, wouldn't that be a better way to go?

[bsz_semihalf.com]

In D4214#94222, @ian wrote:

I think you wouldn't need this if you enabled ARM_INTRNG for the platform, wouldn't that be a better way to go?

I'm working on ARM_INTRNG on this platform on current, it's almost ready. So it's up to you if you want this commit in tree. Personally, I think it's better to have a fallback solution working, but that's not really required.

[andrew]

sys/arm/mv/mv_common.c

2187–2190 ↗

(On Diff #10342)

Why not add gic_decode_fdt to fdt_pic_table?

[bsz_semihalf.com]

Added gic_decode_fdt to fdt_pic_table instead.

[• ian]

In D4214#94387, @bsz_semihalf.com wrote:

In D4214#94222, @ian wrote:

I think you wouldn't need this if you enabled ARM_INTRNG for the platform, wouldn't that be a better way to go?

I'm working on ARM_INTRNG on this platform on current, it's almost ready. So it's up to you if you want this commit in tree. Personally, I think it's better to have a fallback solution working, but that's not really required.

Good point, having a fallback is good idea, and easy to do.