From: <Kavyasree.Kotagiri@microchip.com>
To: <lee@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
<robh+dt@kernel.org>
Cc: devicetree@vger.kernel.org, alexandre.belloni@bootlin.com,
linux-kernel@vger.kernel.org, Sergiu.Moga@microchip.com,
UNGLinuxDriver@microchip.com, Claudiu.Beznea@microchip.com,
linux-arm-kernel@lists.infradead.org
Subject: RE: [PATCH v9 0/3] Add support for lan966x flexcom chip-select configuration
Date: Tue, 27 Sep 2022 09:11:47 +0000 [thread overview]
Message-ID: <CO1PR11MB4865F20789584A034BBDDEF092559@CO1PR11MB4865.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20220916075744.1879428-1-kavyasree.kotagiri@microchip.com>
> This patch series converts atmel-flexcom bindings into json-schema format.
> Adds support for lan966x flexcom chip-select configurations and its
> DT bindings.
>
> v8 -> v9:
> - Use full schema path for i2c bindings.
>
Hi Rob & Krzysztof,
Please let me know your comments on v9 patch series.
Thanks,
Kavya
> v7 -> v8:
> - Added back patternProperties for child nodes.
> - Changed compatible string to microchip,lan9668-flexcom.
>
> v6 -> v7:
> - Change filename to atmel,sama5d2-flexcom.yaml
> - Add #address-cells, #size-cells to flexcom node - Fixed warnings.
>
> v5 -> v6:
> - Removed spi node from example as suggested by Rob and
> also pattern properties(spi dt-bindings conversion to yaml patch is under
> review).
> https://patchwork.kernel.org/project/linux-arm-
> kernel/patch/20220629125804.137099-1-sergiu.moga@microchip.com/
> Once this is accepted, I will add back spi example through new patch.
>
> v4 -> v5:
> - Fix indentations of DT example.
> - Fix dt-schema errors - removed minItems, maxItems for allOf:if:then
> "reg" property as it is not required.
>
> v3 -> v4:
> - Fix dtschema errors.
> - Add a condition to flexcom chip-selects configuration as chip-select
> lines are optional.
>
> v2 -> v3:
> - changed IRQ flag in dt-bindings example.
> - added reg property specific to lan66x which is missed in v2.
> - used goto label for clk_disable in error cases.
>
> v1 -> v2:
> - minor fix in title of dt-bindings.
> - Modified new dt properties usage in atmel,flexcom.yaml.
> - Used GENMASK and macros for maximum allowed values.
> - Use u32 values for flexcom chipselects instead of strings.
> - disable clock in case of errors.
>
> Kavyasree Kotagiri (3):
> dt-bindings: mfd: Convert atmel-flexcom to json-schema
> dt-bindings: mfd: atmel,sama5d2-flexcom: Add new compatible string for
> lan966x
> mfd: atmel-flexcom: Add support for lan966x flexcom chip-select
> configuration
>
> .../bindings/mfd/atmel,sama5d2-flexcom.yaml | 155 ++++++++++++++++++
> .../devicetree/bindings/mfd/atmel-flexcom.txt | 63 -------
> drivers/mfd/atmel-flexcom.c | 94 ++++++++++-
> 3 files changed, 248 insertions(+), 64 deletions(-)
> create mode 100644
> Documentation/devicetree/bindings/mfd/atmel,sama5d2-flexcom.yaml
> delete mode 100644 Documentation/devicetree/bindings/mfd/atmel-
> flexcom.txt
>
> --
> 2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2022-09-27 9:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-16 7:57 [PATCH v9 0/3] Add support for lan966x flexcom chip-select configuration Kavyasree Kotagiri
2022-09-16 7:57 ` [PATCH v9 1/3] dt-bindings: mfd: Convert atmel-flexcom to json-schema Kavyasree Kotagiri
2022-09-19 5:27 ` Kavyasree.Kotagiri
2022-10-03 7:56 ` Krzysztof Kozlowski
2022-10-03 7:58 ` Krzysztof Kozlowski
2022-09-16 7:57 ` [PATCH v9 2/3] dt-bindings: mfd: atmel,sama5d2-flexcom: Add new compatible string for lan966x Kavyasree Kotagiri
2022-09-16 7:57 ` [PATCH v9 3/3] mfd: atmel-flexcom: Add support for lan966x flexcom chip-select configuration Kavyasree Kotagiri
2022-10-24 12:25 ` Lee Jones
2022-09-27 9:11 ` Kavyasree.Kotagiri [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CO1PR11MB4865F20789584A034BBDDEF092559@CO1PR11MB4865.namprd11.prod.outlook.com \
--to=kavyasree.kotagiri@microchip.com \
--cc=Claudiu.Beznea@microchip.com \
--cc=Sergiu.Moga@microchip.com \
--cc=UNGLinuxDriver@microchip.com \
--cc=alexandre.belloni@bootlin.com \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lee@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).