c815fa4511
This patch permits interrupts to be generated by both the I2C and GPIO controllers for simultaneously-executing applications. The controllers share a single interrupt pin, INTC. Prior to this patch, quarkX1000_gpio_init() routed INTA to PIRQC and IRQ 10 (due to an incorrect assumption that INTA is connected to the GPIO controller), and quarkX1000_i2c_init() routed INTC to PIRQC and IRQ 9. The I2C controller initialization is a prerequisite for GPIO initialization, so the final configuration was that INTA and INTC were both routed to PIRQC and IRQ 10. Thus, only the GPIO ISR was being invoked, even if the I2C controller was actually responsible for the interrupt. This patch refactors the I2C and GPIO ISR setup and handler code so that the shared portions are combined in cpu/x86/drivers/legacy_pc/shared-isr.[ch]. The I2C and GPIO drivers communicate their interrupt information to the shared component by placing structures in a specific section of the binary.
68 lines
2.9 KiB
C
68 lines
2.9 KiB
C
/*
|
|
* Copyright (C) 2016, Intel Corporation. All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions
|
|
* are met:
|
|
* 1. Redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer.
|
|
* 2. Redistributions in binary form must reproduce the above copyright
|
|
* notice, this list of conditions and the following disclaimer in the
|
|
* documentation and/or other materials provided with the distribution.
|
|
*
|
|
* 3. Neither the name of the copyright holder nor the names of its
|
|
* contributors may be used to endorse or promote products derived
|
|
* from this software without specific prior written permission.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
|
|
* ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
|
|
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
|
|
* FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
|
|
* COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
|
|
* INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
|
|
* (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
|
|
* SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
|
|
* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
|
|
* STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
|
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
|
|
* OF THE POSSIBILITY OF SUCH DAMAGE.
|
|
*/
|
|
|
|
#ifndef CPU_X86_DRIVERS_LEGACY_PC_SHARED_ISR_H_
|
|
#define CPU_X86_DRIVERS_LEGACY_PC_SHARED_ISR_H_
|
|
|
|
#include <stdbool.h>
|
|
#include "pci.h"
|
|
|
|
/**
|
|
* The handler function should return true if and only if it handled the
|
|
* interrupt.
|
|
*/
|
|
typedef bool (*shared_isr_handler_t)(void);
|
|
|
|
typedef struct shared_isr_client {
|
|
uint8_t irq;
|
|
IRQAGENT agent;
|
|
INTR_PIN pin;
|
|
PIRQ pirq;
|
|
shared_isr_handler_t handler;
|
|
} shared_isr_client_t;
|
|
|
|
/* Unlike a non-shared interrupt handler function, an individual interrupt
|
|
* handler for a shared interrupt must not issue an EOI. The EOI is issued by
|
|
* the shared-isr subsystem.
|
|
*/
|
|
#define DEFINE_SHARED_IRQ(irq_, agent_, pin_, pirq_, handler_) \
|
|
static struct shared_isr_client \
|
|
__attribute__((used, section(".shared_isr_data"))) _shared_irq_##irq_ = { \
|
|
.irq = irq_, \
|
|
.agent = agent_, \
|
|
.pin = pin_, \
|
|
.pirq = pirq_, \
|
|
.handler = handler_ \
|
|
}
|
|
|
|
void shared_isr_init(void);
|
|
|
|
#endif /* CPU_X86_DRIVERS_LEGACY_PC_SHARED_ISR_H_ */
|