On Tue, Jul 30, 2019 at 03:28:05PM -0400, Dimitri John Ledkov wrote:
From: AArch64 Laptops aarch64laptops@gmail.com
This is squashed merged of multiple patches from https://github.com/aarch64-laptops/linux/tree/laptops-ubuntu
Note that some of these, with different names are getting upstreamed targetted for v5.4. Once those are all merged and available, this patch will become obsolete and will be dropped.
Are they in linux-next or a maintainer tree already? If so I think it would be preferable to cherry pick the individual patches.
Otherwise, adding some device trees seems pretty harmless. But one comment ...
<snip>
diff --git a/drivers/regulator/qcom-rpmh-regulator.c b/drivers/regulator/qcom-rpmh-regulator.c index b2c2d01d1637..aa0272497be4 100644 --- a/drivers/regulator/qcom-rpmh-regulator.c +++ b/drivers/regulator/qcom-rpmh-regulator.c @@ -207,7 +207,7 @@ static int rpmh_regulator_is_enabled(struct regulator_dev *rdev) { struct rpmh_vreg *vreg = rdev_get_drvdata(rdev);
- return vreg->enabled;
- return vreg->enabled > 0;
}
This is clearly a bug in the driver. I think it should be a separate patch from the dts files, and it should be able to go into 5.3.
Thanks, Seth