More IRQs missing from RP2350.svd

[lurch]

Similar to #1898 , but slightly different...

I recently noticed that Table 94 in https://datasheets.raspberrypi.com/rp2350/rp2350-datasheet.pdf (on page 83) lists IRQ numbers 40 and 41 as PROC0_IRQ_CTI and PROC1_IRQ_CTI, but these appear to be missing from the RP2350.svd.

[matiasilva]

The CTI enables chip debug. The debug hardware is not bus addressable, so I'm not sure it belongs in the SVD? We never added the RISC-V registers to the SVD, for example.

[lurch]

This is beyond my level of knowledge. Perhaps @liamfraser can provide more info?

[matiasilva]

I'll rename the existing virtual peripheral and add all IRQs there

[matiasilva]

And we get a peripheral called VIRTUAL that looks like this:

        <name>VIRTUAL</name>
        <baseAddress>0x00000000</baseAddress>
        <interrupt>
            <name>PROC0_IRQ_CTI</name>
            <value>40</value>
        </interrupt>
        <interrupt>
            <name>PROC1_IRQ_CTI</name>
            <value>41</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_0</name>
            <value>46</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_1</name>
            <value>47</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_2</name>
            <value>48</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_3</name>
            <value>49</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_4</name>
            <value>50</value>
        </interrupt>
        <interrupt>
            <name>SPARE_IRQ_5</name>
            <value>51</value>
        </interrupt>

It's reassuring that no other IRQs showed up in this list, meaning we haven't missed any others.

[matiasilva]

This is now generating internally and ready for SDK 2.2

[lurch]

We probably ought to mention this in the upcoming SDK release-notes, in case anybody was relying on the old "peripheral name" in the SVD?

[kilograham]

closing for tracking purposes as it will be in 2.2.0 even though it isn't in develop yet

[kilograham]

We probably ought to mention this in the upcoming SDK release-notes, in case anybody was relying on the old "peripheral name" in the SVD?

@matiasilva can you clarify what affect this change would have on existing users? what has changed from->to

[matiasilva]

@kilograham Before 2.2:

• PROC0_IRQ_CTI and PROC1_IRQ_CTI not in SVD
• SPARE_IRQ_ under a virtual (baseAddress = 0) peripheral called SPARE_IRQ

After 2.2:

• PROC0_IRQ_CTI, PROC1_IRQ_CTI and SPARE_IRQ_ all under one virtual peripheral called VIRTUAL