Re: [PATCH 2/2] pwm: bcm-kona: apply pwm settings on enable

2018-11-08 Thread Tim Kryger
On Thu, Nov 8, 2018 at 2:59 AM Uwe Kleine-König wrote: > > Hello, > > adding Tim Kryger as the initial author of the bcm-kona driver to Cc:. > Maybe he can shed some light to the questions below? > > On Thu, Nov 08, 2018 at 11:47:17AM +0100, Clément Péron wrote: > >

Re: [linux-sunxi] Designware UART bug

2017-05-03 Thread Tim Kryger
On Wed, May 3, 2017 at 8:40 AM, Olliver Schinagl wrote: > Hey Tim, > > Ok, so as far as I understand (from the datasheet) the intended way to do > this would be to check for the BUSY IRQ & USR[0] IRQ and if it is busy, > (re-write) the LCR. We no longer do this because it did not work due to the

Re: [linux-sunxi] Designware UART bug

2017-05-03 Thread Tim Kryger
On Wed, May 3, 2017 at 4:26 AM, Olliver Schinagl wrote: > Hey Chen-Yu > > On 03-05-17 12:40, Chen-Yu Tsai wrote: >> >> On Wed, May 3, 2017 at 6:17 PM, Olliver Schinagl >> wrote: >>> >>> Hey Jamie, >>> >>> Several years ago you wrote the glue-code [0] for the DW 8250 IP. Over >>> the >>> years var

Re: [PATCH v0 2/4] i2c: bcm-kona: Use complete() instead of complete_all()

2016-08-04 Thread Tim Kryger
ttern of the completion is: > > bcm_kona_send_i2c_cmd() > reinit_completion() > ... > bcm_kona_i2c_send_cmd_to_ctrl() > ... > wait_for_completion_timeout() > > Signed-off-by: Daniel Wagner Reviewed-by: Tim Kryger > --- > drivers/i2c/busses/i2c-bcm-

Re: [PATCH v8 1/5] drivers: pwm: core: Add pwmchip_add_inversed

2015-06-14 Thread Tim Kryger
On Fri, Jun 12, 2015 at 2:45 AM, Thierry Reding wrote: > On Tue, May 26, 2015 at 01:08:16PM -0700, Jonathan Richardson wrote: >> From: Tim Kryger >> >> Add a new function to register a PWM chip with channels that have their >> initial polarity as inversed. This bene

Re: [PATCH v8 4/5] pwm: kona: Add debug info to config function

2015-05-30 Thread Tim Kryger
On Tue, May 26, 2015 at 1:08 PM, Jonathan Richardson wrote: > Adds debugging info to config function where duty cycle and period > are calculated and verified. > > Signed-off-by: Jonathan Richardson > --- > drivers/pwm/pwm-bcm-kona.c | 25 +++-- > 1 file changed, 23 inserti

Re: [PATCH v8 3/5] pwm: kona: Fix incorrect config, disable, and polarity procedures

2015-05-30 Thread Tim Kryger
r 400ns. This corrects an issue where occasionally a requested change was not properly reflected in the PWM output. Otherwise, this patch looks reasonable so Reviewed-by: Tim Kryger > > Reviewed-by: Arun Ramamurthy > Reviewed-by: Scott Branden > Tested-by: Scott Branden

Re: [PATCH v7 3/5] pwm: kona: Fix incorrect config, disable, and polarity procedures

2015-05-25 Thread Tim Kryger
On Thu, May 21, 2015 at 3:50 PM, Jonathan Richardson wrote: > If this is correct - polarity changes affect the output signal > immediately, then I can change our driver. Could you confirm first this > is what we want? Yes. This seems best. Please do. -Tim -- To unsubscribe from this list: sen

Re: [PATCH v7 3/5] pwm: kona: Fix incorrect config, disable, and polarity procedures

2015-05-17 Thread Tim Kryger
On Tue, May 12, 2015 at 4:28 PM, Jonathan Richardson wrote: > The polarity procedure no longer applies the settings to change the > output signal because it can't be called when the pwm is enabled anyway. > The polarity is only updated in the control register. The correct > polarity will be appli

Re: [PATCH v6 3/4] pwm: kona: Fix incorrect config, disable, and polarity procedures

2015-05-06 Thread Tim Kryger
On Fri, Apr 10, 2015 at 11:58 AM, Jonathan Richardson wrote: > The polarity procedure no longer applies the settings to change the > output signal because it can't be called when the pwm is enabled anyway. > The polarity is only updated in the control register. The correct > polarity will be appl

Re: [RFC] With 8250 Designware UART, if writes to the LCR failed the kernel will hung up

2015-03-09 Thread Tim Kryger
On Mon, Mar 9, 2015 at 8:05 AM, Alan Cox wrote: > Ah no - I meant what is their official software workaround for existing > parts with the bug ? Presumably they have an errata document that > discusses this and the correct methods they recommend to avoid the > hang ? As far as I know, the only a

Re: [RFC] With 8250 Designware UART, if writes to the LCR failed the kernel will hung up

2015-03-09 Thread Tim Kryger
On Mon, Mar 9, 2015 at 6:32 AM, Alan Cox wrote: >> Maybe the next release of the board we will upgrade the serial block to the >> new version. >> but the issue is that how we circumvent this problem in kernel? > > What is the official vendor workaround ? They introduced a UART_16550_COMPATIBLE o

Re: [RFC] With 8250 Designware UART, if writes to the LCR failed the kernel will hung up

2015-03-06 Thread Tim Kryger
You only hit the silicon bug if you bombard the uart with characters and simultaneously request a baud rate or framing change. I'm not sure why you would do either to the uart console. Is it possible your host machine is doing something weird? If you have the leverage, remind the SoC vendor to u

[PATCH] mmc: sdhci: Set SDHCI_POWER_ON with external vmmc

2015-01-13 Thread Tim Kryger
Nvidia boards caused by 5222161 mmc: sdhci: Improve external VDD regulator support. Signed-off-by: Tim Kryger Tested-by: Bjorn Andersson --- drivers/mmc/host/sdhci.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index ada1a3e..59a328a

Re: Possible regression with commit 52221610d

2015-01-13 Thread Tim Kryger
On Sun, Dec 21, 2014 at 7:01 PM, Tim Kryger wrote: > On Wed, Dec 17, 2014 at 11:57 AM, Bjorn Andersson wrote: > >> I'm somewhat puzzled to what benefit 52221610d brings after bringing >> back the write of BIT(0). Is it just that we don't hit the BUG() on >> n

Re: Possible regression with commit 52221610d

2015-01-13 Thread Tim Kryger
On Mon, Jan 12, 2015 at 2:31 AM, Ulf Hansson wrote: > On 5 January 2015 at 20:52, Bjorn Andersson wrote: >> On Sun, Dec 21, 2014 at 7:01 PM, Tim Kryger wrote: >>> On Wed, Dec 17, 2014 at 11:57 AM, Bjorn Andersson wrote: >> [..] >>>> Non-the-less, feel free t

Re: Possible regression with commit 52221610d

2015-01-13 Thread Tim Kryger
On Mon, Jan 5, 2015 at 11:52 AM, Bjorn Andersson wrote: > On Sun, Dec 21, 2014 at 7:01 PM, Tim Kryger wrote: >> On Wed, Dec 17, 2014 at 11:57 AM, Bjorn Andersson wrote: > [..] >>> Non-the-less, feel free to propose a patch and I will give it a test. >> >> Le

Re: [PATCH v4 2/3] pwm: kona: Remove setting default smooth type and polarity for all channels

2015-01-04 Thread Tim Kryger
On Sun, Jan 4, 2015 at 5:12 PM, Tim Kryger wrote: > On Tue, Dec 30, 2014 at 2:43 PM, Jonathan Richardson > wrote: >> Setting the default polarity in probe to normal for all channels caused >> the speaker pwm channel to click. The polarity does need to be set to >> norma

[PATCH 2/2] drivers: pwm: bcm-kona: Dont set polarity in probe

2015-01-04 Thread Tim Kryger
Signed-off-by: Scott Branden Signed-off-by: Tim Kryger --- This is based on Arun's patch (originally posted by Scott) https://lkml.org/lkml/2014/11/25/1019 I modified it to use the pwmchip_add_inversed function so the polarity could be safely left at the power on default and updated the c

[PATCH 0/2] Allow registration of default inversed polarity PWMs

2015-01-04 Thread Tim Kryger
This series alters the PWM core to allow the registration of controllers that begin with inversed polarity output. It also modifies the Broadcom Kona driver to take advantage of this new call. Arun Ramamurthy (1): drivers: pwm: bcm-kona: Dont set polarity in probe Tim Kryger (1): drivers

[PATCH 1/2] drivers: pwm: core: Add pwmchip_add_inversed

2015-01-04 Thread Tim Kryger
Add a new function to register a PWM chip with channels that have their initial polarity as inversed. This benefits drivers of controllers that by default operate with inversed polarity by removing the need to modify the polarity during initialization. Signed-off-by: Tim Kryger --- drivers/pwm

Re: [PATCH v4 2/3] pwm: kona: Remove setting default smooth type and polarity for all channels

2015-01-04 Thread Tim Kryger
On Tue, Dec 30, 2014 at 2:43 PM, Jonathan Richardson wrote: > Setting the default polarity in probe to normal for all channels caused > the speaker pwm channel to click. The polarity does need to be set to > normal because the hw default is inversed whereas the pwm framework > defaults to normal.

Re: Possible regression with commit 52221610d

2014-12-21 Thread Tim Kryger
On Wed, Dec 17, 2014 at 11:57 AM, Bjorn Andersson wrote: > I'm somewhat puzzled to what benefit 52221610d brings after bringing > back the write of BIT(0). Is it just that we don't hit the BUG() on > non-standard voltages? It is to allow the use of external regulators that are capable of supplyi

Re: [PATCH v3 1/2] pwm: kona: Fix incorrect enable, config, and disable procedures

2014-12-20 Thread Tim Kryger
On Wed, Dec 17, 2014 at 10:46 AM, Jonathan Richardson wrote: > The config procedure doesn't follow the spec which periodically results > in failing to enable the output signal. This happens one in ten or > twenty attempts. Following the spec and adding a 400ns delay in the > approp

Re: Possible regression with commit 52221610d

2014-12-16 Thread Tim Kryger
d to bit zero of that same register. The driver should be updated to configure bit zero appropriately even when an external regulator is used. If you like, I can propose a patch or if you have one ready, I will be happy to review yours. Thanks, Tim Kryger -- To unsubscribe from this list: se

Re: [PATCH v2 1/2] pwm: kona: Fix incorrect enable, config, and disable procedures

2014-12-14 Thread Tim Kryger
and trigger bit are both low, the output is constant high. If you look at the PWM output on a scope you will see it go high for 400 ns during your disable even if the duty prior to the disable was zero. How are you testing your proposed changes? Thanks, Tim Kryger -- To unsubscribe from this l

Re: Possible regression with commit 52221610d

2014-12-14 Thread Tim Kryger
On Sat, Dec 13, 2014 at 11:22 PM, Bjorn Andersson wrote: > On Tue, Nov 4, 2014 at 7:31 AM, Tim Kryger wrote: >> On Tue, Nov 4, 2014 at 1:00 AM, Alexandre Courbot >> wrote: >>> Hi Tim, thanks for your reply! >>> >>> On 11/04/2014 02:28 PM, Tim Kryger wr

Re: [PATCH v2 1/4] pwm: kona: Remove setting default smooth type and polarity for all channels

2014-12-06 Thread Tim Kryger
riod) but > may get a non smooth transition. So it's important to follow the spec > here. We don't want non-smooth transitions. Please provide your rationale for requiring smooth transitions. Thanks, Tim Kryger -- To unsubscribe from this list: send the line "unsubscribe lin

Re: [PATCH 2/4] clk: iproc: add initial common clock support

2014-12-06 Thread Tim Kryger
On Thu, Dec 4, 2014 at 1:43 PM, Ray Jui wrote: > This adds basic and generic support for various iProc PLLs and clocks > including the ARMPLL, GENPLL, LCPLL, MIPIPLL, and ASIU clocks. > > SoCs under the iProc architecture can define their specific register > offsets and clock parameters for their

Re: [PATCH v2 3/4] pwm: kona: Fix enable, disable and config procedures

2014-12-01 Thread Tim Kryger
On Mon, Dec 1, 2014 at 11:37 AM, Arun Ramamurthy wrote: > > > On 14-11-28 06:30 PM, Tim Kryger wrote: >> >> On Fri, Nov 28, 2014 at 3:49 PM, Arun Ramamurthy >> wrote: >>> >>> >>> >>> On 14-11-25 11:29 PM, Tim Kryger wrote:

Re: [PATCH v2 1/4] pwm: kona: Remove setting default smooth type and polarity for all channels

2014-11-28 Thread Tim Kryger
On Fri, Nov 28, 2014 at 5:19 PM, Arun Ramamurthy wrote: > > > On 14-11-28 05:08 PM, Tim Kryger wrote: >> >> On Fri, Nov 28, 2014 at 3:47 PM, Arun Ramamurthy >> wrote: >>> >>> >>> >>> On 14-11-25 09:51 PM, Tim Kryger wrote:

Re: [PATCH v2 3/4] pwm: kona: Fix enable, disable and config procedures

2014-11-28 Thread Tim Kryger
On Fri, Nov 28, 2014 at 3:49 PM, Arun Ramamurthy wrote: > > > On 14-11-25 11:29 PM, Tim Kryger wrote: >> >> On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden >> wrote: >>> >>> From: Arun Ramamurthy >>> >>> - Added helper functions to

Re: [PATCH v2 2/4] pwm: kona: Fix incorrect enable after channel polarity change

2014-11-28 Thread Tim Kryger
On Fri, Nov 28, 2014 at 3:48 PM, Arun Ramamurthy wrote: > > > On 14-11-25 10:22 PM, Tim Kryger wrote: >> >> On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden >> wrote: >>> >>> From: Arun Ramamurthy >>> >>> The pwm core code requ

Re: [PATCH v2 1/4] pwm: kona: Remove setting default smooth type and polarity for all channels

2014-11-28 Thread Tim Kryger
On Fri, Nov 28, 2014 at 3:47 PM, Arun Ramamurthy wrote: > > > On 14-11-25 09:51 PM, Tim Kryger wrote: >> >> On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden >> wrote: >>> >>> From: Arun Ramamurthy >>> >>> The probe routine unnec

Re: [PATCH v2 3/4] pwm: kona: Fix enable, disable and config procedures

2014-11-25 Thread Tim Kryger
On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden wrote: > From: Arun Ramamurthy > > - Added helper functions to set and clear smooth and trigger bits > - Added 400ns delays when clearing and setting trigger bit as requied > by spec > - Added helper function to write prescale and other settings >

Re: [PATCH v2 2/4] pwm: kona: Fix incorrect enable after channel polarity change

2014-11-25 Thread Tim Kryger
On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden wrote: > From: Arun Ramamurthy > > The pwm core code requires a separate call for enabling the channel > and hence the driver does not need to set pwm_trigger after a > polarity change The framework does restrict when polarity changes can occur but

Re: [PATCH v2 1/4] pwm: kona: Remove setting default smooth type and polarity for all channels

2014-11-25 Thread Tim Kryger
On Tue, Nov 25, 2014 at 11:40 AM, Scott Branden wrote: > From: Arun Ramamurthy > > The probe routine unnecessarily sets the smooth type and polarity for > all channels. This causes the channel for the speaker to click at the same > time the backlight turns on. The smooth type and polarity should

Re: Possible regression with commit 52221610d

2014-11-05 Thread Tim Kryger
On Wed, Nov 5, 2014 at 12:10 AM, Alexandre Courbot wrote: > On 11/05/2014 12:31 AM, Tim Kryger wrote: >> >> On Tue, Nov 4, 2014 at 1:00 AM, Alexandre Courbot >> wrote: >>> >>> Hi Tim, thanks for your reply! >>> >>> On 11/04/2014 02:28 PM,

Re: Possible regression with commit 52221610d

2014-11-04 Thread Tim Kryger
On Tue, Nov 4, 2014 at 1:00 AM, Alexandre Courbot wrote: > Hi Tim, thanks for your reply! > > On 11/04/2014 02:28 PM, Tim Kryger wrote: >> >> On Mon, Nov 3, 2014 at 7:05 PM, Alexandre Courbot >> wrote: >>> >>> Hi guys, >>> >>> On

Re: Possible regression with commit 52221610d

2014-11-03 Thread Tim Kryger
patch you proposed would break Exynos4210 so I don't think it is appropriate. Do you understand why this code block is executed on your hardware? I wouldn't expect it. Can you provide the relevant parts of the log before the problem occurs? Thanks, Tim Kryger -- 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 the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/1] mmc: core: Use regulator_get_voltage() if OCR mask is empty.

2014-08-17 Thread Tim Kryger
On Fri, Aug 15, 2014 at 3:29 PM, Mark Brown wrote: > On Fri, Aug 15, 2014 at 07:19:41AM -0700, Tim Kryger wrote: > >> That is a little different from my suggestion where the constraints >> check is skipped when the regulator output is fixed. It effectively >> does thi

Re: [PATCH 1/1] mmc: core: Use regulator_get_voltage() if OCR mask is empty.

2014-08-15 Thread Tim Kryger
On Fri, Aug 15, 2014 at 12:48 AM, Javier Martinez Canillas wrote: > Hello Tim, > > On 08/15/2014 07:36 AM, Tim Kryger wrote: >> On Thu, Aug 14, 2014 at 8:19 AM, Mark Brown wrote: >> >>> Right, there's two things going on here. One is that as you describe we &

Re: [PATCH 1/1] mmc: core: Use regulator_get_voltage() if OCR mask is empty.

2014-08-14 Thread Tim Kryger
e its voltage? It seems odd to make callers be the ones to handle this subtlety. Thanks, Tim Kryger -- 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 the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/1] mmc: core: Use regulator_get_voltage() if OCR mask is empty.

2014-08-14 Thread Tim Kryger
On Thu, Aug 14, 2014 at 5:39 AM, Javier Martinez Canillas wrote: > The operation conditions register (OCR) stores the voltage > profile of the card, however the list of possible voltages > is restricted by the voltage range supported by the supply > used as VCC/VDD. So in mmc_vddrange_to_ocrmask()

[PATCH v2] mmc: core: Remove fixed voltage regulator logic

2014-08-11 Thread Tim Kryger
There is no need for regulator consumers to include special logic for fixed voltage regulators as they support regulator_set_voltage() just like their non-fixed regulator counterparts. Signed-off-by: Tim Kryger --- Changes in v2: - Remove query of the current voltage, just set the desired

[RESEND PATCH] mmc: core: Remove fixed voltage regulator logic

2014-07-29 Thread Tim Kryger
There is no need for regulator consumers to include special logic for fixed voltage regulators as they support regulator_set_voltage() just like their non-fixed regulator counterparts. Signed-off-by: Tim Kryger --- Since this eliminates logic that was concealing a bug in how the SDHCI driver

[PATCH] mmc: core: Remove fixed voltage regulator logic

2014-07-22 Thread Tim Kryger
There is no need for regulator consumers to include special logic for fixed voltage regulators as they support regulator_set_voltage() just like their non-fixed regulator counterparts. Signed-off-by: Tim Kryger --- Since this eliminates logic that was concealing a bug in how the SDHCI driver

[PATCH] mmc: sdhci: Improve external VDD regulator support

2014-06-25 Thread Tim Kryger
ff-by: Tim Kryger Tested-by: Sachin Kamat --- This change corrects a regression observed on Exynos platforms that was triggered by "mmc: sdhci: Use mmc core regulator infrastucture" https://www.mail-archive.com/linux-mmc@vger.kernel.org/msg27101.html drivers/mmc/host/sdhci.c | 16

Re: [PATCH 1/3] regulator: bcm590xx: remove unnecessary OOM messages

2014-06-02 Thread Tim Kryger
On Mon, Jun 2, 2014 at 5:50 AM, Mark Brown wrote: > On Mon, Jun 02, 2014 at 12:12:20AM -0700, Tim Kryger wrote: > >> The probe method of this driver, on the other hand, performs several >> allocations and the error messages you intend to remove conveniently >> pinpoint

Re: [PATCH 1/3] regulator: bcm590xx: remove unnecessary OOM messages

2014-06-02 Thread Tim Kryger
The probe method of this driver, on the other hand, performs several allocations and the error messages you intend to remove conveniently pinpoint which one failed. While the offsets in the trace could be used to derive the same information, I am skeptical that is enough to justify removing the messages. Thanks, Tim Kryger -- 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 the FAQ at http://www.tux.org/lkml/

Re: [PATCH] mmc: sdhci: Use mmc core regulator infrastucture

2014-05-06 Thread Tim Kryger
On Fri, Apr 25, 2014 at 1:15 AM, Ulf Hansson wrote: > On 25 April 2014 00:36, Tim Kryger wrote: >> Switch the common SDHCI code over to use mmc_host's regulator pointers >> and remove the ones in the sdhci_host structure. Additionally, use the >> common mmc_regulator_

Re: [PATCH v2] mmc: core: Improve support for deferred regulators

2014-05-06 Thread Tim Kryger
On Tue, May 6, 2014 at 3:57 PM, Tim Kryger wrote: > Callers of mmc_regulator_get_supply could benefit from knowing if either > of the regulators are present but not yet available. Since callers do > not currently examine the return value, modify this function to return > zero or -EPR

[PATCH v2] mmc: core: Improve support for deferred regulators

2014-05-06 Thread Tim Kryger
callers check vmmc/vqmmc using IS_ERR and can deal with absent regulators, switch to devm_regulator_get_optional. This has the added benefit of allowing this function to behave correctly even in the !CONFIG_REGULATOR case such that the stub can be removed. Signed-off-by: Tim Kryger --- drivers

Re: [PATCH RESEND] mmc: core: Improve support for deferred regulators

2014-04-29 Thread Tim Kryger
On Fri, Apr 25, 2014 at 2:33 PM, Andrew Bresticker wrote: >> >> - if (IS_ERR(supply)) >> - return PTR_ERR(supply); >> + if (IS_ERR(mmc->supply.vmmc)) { >> + if (PTR_ERR(mmc->supply.vmmc) == -EPROBE_DEFER) >> + return -EPROBE_DEFER; >> +

Re: [PATCH v6 RESEND 0/5] Add Broadcom Kona PWM Support

2014-04-28 Thread Tim Kryger
On Mon, Apr 28, 2014 at 4:07 AM, Thierry Reding wrote: > On Fri, Apr 25, 2014 at 11:31:10AM -0700, Tim Kryger wrote: >> This series introduces the driver for the Kona PWM controller found in >> Broadcom mobile SoCs like bcm281xx and updates the device tree and the >> defco

[PATCH RESEND] mmc: core: Improve support for deferred regulators

2014-04-25 Thread Tim Kryger
eturn value and may continue to use IS_ERR on the vmmc/vqmmc values to determine if those regulators are available. Furthermore, since all of these callers are capable of dealing with absent regulators, switch over to use the non-optional variety of the regulator get call. Signed-off-by: Tim K

[PATCH v6 RESEND 3/5] ARM: dts: Declare the PWM for bcm11351 (bcm281xx)

2014-04-25 Thread Tim Kryger
Add the device tree node for the PWM on bcm11351 SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm11351.dtsi |8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351

[PATCH v6 RESEND 4/5] ARM: dts: Enable the PWM for bcm28155 AP board

2014-04-25 Thread Tim Kryger
Mark the PWM as enabled on the bcm28155 AP board. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm28155-ap.dts |4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/bcm28155-ap.dts b/arch/arm/boot/dts/bcm28155-ap.dts

[PATCH v6 RESEND 1/5] Documentation: dt: Add Kona PWM binding

2014-04-25 Thread Tim Kryger
Add the binding description for the Kona PWM controller found on Broadcom's mobile SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- .../devicetree/bindings/pwm/bcm-kona-pwm.txt | 21 1 file changed, 21 insertions(+) c

[PATCH v6 RESEND 5/5] ARM: bcm_defconfig: Enable PWM and Backlight

2014-04-25 Thread Tim Kryger
Enable PWM drivers and the PWM-based backlight driver. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/configs/bcm_defconfig |3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/configs/bcm_defconfig b/arch/arm/configs/bcm_defconfig index

[PATCH v6 RESEND 2/5] pwm: kona: Introduce Kona PWM controller support

2014-04-25 Thread Tim Kryger
Add support for the six-channel Kona PWM controller found on Broadcom mobile SoCs like bcm281xx. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- drivers/pwm/Kconfig|9 ++ drivers/pwm/Makefile |1 + drivers/pwm/pwm-bcm-kona.c | 319

[PATCH v6 RESEND 0/5] Add Broadcom Kona PWM Support

2014-04-25 Thread Tim Kryger
cient - Made prescale and duty computation clearer - Moved Makefile addition to keep alphabetical - Split complex lines into multiple steps Tim Kryger (5): Documentation: dt: Add Kona PWM binding pwm: kona: Introduce Kona PWM controller support ARM: dts: Declare the PWM for bcm11351 (bcm

[PATCH] mmc: sdhci: Use mmc core regulator infrastucture

2014-04-24 Thread Tim Kryger
ltage ranges supported which ensures ocr_avail is set correctly while allowing the use of regulators that can't provide exactly 1.8v, 3.0v, or 3.3v. Signed-off-by: Tim Kryger --- This patch is the same as the following series only squashed together. https://lkml.org/lkml/2014/4/17/653 d

Re: [PATCH 0/2] SDHCI should use more common MMC structs and functions

2014-04-24 Thread Tim Kryger
On Thu, Apr 24, 2014 at 1:34 AM, Ulf Hansson wrote: > On 18 April 2014 02:46, Tim Kryger wrote: >> This series updates SDHCI to use the common regulator infrastructure that mmc >> core provides. >> >> Tim Kryger (2): >> mmc: sdhci: Use supplies in common mmc

[PATCH v2] mmc: core: Try other signal levels during power up

2014-04-24 Thread Tim Kryger
but when that can't be set, 1.8v and 1.2v signalling also should be attempted. This is especially important when an external regulator with a limited range is used to supply VCCQ to an eMMC part. Signed-off-by: Tim Kryger --- Changes since v1: - Eliminated temporary err variable and added

Re: [PATCH] mmc: core: Try other signal levels during power up

2014-04-24 Thread Tim Kryger
On Thu, Apr 24, 2014 at 1:38 AM, Ulf Hansson wrote: > On 18 April 2014 20:58, Tim Kryger wrote: >> void mmc_power_up(struct mmc_host *host, u32 ocr) >> { >> + int err; >> + > > The variable err is not needed, since we don't plan on printing it

Re: [PATCH] regulator: Allow set voltage on fixed regulators

2014-04-22 Thread Tim Kryger
On Fri, Apr 18, 2014 at 11:52 AM, Mark Brown wrote: > On Fri, Apr 18, 2014 at 11:30:10AM -0700, Tim Kryger wrote: >> If a regulator consumer requests a voltage range that can be satisfied, >> the return value should indicate success even if that regulator has a >> fixed vol

Re: [PATCH] mmc: sdhci: Set ocr_avail directly based on vmmc

2014-04-18 Thread Tim Kryger
On Wed, Apr 16, 2014 at 12:20 AM, Ulf Hansson wrote: > On 15 April 2014 19:09, Tim Kryger wrote: >> On Fri, Apr 11, 2014 at 1:15 AM, Ulf Hansson wrote: >>> A few times I have suggested to switch to use the >>> mmc_regulator_get_supply() API to simplify and cons

Re: [PATCH] regulator: Allow set voltage on fixed regulators

2014-04-18 Thread Tim Kryger
On Fri, Apr 18, 2014 at 11:52 AM, Mark Brown wrote: > On Fri, Apr 18, 2014 at 11:30:10AM -0700, Tim Kryger wrote: >> If a regulator consumer requests a voltage range that can be satisfied, >> the return value should indicate success even if that regulator has a >> fixed vol

[PATCH] mmc: core: Try other signal levels during power up

2014-04-18 Thread Tim Kryger
but when that can't be set, 1.8v and 1.2v signalling also should be attempted. This is especially important when an external regulator with a limited range is used to supply VCCQ to an eMMC part. Signed-off-by: Tim Kryger --- drivers/mmc/core/core.c | 10 +- 1 file changed, 9 inser

[PATCH] regulator: Allow set voltage on fixed regulators

2014-04-18 Thread Tim Kryger
with constraints that include a positive voltage. Signed-off-by: Tim Kryger --- drivers/regulator/of_regulator.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/regulator/of_regulator.c b/drivers/regulator/of_regulator.c index ea4f36f..a205d62 100644 --- a/drivers

[PATCH v6 0/5] Add Broadcom Kona PWM Support

2014-04-18 Thread Tim Kryger
cient - Made prescale and duty computation clearer - Moved Makefile addition to keep alphabetical - Split complex lines into multiple steps Tim Kryger (5): Documentation: dt: Add Kona PWM binding pwm: kona: Introduce Kona PWM controller support ARM: dts: Declare the PWM for bcm11351 (bcm

[PATCH v6 2/5] pwm: kona: Introduce Kona PWM controller support

2014-04-18 Thread Tim Kryger
Add support for the six-channel Kona PWM controller found on Broadcom mobile SoCs like bcm281xx. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- drivers/pwm/Kconfig|9 ++ drivers/pwm/Makefile |1 + drivers/pwm/pwm-bcm-kona.c | 319

[PATCH v6 5/5] ARM: bcm_defconfig: Enable PWM and Backlight

2014-04-18 Thread Tim Kryger
Enable PWM drivers and the PWM-based backlight driver. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/configs/bcm_defconfig |3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/configs/bcm_defconfig b/arch/arm/configs/bcm_defconfig index

[PATCH v6 1/5] Documentation: dt: Add Kona PWM binding

2014-04-18 Thread Tim Kryger
Add the binding description for the Kona PWM controller found on Broadcom's mobile SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- .../devicetree/bindings/pwm/bcm-kona-pwm.txt | 21 1 file changed, 21 insertions(+) c

[PATCH v6 3/5] ARM: dts: Declare the PWM for bcm11351 (bcm281xx)

2014-04-18 Thread Tim Kryger
Add the device tree node for the PWM on bcm11351 SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm11351.dtsi |8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351

[PATCH v6 4/5] ARM: dts: Enable the PWM for bcm28155 AP board

2014-04-18 Thread Tim Kryger
Mark the PWM as enabled on the bcm28155 AP board. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm28155-ap.dts |4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/bcm28155-ap.dts b/arch/arm/boot/dts/bcm28155-ap.dts

[PATCH 1/2] mmc: sdhci: Use supplies in common mmc_host struct

2014-04-17 Thread Tim Kryger
Switch the common SDHCI code over to use mmc_host's regulator pointers and remove the ones in the sdhci_host structure. Signed-off-by: Tim Kryger --- drivers/mmc/host/sdhci.c | 71 - include/linux/mmc/sdhci.h |3 -- 2 files change

[PATCH 0/2] SDHCI should use more common MMC structs and functions

2014-04-17 Thread Tim Kryger
This series updates SDHCI to use the common regulator infrastructure that mmc core provides. Tim Kryger (2): mmc: sdhci: Use supplies in common mmc_host struct mmc: sdhci: Use common mmc_regulator_get_supply drivers/mmc/host/sdhci.c | 96

[PATCH 2/2] mmc: sdhci: Use common mmc_regulator_get_supply

2014-04-17 Thread Tim Kryger
dress this by relaxing the range checks in the SDHCI driver but it set the capabilities as an intermediate step which meant ocr_avail could get bits set for unsupported voltages. Signed-off-by: Tim Kryger --- drivers/mmc/host/sdhci.c | 67 +- 1 file ch

[PATCH] mmc: core: Improve support for deferred regulators

2014-04-17 Thread Tim Kryger
eturn value and may continue to use IS_ERR on the vmmc/vqmmc values to determine if those regulators are available. Furthermore, since all of these callers are capable of dealing with absent regulators, switch over to use the non-optional variety of the regulator get call. Signed-off-by: Tim K

[PATCH] regulator: core: Return error in get optional stub

2014-04-17 Thread Tim Kryger
call regulator_is_supported_voltage. This improves efficiency as well as eliminates the possibility for a very subtle bug. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder --- This change was proposed here: https://lkml.org/lkml/2014/4/16/627 include/linux/regulator/consumer.h |4 ++-- 1

Re: [PATCH 3/4] mmc: sdhci: defer probing on regulator_get_optional() failures

2014-04-15 Thread Tim Kryger
On Mon, Apr 14, 2014 at 6:42 PM, Andrew Bresticker wrote: > If regulator_get_optional() returns EPROBE_DEFER, it indicates > that the regulator may show up later (e.g. the DT property is > present but the corresponding regulator may not have probed). > Instead of continuing without the regulator,

Re: [PATCH] mmc: sdhci: Set ocr_avail directly based on vmmc

2014-04-15 Thread Tim Kryger
On Fri, Apr 11, 2014 at 1:15 AM, Ulf Hansson wrote: > I have seen some patches around lately touching the code for handling > the regulators (vcc and vccq) in sdhci. Was it this patch you were thinking of or something else? http://www.spinics.net/lists/linux-mmc/msg25640.html > A few times I h

Re: [PATCH] mmc: sdhci: Set ocr_avail directly based on vmmc

2014-04-11 Thread Tim Kryger
On Fri, Apr 11, 2014 at 1:15 AM, Ulf Hansson wrote: > On 11 April 2014 01:31, Tim Kryger wrote: >> +static unsigned int sdhci_get_ocr_avail_from_vmmc(struct sdhci_host *host) >> +{ >> + unsigned int ocr_avail = 0; >> + struct regulator *vmmc = hos

[PATCH] gpio: bcm-kona: Add irq_set_wake method

2014-04-10 Thread Tim Kryger
If a consumer of a GPIO-based IRQ identifies itself as wanting to have the capability to wake up the system through that interrupt, update the flags to keep that interrupt enabled in suspend. Signed-off-by: Tim Kryger --- drivers/gpio/gpio-bcm-kona.c | 19 +++ 1 file changed

[PATCH] mmc: sdhci: Set ocr_avail directly based on vmmc

2014-04-10 Thread Tim Kryger
ed to relax the range checks but because it relied on setting capabilities as an intermediate step, ocr_avail could easily get a bit set that the host couldn't support. Signed-off-by: Tim Kryger --- drivers/mmc/host/sdhci.c | 107 +- 1 file c

[PATCH] regulator: bcm590xx: Set n_voltages for linear reg

2014-04-10 Thread Tim Kryger
Fix the macro used to define linear range regulators to include the number of voltages. Signed-off-by: Tim Kryger --- drivers/regulator/bcm590xx-regulator.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/regulator/bcm590xx-regulator.c b/drivers/regulator/bcm590xx-regulator.c

[RESEND PATCH v5 0/5] Add Broadcom Kona PWM Support

2014-04-09 Thread Tim Kryger
//lkml.org/lkml/2014/2/14/451 Tim Kryger (5): Documentation: dt: Add Kona PWM binding pwm: kona: Introduce Kona PWM controller support ARM: dts: Declare the PWM for bcm11351 (bcm281xx) ARM: dts: Enable the PWM for bcm28155 AP board ARM: bcm_defconfig: Enable PWM and Backlight .../de

[RESEND PATCH v5 3/5] ARM: dts: Declare the PWM for bcm11351 (bcm281xx)

2014-04-09 Thread Tim Kryger
Add the device tree node for the PWM on bcm11351 SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm11351.dtsi |8 1 file changed, 8 insertions(+) diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351

[RESEND PATCH v5 1/5] Documentation: dt: Add Kona PWM binding

2014-04-09 Thread Tim Kryger
Add the binding description for the Kona PWM controller found on Broadcom's mobile SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- .../devicetree/bindings/pwm/bcm-kona-pwm.txt | 21 1 file changed, 21 insertions(+) c

[RESEND PATCH v5 4/5] ARM: dts: Enable the PWM for bcm28155 AP board

2014-04-09 Thread Tim Kryger
Mark the PWM as enabled on the bcm28155 AP board. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm28155-ap.dts |4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/bcm28155-ap.dts b/arch/arm/boot/dts/bcm28155-ap.dts

[RESEND PATCH v5 5/5] ARM: bcm_defconfig: Enable PWM and Backlight

2014-04-09 Thread Tim Kryger
Enable PWM drivers and the PWM-based backlight driver. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/configs/bcm_defconfig |3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/configs/bcm_defconfig b/arch/arm/configs/bcm_defconfig index

[RESEND PATCH v5 2/5] pwm: kona: Introduce Kona PWM controller support

2014-04-09 Thread Tim Kryger
Add support for the six-channel Kona PWM controller found on Broadcom mobile SoCs like bcm281xx. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- drivers/pwm/Kconfig|9 ++ drivers/pwm/Makefile |1 + drivers/pwm/pwm-bcm-kona.c | 319

Re: [rfc]pwm: add BCM2835 PWM driver

2014-04-09 Thread Tim Kryger
On Tue, Apr 8, 2014 at 6:27 PM, Stephen Warren wrote: > On 04/08/2014 05:02 PM, Tim Kryger wrote: >> On Thu, Apr 3, 2014 at 6:44 AM, Bart Tanghe >> wrote: >>> need some recommendation >>> the memory mapped io registers of the bcm2835 pwm hardware are sprea

Re: [rfc]pwm: add BCM2835 PWM driver

2014-04-08 Thread Tim Kryger
On Thu, Apr 3, 2014 at 6:44 AM, Bart Tanghe wrote: > need some recommendation > the memory mapped io registers of the bcm2835 pwm hardware are spreaded > over the memory mapped io > gpio config 0x2024 - clk config 0x201010A0 - pwm configuration 0x2020C000 > to handle this, I've used the base a

Re: [PATCHv3 1/3] pwm: make the PWM_POLARITY flag in DTB optional

2014-04-08 Thread Tim Kryger
On Mon, Apr 7, 2014 at 10:02 PM, Lothar Waßmann wrote: > Thierry Reding wrote: >> No. You cannot emulate polarity inversion in software. >> > Why not? > > duty_ns = period_ns - duty_ns; Since I made the same mistake, I will pass along the pointer Thierry gave me. In include/linux/pwm.h the sec

Re: [PATCH 2/5] ARM: add SMP support for Broadcom mobile SoCs

2014-04-04 Thread Tim Kryger
On Thu, Apr 3, 2014 at 7:18 PM, Alex Elder wrote: > diff --git a/arch/arm/mach-bcm/platsmp.c b/arch/arm/mach-bcm/platsmp.c > new file mode 100644 > index 000..46a64f2 > --- /dev/null > +++ b/arch/arm/mach-bcm/platsmp.c > +/* Size of mapped Cortex A9 SCU address space */ > +#define SCU_SIZE

[RESEND PATCH v5 4/5] ARM: dts: Enable the PWM for bcm28155 AP board

2014-04-02 Thread Tim Kryger
Mark the PWM as enabled on the bcm28155 AP board. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- arch/arm/boot/dts/bcm28155-ap.dts |4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/bcm28155-ap.dts b/arch/arm/boot/dts/bcm28155-ap.dts

[RESEND PATCH v5 1/5] Documentation: dt: Add Kona PWM binding

2014-04-02 Thread Tim Kryger
Add the binding description for the Kona PWM controller found on Broadcom's mobile SoCs. Signed-off-by: Tim Kryger Reviewed-by: Alex Elder Reviewed-by: Markus Mayer --- .../devicetree/bindings/pwm/bcm-kona-pwm.txt | 21 1 file changed, 21 insertions(+) c

  1   2   3   >