lkml.org 
[lkml]   [2015]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 1/2] Regulator: mc34708: Add DT binding documentation
From
Hello Martin,

I haven't looked at your regulator driver in detail so this is not a
full review but I just noticed something so I wanted mention.

> +
> +
> +Example:
> +&i2c3 {
> + pmic: mc34708@08 {
> + compatible = "fsl,mc34708";
> + reg = <0x08>;
> + regulators {
> +#define PMIC_REGMODE_SW_PFM 1
> +#define PMIC_REGMODE_SW_AUTO 2
> +#define PMIC_REGMODE_SW_PWM 3
> +#define PMIC_REGMODE_LDO_LP 1
> +#define PMIC_REGMODE_LDO_NORMAL 2
> +

I see that you are defining these constants in your driver too (albeit
with slightly different names). These kind of constants that are
shared between the driver implementation and the Device Tree source
files should be added to include/dt-bindings/regulator/ and included
in both the driver and the DTS.

Best regards,
Javier


\
 
 \ /
  Last update: 2015-05-11 12:21    [W:0.174 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site