[tip:irq/core] genirq: Fix /proc/interrupts output alignment

2017-02-10 Thread tip-bot for H Hartley Sweeten
Commit-ID: f435da416beaacc8934fc21820d9488269b39c98 Gitweb: http://git.kernel.org/tip/f435da416beaacc8934fc21820d9488269b39c98 Author: H Hartley Sweeten <hswee...@visionengravers.com> AuthorDate: Fri, 10 Feb 2017 09:54:16 -0700 Committer: Thomas Gleixner <t...@linutronix.de>

[tip:irq/core] genirq: Fix /proc/interrupts output alignment

2017-02-10 Thread tip-bot for H Hartley Sweeten
Commit-ID: f435da416beaacc8934fc21820d9488269b39c98 Gitweb: http://git.kernel.org/tip/f435da416beaacc8934fc21820d9488269b39c98 Author: H Hartley Sweeten AuthorDate: Fri, 10 Feb 2017 09:54:16 -0700 Committer: Thomas Gleixner CommitDate: Fri, 10 Feb 2017 20:17:52 +0100 genirq: Fix /proc

[PATCH] getirq: fix /proc/interrupts output alignment

2017-02-10 Thread H Hartley Sweeten
If the irq_desc being output does not have a domain the information following the 'name' is not aligned correctly. Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com> Cc: Thomas Gleixner <t...@linutronix.de> --- kernel/irq/proc.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH] getirq: fix /proc/interrupts output alignment

2017-02-10 Thread H Hartley Sweeten
If the irq_desc being output does not have a domain the information following the 'name' is not aligned correctly. Signed-off-by: H Hartley Sweeten Cc: Thomas Gleixner --- kernel/irq/proc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/kernel/irq/proc.c b/kernel/irq/proc.c index

[PATCH 0/2] watchdog: cleanup ep93xx platform drivers

2017-01-30 Thread H Hartley Sweeten
The ep93xx_wdt driver is used by EP93xx based platforms for the internal watchdog of the EP93xx processor. The TS-72xx platforms have an additional watchdog provided by the CPLD on those boards. Cleanup both drivers. H Hartley Sweeten (2): watchdog: ep93xx_wdt: cleanup and let the core handle

[PATCH 0/2] watchdog: cleanup ep93xx platform drivers

2017-01-30 Thread H Hartley Sweeten
The ep93xx_wdt driver is used by EP93xx based platforms for the internal watchdog of the EP93xx processor. The TS-72xx platforms have an additional watchdog provided by the CPLD on those boards. Cleanup both drivers. H Hartley Sweeten (2): watchdog: ep93xx_wdt: cleanup and let the core handle

[PATCH 1/2] watchdog: ep93xx_wdt: cleanup and let the core handle the heartbeat

2017-01-30 Thread H Hartley Sweeten
Cleanup this driver and remove the 200ms heartbeat timer. The core now has the ability to handle the heartbeat. Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com> Cc: Wim Van Sebroeck <w...@iguana.be> Cc: Guenter Roeck <li...@roeck-us.net> --- drivers/watchdog/

[PATCH 1/2] watchdog: ep93xx_wdt: cleanup and let the core handle the heartbeat

2017-01-30 Thread H Hartley Sweeten
Cleanup this driver and remove the 200ms heartbeat timer. The core now has the ability to handle the heartbeat. Signed-off-by: H Hartley Sweeten Cc: Wim Van Sebroeck Cc: Guenter Roeck --- drivers/watchdog/ep93xx_wdt.c | 115 +- 1 file changed, 46

[PATCH 2/2] watchdog: ts72xx_wdt: convert driver to watchdog core

2017-01-30 Thread H Hartley Sweeten
Cleanup this driver and convert it to use the watchdog framework API. Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com> Cc: Mika Westerberg <mika.westerb...@iki.fi> Cc: Wim Van Sebroeck <w...@iguana.be> Cc: Guenter Roeck <li...@roeck-us.net> --- drivers/wat

[PATCH 2/2] watchdog: ts72xx_wdt: convert driver to watchdog core

2017-01-30 Thread H Hartley Sweeten
Cleanup this driver and convert it to use the watchdog framework API. Signed-off-by: H Hartley Sweeten Cc: Mika Westerberg Cc: Wim Van Sebroeck Cc: Guenter Roeck --- drivers/watchdog/ts72xx_wdt.c | 447 +- 1 file changed, 93 insertions(+), 354

[PATCH 2/2 v5] usb: ohci: remove ep93xx bus glue platform driver

2013-10-21 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten Acked-by: Alan Stern Cc: Ryan Mallon Cc: Lennert

[PATCH 2/2 v5] usb: ohci: remove ep93xx bus glue platform driver

2013-10-21 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Alan Stern st

[PATCH 1/2 v4] ARM: ep93xx_defconfig: cleanup ep93xx_defconfig

2013-10-17 Thread H Hartley Sweeten
Generate ep93xx_defconfig by doing: make ep93xx_defconfig make savedefconfig mv defconfig arch/arm/configs/ep93xx_defconfig No function change. This just refreshes the ep93xx_defconfig to make it easier and cleaner when adding new entries. Signed-off-by: H Hartley Sweeten Acked-by: Ryan Mallon

[PATCH 2/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten Acked-by: Alan Stern Cc: Ryan Mallon Cc: Lennert

[PATCH 0/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
of the Generic OHCI driver for a platform device. v2: Use the (*power_off) callback for (*power_suspend), as suggested by Alan Stern. Remove the Kconfig change to USB_OHCI_HCD_PLATFORM and refresh the ep93xx_defconfig to enable this option, as suggested by Alan Stern. H Hartley

[PATCH 2/2 v3] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Alan Stern Cc: Lennert

[PATCH 1/2 v3] ARM: ep93xx_defconfig: cleanup ep93xx_defconfig

2013-10-17 Thread H Hartley Sweeten
Generate ep93xx_defconfig by doing: make ep93xx_defconfig make savedefconfig mv defconfig arch/arm/configs/ep93xx_defconfig No functional change. This just refreshes the ep93xx_defconfig to make it easier and cleaner when adding new entries. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc

[PATCH 0/2 v3] usb: ohci: remove ep93xx bus glue driver

2013-10-17 Thread H Hartley Sweeten
the (*power_off) callback for (*power_suspend), as suggested by Alan Stern. Remove the Kconfig change to USB_OHCI_HCD_PLATFORM and refresh the ep93xx_defconfig to enable this option, as suggested by Alan Stern. H Hartley Sweeten (2): ARM: ep93xx_defconfig: cleanup ep93xx_defconfig usb

[PATCH 0/2 v3] usb: ohci: remove ep93xx bus glue driver

2013-10-17 Thread H Hartley Sweeten
the (*power_off) callback for (*power_suspend), as suggested by Alan Stern. Remove the Kconfig change to USB_OHCI_HCD_PLATFORM and refresh the ep93xx_defconfig to enable this option, as suggested by Alan Stern. H Hartley Sweeten (2): ARM: ep93xx_defconfig: cleanup ep93xx_defconfig usb

[PATCH 1/2 v3] ARM: ep93xx_defconfig: cleanup ep93xx_defconfig

2013-10-17 Thread H Hartley Sweeten
Generate ep93xx_defconfig by doing: make ep93xx_defconfig make savedefconfig mv defconfig arch/arm/configs/ep93xx_defconfig No functional change. This just refreshes the ep93xx_defconfig to make it easier and cleaner when adding new entries. Signed-off-by: H Hartley Sweeten hswee

[PATCH 2/2 v3] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal

[PATCH 0/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
of the Generic OHCI driver for a platform device. v2: Use the (*power_off) callback for (*power_suspend), as suggested by Alan Stern. Remove the Kconfig change to USB_OHCI_HCD_PLATFORM and refresh the ep93xx_defconfig to enable this option, as suggested by Alan Stern. H Hartley

[PATCH 2/2 v4] usb: ohci: remove ep93xx bus glue platform driver

2013-10-17 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Enable CONFIG_OHCI_HCD_PLATFORM in the ep93xx_defconfig so that USB is still enabled by default on the EP93xx platform. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Alan Stern st

[PATCH 1/2 v4] ARM: ep93xx_defconfig: cleanup ep93xx_defconfig

2013-10-17 Thread H Hartley Sweeten
Generate ep93xx_defconfig by doing: make ep93xx_defconfig make savedefconfig mv defconfig arch/arm/configs/ep93xx_defconfig No function change. This just refreshes the ep93xx_defconfig to make it easier and cleaner when adding new entries. Signed-off-by: H Hartley Sweeten hswee

[PATCH v2] pwm: add ep93xx PWM support

2013-10-15 Thread H Hartley Sweeten
= c2port/ diff --git a/drivers/misc/ep93xx_pwm.c b/drivers/misc/ep93xx_pwm.c deleted file mode 100644 index cdb67a9..000 --- a/drivers/misc/ep93xx_pwm.c +++ /dev/null @@ -1,286 +0,0 @@ -/* - * Simple PWM driver for EP93XX - * - * (c) Copyright 2009 Matthieu Crapet - * (c) Copyright 200

[PATCH v2] pwm: add ep93xx PWM support

2013-10-15 Thread H Hartley Sweeten
for EP93XX - * - * (c) Copyright 2009 Matthieu Crapet mcra...@gmail.com - * (c) Copyright 2009 H Hartley Sweeten hswee...@visionengravers.com - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License

[PATCH] usb: ohci: remove ep93xx bus glue platform driver

2013-10-14 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Greg Kroah-Hartman Cc: Ryan Mallon --- arch/arm/mach-ep93xx/clock.c | 2 +- arch/arm/mach-ep93xx/core.c| 45 +- drivers/usb/host

[PATCH] pwm: add ep93xx PWM support

2013-10-14 Thread H Hartley Sweeten
Remove the non-standard EP93xx pwm driver in drivers/misc and add a new driver for the PWM chips on the EP93xx platforms based on the PWM framework. These PWM chips each support 1 PWM channel with programmable duty cycle, frequency, and polarity inversion. Signed-off-by: H Hartley Sweeten Cc

[PATCH] pwm: add ep93xx PWM support

2013-10-14 Thread H Hartley Sweeten
Remove the non-standard EP93xx pwm driver in drivers/misc and add a new driver for the PWM chips on the EP93xx platforms based on the PWM framework. These PWM chips each support 1 PWM channel with programmable duty cycle, frequency, and polarity inversion. Signed-off-by: H Hartley Sweeten hswee

[PATCH] usb: ohci: remove ep93xx bus glue platform driver

2013-10-14 Thread H Hartley Sweeten
Convert ep93xx to use the OHCI platform driver and remove the ohci-ep93xx bus glue driver. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Alan Stern st...@rowland.harvard.edu Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: Ryan Mallon rmal...@gmail.com --- arch/arm/mach

RE: [PATCH] gpio: (gpio-pca953x) move header to linux/platform_data/

2013-08-16 Thread H Hartley Sweeten
On Friday, August 16, 2013 8:02 AM, Vivien Didelot wrote: > Hi Linus, > > You wrote: > > But: >> >>> drivers/gpio/gpio-pca953x.c | 2 +- >> (...) >>> diff --git a/drivers/gpio/gpio-pca953x.c >>> b/drivers/gpio/gpio-pca953x.c >>> index 426c51d..8804aec 100644 >>> ---

RE: [PATCH] gpio: (gpio-pca953x) move header to linux/platform_data/

2013-08-16 Thread H Hartley Sweeten
On Friday, August 16, 2013 8:02 AM, Vivien Didelot wrote: Hi Linus, You wrote: But: drivers/gpio/gpio-pca953x.c | 2 +- (...) diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c index 426c51d..8804aec 100644 --- a/drivers/gpio/gpio-pca953x.c +++

RE: [PATCH] gpio: (gpio-pca953x) move header to linux/platform_data/

2013-07-31 Thread H Hartley Sweeten
9307.c| 2 +- For the ep93xx change: Acked-by: H Hartley Sweeten Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read th

RE: [PATCH] gpio: (gpio-pca953x) move header to linux/platform_data/

2013-07-31 Thread H Hartley Sweeten
/vision_ep9307.c| 2 +- For the ep93xx change: Acked-by: H Hartley Sweeten hswee...@visionengravers.com Thanks -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

RE: [PATCH v2 0/4] usb: ohci-ep93xx: do a bit of housecleaning

2013-07-26 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:52 PM, H Hartley Sweeten wrote: > Tidy up this driver a bit. > > v2: redo patch 4 based on suggestions from Alan Stern > > H Hartley Sweeten (4): > usb: ohci-ep93xx: use devm_ioremap_resource() > usb: ohci-ep93xx: use platform_get_irq() >

RE: [PATCH v2 0/4] usb: ohci-ep93xx: do a bit of housecleaning

2013-07-26 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:52 PM, H Hartley Sweeten wrote: Tidy up this driver a bit. v2: redo patch 4 based on suggestions from Alan Stern H Hartley Sweeten (4): usb: ohci-ep93xx: use devm_ioremap_resource() usb: ohci-ep93xx: use platform_get_irq() usb: ohci-ep93xx: use devm_clk_get

RE: [PATCH 22/36] staging: comedi: convert class code to use dev_groups

2013-07-24 Thread H Hartley Sweeten
On Wednesday, July 24, 2013 3:05 PM, Greg Kroah-Hartman wrote: > > The dev_attrs field of struct class is going away soon, dev_groups > should be used instead. This converts the comedi class code to use the > correct field. > > Cc: Ian Abbott > Cc: H Hartley Sweeten > S

RE: [PATCH 22/36] staging: comedi: convert class code to use dev_groups

2013-07-24 Thread H Hartley Sweeten
On Wednesday, July 24, 2013 3:05 PM, Greg Kroah-Hartman wrote: The dev_attrs field of struct class is going away soon, dev_groups should be used instead. This converts the comedi class code to use the correct field. Cc: Ian Abbott abbo...@mev.co.uk Cc: H Hartley Sweeten hswee

[PATCH] ASoC: ep93xx: fix build of ep93xx-ac97.c

2013-07-19 Thread H Hartley Sweeten
the member 'port'. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Lars-Peter Clausen Cc: Mark Brown Cc: Liam Girdwood Cc: Jaroslav Kysela Cc: Takashi Iwai --- sound/soc/cirrus/ep93xx-ac97.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/cirrus/ep93xx-ac97

[PATCH] ASoC: ep93xx: fix build of ep93xx-ac97.c

2013-07-19 Thread H Hartley Sweeten
the member 'port'. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Lars-Peter Clausen l...@metafoo.de Cc: Mark Brown broo...@kernel.org Cc: Liam Girdwood lgirdw...@gmail.com Cc: Jaroslav Kysela pe...@perex.cz Cc: Takashi Iwai ti...@suse.de --- sound

[PATCH v2 11/11] spi: spi-ep93xx: use master->cur_msg for in-flight message

2013-07-08 Thread H Hartley Sweeten
Instead of carrying the in-flight message in the driver private data, use the cur_msg pointer that is already setup by the core. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 88

RE: [PATCH v2 09/11] spi: spi-ep93xx: move the clock divider calcs into ep93xx_spi_chip_setup()

2013-07-08 Thread H Hartley Sweeten
On Monday, July 08, 2013 5:10 AM, Mark Brown wrote: > On Tue, Jul 02, 2013 at 10:10:29AM -0700, H Hartley Sweeten wrote: >> The divider values stored in the per chip data are only used to set the >> registers in the hardware to generate the desired SPI clock. Since these >&g

[PATCH v2 10/11] spi: spi-ep93xx: convert to the queued driver infrastructure

2013-07-08 Thread H Hartley Sweeten
The SPI core provides infrastructure for standard message queueing. Use that instead of handling it in the driver. Signed-off-by: H Hartley Sweeten Acked-by: Mika Westerberg Cc: Ryan Mallon Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 165

[PATCH v2 10/11] spi: spi-ep93xx: convert to the queued driver infrastructure

2013-07-08 Thread H Hartley Sweeten
The SPI core provides infrastructure for standard message queueing. Use that instead of handling it in the driver. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Mika Westerberg mika.westerb...@iki.fi Cc: Ryan Mallon rmal...@gmail.com Cc: Mark Brown broo...@kernel.org Cc

RE: [PATCH v2 09/11] spi: spi-ep93xx: move the clock divider calcs into ep93xx_spi_chip_setup()

2013-07-08 Thread H Hartley Sweeten
On Monday, July 08, 2013 5:10 AM, Mark Brown wrote: On Tue, Jul 02, 2013 at 10:10:29AM -0700, H Hartley Sweeten wrote: The divider values stored in the per chip data are only used to set the registers in the hardware to generate the desired SPI clock. Since these are calculated per transfer

[PATCH v2 11/11] spi: spi-ep93xx: use master-cur_msg for in-flight message

2013-07-08 Thread H Hartley Sweeten
Instead of carrying the in-flight message in the driver private data, use the cur_msg pointer that is already setup by the core. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo

RE: [PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-03 Thread H Hartley Sweeten
On Wednesday, July 03, 2013 11:21 AM, Mark Brown wrote: > On Tue, Jul 02, 2013 at 10:08:21AM -0700, H Hartley Sweeten wrote: >> Use devm_clk_get() so that the clk_put() happens automatically when >> the last reference to this driver is dropped. > > This doesn't apply against

RE: [PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-03 Thread H Hartley Sweeten
On Wednesday, July 03, 2013 11:21 AM, Mark Brown wrote: > On Tue, Jul 02, 2013 at 10:08:21AM -0700, H Hartley Sweeten wrote: >> Use devm_clk_get() so that the clk_put() happens automatically when >> the last reference to this driver is dropped. > > This doesn't apply against

RE: [PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-03 Thread H Hartley Sweeten
On Wednesday, July 03, 2013 11:21 AM, Mark Brown wrote: On Tue, Jul 02, 2013 at 10:08:21AM -0700, H Hartley Sweeten wrote: Use devm_clk_get() so that the clk_put() happens automatically when the last reference to this driver is dropped. This doesn't apply against current code - always submit

RE: [PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-03 Thread H Hartley Sweeten
On Wednesday, July 03, 2013 11:21 AM, Mark Brown wrote: On Tue, Jul 02, 2013 at 10:08:21AM -0700, H Hartley Sweeten wrote: Use devm_clk_get() so that the clk_put() happens automatically when the last reference to this driver is dropped. This doesn't apply against current code - always submit

[PATCH v2 06/11] spi: spi-ep93xx: remove dev_err() for kzalloc() failure

2013-07-02 Thread H Hartley Sweeten
The kzalloc() failure will have already output a message. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/spi/spi-ep93xx.c b

[PATCH v2 09/11] spi: spi-ep93xx: move the clock divider calcs into ep93xx_spi_chip_setup()

2013-07-02 Thread H Hartley Sweeten
all into ep93xx_spi_chip_setup() and return the dividers thru pointers. Remove the divider values from the per chip data structure. Signed-off-by: H Hartley Sweeten Acked-by: Mika Westerberg Cc: Ryan Mallon Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c |

[PATCH v2 08/11] spi: spi-ep93xx: don't bother calculating the divisors in ep93xx_spi_setup()

2013-07-02 Thread H Hartley Sweeten
The divisors needed to generate the SPI clock are calculated per transfer based on the t->speed_hz. There is no reason to calculate them in ep93xx_spi_setup(). Signed-off-by: H Hartley Sweeten Acked-by: Mika Westerberg Cc: Ryan Mallon Cc: Mark Brown Cc: Grant Likely --- drivers/spi/

[PATCH v2 07/11] spi: spi-ep93xx: remove 'dss' from per chip private data

2013-07-02 Thread H Hartley Sweeten
tup() in ep93xx_spi_process_transfer() and remove 'dss' from the per chip private data. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/

[PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-02 Thread H Hartley Sweeten
Use devm_clk_get() so that the clk_put() happens automatically when the last reference to this driver is dropped. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 9 +++-- 1 file changed, 3 insertions

[PATCH v2 04/11] spi: spi-ep93xx: get platform resources early in (*probe)

2013-07-02 Thread H Hartley Sweeten
Get the platform resources early in the (*probe) to minimize the number of goto's in the error path. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 26 -- 1 file changed, 12

[PATCH v2 03/11] spi: spi-ep93xx: remove bits_per_word() helper

2013-07-02 Thread H Hartley Sweeten
Check t->bits_per_word directly and remove the inline helper function. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --

[PATCH v2 02/11] spi: spi-ep93xx: use read,write instead of __raw_* variants

2013-07-02 Thread H Hartley Sweeten
The memory resource used by this driver is ioremap()'d and the normal read,write calls can be used instead of the __raw_* variants. Also, remove the inline tag on the helper functions and let the compiler decide if they are inlined. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika

[PATCH v2 01/11] spi: spi-ep93xx: always handle transfer specific settings

2013-07-02 Thread H Hartley Sweeten
for each transfer in the message. Remove the redundant ep93xx_spi_chp_setup() in ep93xx_spi_process_transfer() which just initializes the hardware to the "default" based on the SPI device. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Gr

[PATCH v2 00/11] spi: spi-ep93xx: tidy up and convert to queued driver infrastructure

2013-07-02 Thread H Hartley Sweeten
here only for completeness. H Hartley Sweeten (11): spi: spi-ep93xx: always handle transfer specific settings spi: spi-ep93xx: use read,write instead of __raw_* variants spi: spi-ep93xx: remove bits_per_word() helper spi: spi-ep93xx: get platform resources early in (*probe) spi: spi

[PATCH v2 00/11] spi: spi-ep93xx: tidy up and convert to queued driver infrastructure

2013-07-02 Thread H Hartley Sweeten
here only for completeness. H Hartley Sweeten (11): spi: spi-ep93xx: always handle transfer specific settings spi: spi-ep93xx: use read,write instead of __raw_* variants spi: spi-ep93xx: remove bits_per_word() helper spi: spi-ep93xx: get platform resources early in (*probe) spi: spi

[PATCH v2 01/11] spi: spi-ep93xx: always handle transfer specific settings

2013-07-02 Thread H Hartley Sweeten
for each transfer in the message. Remove the redundant ep93xx_spi_chp_setup() in ep93xx_spi_process_transfer() which just initializes the hardware to the default based on the SPI device. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika

[PATCH v2 02/11] spi: spi-ep93xx: use read,write instead of __raw_* variants

2013-07-02 Thread H Hartley Sweeten
The memory resource used by this driver is ioremap()'d and the normal read,write calls can be used instead of the __raw_* variants. Also, remove the inline tag on the helper functions and let the compiler decide if they are inlined. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com

[PATCH v2 03/11] spi: spi-ep93xx: remove bits_per_word() helper

2013-07-02 Thread H Hartley Sweeten
Check t-bits_per_word directly and remove the inline helper function. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo...@kernel.org Cc: Grant Likely grant.lik...@linaro.org --- drivers

[PATCH v2 05/11] spi: spi-ep93xx: use devm_clk_get()

2013-07-02 Thread H Hartley Sweeten
Use devm_clk_get() so that the clk_put() happens automatically when the last reference to this driver is dropped. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo...@kernel.org Cc: Grant

[PATCH v2 04/11] spi: spi-ep93xx: get platform resources early in (*probe)

2013-07-02 Thread H Hartley Sweeten
Get the platform resources early in the (*probe) to minimize the number of goto's in the error path. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo...@kernel.org Cc: Grant Likely

[PATCH v2 07/11] spi: spi-ep93xx: remove 'dss' from per chip private data

2013-07-02 Thread H Hartley Sweeten
() in ep93xx_spi_process_transfer() and remove 'dss' from the per chip private data. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo...@kernel.org Cc: Grant Likely grant.lik...@linaro.org --- drivers/spi/spi

[PATCH v2 08/11] spi: spi-ep93xx: don't bother calculating the divisors in ep93xx_spi_setup()

2013-07-02 Thread H Hartley Sweeten
The divisors needed to generate the SPI clock are calculated per transfer based on the t-speed_hz. There is no reason to calculate them in ep93xx_spi_setup(). Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Mika Westerberg mika.westerb...@iki.fi Cc: Ryan Mallon rmal

[PATCH v2 09/11] spi: spi-ep93xx: move the clock divider calcs into ep93xx_spi_chip_setup()

2013-07-02 Thread H Hartley Sweeten
into ep93xx_spi_chip_setup() and return the dividers thru pointers. Remove the divider values from the per chip data structure. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Mika Westerberg mika.westerb...@iki.fi Cc: Ryan Mallon rmal...@gmail.com Cc: Mark Brown broo...@kernel.org

[PATCH v2 06/11] spi: spi-ep93xx: remove dev_err() for kzalloc() failure

2013-07-02 Thread H Hartley Sweeten
The kzalloc() failure will have already output a message. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Ryan Mallon rmal...@gmail.com Cc: Mika Westerberg mika.westerb...@iki.fi Cc: Mark Brown broo...@kernel.org Cc: Grant Likely grant.lik...@linaro.org --- drivers/spi/spi

[PATCH v2 3/4] usb: ohci-ep93xx: use devm_clk_get()

2013-07-01 Thread H Hartley Sweeten
Use devm_clk_get() to make the code a bit cleaner and simpler. This also fixes a bug where a clk_put() is not done if usb_add_hcd() fails. Signed-off-by: H Hartley Sweeten Acked-by: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 4 +--- 1 file

[PATCH v2 4/4] usb: ohci-ep93xx: tidy up driver (*probe) and (*remove)

2013-07-01 Thread H Hartley Sweeten
and simply call clk_enable() and clk_disable() directly. The extra level of redirection does not add any clarity. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 128 + 1

[PATCH v2 2/4] usb: ohci-ep93xx: use platform_get_irq()

2013-07-01 Thread H Hartley Sweeten
Use platform_get_irq() instead of accessing the platform_device resources directly. Signed-off-by: H Hartley Sweeten Acked-by: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

[PATCH v2 0/4] usb: ohci-ep93xx: do a bit of housecleaning

2013-07-01 Thread H Hartley Sweeten
Tidy up this driver a bit. v2: redo patch 4 based on suggestions from Alan Stern H Hartley Sweeten (4): usb: ohci-ep93xx: use devm_ioremap_resource() usb: ohci-ep93xx: use platform_get_irq() usb: ohci-ep93xx: use devm_clk_get() usb: ohci-ep93xx: tidy up driver (*probe) and (*remove

[PATCH v2 1/4] usb: ohci-ep93xx: use devm_ioremap_resource()

2013-07-01 Thread H Hartley Sweeten
Use devm_ioremap_resource() to make the code a bit cleaner and simpler. Signed-off-by: H Hartley Sweeten Acked-by: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 35 ++- 1 file changed, 14 insertions(+), 21

RE: [PATCH 2/8] spi: spi-ep93xx: use bits_per_word_mask

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 11:23 AM, H Hartley Sweeten wrote: > On Sunday, June 30, 2013 9:14 AM, Mika Westerberg wrote: >> Please use SPI_BPW_RANGE_MASK(4, 16) here. > > I thought there was a macro for this, just couldn't find it. I'll fix this. Doh... This patch is already in li

RE: [PATCH 2/8] spi: spi-ep93xx: use bits_per_word_mask

2013-07-01 Thread H Hartley Sweeten
On Sunday, June 30, 2013 9:14 AM, Mika Westerberg wrote: > On Fri, Jun 28, 2013 at 11:43:07AM -0700, H Hartley Sweeten wrote: >> This driver supports 16 to 4 bits per work. Set the bits_per_word_mask >> to allows the spi core to handle validation. >> >> Signed-off-b

RE: [PATCH 1/8] spi: spi-ep93xx: use read,write instead of __raw_* variants

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:58 AM, Mark Brown wrote: > On Sat, Jun 29, 2013 at 09:15:09AM +1000, Ryan Mallon wrote: >> On 29/06/13 04:42, H Hartley Sweeten wrote: > >>> -static inline u8 >>> -ep93xx_spi_read_u8(const struct ep93xx_spi *spi, u16 reg) >>> -{ &g

RE: [PATCH 3/8] spi: spi-ep93xx: always handle transfer specific settings

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:28 AM, Mark Brown wrote: > On Fri, Jun 28, 2013 at 11:43:34AM -0700, H Hartley Sweeten wrote: >> __spi_async(), which starts every SPI message transfer, initializes >> the bits_per_word and max speed for every transfer in the message. >> Since

RE: [PATCH 3/8] spi: spi-ep93xx: always handle transfer specific settings

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:28 AM, Mark Brown wrote: On Fri, Jun 28, 2013 at 11:43:34AM -0700, H Hartley Sweeten wrote: __spi_async(), which starts every SPI message transfer, initializes the bits_per_word and max speed for every transfer in the message. Since the conditional test

RE: [PATCH 1/8] spi: spi-ep93xx: use read,write instead of __raw_* variants

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 3:58 AM, Mark Brown wrote: On Sat, Jun 29, 2013 at 09:15:09AM +1000, Ryan Mallon wrote: On 29/06/13 04:42, H Hartley Sweeten wrote: -static inline u8 -ep93xx_spi_read_u8(const struct ep93xx_spi *spi, u16 reg) -{ - return __raw_readb(spi-regs_base + reg

RE: [PATCH 2/8] spi: spi-ep93xx: use bits_per_word_mask

2013-07-01 Thread H Hartley Sweeten
On Sunday, June 30, 2013 9:14 AM, Mika Westerberg wrote: On Fri, Jun 28, 2013 at 11:43:07AM -0700, H Hartley Sweeten wrote: This driver supports 16 to 4 bits per work. Set the bits_per_word_mask to allows the spi core to handle validation. Signed-off-by: H Hartley Sweeten hswee

RE: [PATCH 2/8] spi: spi-ep93xx: use bits_per_word_mask

2013-07-01 Thread H Hartley Sweeten
On Monday, July 01, 2013 11:23 AM, H Hartley Sweeten wrote: On Sunday, June 30, 2013 9:14 AM, Mika Westerberg wrote: Please use SPI_BPW_RANGE_MASK(4, 16) here. I thought there was a macro for this, just couldn't find it. I'll fix this. Doh... This patch is already in linux-next. I'll need

[PATCH v2 0/4] usb: ohci-ep93xx: do a bit of housecleaning

2013-07-01 Thread H Hartley Sweeten
Tidy up this driver a bit. v2: redo patch 4 based on suggestions from Alan Stern H Hartley Sweeten (4): usb: ohci-ep93xx: use devm_ioremap_resource() usb: ohci-ep93xx: use platform_get_irq() usb: ohci-ep93xx: use devm_clk_get() usb: ohci-ep93xx: tidy up driver (*probe) and (*remove

[PATCH v2 1/4] usb: ohci-ep93xx: use devm_ioremap_resource()

2013-07-01 Thread H Hartley Sweeten
Use devm_ioremap_resource() to make the code a bit cleaner and simpler. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Alan Stern st...@rowland.harvard.edu Cc: Lennert Buytenhek ker...@wantstofly.org Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/usb/host

[PATCH v2 4/4] usb: ohci-ep93xx: tidy up driver (*probe) and (*remove)

2013-07-01 Thread H Hartley Sweeten
and simply call clk_enable() and clk_disable() directly. The extra level of redirection does not add any clarity. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Cc: Alan Stern st...@rowland.harvard.edu Cc: Lennert Buytenhek ker...@wantstofly.org Cc: Greg Kroah-Hartman gre

[PATCH v2 2/4] usb: ohci-ep93xx: use platform_get_irq()

2013-07-01 Thread H Hartley Sweeten
Use platform_get_irq() instead of accessing the platform_device resources directly. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Alan Stern st...@rowland.harvard.edu Cc: Lennert Buytenhek ker...@wantstofly.org Cc: Greg Kroah-Hartman gre...@linuxfoundation.org

[PATCH v2 3/4] usb: ohci-ep93xx: use devm_clk_get()

2013-07-01 Thread H Hartley Sweeten
Use devm_clk_get() to make the code a bit cleaner and simpler. This also fixes a bug where a clk_put() is not done if usb_add_hcd() fails. Signed-off-by: H Hartley Sweeten hswee...@visionengravers.com Acked-by: Alan Stern st...@rowland.harvard.edu Cc: Lennert Buytenhek ker...@wantstofly.org Cc

[PATCH 4/4] usb: ohci-ep93xx: tidy up ohci_hcd_ep93xx_drv_probe()

2013-06-28 Thread H Hartley Sweeten
Refactor the function a bit to remove the need for the local variable. The extern prototype of usb_disabled() is not needed. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 11 +++ 1 file changed, 3

[PATCH 3/4] usb: ohci-ep93xx: use devm_clk_get()

2013-06-28 Thread H Hartley Sweeten
Use devm_clk_get() to make the code a bit cleaner and simpler. This also fixes a bug where a clk_put() is not done if usb_add_hcd() fails. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 4 +--- 1 file changed

[PATCH 2/4] usb: ohci-ep93xx: use platform_get_irq()

2013-06-28 Thread H Hartley Sweeten
Use platform_get_irq() instead of accessing the platform_device resources directly. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH 1/4] usb: ohci-ep93xx: use devm_ioremap_resource()

2013-06-28 Thread H Hartley Sweeten
Use devm_ioremap_resource() to make the code a bit cleaner and simpler. Signed-off-by: H Hartley Sweeten Cc: Alan Stern Cc: Lennert Buytenhek Cc: Greg Kroah-Hartman --- drivers/usb/host/ohci-ep93xx.c | 35 ++- 1 file changed, 14 insertions(+), 21 deletions

[PATCH 0/4] usb: ohci-ep93xx: do a bit of house cleaning

2013-06-28 Thread H Hartley Sweeten
Tidy up this driver a bit. H Hartley Sweeten (4): usb: ohci-ep93xx: use devm_ioremap_resource() usb: ohci-ep93xx: use platform_get_irq() usb: ohci-ep93xx: use devm_clk_get() usb: ohci-ep93xx: tidy up ohci_hcd_ep93xx_drv_probe() drivers/usb/host/ohci-ep93xx.c | 60

Q: mmc au size

2013-06-28 Thread H Hartley Sweeten
Hello all, I have a question concerning the "au" size of a SD Card. I have an old Kingston 64MB SD-Card and lately I have noticed this kernel message when I use the card: mmc0: SD Status: Invalid Allocation Unit size. This is due to the following commit: commit

RE: [PATCH 3/8] spi: spi-ep93xx: always handle transfer specific settings

2013-06-28 Thread H Hartley Sweeten
On Friday, June 28, 2013 4:18 PM, Ryan Mallon wrote: > On 29/06/13 04:43, H Hartley Sweeten wrote: >> __spi_async(), which starts every SPI message transfer, initializes >> the bits_per_word and max speed for every transfer in the message. >> Since

Q: mmc au size

2013-06-28 Thread H Hartley Sweeten
Hello Chris, I have a question concerning the "au" size of a SD Card. I have an old Kingston 64MB SD-Card and lately I have noticed this kernel message when I use the card: mmc0: SD Status: Invalid Allocation Unit size. This is due to the following commit: commit

[PATCH 8/8] spi: spi-ep93xx: convert to the queued driver infrastructure

2013-06-28 Thread H Hartley Sweeten
The SPI core provides infrastructure for standard message queueing. Use that instead of handling it in the driver. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 164

[PATCH 7/8] spi: spi-ep93xx: move the clock divider calcs into ep93xx_spi_chip_setup()

2013-06-28 Thread H Hartley Sweeten
all into ep93xx_spi_chip_setup() and return the dividers thru pointers. Remove the divider values from the per chip data structure. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c |

[PATCH 6/8] spi: spi-ep93xx: don't bother calculating the divisors in ep93xx_spi_setup()

2013-06-28 Thread H Hartley Sweeten
The divisors needed to generate the SPI clock are calculated per transfer based on the t->speed_hz. There is no reason to calculate them in ep93xx_spi_setup(). Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93x

[PATCH 5/8] spi: spi-ep93xx: remove 'dss' from per chip private data

2013-06-28 Thread H Hartley Sweeten
tup() in ep93xx_spi_process_transfer() and remove 'dss' from the per chip private data. Signed-off-by: H Hartley Sweeten Cc: Ryan Mallon Cc: Mika Westerberg Cc: Mark Brown Cc: Grant Likely --- drivers/spi/spi-ep93xx.c | 20 +++- 1 file changed, 7 insertions(+), 13 deletions(-) diff --git a/driv

  1   2   3   4   5   6   7   8   9   10   >