Received-SPF: pass (google.com: domain of linux-kernel+bounces-140265-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: "Hennerich, Michael" <Michael.Hennerich@analog.com>
To: =?iso-8859-1?Q?Uwe_Kleine-K=F6nig?= <u.kleine-koenig@pengutronix.de>,
        Trevor Gamblin <tgamblin@baylibre.com>
CC: "linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Sa, Nuno"
	<Nuno.Sa@analog.com>,
        "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>,
        "conor+dt@kernel.org" <conor+dt@kernel.org>,
        David Lechner
	<dlechner@baylibre.com>,
        Drew Fustini <dfustini@baylibre.com>,
        Krzysztof
 Kozlowski <krzysztof.kozlowski@linaro.org>
Subject: RE: [PATCH 1/2 v4] dt-bindings: pwm: Add AXI PWM generator
Thread-Topic: [PATCH 1/2 v4] dt-bindings: pwm: Add AXI PWM generator
Thread-Index: AQHaa/6i41R8h8tWz0mYC6f6IcDfdLFjHzqAgAABauA=
Date: Thu, 11 Apr 2024 10:38:28 +0000
Message-ID: 
 <CO3PR03MB67746D2CB47F5D9D9F96543D8E052@CO3PR03MB6774.namprd03.prod.outlook.com>
References: <20240301173343.1086332-1-tgamblin@baylibre.com>
 <20240301173343.1086332-2-tgamblin@baylibre.com>
 <mf3riwux5cwr63w6i3n5mh4g2yrn4xeywbhce3izldahrb7lw6@4tvyt6pkfixr>
In-Reply-To: <mf3riwux5cwr63w6i3n5mh4g2yrn4xeywbhce3izldahrb7lw6@4tvyt6pkfixr>
Accept-Language: en-US, de-DE
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?iso-8859-1?Q?1l8J3FXYg4ODMcLx09p/iwBXf0f/2piHzMxjLZicnD4pdjoljiOBqR4r9z?=
 =?iso-8859-1?Q?WwgBLChAtKXc6F7JPiCrLdbaJBOAnOvBXsXnbG3C0aO7rpKQ7hsQbgnZel?=
 =?iso-8859-1?Q?6DsjW1j/KqxDnCEtmZ0nmMkoiwCY2O+FhfFyj3nBFafuYz6IfLv2egkqBa?=
 =?iso-8859-1?Q?7IwwlPOQOosoAvQEFCthqMfMHAZNHEPYFTSvp7g7ELBnHAGUPmv7buQeGR?=
 =?iso-8859-1?Q?Ug3GSFI2NrmvDmrd8CsURFGKFkAQ1CtpiAm64aUPn1uGv8ZhXFlphPXtfL?=
 =?iso-8859-1?Q?pdaZ74LGEpESOInpYrUzjA5ERNrB3sD6ukHLl7o9PaGMqszsuAiVz4z1B7?=
 =?iso-8859-1?Q?bi6Cgyb1hJxBM5s2t8KxpTfrDvIOfDOJsi6rrQP4GexsDYEH4M01KdElcJ?=
 =?iso-8859-1?Q?fUgNAt1bivC+fEqrJzGPkUdAUlMHIomP7trFiL0IZoE1FL36PvCSeu7Nty?=
 =?iso-8859-1?Q?zXiSePc2fLBZxIgJijBJq3Ia7YmAqJPscR9osOidWebUHWDlqmM7iEyq1n?=
 =?iso-8859-1?Q?fyvmkYCLk1GXOke8GsWNrrCAgcTMOLv5FgY+p4ZOSHhC0td3KtaCvhRYSO?=
 =?iso-8859-1?Q?RWQxkollbP831LDsewATZilzYlfzM4shVCx+y4aQiEuvBrSoMoOGvaQCpL?=
 =?iso-8859-1?Q?oAoE7nSYbHUt4rVuVRPxq0otBMao1CbpkTFti9NMY1BtS0Fy8KZ2T4WlFo?=
 =?iso-8859-1?Q?ILKYIe7yXcSpE6nXyZDsjU+fMxICiv/ydbwtimvSro6n5VyanCyDm8xTok?=
 =?iso-8859-1?Q?wk0yHboU8xSXWrl/dzhzAxUaAZSxu3axgwVmq2a3WGNVTpigs3A3KTzcAG?=
 =?iso-8859-1?Q?FCGZflcE5iyv/p6cKsN4GK0iKoZk3LV92jp7e395x1/499B+eZEkxrWcFd?=
 =?iso-8859-1?Q?W0EZEujDr9LoBdqHpGdGLwKMEa64Px0Fx5CYfP8SVKMjduzAd0d9ZtRC65?=
 =?iso-8859-1?Q?W5g6gZbIQinhQ12x+jPDAiOgR+afREhSRzcQdhuBiMTm3fTsmoTy3ugkHK?=
 =?iso-8859-1?Q?hRg4fZCHjnT9LXCfE78VS0Q4oQvATRWIdYpoZJVxfwVs/NQZXG5gM9st72?=
 =?iso-8859-1?Q?V5qEm+ZlVPeoHqyXvnE6gHCxMBKlZYqi82yIhyqdMxhq7o0CAlKbjamSzi?=
 =?iso-8859-1?Q?oGtGNnInMZCa5zRDotcFvHkHrU7D1ecn+POAGHdbwv3myy8r+6HeRwX3hD?=
 =?iso-8859-1?Q?/juhKwzk58NxsKgKUnqwTxmi8iec2C6e7KQlU98T1TZEzKsYsRlA7A0czG?=
 =?iso-8859-1?Q?RJmsBUVMdx3pdn4AzfjKp09hCwp3nxpf2OkS6q6lwejIv+Rt+g4J4atLji?=
 =?iso-8859-1?Q?c6mu4QFoCVMIJsXiKv5yHSWQOPwewqkhcBJMsK9HC7fBi/6g204BXL/xyR?=
 =?iso-8859-1?Q?4jyqyfSYzk2kOrAOJ76UxrAHYng/FTw0Ghp221PJFp75wwk1T0Itwft6V0?=
 =?iso-8859-1?Q?/Ye5dye7NAfSlpAVlRZ0s6wcwyqyISW0Qjd1dJQZkjw+eROnwjINXmd+hS?=
 =?iso-8859-1?Q?5l54xonQ0+rzcLpV/wNX64zsrUzLki+v9NAMZSoSNLQPfIng6Q5zWiavXc?=
 =?iso-8859-1?Q?S6NCCRSNPcNuaWrBwaSX7XR12xq7Haj/06XQLclvYRDcpgCRskL4lPGe2X?=
 =?iso-8859-1?Q?L/CvwpLk0lDFsVAWu6v6xHZtcwixSFoqWi6MPaRSDOpUAH/xQlN/ML2Q?=
 =?iso-8859-1?Q?=3D=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO3PR03MB6774.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2798bdbd-5e12-4a52-2875-08dc5a13866e
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Apr 2024 10:38:28.5505
 (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: FMXjpxu8F0NTd27txlCVpanuxWhar8H9ZDYDS7cLj0TIxryZNUuB4PfDQ7wpWCV1q33MGfZZR0Of3HakQ8LB5+MhYgKJBxcvZ0l7+Vc8mG4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR03MB7213
X-Proofpoint-GUID: uZOU2GT_W9HYHLB_gd2idwP9yNCVBJ9x
X-Proofpoint-ORIG-GUID: uZOU2GT_W9HYHLB_gd2idwP9yNCVBJ9x
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-04-11_04,2024-04-09_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 mlxscore=0
 priorityscore=1501 phishscore=0 adultscore=0 mlxlogscore=999
 malwarescore=0 spamscore=0 clxscore=1011 bulkscore=0 suspectscore=0
 lowpriorityscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2404010003 definitions=main-2404110076


> -----Original Message-----
> From: Uwe Kleine-K=F6nig <u.kleine-koenig@pengutronix.de>
> Sent: Thursday, April 11, 2024 12:33 PM
> To: Trevor Gamblin <tgamblin@baylibre.com>
> Cc: linux-pwm@vger.kernel.org; linux-kernel@vger.kernel.org; Hennerich,
> Michael <Michael.Hennerich@analog.com>; Sa, Nuno
> <Nuno.Sa@analog.com>; devicetree@vger.kernel.org; robh+dt@kernel.org;
> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; David Lechner
> <dlechner@baylibre.com>; Drew Fustini <dfustini@baylibre.com>; Krzysztof
> Kozlowski <krzysztof.kozlowski@linaro.org>
> Subject: Re: [PATCH 1/2 v4] dt-bindings: pwm: Add AXI PWM generator
>=20
>=20
> Hello,
>=20
> On Fri, Mar 01, 2024 at 12:33:41PM -0500, Trevor Gamblin wrote:
> > From: Drew Fustini <dfustini@baylibre.com>
> >
> > Add Analog Devices AXI PWM generator.
> >
> > Link: https://wiki.analog.com/resources/fpga/docs/axi_pwm_gen
> > Signed-off-by: Drew Fustini <dfustini@baylibre.com>
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Co-developed-by: Trevor Gamblin <tgamblin@baylibre.com>
> > Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
> > ---
> > v4 changes: None (rebased, added maintainer's previous Reviewed-by)
> > v3 changes: None (rebased, added maintainer's previous Reviewed-by)
> >
> > v2 changes:
> > * Address feedback for driver and device tree in v1:
> >   * Relocate "unevaluatedProperties" in device tree binding
> >   * Remove redundant "bindings for" in description
> >
> > ---
> >  .../bindings/pwm/adi,axi-pwmgen.yaml          | 48 +++++++++++++++++++
> >  MAINTAINERS                                   |  8 ++++
> >  2 files changed, 56 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
> > b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
> > new file mode 100644
> > index 000000000000..63461920a362
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/pwm/adi,axi-pwmgen.yaml
> > @@ -0,0 +1,48 @@
> > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause %YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/pwm/adi,axi-pwmgen.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Analog Devices AXI PWM generator
> > +
> > +maintainers:
> > +  - Michael Hennerich <Michael.Hennerich@analog.com>
> > +  - Nuno S=E1 <nuno.sa@analog.com>
>=20
> Would be great to get an ack from these two.

Looks good to me.

Acked-by: Michael Hennerich <michael.hennerich@analog.com>

> Otherwise looks fine to me.
>=20
> Best regards
> Uwe
>=20
> --
> Pengutronix e.K.                           | Uwe Kleine-K=F6nig          =
  |
> Industrial Linux Solutions                 | https://www.pengutronix.de/ =
|