All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Krzysztof Kozlowski <krzk@kernel.org>
Cc: Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Tudor Ambarus <tudor.ambarus@linaro.org>,
	Pratyush Yadav <pratyush@kernel.org>,
	Michael Walle <michael@walle.cc>,
	linux-mtd@lists.infradead.org, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	devicetree@vger.kernel.org,
	Chris Packham <Chris.Packham@alliedtelesis.co.nz>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Alexandre Torgue <alexandre.torgue@foss.st.com>,
	AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>,
	Brian Norris <computersforpeace@gmail.com>,
	Chen-Yu Tsai <wens@csie.org>,
	Christophe Kerello <christophe.kerello@foss.st.com>,
	Heiko Stuebner <heiko@sntech.de>,
	Jernej Skrabec <jernej.skrabec@gmail.com>,
	Kamal Dasu <kdasu.kdev@gmail.com>,
	Liang Yang <liang.yang@amlogic.com>,
	Manivannan Sadhasivam <mani@kernel.org>,
	Masahiro Yamada <yamada.masahiro@socionext.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Maxime Coquelin <mcoquelin.stm32@gmail.com>,
	Maxime Ripard <mripard@kernel.org>,
	Paul Cercueil <paul@crapouillou.net>,
	Samuel Holland <samuel@sholland.org>,
	Vadivel Murugan <vadivel.muruganx.ramuthevar@linux.intel.com>,
	Xiangsheng Hou <xiangsheng.hou@mediatek.com>
Subject: Re: [PATCH v2 00/17] Prevent NAND chip unevaluated properties
Date: Thu, 15 Jun 2023 10:26:09 +0200	[thread overview]
Message-ID: <20230615102609.0aeea6ab@xps-13> (raw)
In-Reply-To: <20230613134350.3cab6780@xps-13>

Hi Krzysztof,

miquel.raynal@bootlin.com wrote on Tue, 13 Jun 2023 13:43:50 +0200:

> Hi Krzysztof,
> 
> krzk@kernel.org wrote on Sat, 10 Jun 2023 11:27:55 +0200:
> 
> > On 06/06/2023 19:52, Miquel Raynal wrote:  
> > > As discussed with Krzysztof and Chris, it seems like each NAND
> > > controller binding should actually restrain the properties allowed in
> > > the NAND chip node with its own "unevaluatedProperties: false". This
> > > only works if we reference a yaml schema which contains all the possible
> > > properties *in the NAND chip node*. Indeed, the NAND controller yaml
> > > schema contains properties for the NAND chip which are not evaluated
> > > with this construction.
> > > 
> > > Link: https://lore.kernel.org/all/a23dd485-a3d9-e31f-be3e-0ab293fcfc4a@linaro.org/    
> > 
> > Please rebase on latest kernel. This is some very old tree if you CC
> > this address.  
> 
> Sorry for the trouble, it was a hardcoded address on my side. I've
> updated the address to
> 
> 	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> 
> The series is based on v6.4-rc1. Against what branch/tag do you want it
> to be rebased?

When I asked this question I forgot I usually apply mtd bindings in my
own tree :) I usually do it after receiving at least one R-by on the
whole series, but here as I'm the author I would just like to get
confirmation from you Krzysztof that I can go ahead (or if you prefer
to take it, or maybe want to give this another look later).

Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

WARNING: multiple messages have this Message-ID (diff)
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Krzysztof Kozlowski <krzk@kernel.org>
Cc: Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Tudor Ambarus <tudor.ambarus@linaro.org>,
	Pratyush Yadav <pratyush@kernel.org>,
	Michael Walle <michael@walle.cc>,
	linux-mtd@lists.infradead.org, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	devicetree@vger.kernel.org,
	Chris Packham <Chris.Packham@alliedtelesis.co.nz>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Alexandre Torgue <alexandre.torgue@foss.st.com>,
	AngeloGioacchino Del Regno 
	<angelogioacchino.delregno@collabora.com>,
	Brian Norris <computersforpeace@gmail.com>,
	Chen-Yu Tsai <wens@csie.org>,
	Christophe Kerello <christophe.kerello@foss.st.com>,
	Heiko Stuebner <heiko@sntech.de>,
	Jernej Skrabec <jernej.skrabec@gmail.com>,
	Kamal Dasu <kdasu.kdev@gmail.com>,
	Liang Yang <liang.yang@amlogic.com>,
	Manivannan Sadhasivam <mani@kernel.org>,
	Masahiro Yamada <yamada.masahiro@socionext.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Maxime Coquelin <mcoquelin.stm32@gmail.com>,
	Maxime Ripard <mripard@kernel.org>,
	Paul Cercueil <paul@crapouillou.net>,
	Samuel Holland <samuel@sholland.org>,
	Vadivel Murugan <vadivel.muruganx.ramuthevar@linux.intel.com>,
	Xiangsheng Hou <xiangsheng.hou@mediatek.com>
Subject: Re: [PATCH v2 00/17] Prevent NAND chip unevaluated properties
Date: Thu, 15 Jun 2023 10:26:09 +0200	[thread overview]
Message-ID: <20230615102609.0aeea6ab@xps-13> (raw)
In-Reply-To: <20230613134350.3cab6780@xps-13>

Hi Krzysztof,

miquel.raynal@bootlin.com wrote on Tue, 13 Jun 2023 13:43:50 +0200:

> Hi Krzysztof,
> 
> krzk@kernel.org wrote on Sat, 10 Jun 2023 11:27:55 +0200:
> 
> > On 06/06/2023 19:52, Miquel Raynal wrote:  
> > > As discussed with Krzysztof and Chris, it seems like each NAND
> > > controller binding should actually restrain the properties allowed in
> > > the NAND chip node with its own "unevaluatedProperties: false". This
> > > only works if we reference a yaml schema which contains all the possible
> > > properties *in the NAND chip node*. Indeed, the NAND controller yaml
> > > schema contains properties for the NAND chip which are not evaluated
> > > with this construction.
> > > 
> > > Link: https://lore.kernel.org/all/a23dd485-a3d9-e31f-be3e-0ab293fcfc4a@linaro.org/    
> > 
> > Please rebase on latest kernel. This is some very old tree if you CC
> > this address.  
> 
> Sorry for the trouble, it was a hardcoded address on my side. I've
> updated the address to
> 
> 	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> 
> The series is based on v6.4-rc1. Against what branch/tag do you want it
> to be rebased?

When I asked this question I forgot I usually apply mtd bindings in my
own tree :) I usually do it after receiving at least one R-by on the
whole series, but here as I'm the author I would just like to get
confirmation from you Krzysztof that I can go ahead (or if you prefer
to take it, or maybe want to give this another look later).

Thanks,
Miquèl

  reply	other threads:[~2023-06-15  8:26 UTC|newest]

Thread overview: 98+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-06 17:52 [PATCH v2 00/17] Prevent NAND chip unevaluated properties Miquel Raynal
2023-06-06 17:52 ` Miquel Raynal
2023-06-06 17:52 ` [PATCH v2 01/17] dt-bindings: mtd: Accept nand related node names Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-19  6:28   ` Krzysztof Kozlowski
2023-06-19  6:28     ` Krzysztof Kozlowski
2023-06-06 17:52 ` [PATCH v2 02/17] dt-bindings: mtd: Create a file for raw NAND chip properties Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-19  6:28   ` Krzysztof Kozlowski
2023-06-19  6:28     ` Krzysztof Kozlowski
2023-06-19  6:59     ` Miquel Raynal
2023-06-19  6:59       ` Miquel Raynal
2023-06-06 17:52 ` [PATCH v2 03/17] dt-bindings: mtd: Mark nand-ecc-placement deprecated Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-19  6:34   ` Krzysztof Kozlowski
2023-06-19  6:34     ` Krzysztof Kozlowski
2023-06-06 17:52 ` [PATCH v2 04/17] dt-bindings: mtd: Describe nand-ecc-mode Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-19  6:51   ` Krzysztof Kozlowski
2023-06-19  6:51     ` Krzysztof Kozlowski
2023-06-06 17:52 ` [PATCH v2 05/17] dt-bindings: mtd: qcom: Fix a property position Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-19  6:54   ` Krzysztof Kozlowski
2023-06-19  6:54     ` Krzysztof Kozlowski
2023-06-19  7:19     ` Miquel Raynal
2023-06-19  7:19       ` Miquel Raynal
2023-06-06 17:52 ` [PATCH v2 06/17] dt-bindings: mtd: qcom: Prevent NAND chip unevaluated properties Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 22:27   ` Chris Packham
2023-06-06 22:27     ` Chris Packham
2023-06-07  7:53     ` Miquel Raynal
2023-06-07  7:53       ` Miquel Raynal
2023-06-07 20:47       ` Chris Packham
2023-06-07 20:47         ` Chris Packham
2023-06-06 17:52 ` [PATCH v2 07/17] dt-bindings: mtd: ingenic: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-07  9:33     ` Miquel Raynal
2023-06-07  9:33       ` Miquel Raynal
2023-06-08 15:20       ` Rob Herring
2023-06-08 15:20         ` Rob Herring
2023-06-09  8:36         ` Miquel Raynal
2023-06-09  8:36           ` Miquel Raynal
2023-06-06 17:52 ` [PATCH v2 08/17] dt-bindings: mtd: sunxi: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 09/17] dt-bindings: mtd: meson: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 10/17] dt-bindings: mtd: brcmnand: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 11/17] dt-bindings: mtd: denali: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 12/17] dt-bindings: mtd: intel: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 13/17] dt-bindings: mtd: rockchip: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 14/17] dt-bindings: mtd: stm32: " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 17:52 ` [PATCH v2 15/17] dt-bindings: mtd: mediatek: Reference raw-nand-chip.yaml Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 16/17] dt-bindings: mtd: mediatek: Prevent NAND chip unevaluated properties Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-06 18:52   ` Rob Herring
2023-06-06 18:52     ` Rob Herring
2023-06-06 17:52 ` [PATCH v2 17/17] dt-bindings: mtd: ti,am654: Prevent " Miquel Raynal
2023-06-06 17:52   ` Miquel Raynal
2023-06-09 20:51 ` [PATCH v2 00/17] Prevent NAND chip " Rob Herring
2023-06-09 20:51   ` Rob Herring
2023-06-10  9:27 ` Krzysztof Kozlowski
2023-06-10  9:27   ` Krzysztof Kozlowski
2023-06-13 11:43   ` Miquel Raynal
2023-06-13 11:43     ` Miquel Raynal
2023-06-15  8:26     ` Miquel Raynal [this message]
2023-06-15  8:26       ` Miquel Raynal
2023-09-27  7:20 ` Maxime Ripard
2023-09-27  7:20   ` Maxime Ripard
2023-09-27 15:35   ` Maxime Ripard
2023-09-27 15:35     ` Maxime Ripard
2023-09-27  7:20 ` Maxime Ripard
2023-09-27  7:20   ` Maxime Ripard
2023-09-27  7:20 ` Maxime Ripard
2023-09-27  7:20   ` Maxime Ripard
2023-09-27  7:21 ` Maxime Ripard
2023-09-27  7:21   ` Maxime Ripard

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=20230615102609.0aeea6ab@xps-13 \
    --to=miquel.raynal@bootlin.com \
    --cc=Chris.Packham@alliedtelesis.co.nz \
    --cc=alexandre.torgue@foss.st.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=christophe.kerello@foss.st.com \
    --cc=computersforpeace@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=heiko@sntech.de \
    --cc=jernej.skrabec@gmail.com \
    --cc=kdasu.kdev@gmail.com \
    --cc=krzk+dt@kernel.org \
    --cc=krzk@kernel.org \
    --cc=liang.yang@amlogic.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=mani@kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=michael@walle.cc \
    --cc=mripard@kernel.org \
    --cc=paul@crapouillou.net \
    --cc=pratyush@kernel.org \
    --cc=richard@nod.at \
    --cc=robh+dt@kernel.org \
    --cc=samuel@sholland.org \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=tudor.ambarus@linaro.org \
    --cc=vadivel.muruganx.ramuthevar@linux.intel.com \
    --cc=vigneshr@ti.com \
    --cc=wens@csie.org \
    --cc=xiangsheng.hou@mediatek.com \
    --cc=yamada.masahiro@socionext.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.