[PATCH v3 1/3] ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels

2020-10-21 Thread Alexandru Stan
ost a noop/cleanup now, but it will be required in the future. Signed-off-by: Alexandru Stan --- arch/arm/boot/dts/rk3288-veyron-jaq.dts| 2 +- arch/arm/boot/dts/rk3288-veyron-minnie.dts | 2 +- arch/arm/boot/dts/rk3288-veyron-tiger.dts | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)

[PATCH v3 3/3] backlight: pwm_bl: Fix interpolation

2020-10-21 Thread Alexandru Stan
tervals (num-interpolated-steps sized, as opposed to before where we were at the mercy of quantization). END Signed-off-by: Alexandru Stan --- drivers/video/backlight/pwm_bl.c | 70 ++-- 1 file changed, 31 insertions(+), 39 deletions(-) diff --git a/drivers/vid

[PATCH v3 2/3] arm64: dts: qcom: trogdor: Add brightness-levels

2020-10-21 Thread Alexandru Stan
avoid sending it anything lower than that. Note: Ideally this patch should be coupled with the driver change from "backlight: pwm_bl: Fix interpolation", but it can work without it, without looking too ugly. Signed-off-by: Alexandru Stan --- arch/arm64/boot/dts/qcom/sc7180-trogdo

[PATCH v3 0/3] PWM backlight interpolation adjustments

2020-10-21 Thread Alexandru Stan
ation", userspace works just fine without it because it already knows how to use bl_power for turning off the display. - Added brightness-levels to trogdor as well, now the dts is upstream. Alexandru Stan (3): ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels arm64: dts: qco

Re: [PATCH v2 1/3] backlight: pwm_bl: Fix interpolation

2020-10-21 Thread Alexandru Stan
On Wed, 14 Oct 2020 at 23:55, Geert Uytterhoeven wrote: > > Hi Alexandru, > > On Tue, Oct 13, 2020 at 1:57 PM Alexandru Stan wrote: > > Whenever num-interpolated-steps was larger than the distance > > between 2 consecutive brightness levels the table would get really >

[PATCH v2 2/3] ARM: dts: rockchip: veyron: Remove 0 point from brightness-levels

2020-10-13 Thread Alexandru Stan
between 0 and 3, looking really bad in userspace. We don't need the 0% point, userspace seems to handle this just fine because it uses the bl_power property to turn off the display. Signed-off-by: Alexandru Stan --- arch/arm/boot/dts/rk3288-veyron-jaq.dts| 2 +- arch/arm/boot/dts/rk3

[PATCH v2 3/3] arm64: dts: qcom: trogdor: Add brightness-levels

2020-10-13 Thread Alexandru Stan
Now that we have better interpolation for the backlight ("backlight: pwm_bl: Fix interpolation"), we can now add the curve to the trogdor boards, being careful to crop the low end. Signed-off-by: Alexandru Stan --- arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 9 + 1 file

[PATCH v2 1/3] backlight: pwm_bl: Fix interpolation

2020-10-13 Thread Alexandru Stan
ts available (read: values in the table would appear more than once). This should match the expected behavior much more closely. Signed-off-by: Alexandru Stan --- drivers/video/backlight/pwm_bl.c | 70 ++-- 1 file changed, 31 insertions(+), 39 deletions(-) diff --git a/d

[PATCH v2 0/3] PWM backlight interpolation adjustments

2020-10-13 Thread Alexandru Stan
ially add 0% during interpolation", userspace works just fine without it because it already knows how to use bl_power for turning off the display. - Added brightness-levels to trogdor as well, now the dts is upstream. Alexandru Stan (3): backlight: pwm_bl: Fix interpolation ARM: dts

[PATCH 0/3] PWM backlight interpolation adjustments

2020-07-20 Thread Alexandru Stan
to do this conditionally because it seems some devices like to have the scale inverted: % git grep "brightness-levels\s*=\s*<\s*[1-9]"|cat arch/arm/boot/dts/tegra124-apalis-eval.dts: brightness-levels = <255 231 223 207 191 159 127 0>; Alexandru Stan (3): backlight:

[PATCH 1/3] backlight: pwm_bl: Fix interpolation

2020-07-20 Thread Alexandru Stan
ts available (read: values in the table would appear more than once). This should match the expected behavior much more closely. Reviewed-by: Douglas Anderson Reviewed-by: Matthias Kaehlcke Signed-off-by: Alexandru Stan --- drivers/video/backlight/pwm_bl.c | 70 ++-- 1

[PATCH 2/3] backlight: pwm_bl: Artificially add 0% during interpolation

2020-07-20 Thread Alexandru Stan
Some displays need the low end of the curve cropped in order to make them happy. In that case we still want to have the 0% point, even though anything between 0% and 5%(example) would be skipped. Signed-off-by: Alexandru Stan --- drivers/video/backlight/pwm_bl.c | 8 1 file changed, 8

[PATCH 3/3] ARM: dts: rockchip: Remove 0 point in backlight

2020-07-20 Thread Alexandru Stan
between 0 and 3, looking really bad in userspace. We'll still have the 0% point available to userspace, "backlight: pwm_bl: Artificially add 0% during interpolation" takes care of that. Signed-off-by: Alexandru Stan --- arch/arm/boot/dts/rk3288-veyron-jaq.dts| 2 +- arch/arm/bo

Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.

2018-03-07 Thread Alexandru Stan
On Wed, Mar 7, 2018 at 11:34 AM, Rob Herring wrote: > On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: >> The extcon property is used to detect the cable-state but some boards >> just connect the type-c phy to a regular USB-A connector without any >>

Re: [PATCH v2 2/6] dt-bindings: phy-rockchip-typec: move extcon property to be optional.

2018-03-07 Thread Alexandru Stan
On Wed, Mar 7, 2018 at 11:34 AM, Rob Herring wrote: > On Thu, Mar 01, 2018 at 04:25:11PM +0100, Enric Balletbo i Serra wrote: >> The extcon property is used to detect the cable-state but some boards >> just connect the type-c phy to a regular USB-A connector without any >> power-delivery and thus

Re: [PATCH v2 00/10] mfd: cros_ec: Add multi EC and proto v3 support

2015-05-11 Thread Alexandru Stan
just a standard i2c device directly connected to the AP. Alexandru Stan (amstan) -- 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 v2 00/10] mfd: cros_ec: Add multi EC and proto v3 support

2015-05-11 Thread Alexandru Stan
to the AP. Alexandru Stan (amstan) -- 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/4] regulator: core: Support trying to get close to a certain voltage

2014-12-10 Thread Alexandru Stan
). The correct voltage would be 1.79V It's unfortunate that regulator_set_voltage was designed to always pick the lowest voltage in that range. I understand that it's a power efficiency thing, but it's not ideal in cases like this. Alexandru Stan (amstan) On Wed, Dec 10, 2014 at 3:53 PM, Mark Brown

Re: [PATCH 1/4] regulator: core: Support trying to get close to a certain voltage

2014-12-10 Thread Alexandru Stan
). The correct voltage would be 1.79V It's unfortunate that regulator_set_voltage was designed to always pick the lowest voltage in that range. I understand that it's a power efficiency thing, but it's not ideal in cases like this. Alexandru Stan (amstan) On Wed, Dec 10, 2014 at 3:53 PM, Mark Brown

Re: [PATCH v3] mmc: dw_mmc: add quirk for broken data transfer over scheme

2014-12-05 Thread Alexandru Stan
msecs_to_jiffies(drto_ms)); + } break; + } host->data = NULL; set_bit(EVENT_DATA_COMPLETE, >completed_events); Alexandru Stan (amstan) On Tue, Dec 2, 2014 at 9:08 P

Re: [PATCH v3] mmc: dw_mmc: add quirk for broken data transfer over scheme

2014-12-05 Thread Alexandru Stan
)); + } break; + } host-data = NULL; set_bit(EVENT_DATA_COMPLETE, host-completed_events); Alexandru Stan (amstan) On Tue, Dec 2, 2014 at 9:08 PM, Doug Anderson diand

Re: [PATCH] mmc: dw_mmc: try pick the exact same voltage as vmmc for vqmmc

2014-11-25 Thread Alexandru Stan
ing (you're essentially powering the sd card through the IO pins, or the SD card is powering the IO block on the AP). Alexandru Stan (amstan) On Mon, Nov 24, 2014 at 9:36 PM, Doug Anderson wrote: > Addy, > > On Mon, Nov 24, 2014 at 6:38 PM, Addy wrote: >>> In worst case scenari

Re: [PATCH] mmc: dw_mmc: try pick the exact same voltage as vmmc for vqmmc

2014-11-25 Thread Alexandru Stan
(you're essentially powering the sd card through the IO pins, or the SD card is powering the IO block on the AP). Alexandru Stan (amstan) On Mon, Nov 24, 2014 at 9:36 PM, Doug Anderson diand...@chromium.org wrote: Addy, On Mon, Nov 24, 2014 at 6:38 PM, Addy addy...@rock-chips.com wrote: In worst

Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Alexandru Stan
CLK_SDMMC) is twice as fast as the real clock(see RK3288_CLKGEN_DIV in drivers/mmc/host/dw_mmc-rockchip.c), since that clock goes to a /2 divider to be able to do the 90 degree clock phases. Alexandru Stan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in th

Re: [PATCH v2 1/2] clk: rockchip: add bindings for the mmc clock phases

2014-11-17 Thread Alexandru Stan
(eg SCLK_SDMMC) is twice as fast as the real clock(see RK3288_CLKGEN_DIV in drivers/mmc/host/dw_mmc-rockchip.c), since that clock goes to a /2 divider to be able to do the 90 degree clock phases. Alexandru Stan -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body