lkml.org 
[lkml]   [2019]   [Jun]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 6/8] regulator: core: Don't attach generic coupler to Tegra SoC regulators
On Tue, Jun 04, 2019 at 02:59:02AM +0300, Dmitry Osipenko wrote:

> static int generic_coupler_attach(struct regulator_coupler *coupler,
> struct regulator_dev *rdev)
> {
> + /*
> + * Generic coupler isn't suitable for NVIVIA Tegra SoC's, at least
> + * for now. Hence filter out the unwanted regulators as they shall be
> + * managed by a platform-specific coupler.
> + */
> + if (of_property_read_bool(rdev->dev.of_node, "tegra-core-regulator") ||
> + of_property_read_bool(rdev->dev.of_node, "tegra-rtc-regulator") ||
> + of_property_read_bool(rdev->dev.of_node, "tegra-cpu-regulator"))
> + return -EPERM;
> +

I'm having a hard time loving this as it requires explicit DT changes
for implementation. I'm thinking that since the couplers are going to
need to be built in it'd be better to make sure that any custom ones get
registered first and then only bind the generic coupler to anything they
reject.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2019-06-17 17:45    [W:0.287 / U:0.328 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site