mbox series

[v6,0/3] Stacked/parallel memories bindings

Message ID 20220126112608.955728-1-miquel.raynal@bootlin.com
Headers show
Series Stacked/parallel memories bindings | expand

Message

Miquel Raynal Jan. 26, 2022, 11:26 a.m. UTC
Hello Rob, Mark, Tudor & Pratyush,

Here is a sixth versions for these bindings, which applies on top of
the v5.17-rc1 now that Pratyush's work as been merged.
(https://lore.kernel.org/all/20211109181911.2251-1-p.yadav@ti.com/)

Cheers,
Miquèl

Changes in v6:
* Added Pratyush's acks.
* The tooling now validates the binding (updating dt-schema is
  necesary).
* Updated the maxItems field to 4 as a "big enough value" as discussed.

Changes in v5:
* Used the uint64-array instead of the matrix type.
* Updated the example as well to use a single "/bits/ 64" cast because
  doing it twice, despite being supported by the language itself, is not
  yet something that we can use for describing bindings.

Changes in v4:
* Changed the type of properties to uint64-arrays in order to be able to
  describe the size of each element in the array.
* Updated the example accordingly.

Changes in v3:
* Rebased on top of Pratyush's recent changes.
* Dropped the commit allowing to provide two reg entries on the node
  name.
* Dropped the commit referencing spi-controller.yaml from
  jedec,spi-nor.yaml, now replaced by spi-peripheral-props.yaml and
  already done in Pratyush's series.
* Added Rob's Ack.
* Enhanced a commit message.
* Moved the new properties to the new SPI peripheral binding file.

Changes in v2:
* Dropped the dtc changes for now.
* Moved the properties in the device's nodes, not the controller's.
* Dropped the useless #address-cells change.
* Added a missing "minItems".
* Moved the new properties in the spi-controller.yaml file.
* Added an example using two stacked memories in the
  spi-controller.yaml file.
* Renamed the properties to drop the Xilinx prefix.
* Added a patch to fix the spi-nor jedec yaml file.

Miquel Raynal (3):
  dt-bindings: mtd: spi-nor: Allow two CS per device
  spi: dt-bindings: Describe stacked/parallel memories modes
  spi: dt-bindings: Add an example with two stacked flashes

 .../bindings/mtd/jedec,spi-nor.yaml           |  3 ++-
 .../bindings/spi/spi-controller.yaml          |  7 ++++++
 .../bindings/spi/spi-peripheral-props.yaml    | 25 +++++++++++++++++++
 3 files changed, 34 insertions(+), 1 deletion(-)

Comments

Michal Simek Feb. 18, 2022, 11:53 a.m. UTC | #1
Hi Mark

On 1/26/22 12:26, Miquel Raynal wrote:
> Hello Rob, Mark, Tudor & Pratyush,
> 
> Here is a sixth versions for these bindings, which applies on top of
> the v5.17-rc1 now that Pratyush's work as been merged.
> (https://lore.kernel.org/all/20211109181911.2251-1-p.yadav@ti.com/)
> 
> Cheers,
> Miquèl
> 
> Changes in v6:
> * Added Pratyush's acks.
> * The tooling now validates the binding (updating dt-schema is
>    necesary).
> * Updated the maxItems field to 4 as a "big enough value" as discussed.
> 
> Changes in v5:
> * Used the uint64-array instead of the matrix type.
> * Updated the example as well to use a single "/bits/ 64" cast because
>    doing it twice, despite being supported by the language itself, is not
>    yet something that we can use for describing bindings.
> 
> Changes in v4:
> * Changed the type of properties to uint64-arrays in order to be able to
>    describe the size of each element in the array.
> * Updated the example accordingly.
> 
> Changes in v3:
> * Rebased on top of Pratyush's recent changes.
> * Dropped the commit allowing to provide two reg entries on the node
>    name.
> * Dropped the commit referencing spi-controller.yaml from
>    jedec,spi-nor.yaml, now replaced by spi-peripheral-props.yaml and
>    already done in Pratyush's series.
> * Added Rob's Ack.
> * Enhanced a commit message.
> * Moved the new properties to the new SPI peripheral binding file.
> 
> Changes in v2:
> * Dropped the dtc changes for now.
> * Moved the properties in the device's nodes, not the controller's.
> * Dropped the useless #address-cells change.
> * Added a missing "minItems".
> * Moved the new properties in the spi-controller.yaml file.
> * Added an example using two stacked memories in the
>    spi-controller.yaml file.
> * Renamed the properties to drop the Xilinx prefix.
> * Added a patch to fix the spi-nor jedec yaml file.
> 
> Miquel Raynal (3):
>    dt-bindings: mtd: spi-nor: Allow two CS per device
>    spi: dt-bindings: Describe stacked/parallel memories modes
>    spi: dt-bindings: Add an example with two stacked flashes
> 
>   .../bindings/mtd/jedec,spi-nor.yaml           |  3 ++-
>   .../bindings/spi/spi-controller.yaml          |  7 ++++++
>   .../bindings/spi/spi-peripheral-props.yaml    | 25 +++++++++++++++++++
>   3 files changed, 34 insertions(+), 1 deletion(-)
> 

Can you please pick up this series?

Thanks,
Michal
Mark Brown Feb. 21, 2022, 3:24 p.m. UTC | #2
On Wed, 26 Jan 2022 12:26:04 +0100, Miquel Raynal wrote:
> Here is a sixth versions for these bindings, which applies on top of
> the v5.17-rc1 now that Pratyush's work as been merged.
> (https://lore.kernel.org/all/20211109181911.2251-1-p.yadav@ti.com/)
> 
> Cheers,
> Miquèl
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next

Thanks!

[1/3] dt-bindings: mtd: spi-nor: Allow two CS per device
      commit: b252ada293d5d30566121c61fa7552e74396d533
[2/3] spi: dt-bindings: Describe stacked/parallel memories modes
      commit: e2edd1b64f1c79e8abda365149ed62a2a9a494b4
[3/3] spi: dt-bindings: Add an example with two stacked flashes
      commit: eba5368503b4291db7819512600fa014ea17c5a8

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark