mbox series

[v8,0/3] Add support for lan966x flexcom chip-select configuration

Message ID 20220708115619.254073-1-kavyasree.kotagiri@microchip.com (mailing list archive)
Headers show
Series Add support for lan966x flexcom chip-select configuration | expand

Message

Kavyasree Kotagiri July 8, 2022, 11:56 a.m. UTC
This patch series converts atmel-flexcom bindings into json-schema format.
Adds support for lan966x flexcom chip-select configurations and its
DT bindings.

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

Comments

Kavyasree Kotagiri Sept. 5, 2022, 7:53 a.m. UTC | #1
Hi Lee,

Are you planning to take this patch series for 6.1 or Can we include them through arm-soc pull-request?

Thanks,
Kavya

> -----Original Message-----
> From: Kavyasree Kotagiri [mailto:kavyasree.kotagiri@microchip.com]
> Sent: Friday, July 8, 2022 5:26 PM
> To: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; Nicolas Ferre -
> M43238 <Nicolas.Ferre@microchip.com>; alexandre.belloni@bootlin.com;
> Claudiu Beznea - M18063 <Claudiu.Beznea@microchip.com>;
> UNGLinuxDriver <UNGLinuxDriver@microchip.com>
> Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> kernel@vger.kernel.org; Kavyasree Kotagiri - I30978
> <Kavyasree.Kotagiri@microchip.com>
> Subject: [PATCH v8 0/3] Add support for lan966x flexcom chip-select
> configuration
> 
> This patch series converts atmel-flexcom bindings into json-schema format.
> Adds support for lan966x flexcom chip-select configurations and its
> DT bindings.
> 
> 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
Lee Jones Sept. 5, 2022, 2:55 p.m. UTC | #2
On Mon, 05 Sep 2022, Kavyasree.Kotagiri@microchip.com wrote:

> Hi Lee,
> 
> Are you planning to take this patch series for 6.1 or Can we include them through arm-soc pull-request?

I don't seem to have visibility.

Have I Acked it already?

BTW, this is my old email address.

Please use the one in my signature instead.

> > -----Original Message-----
> > From: Kavyasree Kotagiri [mailto:kavyasree.kotagiri@microchip.com]
> > Sent: Friday, July 8, 2022 5:26 PM
> > To: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; Nicolas Ferre -
> > M43238 <Nicolas.Ferre@microchip.com>; alexandre.belloni@bootlin.com;
> > Claudiu Beznea - M18063 <Claudiu.Beznea@microchip.com>;
> > UNGLinuxDriver <UNGLinuxDriver@microchip.com>
> > Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> > kernel@vger.kernel.org; Kavyasree Kotagiri - I30978
> > <Kavyasree.Kotagiri@microchip.com>
> > Subject: [PATCH v8 0/3] Add support for lan966x flexcom chip-select
> > configuration
> > 
> > This patch series converts atmel-flexcom bindings into json-schema format.
> > Adds support for lan966x flexcom chip-select configurations and its
> > DT bindings.
> > 
> > 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
> > 
>
Kavyasree Kotagiri Sept. 7, 2022, 10:50 a.m. UTC | #3
> 
> > Hi Lee,
> >
> > Are you planning to take this patch series for 6.1 or Can we include them
> through arm-soc pull-request?
> 
> I don't seem to have visibility.
> 
Please find the patches below:
https://lore.kernel.org/lkml/20220708115619.254073-1-kavyasree.kotagiri@microchip.com/T/

> Have I Acked it already?
> 
No. 

> BTW, this is my old email address.
> 
> Please use the one in my signature instead.
> 
> > > -----Original Message-----
> > > From: Kavyasree Kotagiri [mailto:kavyasree.kotagiri@microchip.com]
> > > Sent: Friday, July 8, 2022 5:26 PM
> > > To: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; Nicolas
> Ferre -
> > > M43238 <Nicolas.Ferre@microchip.com>;
> alexandre.belloni@bootlin.com;
> > > Claudiu Beznea - M18063 <Claudiu.Beznea@microchip.com>;
> > > UNGLinuxDriver <UNGLinuxDriver@microchip.com>
> > > Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
> linux-
> > > kernel@vger.kernel.org; Kavyasree Kotagiri - I30978
> > > <Kavyasree.Kotagiri@microchip.com>
> > > Subject: [PATCH v8 0/3] Add support for lan966x flexcom chip-select
> > > configuration
> > >
> > > This patch series converts atmel-flexcom bindings into json-schema
> format.
> > > Adds support for lan966x flexcom chip-select configurations and its
> > > DT bindings.
> > >
> > > 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
> > >
> >
> 
> --
> DEPRECATED: Please use lee@kernel.org
Lee Jones Sept. 8, 2022, 7:30 a.m. UTC | #4
On Wed, 07 Sep 2022, Kavyasree.Kotagiri@microchip.com wrote:

> > 
> > > Hi Lee,
> > >
> > > Are you planning to take this patch series for 6.1 or Can we include them
> > through arm-soc pull-request?
> > 
> > I don't seem to have visibility.
> > 
> Please find the patches below:
> https://lore.kernel.org/lkml/20220708115619.254073-1-kavyasree.kotagiri@microchip.com/T/

I need patches in my inbox to apply them.

> > Have I Acked it already?
> > 
> No. 

Then please submit a [RESEND] - with the full Cc list this time.