[PATCH 01/13] [ARM] pxa/palm: Introduce Palm27x
Marek Vasut
marek.vasut at gmail.com
Thu Jul 29 11:54:02 EDT 2010
Dne Čt 29. července 2010 12:04:55 Mike Rapoport napsal(a):
> Marek Vasut wrote:
> > This contains common code for Palm LD, TX, T5, Z72, Treo680, Centro
> >
> > Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> > ---
> >
> > arch/arm/mach-pxa/Kconfig | 3 +
> > arch/arm/mach-pxa/Makefile | 1 +
> > arch/arm/mach-pxa/include/mach/palm27x.h | 48 +++
> > arch/arm/mach-pxa/palm27x.c | 475
> > ++++++++++++++++++++++++++++++ 4 files changed, 527 insertions(+), 0
> > deletions(-)
> > create mode 100644 arch/arm/mach-pxa/include/mach/palm27x.h
> > create mode 100644 arch/arm/mach-pxa/palm27x.c
> >
> > diff --git a/arch/arm/mach-pxa/Kconfig b/arch/arm/mach-pxa/Kconfig
> > index 21d1224..6f04f21 100644
> > --- a/arch/arm/mach-pxa/Kconfig
> > +++ b/arch/arm/mach-pxa/Kconfig
> > @@ -347,6 +347,9 @@ config ARCH_PXA_PALM
> >
> > bool "PXA based Palm PDAs"
> > select HAVE_PWM
> >
> > +config MACH_PALM27X
> > + bool
> > +
> >
> > config MACH_PALMTE2
> >
> > bool "Palm Tungsten|E2"
> > default y
> >
> > diff --git a/arch/arm/mach-pxa/Makefile b/arch/arm/mach-pxa/Makefile
> > index cb408ef..85c7fb3 100644
> > --- a/arch/arm/mach-pxa/Makefile
> > +++ b/arch/arm/mach-pxa/Makefile
> > @@ -75,6 +75,7 @@ obj-$(CONFIG_PXA_EZX) += ezx.o
> >
> > obj-$(CONFIG_MACH_MP900C) += mp900.o
> > obj-$(CONFIG_MACH_PALMTE2) += palmte2.o
> > obj-$(CONFIG_MACH_PALMTC) += palmtc.o
> >
> > +obj-$(CONFIG_MACH_PALM27X) += palm27x.o
> >
> > obj-$(CONFIG_MACH_PALMT5) += palmt5.o
> > obj-$(CONFIG_MACH_PALMTX) += palmtx.o
> > obj-$(CONFIG_MACH_PALMZ72) += palmz72.o
> >
> > diff --git a/arch/arm/mach-pxa/include/mach/palm27x.h
> > b/arch/arm/mach-pxa/include/mach/palm27x.h new file mode 100644
> > index 0000000..94ae6d9
> > --- /dev/null
> > +++ b/arch/arm/mach-pxa/include/mach/palm27x.h
>
> Does it have to be in arch/arm/mach-pxa/include/mach/? I'd move it to
> arch/arm/mach-pxa/ to reduce the header visibility
>
> > @@ -0,0 +1,48 @@
> > +/*
> > + * Common functions for Palm LD, T5, TX, Z72
> > + *
> > + * Copyright (C) 2010
> > + * Marek Vasut <marek.vasut at gmail.com>
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + *
> > + */
> > +
> > +struct palm27x_handheld {
> > + /* SD/MMC */
> > + int mmc_detect;
> > + int mmc_ro;
> > + int mmc_power;
> > +
> > + /* PM */
> > + unsigned long pm_str_base;
> > +
> > + /* UDC */
> > + int udc_detect;
> > + int udc_pullup;
> > +
> > + /* IrDA */
> > + int irda_pwdn;
> > +
> > + /* Battery */
> > + int batt_minv;
> > + int batt_maxv;
> > +
> > + /* Audio */
> > + int jack_gpio;
> > +
> > + /* Backlight */
> > + int bl_bl;
> > + int bl_lcd;
> > +
> > + /* Power supply */
> > + int power_ac;
> > + int power_usb;
> > +
> > + /* LCD */
> > + int lcd_power;
> > +};
> > +
> > +extern void __init palm27x_common_init(struct palm27x_handheld *);
> > diff --git a/arch/arm/mach-pxa/palm27x.c b/arch/arm/mach-pxa/palm27x.c
> > new file mode 100644
> > index 0000000..ff5ee8c
> > --- /dev/null
> > +++ b/arch/arm/mach-pxa/palm27x.c
> > @@ -0,0 +1,475 @@
> > +/*
> > + * Common code for Palm LD, T5, TX, Z72
> > + *
> > + * Copyright (C) 2010
> > + * Marek Vasut <marek.vasut at gmail.com>
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2 as
> > + * published by the Free Software Foundation.
> > + *
> > + */
> > +
> > +#include <linux/platform_device.h>
> > +#include <linux/delay.h>
> > +#include <linux/irq.h>
> > +#include <linux/gpio_keys.h>
> > +#include <linux/input.h>
> > +#include <linux/pda_power.h>
> > +#include <linux/pwm_backlight.h>
> > +#include <linux/gpio.h>
> > +#include <linux/wm97xx.h>
> > +#include <linux/power_supply.h>
> > +#include <linux/usb/gpio_vbus.h>
> > +
> > +#include <asm/mach-types.h>
> > +#include <asm/mach/arch.h>
> > +#include <asm/mach/map.h>
> > +
> > +#include <mach/pxa27x.h>
> > +#include <mach/audio.h>
> > +#include <mach/mmc.h>
> > +#include <mach/pxafb.h>
> > +#include <mach/irda.h>
> > +#include <mach/udc.h>
> > +#include <mach/palmasoc.h>
> > +#include <mach/palm27x.h>
> > +
> > +#include "generic.h"
> > +#include "devices.h"
> > +
> > +/***********************************************************************
> > ******* + * SD/MMC card controller
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_MMC_PXA) || defined(CONFIG_MMC_PXA_MODULE)
> > +static struct pxamci_platform_data palm27x_mci_platform_data = {
> > + .ocr_mask = MMC_VDD_32_33 | MMC_VDD_33_34,
> > + .detect_delay_ms = 200,
> > +};
> > +
> > +static void __init palm27x_mmc_init(int detect, int ro, int power)
> > +{
> > + palm27x_mci_platform_data.gpio_card_detect = detect;
> > + palm27x_mci_platform_data.gpio_card_ro = ro;
> > + palm27x_mci_platform_data.gpio_power = power;
> > +
> > + if (machine_is_palmz72() || machine_is_centro())
> > + palm27x_mci_platform_data.gpio_power_invert = 1;
> > +
> > + pxa_set_mci_info(&palm27x_mci_platform_data);
> > +}
> > +#else
> > +static inline void palm27x_mmc_init(int detect, int ro, int power) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * Power management - standby
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_SUSPEND)
> > +static void __init palm27x_pm_init(unsigned long str_base)
> > +{
> > + static const unsigned long resume[] = {
> > + 0xe3a00101, /* mov r0, #0x40000000 */
> > + 0xe380060f, /* orr r0, r0, #0x00f00000 */
> > + 0xe590f008, /* ldr pc, [r0, #0x08] */
> > + };
> > +
> > + /*
> > + * Copy the bootloader.
> > + * NOTE: PalmZ72 uses a different wakeup method!
> > + */
> > + if (!machine_is_palmz72())
> > + memcpy(phys_to_virt(str_base), resume, sizeof(resume));
> > +}
> > +#else
> > +static inline void palm27x_pm_init(unsigned long str_base) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * Framebuffer
> > +
> > ************************************************************************
> > ******/ +/*
> > + * NOTE: PalmZ72 has so called HiRes screen -- 320x320
> > + * The rest of devices has HiRes+ screen -- 320x480
> > + */
> > +#if defined(CONFIG_FB_PXA) || defined(CONFIG_FB_PXA_MODULE)
> > +static struct pxafb_mode_info palm27x_modes[] = {
> > + {
> > + .pixclock = 57692,
> > + .xres = 320,
> > + .yres = 480,
> > + .bpp = 16,
> > +
> > + .left_margin = 32,
> > + .right_margin = 1,
> > + .upper_margin = 7,
> > + .lower_margin = 1,
> > +
> > + .hsync_len = 4,
> > + .vsync_len = 1,
> > + }, {
> > + .pixclock = 115384,
> > + .xres = 320,
> > + .yres = 320,
> > + .bpp = 16,
> > +
> > + .left_margin = 27,
> > + .right_margin = 7,
> > + .upper_margin = 7,
> > + .lower_margin = 8,
> > +
> > + .hsync_len = 6,
> > + .vsync_len = 1,
> > + }, {
> > + .pixclock = 86538,
> > + .xres = 320,
> > + .yres = 320,
> > + .bpp = 16,
> > +
> > + .left_margin = 20,
> > + .right_margin = 8,
> > + .upper_margin = 8,
> > + .lower_margin = 5,
> > +
> > + .hsync_len = 4,
> > + .vsync_len = 1,
> > + }
> > +};
>
> I think that keeping three structures is more readable than the array, i.e
> static struct pxafb_mode_info palmz72_mode = {
> ...
> };
>
> static struct pxafb_mode_info palmz72_mode = {
> ...
> };
>
> static struct pxafb_mode_info treo_centro_mode = {
> ...
> };
>
> static struct pxafb_mode_info palmz72x_mode = {
> ...
> };
>
> > +static struct pxafb_mach_info palm27x_lcd_screen = {
> > + .num_modes = 1,
> > + .lcd_conn = LCD_COLOR_TFT_16BPP | LCD_PCLK_EDGE_FALL,
> > +};
> > +
> > +static int palm27x_lcd_power;
> > +static void palm27x_lcd_ctl(int on, struct fb_var_screeninfo *info)
> > +{
> > + gpio_set_value(palm27x_lcd_power, on);
> > +}
> > +
> > +static void __init palm27x_lcd_init(int power)
> > +{
> > + if (machine_is_palmz72())
> > + palm27x_lcd_screen.modes = &palm27x_modes[1];
> > + else if (machine_is_treo680() || machine_is_centro())
> > + palm27x_lcd_screen.modes = &palm27x_modes[2];
> > + else
> > + palm27x_lcd_screen.modes = &palm27x_modes[0];
> > +
> > + if (gpio_is_valid(power)) {
> > + if (!gpio_request(power, "LCD power")) {
> > + pr_err("Palm27x: failed to claim lcd power gpio!\n");
> > + return;
> > + }
> > + if (!gpio_direction_output(power, 1)) {
> > + pr_err("Palm27x: lcd power configuration failed!\n");
> > + return;
> > + }
> > + palm27x_lcd_power = power;
> > + palm27x_lcd_screen.pxafb_lcd_power = palm27x_lcd_ctl;
> > + }
> > +
> > + set_pxa_fb_info(&palm27x_lcd_screen);
> > +}
> > +
> > +#else
> > +static inline void palm27x_lcd_init(int power) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * USB Gadget
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_USB_GADGET_PXA27X) || \
> > + defined(CONFIG_USB_GADGET_PXA27X_MODULE)
> > +static struct gpio_vbus_mach_info palm27x_udc_info = {
> > + .gpio_vbus_inverted = 1,
> > +};
> > +
> > +static struct platform_device palm27x_gpio_vbus = {
> > + .name = "gpio-vbus",
> > + .id = -1,
> > + .dev = {
> > + .platform_data = &palm27x_udc_info,
> > + },
> > +};
> > +
> > +static void __init palm27x_udc_init(int vbus, int pullup)
> > +{
> > + if (machine_is_palmld())
> > + return;
> > +
> > + palm27x_udc_info.gpio_vbus = vbus;
> > + palm27x_udc_info.gpio_pullup = pullup;
> > +
> > + if (machine_is_palmz72())
> > + palm27x_udc_info.gpio_vbus_inverted = 0;
> > +
> > + if (!gpio_request(pullup, "USB Pullup")) {
> > + gpio_direction_output(pullup,
> > + palm27x_udc_info.gpio_vbus_inverted);
> > + gpio_free(pullup);
> > + } else
> > + return;
> > +
> > + platform_device_register(&palm27x_gpio_vbus);
> > +}
> > +#else
> > +static inline void palm27x_udc_init(int vbus, int pullup) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * IrDA
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_IRDA) || defined(CONFIG_IRDA_MODULE)
> > +static struct pxaficp_platform_data palm27x_ficp_platform_data = {
> > + .transceiver_cap = IR_SIRMODE | IR_OFF,
> > +};
> > +
> > +static void __init palm27x_irda_init(int pwdn)
> > +{
> > + palm27x_ficp_platform_data.gpio_pwdown = pwdn;
> > + pxa_set_ficp_info(&palm27x_ficp_platform_data);
> > +}
> > +#else
> > +static inline void palm27x_irda_init(int pwdn) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * WM97xx audio, battery
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_TOUCHSCREEN_WM97XX) || \
> > + defined(CONFIG_TOUCHSCREEN_WM97XX_MODULE)
> > +static struct wm97xx_batt_pdata palm27x_batt_pdata = {
> > + .batt_aux = WM97XX_AUX_ID3,
> > + .temp_aux = WM97XX_AUX_ID2,
> > + .charge_gpio = -1,
> > + .batt_mult = 1000,
> > + .batt_div = 414,
> > + .temp_mult = 1,
> > + .temp_div = 1,
> > + .batt_tech = POWER_SUPPLY_TECHNOLOGY_LIPO,
> > + .batt_name = "main-batt",
> > +};
> > +
> > +static struct wm97xx_pdata palm27x_wm97xx_pdata = {
> > + .batt_pdata = &palm27x_batt_pdata,
> > +};
> > +
> > +static pxa2xx_audio_ops_t palm27x_ac97_pdata = {
> > + .codec_pdata = { &palm27x_wm97xx_pdata, },
> > +};
> > +
> > +static struct palm27x_asoc_info palm27x_asoc_pdata = {
> > + .jack_gpio = -1,
> > +};
> > +
> > +static struct platform_device palm27x_asoc = {
> > + .name = "palm27x-asoc",
> > + .id = -1,
> > + .dev = {
> > + .platform_data = &palm27x_asoc_pdata,
> > + },
> > +};
> > +
> > +static void __init palm27x_ac97_init(int minv, int maxv, int jack)
> > +{
> > + if (!machine_is_palmz72())
> > + palm27x_ac97_pdata.reset_gpio = 95;
> > + else
> > + palm27x_asoc_pdata.jack_gpio = jack;
> > +
> > + if (machine_is_treo680() || machine_is_centro()) {
> > + palm27x_ac97_pdata.codec_pdata[0] = NULL;
> > + pxa_set_ac97_info(&palm27x_ac97_pdata);
> > + } else {
> > + palm27x_batt_pdata.min_voltage = minv,
> > + palm27x_batt_pdata.max_voltage = maxv,
> > +
> > + pxa_set_ac97_info(&palm27x_ac97_pdata);
> > + platform_device_register(&palm27x_asoc);
> > + }
> > +}
> > +#else
> > +static inline void palm27x_ac97_init(int minv, int maxv, int jack) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * Backlight
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_BACKLIGHT_PWM) ||
> > defined(CONFIG_BACKLIGHT_PWM_MODULE) +struct {
> > + int bl_power;
> > + int lcd_power;
> > +} palm27x_bl_info;
> > +
> > +static int palm27x_backlight_init(struct device *dev)
> > +{
> > + int ret;
> > +
> > + ret = gpio_request(palm27x_bl_info.bl_power, "BL POWER");
> > + if (ret)
> > + goto err;
> > + ret = gpio_direction_output(palm27x_bl_info.bl_power, 0);
> > + if (ret)
> > + goto err2;
> > +
> > + if (gpio_is_valid(palm27x_bl_info.lcd_power)) {
> > + ret = gpio_request(palm27x_bl_info.lcd_power, "LCD POWER");
> > + if (ret)
> > + goto err2;
> > + ret = gpio_direction_output(palm27x_bl_info.lcd_power, 0);
> > + if (ret)
> > + goto err3;
> > + }
> > +
> > + return 0;
> > +err3:
> > + gpio_free(palm27x_bl_info.lcd_power);
> > +err2:
> > + gpio_free(palm27x_bl_info.bl_power);
> > +err:
> > + return ret;
> > +}
> > +
> > +static int palm27x_backlight_notify(struct device *dev, int brightness)
> > +{
> > + gpio_set_value(palm27x_bl_info.bl_power, brightness);
> > + if (gpio_is_valid(palm27x_bl_info.lcd_power))
> > + gpio_set_value(palm27x_bl_info.lcd_power, brightness);
> > + return brightness;
> > +}
> > +
> > +static void palm27x_backlight_exit(struct device *dev)
> > +{
> > + gpio_free(palm27x_bl_info.bl_power);
> > + if (gpio_is_valid(palm27x_bl_info.lcd_power))
> > + gpio_free(palm27x_bl_info.lcd_power);
> > +}
> > +
> > +static struct platform_pwm_backlight_data palm27x_backlight_data = {
> > + .pwm_id = 0,
> > + .max_brightness = 0xfe,
> > + .dft_brightness = 0x7e,
> > + .pwm_period_ns = 3500,
> > + .init = palm27x_backlight_init,
> > + .notify = palm27x_backlight_notify,
> > + .exit = palm27x_backlight_exit,
> > +};
> > +
> > +static struct platform_device palm27x_backlight = {
> > + .name = "pwm-backlight",
> > + .dev = {
> > + .parent = &pxa27x_device_pwm0.dev,
> > + .platform_data = &palm27x_backlight_data,
> > + },
> > +};
> > +
> > +static void __init palm27x_pwm_init(int bl, int lcd)
> > +{
> > + palm27x_bl_info.bl_power = bl;
> > + palm27x_bl_info.lcd_power = lcd;
> > + platform_device_register(&palm27x_backlight);
> > +}
> > +#else
> > +static inline void palm27x_pwm_init(int bl, int lcd) {}
> > +#endif
> > +
> > +/***********************************************************************
> > ******* + * Power supply
> > +
> > ************************************************************************
> > ******/ +#if defined(CONFIG_PDA_POWER) ||
> > defined(CONFIG_PDA_POWER_MODULE) +struct {
> > + int ac_state;
> > + int usb_state;
> > +} palm27x_power_info;
> > +
> > +static int palm27x_power_supply_init(struct device *dev)
> > +{
> > + int ret;
> > +
> > + ret = gpio_request(palm27x_power_info.ac_state, "AC state");
> > + if (ret)
> > + goto err1;
> > + ret = gpio_direction_input(palm27x_power_info.ac_state);
> > + if (ret)
> > + goto err2;
> > +
> > + if (gpio_is_valid(palm27x_power_info.usb_state)) {
> > + ret = gpio_request(palm27x_power_info.usb_state, "USB state");
> > + if (ret)
> > + goto err2;
> > + ret = gpio_direction_input(palm27x_power_info.usb_state);
> > + if (ret)
> > + goto err3;
> > + }
> > +
> > + return 0;
> > +err3:
> > + gpio_free(palm27x_power_info.usb_state);
> > +err2:
> > + gpio_free(palm27x_power_info.ac_state);
> > +err1:
> > + return ret;
> > +}
> > +
> > +static void palm27x_power_supply_exit(struct device *dev)
> > +{
> > + gpio_free(palm27x_power_info.usb_state);
> > + gpio_free(palm27x_power_info.ac_state);
> > +}
> > +
> > +static int palm27x_is_ac_online(void)
> > +{
> > + return gpio_get_value(palm27x_power_info.ac_state);
> > +}
> > +
> > +static int palm27x_is_usb_online(void)
> > +{
> > + return !gpio_get_value(palm27x_power_info.usb_state);
> > +}
> > +static char *palm27x_supplicants[] = {
> > + "main-battery",
> > +};
> > +
> > +static struct pda_power_pdata palm27x_ps_info = {
> > + .init = palm27x_power_supply_init,
> > + .exit = palm27x_power_supply_exit,
> > + .is_ac_online = palm27x_is_ac_online,
> > + .is_usb_online = palm27x_is_usb_online,
> > + .supplied_to = palm27x_supplicants,
> > + .num_supplicants = ARRAY_SIZE(palm27x_supplicants),
> > +};
> > +
> > +static struct platform_device palm27x_power_supply = {
> > + .name = "pda-power",
> > + .id = -1,
> > + .dev = {
> > + .platform_data = &palm27x_ps_info,
> > + },
> > +};
> > +
> > +static void __init palm27x_power_init(int ac, int usb)
> > +{
> > + palm27x_power_info.ac_state = ac;
> > + palm27x_power_info.usb_state = usb;
> > + platform_device_register(&palm27x_power_supply);
> > +}
> > +#else
> > +static inline void palm27x_power_init(int ac, int usb) {}
> > +#endif
> > +
> > +void __init palm27x_common_init(struct palm27x_handheld *palm)
> > +{
> > + pxa_set_ffuart_info(NULL);
> > + pxa_set_btuart_info(NULL);
> > + pxa_set_stuart_info(NULL);
> > +
> > + palm27x_mmc_init(palm->mmc_detect, palm->mmc_ro, palm->mmc_power);
> > + palm27x_pm_init(palm->pm_str_base);
> > + palm27x_lcd_init(palm->lcd_power);
> > + palm27x_udc_init(palm->udc_detect, palm->udc_pullup);
> > + palm27x_irda_init(palm->irda_pwdn);
> > + palm27x_ac97_init(palm->batt_minv, palm->batt_maxv, palm->jack_gpio);
> > + palm27x_pwm_init(palm->bl_bl, palm->bl_lcd);
> > + palm27x_power_init(palm->power_ac, palm->power_usb);
> > +}
Hey Mike,
I'll push the changes you suggested in a separate patch to avoid breaking the
already existing code (I can't currently test this on all affected platforms,
but I should be able to do it once I get another palmld). Especially I'll have
to analyze the PMIC change.
Thanks for the review !
Cheers
More information about the linux-arm-kernel
mailing list