lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <562A3AEB.2080502@redhat.com> Date: Fri, 23 Oct 2015 15:49:31 +0200 From: Hans de Goede <hdegoede@...hat.com> To: Chen-Yu Tsai <wens@...e.org>, Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>, Tomi Valkeinen <tomi.valkeinen@...com>, Rob Herring <robh+dt@...nel.org>, Pawel Moll <pawel.moll@....com>, Mark Rutland <mark.rutland@....com>, Ian Campbell <ijc+devicetree@...lion.org.uk>, Kumar Gala <galak@...eaurora.org>, Maxime Ripard <maxime.ripard@...e-electrons.com> Cc: linux-fbdev@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, Mark Brown <broonie@...nel.org> Subject: Re: [PATCH v4 0/4] simplefb: Add regulator handling support Hi, Patches 1/2 look good and are: Reviewed-by: Hans de Goede <hdegoede@...hat.com> And the approach used for the regulators in them is also: Acked-by: Mark Brown <broonie@...nel.org> So patches 1/2 are ready for merging. Jean-Christophe or Tomi, can you pick up patch 1 & 2 please ? Regards, Hans On 10/23/2015 05:50 AM, Chen-Yu Tsai wrote: > Hi everyone, > > This is v4 of the simplefb regulator support series. This series adds > regulator claiming and enabling support for simplefb. > > Changes since v4: > - Fixed inverted logic when testing the property name. > - Fixed regulator supply name string copy length off by 1. > - Added real world user, MSI Primo 81 dts patches. > > Changes since v3: > - Dropped extra "if" which is always true, leftover from v1. > - Updated commit message of patch 1 > > Sometimes the simplefb display output path consits of external conversion > chips and/or LCD drivers and backlights. These devices normally have > GPIOs to turn them on and/or bring them out of reset, and regulators > supplying power to them. > > While the kernel does not touch unclaimed GPIOs, the regulator core > happily disables unused regulators. Thus we need simplefb to claim > and enable the regulators used throughout the display pipeline. > > The binding supports any named regulator supplies under its device > node. The driver will look through its properties, and claim any > regulators by matching "*-supply", as Mark suggested. > > I've not done a generic helper in the regulator core yet, instead doing > the regulator property handling in the simplefb code for now. > > > Patch 1 adds the regulator properties to the DT binding. > > Patch 2 adds code to the simplefb driver to claim and enable regulators. > Hans, I dropped your Reviewed-by tag from this patch. > > Patch 3 adds labels to the simplefb device nodes in sun6i-a31.dtsi > so we can reference them in the board dts files, like in the next > patch. > > Patch 4 adds a dts file for MSI's Primo 81 tablet. > > > Regards > ChenYu > > Chen-Yu Tsai (3): > dt-bindings: simplefb: Support regulator supply properties > simplefb: Claim and enable regulators > ARM: dts: sun6i: Add simplefb node labels to reference at board level > > Karsten Merker (1): > ARM: dts: sun6i: Add dts file for MSI Primo81 tablet > > .../bindings/video/simple-framebuffer.txt | 13 +- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/sun6i-a31.dtsi | 4 +- > arch/arm/boot/dts/sun6i-a31s-primo81.dts | 255 +++++++++++++++++++++ > drivers/video/fbdev/simplefb.c | 120 +++++++++- > 5 files changed, 386 insertions(+), 7 deletions(-) > create mode 100644 arch/arm/boot/dts/sun6i-a31s-primo81.dts > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists