Re: PWM backlight build warning

2013-04-06 Thread Tony Prisk
On 07/04/13 09:10, Thierry Reding wrote: On Sat, Apr 06, 2013 at 09:02:51PM +0100, Mark Brown wrote: On Sun, Apr 07, 2013 at 06:33:35AM +1200, Tony Prisk wrote: The problem occurs because when CONFIG_PM=y but CONFIG_PM_SLEEP=n: I guess this would be a problem on every driver that has been

PWM backlight build warning

2013-04-06 Thread Tony Prisk
Hi Thierry, Sorry it's taken so long to report this but I've been tied up with other patches recently. I noticed a build warning on the pwm_bl driver in 3.9-rcX but I'm not sure what the correct resolution would be. This patch: commit e2c17bc6f717a8847df2a867caec6ba4fe85f3fc backlight:

PWM backlight build warning

2013-04-06 Thread Tony Prisk
Hi Thierry, Sorry it's taken so long to report this but I've been tied up with other patches recently. I noticed a build warning on the pwm_bl driver in 3.9-rcX but I'm not sure what the correct resolution would be. This patch: commit e2c17bc6f717a8847df2a867caec6ba4fe85f3fc backlight:

Re: PWM backlight build warning

2013-04-06 Thread Tony Prisk
On 07/04/13 09:10, Thierry Reding wrote: On Sat, Apr 06, 2013 at 09:02:51PM +0100, Mark Brown wrote: On Sun, Apr 07, 2013 at 06:33:35AM +1200, Tony Prisk wrote: The problem occurs because when CONFIG_PM=y but CONFIG_PM_SLEEP=n: I guess this would be a problem on every driver that has been

Re: [PATCHv3 0/6] fb: vt8500: patches for 3.10

2013-04-03 Thread Tony Prisk
On 03/04/13 07:20, Tony Prisk wrote: V3 Changes: Tidied up patch 1 as requested by Jean-Christophe - Makes the patch much simpler. V2 Changes: Split the SoC and board portions of the binding into their respective files. Removed the forced non-interlaced code from wm8505fb.c and vt8500lcdfb.c

[GIT PULL] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
from Stephen W. Stephen Warren (1): pinctrl: bcm2835: make use of of_property_read_u32_index() Tony Prisk (7): of: Add support for reading a u32 from a multi-value property. of: Remove duplicated code for validating

Re: [PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
On 04/04/13 10:23, Stephen Warren wrote: On 04/03/2013 07:40 AM, Linus Walleij wrote: On Wed, Apr 3, 2013 at 8:24 AM, Tony Prisk wrote: v5 changes: Changed as requested by Rob Herring: Added of_find_property_value_of_size() to drivers/of/base.c to remove some code that was being duplicated

Re: [PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
On 04/04/13 02:40, Linus Walleij wrote: On Wed, Apr 3, 2013 at 8:24 AM, Tony Prisk wrote: v5 changes: Changed as requested by Rob Herring: Added of_find_property_value_of_size() to drivers/of/base.c to remove some code that was being duplicated. Changed as requested by Stephen Warren: Redid

[PATCHv5 2/7] of: Remove duplicated code for validating property and value

2013-04-03 Thread Tony Prisk
EOVERFLOW; This patch adds of_find_property_value_of_size() which performs the equivalent of the above code and removes the instances where it was duplicated in several functions. Reported-by: Rob Herring Signed-off-by: Tony Prisk --- drivers/of/base

[PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (7): of: Add support for reading a u32 from a multi-value property. of: Remove duplicated code for validating property and value arm: vt8500: Increase available GPIOs on arch-vt8500 pinctrl: gpio

[PATCHv5 3/7] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-03 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/Kconfig b/arch

[PATCHv5 6/7] arm: vt8500: Remove gpio devicetree nodes

2013-04-03 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi |7 --- arch

[PATCHv5 7/7] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-03 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv5 5/7] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-03 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9 + arch/arm

[PATCHv5 1/7] of: Add support for reading a u32 from a multi-value property.

2013-04-03 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk Reviewed-by: Stephen Warren --- drivers/of/base.c | 33 + include/linux/of.h |9

Re: [PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
On 04/04/13 10:23, Stephen Warren wrote: On 04/03/2013 07:40 AM, Linus Walleij wrote: On Wed, Apr 3, 2013 at 8:24 AM, Tony Prisk li...@prisktech.co.nz wrote: v5 changes: Changed as requested by Rob Herring: Added of_find_property_value_of_size() to drivers/of/base.c to remove some code

[GIT PULL] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
from Stephen W. Stephen Warren (1): pinctrl: bcm2835: make use of of_property_read_u32_index() Tony Prisk (7): of: Add support for reading a u32 from a multi-value property. of: Remove duplicated code for validating

Re: [PATCHv3 0/6] fb: vt8500: patches for 3.10

2013-04-03 Thread Tony Prisk
On 03/04/13 07:20, Tony Prisk wrote: V3 Changes: Tidied up patch 1 as requested by Jean-Christophe - Makes the patch much simpler. V2 Changes: Split the SoC and board portions of the binding into their respective files. Removed the forced non-interlaced code from wm8505fb.c and vt8500lcdfb.c

[PATCHv5 1/7] of: Add support for reading a u32 from a multi-value property.

2013-04-03 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Stephen Warren swar...@nvidia.com --- drivers/of/base.c | 33

[PATCHv5 5/7] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-03 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9

[PATCHv5 7/7] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-03 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv5 6/7] arm: vt8500: Remove gpio devicetree nodes

2013-04-03 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi

[PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (7): of: Add support for reading a u32 from a multi-value property. of: Remove duplicated code for validating property and value arm: vt8500: Increase available GPIOs on arch-vt8500 pinctrl: gpio

[PATCHv5 3/7] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-03 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCHv5 2/7] of: Remove duplicated code for validating property and value

2013-04-03 Thread Tony Prisk
-EOVERFLOW; This patch adds of_find_property_value_of_size() which performs the equivalent of the above code and removes the instances where it was duplicated in several functions. Reported-by: Rob Herring robherri...@gmail.com Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/of/base.c

Re: [PATCHv5 0/7] arm: vt8500: Add support for pinctrl/gpio module

2013-04-03 Thread Tony Prisk
On 04/04/13 02:40, Linus Walleij wrote: On Wed, Apr 3, 2013 at 8:24 AM, Tony Prisk li...@prisktech.co.nz wrote: v5 changes: Changed as requested by Rob Herring: Added of_find_property_value_of_size() to drivers/of/base.c to remove some code that was being duplicated. Changed as requested

[PATCHv3 4/6] drivers/video/wm8505fb.c: use devm_ functions

2013-04-02 Thread Tony Prisk
block of error-handling code that needs to call fb_dealloc_cmap, and so this is moved up to the place where it is needed, eliminating the need for all gotos and labels in the function. This was suggested by Tony Prisk. The initializations of fbi and ret at the beginning of the function

[PATCHv3 3/6] video: vt8500: Correct descriptions in video/Kconfig

2013-04-02 Thread Tony Prisk
This patch corrects the descriptions for the VIA VT8500 and Wondermedia WM8xxx-series framebuffer drivers to correctly reflect which hardware they support. Signed-off-by: Tony Prisk Reviewed-by: Jean-Christophe Plagniol-Villard --- drivers/video/Kconfig | 11 ++- 1 file changed, 6

[PATCHv3 5/6] video: vt8500: Adjust contrast in wm8505 framebuffer driver.

2013-04-02 Thread Tony Prisk
The contrast value was typo'd on the original commit (0x80 instead of 0x08). Following feedback from an enduser, a value of 0x10 seems more suitable due to the default backlight being <100%. Signed-off-by: Tony Prisk Reviewed-by: Jean-Christophe Plagniol-Villard --- drivers/video/wm8505f

[PATCHv3 6/6] video: fb: vt8500: Convert framebuffer drivers to standardized binding

2013-04-02 Thread Tony Prisk
this as part of the display timing, it is moved into the framebuffer node. I have also taken the opportunity to alphabetise the includes of each driver to avoid double-ups. Signed-off-by: Tony Prisk --- .../devicetree/bindings/video/via,vt8500-fb.txt| 48 -- .../devicetree

[PATCHv3 0/6] fb: vt8500: patches for 3.10

2013-04-02 Thread Tony Prisk
/wm8505fb.c: use devm_ functions Tony Prisk (5): video: vt8500: Make wmt_ge_rops optional video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h video: vt8500: Correct descriptions in video/Kconfig video: vt8500: Adjust contrast in wm8505 framebuffer driver. video: fb: vt8500: Convert

[PATCHv3 2/6] video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h

2013-04-02 Thread Tony Prisk
With the conversion to devicetree only for arch-vt8500, this header is no longer required. This patch removes the #include from the two framebuffer drivers that used it, and the header file. Signed-off-by: Tony Prisk Reviewed-by: Jean-Christophe Plagniol-Villard --- drivers/video/vt8500lcdfb.c

[PATCHv3 1/6] video: vt8500: Make wmt_ge_rops optional

2013-04-02 Thread Tony Prisk
wmt_ge_rops is a seperate driver to vt8500/wm8505 framebuffer driver but is currently a required option. This patch makes accelerated raster ops optional. Signed-off-by: Tony Prisk Reviewed-by: Jean-Christophe Plagniol-Villard --- drivers/video/Kconfig | 22

[PATCHv3 2/6] video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h

2013-04-02 Thread Tony Prisk
With the conversion to devicetree only for arch-vt8500, this header is no longer required. This patch removes the #include from the two framebuffer drivers that used it, and the header file. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Jean-Christophe Plagniol-Villard plagn

[PATCHv3 1/6] video: vt8500: Make wmt_ge_rops optional

2013-04-02 Thread Tony Prisk
wmt_ge_rops is a seperate driver to vt8500/wm8505 framebuffer driver but is currently a required option. This patch makes accelerated raster ops optional. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Jean-Christophe Plagniol-Villard plagn...@jcrosoft.com --- drivers/video/Kconfig

[PATCHv3 0/6] fb: vt8500: patches for 3.10

2013-04-02 Thread Tony Prisk
/wm8505fb.c: use devm_ functions Tony Prisk (5): video: vt8500: Make wmt_ge_rops optional video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h video: vt8500: Correct descriptions in video/Kconfig video: vt8500: Adjust contrast in wm8505 framebuffer driver. video: fb: vt8500: Convert

[PATCHv3 6/6] video: fb: vt8500: Convert framebuffer drivers to standardized binding

2013-04-02 Thread Tony Prisk
this as part of the display timing, it is moved into the framebuffer node. I have also taken the opportunity to alphabetise the includes of each driver to avoid double-ups. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/video/via,vt8500-fb.txt| 48

[PATCHv3 5/6] video: vt8500: Adjust contrast in wm8505 framebuffer driver.

2013-04-02 Thread Tony Prisk
The contrast value was typo'd on the original commit (0x80 instead of 0x08). Following feedback from an enduser, a value of 0x10 seems more suitable due to the default backlight being 100%. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Jean-Christophe Plagniol-Villard plagn

[PATCHv3 4/6] drivers/video/wm8505fb.c: use devm_ functions

2013-04-02 Thread Tony Prisk
. This way there is only one block of error-handling code that needs to call fb_dealloc_cmap, and so this is moved up to the place where it is needed, eliminating the need for all gotos and labels in the function. This was suggested by Tony Prisk. The initializations of fbi and ret at the beginning

[PATCHv3 3/6] video: vt8500: Correct descriptions in video/Kconfig

2013-04-02 Thread Tony Prisk
This patch corrects the descriptions for the VIA VT8500 and Wondermedia WM8xxx-series framebuffer drivers to correctly reflect which hardware they support. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Jean-Christophe Plagniol-Villard plagn...@jcrosoft.com --- drivers/video

[PATCHv4 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-04-01 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi |7 --- arch

[PATCHv4 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-01 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9 + arch/arm

[PATCHv4 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-01 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv4 1/6] of: Add support for reading a u32 from a multi-value property.

2013-04-01 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk Reviewed-by: Stephen Warren --- drivers/of/base.c | 33 + include/linux/of.h |9

[PATCHv4 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-01 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/Kconfig b/arch

[PATCHv4 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-04-01 Thread Tony Prisk
driver to function properly. Patch 2 is independant, but required for the driver to function properly. Patches 3-4 are pinctrl/gpio patches. Patch 5-6 remove the old gpio code which is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (6): of: Add

[PATCHv4 RESEND 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-04-01 Thread Tony Prisk
if necessary. Regards Tony Prisk Tony Prisk (6): of: Add support for reading a u32 from a multi-value property. arm: vt8500: Increase available GPIOs on arch-vt8500 pinctrl: gpio: vt8500: Add pincontrol driver for arch-vt8500 arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl

[PATCHv4 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-01 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/Kconfig b/arch

[PATCHv4 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-01 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv4 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-04-01 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi |7 --- arch

[PATCHv4 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-01 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9 + arch/arm

[PATCHv4 1/6] of: Add support for reading a u32 from a multi-value property.

2013-04-01 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk Reviewed-by: Stephen Warren --- drivers/of/base.c | 33 + include/linux/of.h |9

Re: [PATCHv3 3/6] pinctrl: gpio: vt8500: Add pincontrol driver for arch-vt8500

2013-04-01 Thread Tony Prisk
On 02/04/13 06:06, Stephen Warren wrote: On 03/28/2013 12:10 AM, Tony Prisk wrote: This patch adds support for the GPIO/pinmux controller found on the VIA VT8500 and Wondermedia WM8xxx-series SoCs. Each pin within the controller is capable of operating as a GPIO or as an alternate function

Re: [PATCHv3 3/6] pinctrl: gpio: vt8500: Add pincontrol driver for arch-vt8500

2013-04-01 Thread Tony Prisk
On 02/04/13 06:06, Stephen Warren wrote: On 03/28/2013 12:10 AM, Tony Prisk wrote: This patch adds support for the GPIO/pinmux controller found on the VIA VT8500 and Wondermedia WM8xxx-series SoCs. Each pin within the controller is capable of operating as a GPIO or as an alternate function

[PATCHv4 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-01 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9

[PATCHv4 1/6] of: Add support for reading a u32 from a multi-value property.

2013-04-01 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Stephen Warren swar...@nvidia.com --- drivers/of/base.c | 33

[PATCHv4 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-01 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv4 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-04-01 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi

[PATCHv4 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-01 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCHv4 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-04-01 Thread Tony Prisk
driver to function properly. Patch 2 is independant, but required for the driver to function properly. Patches 3-4 are pinctrl/gpio patches. Patch 5-6 remove the old gpio code which is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (6): of: Add

[PATCHv4 RESEND 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-04-01 Thread Tony Prisk
if necessary. Regards Tony Prisk Tony Prisk (6): of: Add support for reading a u32 from a multi-value property. arm: vt8500: Increase available GPIOs on arch-vt8500 pinctrl: gpio: vt8500: Add pincontrol driver for arch-vt8500 arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl

[PATCHv4 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-04-01 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCHv4 1/6] of: Add support for reading a u32 from a multi-value property.

2013-04-01 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk li...@prisktech.co.nz Reviewed-by: Stephen Warren swar...@nvidia.com --- drivers/of/base.c | 33

[PATCHv4 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-04-01 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9

[PATCHv4 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-04-01 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv4 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-04-01 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi

Re: [PATCH] usb: ehci: mark unlink_empty_async_suspended() as __maybe_unused

2013-03-28 Thread Tony Prisk
On 29/03/13 10:16, Arnd Bergmann wrote: On Thursday 28 March 2013, Arnd Bergmann wrote: Patch 4d053fdac3 "usb: ehci: unlink_empty_async_suspended() only used with CONFIG_PM" tried to hide the unlink_empty_async_suspended function inside of an #ifdef to work around an unused function warning.

Re: [PATCH] pinctrl: bcm2835: make use of of_property_read_u32_index()

2013-03-28 Thread Tony Prisk
rom a multi-value property" by Tony Prisk. > > BTW, I realized why I didn't create that standard API, but wrote custom > prop_u32() instead; the code has already looked up the properties, and > validated their length, so prop_u32() can simply index into the property > data, whereas o

[PATCHv3 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-03-28 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/arm/Kconfig b/arch

[PATCHv3 1/6] of: Add support for reading a u32 from a multi-value property.

2013-03-28 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk --- drivers/of/base.c | 33 + include/linux/of.h |9 + 2 files changed

[PATCHv3 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-03-28 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv3 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-03-28 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9 + arch/arm

[PATCHv3 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-03-28 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi |7 --- arch

[PATCHv3 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-03-28 Thread Tony Prisk
to function properly. Patches 3-4 are pinctrl/gpio patches. Patch 5-6 remove the old gpio code which is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (6): of: Add support for reading a u32 from a multi-value property. arm: vt8500: Increase available GPIOs

Re: [PATCH] usb: ehci: mark unlink_empty_async_suspended() as __maybe_unused

2013-03-28 Thread Tony Prisk
On 29/03/13 10:16, Arnd Bergmann wrote: On Thursday 28 March 2013, Arnd Bergmann wrote: Patch 4d053fdac3 usb: ehci: unlink_empty_async_suspended() only used with CONFIG_PM tried to hide the unlink_empty_async_suspended function inside of an #ifdef to work around an unused function warning. Hi

[PATCHv3 0/6] arm: vt8500: Add support for pinctrl/gpio module

2013-03-28 Thread Tony Prisk
to function properly. Patches 3-4 are pinctrl/gpio patches. Patch 5-6 remove the old gpio code which is no longer required. Patch 2 could go via arm-soc if necessary. Regards Tony Prisk Tony Prisk (6): of: Add support for reading a u32 from a multi-value property. arm: vt8500: Increase available GPIOs

[PATCHv3 4/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

2013-03-28 Thread Tony Prisk
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi files to support the pinctrl driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |9 + arch/arm/boot/dts/wm8505.dtsi |9 + arch/arm/boot/dts/wm8650.dtsi |9

[PATCHv3 5/6] arm: vt8500: Remove gpio devicetree nodes

2013-03-28 Thread Tony Prisk
Remove the gpio related devicetree nodes as these are no longer required with the move to a combined pinctrl/gpio driver. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/boot/dts/vt8500.dtsi |7 --- arch/arm/boot/dts/wm8505.dtsi |7 --- arch/arm/boot/dts/wm8650.dtsi

[PATCHv3 1/6] of: Add support for reading a u32 from a multi-value property.

2013-03-28 Thread Tony Prisk
This patch adds an of_property_read_u32_index() function to allow reading a single indexed u32 value from a property containing multiple u32 values. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/of/base.c | 33 + include/linux/of.h |9

[PATCHv3 6/6] gpio: vt8500: Remove arch-vt8500 gpio driver

2013-03-28 Thread Tony Prisk
With the move to a combined pinctrl/gpio driver, the arch-vt8500 gpio driver is no longer required. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/gpio/gpio-vt8500.txt | 24 -- drivers/gpio/Kconfig |6 - drivers/gpio/Makefile

[PATCHv3 2/6] arm: vt8500: Increase available GPIOs on arch-vt8500

2013-03-28 Thread Tony Prisk
With the inclusion of the pin control driver, more GPIO pins have been identified on the arch-vt8500 SoCs requiring an increase in the available GPIOs. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- arch/arm/Kconfig |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH] pinctrl: bcm2835: make use of of_property_read_u32_index()

2013-03-28 Thread Tony Prisk
property by Tony Prisk. BTW, I realized why I didn't create that standard API, but wrote custom prop_u32() instead; the code has already looked up the properties, and validated their length, so prop_u32() can simply index into the property data, whereas of_property_read_u32_index() needs to go

Re: [PATCH 6/6] video: fb: vt8500: Convert framebuffer drivers to standardized binding

2013-03-27 Thread Tony Prisk
On Wed, 2013-03-27 at 13:10 +0200, Tomi Valkeinen wrote: > Hi, > > On 2013-03-27 10:47, Tony Prisk wrote: > > Now that a display timing binding is available, convert our almost identical > > binding to use the standard binding. > > > > This patch converts

Re: [Bulk] [PATCH] video: fix invalid free of devm_ allocated data

2013-03-27 Thread Tony Prisk
On Thu, 2013-03-28 at 01:55 +0200, Andrei Epure wrote: > The objects allocated by devm_* APIs are managed by devres and are freed when > the device is detached. Hence there is no need to use kfree() explicitly. > Patch found using coccinelle. > > Signed-off-by: Andrei Epure > --- >

Re: [PATCH] video: fixed missing iounmap coccinelle errors

2013-03-27 Thread Tony Prisk
On Thu, 2013-03-28 at 01:43 +0200, Andrei Epure wrote: > Modified or added the necessary goto statements so that the ioremapped > regions get unmapped before return. > > Signed-off-by: Andrei Epure > --- > drivers/video/vt8500lcdfb.c |7 --- > drivers/video/wm8505fb.c|7 ---

[PATCH 4/6] drivers/video/wm8505fb.c: use devm_ functions

2013-03-27 Thread Tony Prisk
block of error-handling code that needs to call fb_dealloc_cmap, and so this is moved up to the place where it is needed, eliminating the need for all gotos and labels in the function. This was suggested by Tony Prisk. The initializations of fbi and ret at the beginning of the function

[PATCH 6/6] video: fb: vt8500: Convert framebuffer drivers to standardized binding

2013-03-27 Thread Tony Prisk
this as part of the display timing, it is moved into the framebuffer node. I have also taken the opportunity to alphabetise the includes of each driver to avoid double-ups. Signed-off-by: Tony Prisk --- .../devicetree/bindings/video/via,vt8500-fb.txt| 48 -- .../devicetree

[PATCH 5/6] video: vt8500: Adjust contrast in wm8505 framebuffer driver.

2013-03-27 Thread Tony Prisk
The contrast value was typo'd on the original commit (0x80 instead of 0x08). Following feedback from an enduser, a value of 0x10 seems more suitable due to the default backlight being <100%. Signed-off-by: Tony Prisk --- drivers/video/wm8505fb.c |2 +- 1 file changed, 1 insertion(+)

[PATCH 1/6] video: vt8500: Make wmt_ge_rops optional

2013-03-27 Thread Tony Prisk
wmt_ge_rops is a seperate driver to vt8500/wm8505 framebuffer driver but is currently a required option. This patch makes accelerated raster ops optional. Signed-off-by: Tony Prisk --- drivers/video/Kconfig | 22 -- drivers/video/vt8500lcdfb.c | 15

[PATCH 2/6] video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h

2013-03-27 Thread Tony Prisk
With the conversion to devicetree only for arch-vt8500, this header is no longer required. This patch removes the #include from the two framebuffer drivers that used it, and the header file. Signed-off-by: Tony Prisk --- drivers/video/vt8500lcdfb.c |2 -- drivers/video

[PATCH 0/6] fb: vt8500: patches for 3.10

2013-03-27 Thread Tony Prisk
Prisk Julia Lawall (1): drivers/video/wm8505fb.c: use devm_ functions Tony Prisk (5): video: vt8500: Make wmt_ge_rops optional video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h video: vt8500: Correct descriptions in video/Kconfig video: vt8500: Adjust contrast in wm8505

[PATCH 3/6] video: vt8500: Correct descriptions in video/Kconfig

2013-03-27 Thread Tony Prisk
This patch corrects the descriptions for the VIA VT8500 and Wondermedia WM8xxx-series framebuffer drivers to correctly reflect which hardware they support. Signed-off-by: Tony Prisk --- drivers/video/Kconfig | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git

[PATCH 3/6] video: vt8500: Correct descriptions in video/Kconfig

2013-03-27 Thread Tony Prisk
This patch corrects the descriptions for the VIA VT8500 and Wondermedia WM8xxx-series framebuffer drivers to correctly reflect which hardware they support. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/video/Kconfig | 11 ++- 1 file changed, 6 insertions(+), 5 deletions

[PATCH 0/6] fb: vt8500: patches for 3.10

2013-03-27 Thread Tony Prisk
Prisk Julia Lawall (1): drivers/video/wm8505fb.c: use devm_ functions Tony Prisk (5): video: vt8500: Make wmt_ge_rops optional video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h video: vt8500: Correct descriptions in video/Kconfig video: vt8500: Adjust contrast in wm8505

[PATCH 2/6] video: vt8500: Remove unused platform_data/video-vt8500lcdfb.h

2013-03-27 Thread Tony Prisk
With the conversion to devicetree only for arch-vt8500, this header is no longer required. This patch removes the #include from the two framebuffer drivers that used it, and the header file. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/video/vt8500lcdfb.c

[PATCH 1/6] video: vt8500: Make wmt_ge_rops optional

2013-03-27 Thread Tony Prisk
wmt_ge_rops is a seperate driver to vt8500/wm8505 framebuffer driver but is currently a required option. This patch makes accelerated raster ops optional. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/video/Kconfig | 22 -- drivers/video/vt8500lcdfb.c

[PATCH 5/6] video: vt8500: Adjust contrast in wm8505 framebuffer driver.

2013-03-27 Thread Tony Prisk
The contrast value was typo'd on the original commit (0x80 instead of 0x08). Following feedback from an enduser, a value of 0x10 seems more suitable due to the default backlight being 100%. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- drivers/video/wm8505fb.c |2 +- 1 file changed, 1

[PATCH 6/6] video: fb: vt8500: Convert framebuffer drivers to standardized binding

2013-03-27 Thread Tony Prisk
this as part of the display timing, it is moved into the framebuffer node. I have also taken the opportunity to alphabetise the includes of each driver to avoid double-ups. Signed-off-by: Tony Prisk li...@prisktech.co.nz --- .../devicetree/bindings/video/via,vt8500-fb.txt| 48

[PATCH 4/6] drivers/video/wm8505fb.c: use devm_ functions

2013-03-27 Thread Tony Prisk
. This way there is only one block of error-handling code that needs to call fb_dealloc_cmap, and so this is moved up to the place where it is needed, eliminating the need for all gotos and labels in the function. This was suggested by Tony Prisk. The initializations of fbi and ret at the beginning

Re: [Bulk] [PATCH] video: fix invalid free of devm_ allocated data

2013-03-27 Thread Tony Prisk
On Thu, 2013-03-28 at 01:55 +0200, Andrei Epure wrote: The objects allocated by devm_* APIs are managed by devres and are freed when the device is detached. Hence there is no need to use kfree() explicitly. Patch found using coccinelle. Signed-off-by: Andrei Epure epure.and...@gmail.com ---

<    1   2   3   4   5   6   7   8   >