Hi Bin,
On 21 August 2015 at 01:18, Bin Meng <bmeng...@gmail.com> wrote:
> Add an api to enable and configure the integrated keyboard controller
> on SMSC LPC47m superio chipset. It also adds several macros to help
> future extension.
>
> Signed-off-by: Bin Meng <bmeng...@gmail.com>
>
> ---
>
> Changes in v2:
> - Change parameter type to uint
>
> arch/x86/include/asm/ibmpc.h | 3 +++
> drivers/misc/smsc_lpc47m.c | 13 ++++++++++++-
> include/smsc_lpc47m.h | 29 +++++++++++++++++++++++++----
> 3 files changed, 40 insertions(+), 5 deletions(-)
>
Acked-by: Simon Glass <s...@chromium.org>
> diff --git a/arch/x86/include/asm/ibmpc.h b/arch/x86/include/asm/ibmpc.h
> index 678cde4..1e9058c 100644
> --- a/arch/x86/include/asm/ibmpc.h
> +++ b/arch/x86/include/asm/ibmpc.h
> @@ -24,4 +24,7 @@
> #define UART0_IRQ 4
> #define UART1_IRQ 3
>
> +#define KBD_IRQ 1
> +#define MSE_IRQ 12
> +
> #endif
> diff --git a/drivers/misc/smsc_lpc47m.c b/drivers/misc/smsc_lpc47m.c
> index 1e50d5b..fcce3a4 100644
> --- a/drivers/misc/smsc_lpc47m.c
> +++ b/drivers/misc/smsc_lpc47m.c
> @@ -22,7 +22,7 @@ static void pnp_exit_conf_state(u16 dev)
> outb(0xaa, port);
> }
>
> -void lpc47m_enable_serial(u16 dev, u16 iobase, u8 irq)
> +void lpc47m_enable_serial(uint dev, uint iobase, uint irq)
For future reference: I think a tidy-up patch should be separate from
adding a new feature - i.e. this sort of patch should really be split
into two. But it's only a minor point.
> {
> pnp_enter_conf_state(dev);
> pnp_set_logical_device(dev);
> @@ -32,3 +32,14 @@ void lpc47m_enable_serial(u16 dev, u16 iobase, u8 irq)
> pnp_set_enable(dev, 1);
> pnp_exit_conf_state(dev);
> }
> +
> +void lpc47m_enable_kbc(uint dev, uint irq0, uint irq1)
> +{
> + pnp_enter_conf_state(dev);
> + pnp_set_logical_device(dev);
> + pnp_set_enable(dev, 0);
> + pnp_set_irq(dev, PNP_IDX_IRQ0, irq0);
> + pnp_set_irq(dev, PNP_IDX_IRQ1, irq1);
> + pnp_set_enable(dev, 1);
> + pnp_exit_conf_state(dev);
> +}
> diff --git a/include/smsc_lpc47m.h b/include/smsc_lpc47m.h
> index 32b069d..419643f 100644
> --- a/include/smsc_lpc47m.h
> +++ b/include/smsc_lpc47m.h
> @@ -7,14 +7,35 @@
> #ifndef _SMSC_LPC47M_H_
> #define _SMSC_LPC47M_H_
>
> +/* I/O address of LPC47M */
> +#define LPC47M_IO_PORT 0x2e
> +
> +/* Logical device number */
> +#define LPC47M_FDC 0 /* Floppy */
> +#define LPC47M_SP2 2 /* Serial Port 2 */
> +#define LPC47M_PP 3 /* Parallel Port */
> +#define LPC47M_SP1 4 /* Serial Port 1 */
> +#define LPC47M_KBC 7 /* Keyboard & Mouse */
> +#define LPC47M_PME 10 /* Power Control */
> +
> /**
> * Configure the base I/O port of the specified serial device and enable the
> * serial device.
> *
> - * @dev: High 8 bits = Super I/O port, low 8 bits = logical device number.
> - * @iobase: Processor I/O port address to assign to this serial device.
> - * @irq: Processor IRQ number to assign to this serial device.
> + * @dev: high 8 bits = super I/O port, low 8 bits = logical device number
> + * @iobase: processor I/O port address to assign to this serial device
> + * @irq: processor IRQ number to assign to this serial device
> + */
> +void lpc47m_enable_serial(uint dev, uint iobase, uint irq);
> +
> +/**
> + * Configure the specified keyboard controller device and enable the keyboard
> + * controller device.
> + *
> + * @dev: high 8 bits = Super I/O port, low 8 bits = logical device number
> + * @irq0: processor IRQ number to assign to keyboard
> + * @irq1: processor IRQ number to assign to mouse
> */
> -void lpc47m_enable_serial(u16 dev, u16 iobase, u8 irq);
> +void lpc47m_enable_kbc(uint dev, uint irq0, uint irq1);
>
> #endif /* _SMSC_LPC47M_H_ */
> --
> 1.8.2.1
>
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
