On Wed, 2019-05-08 at 18:07 +0300, Andy Shevchenko wrote: > On Wed, Apr 24, 2019 at 10:44 AM Lubomir Rintel <lkundrak@xxxxx> wrote: > > Hello! > > > > This is a sixth spin of the patch set that adds support for the Embedded > > Controller on an OLPC XO 1.75 machine. > > > > Compared to the previous version there are only some trivial-ish changes to > > [06/10] and addition of Reviewd-by tags; as indicated in changelogs of > > individual patches. > > > > In my opinion the first seven patches are ready for merge as they are: > > > > [01/10] dt-bindings: olpc,xo1.75-ec: Add OLPC XO-1.75 EC > > [02/10] Platform: OLPC: Remove an unused include > > [03/10] Platform: OLPC: Move EC-specific functionality out from > > [04/10] Platform: OLPC: Avoid a warning if the EC didn't > > [05/10] Platform: OLPC: Use BIT() and GENMASK() for event masks > > [06/10] Platform: OLPC: Add XO-1.75 EC driver > > [07/10] Platform: OLPC: Add a regulator for the DCON > > > > The patch [08/10] relies on changes to the OLPC battery driver that have > > recently landed in the power-supply tree (without them enabling > > CONFIG_BATTERY_OLPC on ARM would break build): > > > > [08/10] power: supply: olpc_battery: Allow building the driver > > > > Sebastian Reichel provided an signed tag of an immutable branch that has > > the necessary bits and was merged into the power-supply tree for 5.2: > > > > git pull git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git psy-olpc-1.75-battery-signed > > > > The patch [10/10] was split of from the rest because Russell King found the > > patch [09/10] it depends on possibly objectionable. They should not be > > applied without further input from Russell: > > > > [09/10] ARM: export arm_pm_restart > > [10/10] Platform: OLPC: Add restart support to XO-1.75 EC > > > > Tested to work on an OLPC XO 1.75 and also tested not to break x86 > > support with an OLPC XO 1 machine. I don't have a XO 1.5, but it's > > unlikely this breaks it when XO 1 works. > > Pushed first 8 to my review and testing queue, thanks! Thank you. Looking at kbuild bot errors, it seems I managed to somehow mess up again. It is not true that just the [08/10] patch relies on the psy-olpc-1.75-battery-signed branch of power-supply tree (see above); in fact [03/10] also does on x86. Would it be possible to include that branch in your testing as well? It is going to be included in the power-supply pull for 5.2. Also, it seems patches [09/10] and [10/10] landed in your queue; they are not ready. Lubo
