Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Chindris, Mihail" <Mihail.Chindris@analog.com>
To:     "Chindris, Mihail" <Mihail.Chindris@analog.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "broonie@kernel.org" <broonie@kernel.org>
CC:     "lars@metafoo.de" <lars@metafoo.de>,
        "Hennerich, Michael" <Michael.Hennerich@analog.com>,
        "jic23@kernel.org" <jic23@kernel.org>,
        "Sa, Nuno" <Nuno.Sa@analog.com>,
        "Bogdan, Dragos" <Dragos.Bogdan@analog.com>
Subject: RE: [PATCH v5 1/2] dt-bindings: iio: dac: Add adi,ad3552r.yaml
Thread-Topic: [PATCH v5 1/2] dt-bindings: iio: dac: Add adi,ad3552r.yaml
Thread-Index: AQHX1Ho2pyvidWqOtkOuCwDisYTFAav5S56w
Date:   Mon, 8 Nov 2021 08:29:13 +0000
Message-ID: <SN4PR03MB679988CBFA34DF338C85549699919@SN4PR03MB6799.namprd03.prod.outlook.com>
References: <20211108082447.116663-1-mihail.chindris@analog.com>
 <20211108082447.116663-2-mihail.chindris@analog.com>
In-Reply-To: <20211108082447.116663-2-mihail.chindris@analog.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?+zg0Ezaojh54jRpMMbY22l2X5x4DuxWpqnuYnQBIK9JLPTPmSf0vhiw7Uc+3?=
 =?us-ascii?Q?+l9DgpSNpZJdnUwDNMP5/e8KBe5PiIpp/dVyEQi2Qd9tX4t5VjtOaPvsEoru?=
 =?us-ascii?Q?SU/ZjJw8SaCysd6Otu2dB8HKvAOBH3pb77eWtYA4tsTXi+yVTG/DQQsjOrcb?=
 =?us-ascii?Q?DtsQ05orYvQ4KqbpqenaRpn3DP+w5BghnitQzIUv9AsgOgPXlVBUAUqMZKg1?=
 =?us-ascii?Q?jqMEM/p30XE5rL+2PqRKeIpqyPju6Y48eKZTWRM3H3PqWJsSREBlwtAWg/YF?=
 =?us-ascii?Q?hrK+JHCEEKujnRRsRckXbnJek5I2xIZO5RChDytxxnDq/H+saRhHauOoMQoa?=
 =?us-ascii?Q?owxCiyzEZBmPEqSx+o1dYDtKyBX10c6L0PVDdTK2Twbcrmlg30qKMG9746uH?=
 =?us-ascii?Q?QeY18WlqKjUEP7jdhpW728Llw3z0oSdBiyRraZevTO1ASDuDxCJ7nYt31PsT?=
 =?us-ascii?Q?kW+XH8W1d6bPQ6XE5uKvdphfDUpygCa/3e0WocU7D8j525f/23jD2r8Ume1D?=
 =?us-ascii?Q?5K0UAf30p3WzgyFDWEz+/H245HKKB2Z89TjVYtaOdYAZ82tSi3hwjx+u6lEN?=
 =?us-ascii?Q?qnF8V66wWMhu0ejic6fzxobs2ZHcZLvGTxhhZqzTjaWfwN/UGxZnRiNJRqlo?=
 =?us-ascii?Q?kP5waTVtpqorCGyaxH5JFU9nfiLKe8e9QBrphONvmYMpYyfUkTpaT0q8ESh0?=
 =?us-ascii?Q?PtKy0GSYaDioD692WQp3zMVty7tpiHYU7mAiiCNAcNchef6beYKXAowiyg6/?=
 =?us-ascii?Q?C37ZqQCl8GXR14JgCjTs5dvAMQp04keDqLyIghcGuE+e1VFa+A/vjgoNBbbB?=
 =?us-ascii?Q?EUTn5yYOn1e96rfcH9c7Hp75jCVa4otOeYZDw0T2DyRAvo778dUf7WV4LC+k?=
 =?us-ascii?Q?1CUEnAjQYf56Fm4+D0yiBDneQkJDvAu0mYXjBk5CQPvkJ2OXo+whLl8YgKlo?=
 =?us-ascii?Q?6KTWywYNrspBRuqma8IbPfrS+5OWeFB5m4Cz/Tf7FzgwUxknqQ2oWtqYbY+C?=
 =?us-ascii?Q?WYcXK4xLnLyzQC/9XIUGyLYEATgfe7R9WFXrYZqT7xbDRv6BJH8cR2v97tRq?=
 =?us-ascii?Q?tspcemF9Sbq3QaBukQGp7ss04edO+BDhMhdl388GX30Bg81hKvfQnMc04CZY?=
 =?us-ascii?Q?gEH3yy44aRBXKDVXZX0+pEnYQYQhBBOPbe02OM8Ow3EriMfisppveO5hocTx?=
 =?us-ascii?Q?OMckrbUZD8Iyo3uCnsVWOdXI/AIZD0sPuvOyZp7nNB7bPxbHuXL+mEE4yxPo?=
 =?us-ascii?Q?0gYOh6/jn+ZAU//NM+E46WwVHCkZHPaTYgz0FqbI5FhWxSPJGH9w0MBvh9Hc?=
 =?us-ascii?Q?Ax8Evu2xlc1FOfCC7ElJ4bthHz9kQlmOFeyNlte/IG9LoHgrL9+mlqosb8xY?=
 =?us-ascii?Q?LnXAW+Gc+pJfUi6fYQXK8SM38qotMJqHxCEZJOIgCue6fxArT9PGU80iiCfV?=
 =?us-ascii?Q?29Vy667iJUMsdSoMTE1xrVM+5XBQVU//g46QkQoOGMPxMJADW4XLz5oezrVo?=
 =?us-ascii?Q?GHasgNnyKVoLm2fOko/4U9FazPIEJolVAgH844MK8n+Yz62JVaDUEqDZJWza?=
 =?us-ascii?Q?dvswh2uxWFb6P5JKXMjfK6BJfVfPIv33Gdx5exAJ/jY6PAjDWDfNNTwV8BZt?=
 =?us-ascii?Q?uu1qU7L8t5p3xDcrqoWo8X3kbdTfHssHTthuYJ2Ynt4/zm/ZZKjejP0kNUt9?=
 =?us-ascii?Q?54HnTiHYi3tiai5evX/K0izXhy75h9fu0f3AFqpk4jhQyR8RRsUWvbrRPE9F?=
 =?us-ascii?Q?FUi6GZDGWQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SN4PR03MB6799.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c93865d7-6f3b-4355-e178-08d9a291d8e2
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Nov 2021 08:29:13.9443
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: kI/4h1mW8gCXOjdYcewyIsynbBP2jVHXzOAAI0tBnvx6BbJRERu6N9M8mgpsLxXpedfqvauRPzTKlPwL3gxQ70h9zo6aMHMs+XvwMV4dwiU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR03MB6580
X-Proofpoint-ORIG-GUID: 4kd3rpvzbcOsA6vlYB00Y_933Emm9Bk8
X-Proofpoint-GUID: 4kd3rpvzbcOsA6vlYB00Y_933Emm9Bk8
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.0.607.475
 definitions=2021-11-08_03,2021-11-03_01,2020-04-07_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501
 malwarescore=0 bulkscore=0 impostorscore=0 spamscore=0 mlxlogscore=999
 adultscore=0 mlxscore=0 phishscore=0 suspectscore=0 clxscore=1015
 lowpriorityscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2110150000 definitions=main-2111080053
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

...
> +patternProperties:
> +  "^channel@([0-1])$":
> +    type: object
> +    description: Configurations of the DAC Channels
> +    properties:
> +      reg:
> +          description: Channel number
> +          enum: [0, 1]
> +
> +      custom-output-range-config:

Copy unanswered comment from V3

> > >=20
> > > Not a generic property so I think this needs an adi prefix. =20
> > > Jonathan
> >
> > I tried with adi prefix but I get weird errors while running dt_binding=
_check for properties with adi prefix and with type:object
> > Do you have any suggestion for this issues?
> >=20
> > Mihail
> >
@Rob
Jonathan

> +        type: object
> +        description: Configuration of custom range when
> +          adi,output-range-microvolt is not present.
> +          The formulas for calculation the output voltages are
> +            Vout_fs =3D 2.5 + [(GainN + Offset/1024) * 2.5 * Rfbx * 1.03=
]
> +            Vout_zs =3D 2.5 - [(GainP + Offset/1024) * 2.5 * Rfbx * 1.03=
]
...