On 17/01/24 9:03 pm, Conor Dooley wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > > ForwardedMessage.eml > > Subject: > Re: [PATCH v2 2/3] dt-bindings: atmel,hlcdc: convert pwm bindings to > json-schema > From: > Conor Dooley <conor@xxxxxxxxxx> > Date: > 17/01/24, 9:03 pm > > To: > Dharma.B@xxxxxxxxxxxxx > CC: > alexandre.belloni@xxxxxxxxxxx, Conor.Dooley@xxxxxxxxxxxxx, > sam@xxxxxxxxxxxx, bbrezillon@xxxxxxxxxx, > maarten.lankhorst@xxxxxxxxxxxxxxx, mripard@xxxxxxxxxx, > tzimmermann@xxxxxxx, airlied@xxxxxxxxx, daniel@xxxxxxxx, > robh+dt@xxxxxxxxxx, krzysztof.kozlowski+dt@xxxxxxxxxx, > conor+dt@xxxxxxxxxx, Nicolas.Ferre@xxxxxxxxxxxxx, > claudiu.beznea@xxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, > devicetree@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, > linux-kernel@xxxxxxxxxxxxxxx, lee@xxxxxxxxxx, thierry.reding@xxxxxxxxx, > u.kleine-koenig@xxxxxxxxxxxxxx, linux-pwm@xxxxxxxxxxxxxxx, > Linux4Microchip@xxxxxxxxxxxxx > > > On Wed, Jan 17, 2024 at 02:43:00AM +0000,Dharma.B@xxxxxxxxxxxxx wrote: >> On 17/01/24 1:40 am, Alexandre Belloni wrote: >>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >>> >>> On 16/01/2024 18:03:19+0000, Conor Dooley wrote: >>>> On Tue, Jan 16, 2024 at 05:07:59PM +0530, Dharma Balasubiramani wrote: >>>>> Convert device tree bindings for Atmel's HLCDC PWM controller to YAML >>>>> format. >>>>> >>>>> Signed-off-by: Dharma Balasubiramani<dharma.b@xxxxxxxxxxxxx> >>>>> --- >>>>> changelog >>>>> v1 -> v2 >>>>> - Remove the explicit copyrights. >>>>> - Modify title (not include words like binding/driver). >>>>> - Modify description actually describing the hardware and not the driver. >>>>> - Remove pinctrl properties which aren't required. >>>>> - Drop parent node and it's other sub-device node which are not related here. >>>>> --- >>>>> .../bindings/pwm/atmel,hlcdc-pwm.yaml | 47 +++++++++++++++++++ >>>>> .../bindings/pwm/atmel-hlcdc-pwm.txt | 29 ------------ >>>>> 2 files changed, 47 insertions(+), 29 deletions(-) >>>>> create mode 100644 Documentation/devicetree/bindings/pwm/atmel,hlcdc-pwm.yaml >>>>> delete mode 100644 Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/pwm/atmel,hlcdc-pwm.yaml b/Documentation/devicetree/bindings/pwm/atmel,hlcdc-pwm.yaml >>>>> new file mode 100644 >>>>> index 000000000000..751122309fa9 >>>>> --- /dev/null >>>>> +++ b/Documentation/devicetree/bindings/pwm/atmel,hlcdc-pwm.yaml >>>>> @@ -0,0 +1,47 @@ >>>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>>> The original file has no license, but was originally written by a >>>> free-electrons employee, so the relicensing here is fine. >>>> >>> I confirm relicensing is fine, even assigning the copyright to >>> Microchip (note that Bootlin is legally the same entity as >>> free-electrons) >> Thanks Conor and Alexandre. >> I will add the copyrights back in v3. > Just to note, in case you misunderstood my original comment here: > What I said had nothing to do with adding a Microchip copyright assignment > to the file, but rather about the fact that your patch relicenses the > binding from GPL v2 to GPL v2 OR BSD 2 Clause. I appreciate the clarification; my initial understanding was not accurate. Thanks > -- With Best Regards, Dharma B.
