Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-05 Thread Rob Herring
On Thu, May 4, 2023 at 11:42 AM Jassi Brar wrote: > > On Thu, 4 May 2023 at 11:31, Ilias Apalodimas > wrote: > > > > Replying to both Jassi and Tom here since it makes more sense, > > > > On Thu, 4 May 2023 at 19:19, Tom Rini wrote: > > > > > > On Thu, May 04, 2023 at 10:39:06AM -0500, Jassi

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-05 Thread Michal Simek
On 5/4/23 17:19, Rob Herring wrote: On Thu, May 4, 2023 at 9:01 AM Jassi Brar wrote: On Thu, 4 May 2023 at 07:08, Ilias Apalodimas wrote: [...] I'm assuming it's per partition type rather than storage medium (e.g. SATA, USB, SD, NAND, SPI-NOR)? GPT, 'fixed-partitions', other DT

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Jassi Brar
On Thu, 4 May 2023 at 11:31, Ilias Apalodimas wrote: > > Replying to both Jassi and Tom here since it makes more sense, > > On Thu, 4 May 2023 at 19:19, Tom Rini wrote: > > > > On Thu, May 04, 2023 at 10:39:06AM -0500, Jassi Brar wrote: > > > On Thu, 4 May 2023 at 10:19, Rob Herring wrote: > >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Ilias Apalodimas
Replying to both Jassi and Tom here since it makes more sense, On Thu, 4 May 2023 at 19:19, Tom Rini wrote: > > On Thu, May 04, 2023 at 10:39:06AM -0500, Jassi Brar wrote: > > On Thu, 4 May 2023 at 10:19, Rob Herring wrote: > > > On Thu, May 4, 2023 at 9:01 AM Jassi Brar > > > wrote: > > > >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Tom Rini
On Thu, May 04, 2023 at 10:39:06AM -0500, Jassi Brar wrote: > On Thu, 4 May 2023 at 10:19, Rob Herring wrote: > > On Thu, May 4, 2023 at 9:01 AM Jassi Brar > > wrote: > > > > > I may be wrong, but I see having fwu properties contained within the > > > fwu node is cleaner than having them

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Jassi Brar
On Thu, 4 May 2023 at 10:19, Rob Herring wrote: > On Thu, May 4, 2023 at 9:01 AM Jassi Brar wrote: > > > I may be wrong, but I see having fwu properties contained within the > > fwu node is cleaner than having them embedded into existing bindings > > (potentially different classes in future).

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Rob Herring
On Thu, May 4, 2023 at 9:01 AM Jassi Brar wrote: > > On Thu, 4 May 2023 at 07:08, Ilias Apalodimas > wrote: > > [...] > > > > > > > I'm assuming it's per partition type rather than storage medium (e.g. > > > > > SATA, USB, SD, NAND, SPI-NOR)? GPT, 'fixed-partitions', other DT > > > > > partition

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Jassi Brar
On Thu, 4 May 2023 at 07:08, Ilias Apalodimas wrote: > [...] > > > > > I'm assuming it's per partition type rather than storage medium (e.g. > > > > SATA, USB, SD, NAND, SPI-NOR)? GPT, 'fixed-partitions', other DT > > > > partition bindings, etc. If so, then I'm really wondering why it's a > > >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Ilias Apalodimas
[...] > > > I'm assuming it's per partition type rather than storage medium (e.g. > > > SATA, USB, SD, NAND, SPI-NOR)? GPT, 'fixed-partitions', other DT > > > partition bindings, etc. If so, then I'm really wondering why it's a > > > standalone tree rather than integrated into those existing

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Sughosh Ganu
hi Ilias, On Thu, 4 May 2023 at 16:15, Ilias Apalodimas wrote: > > Hi Rob, > > On Wed, May 03, 2023 at 12:24:39PM -0500, Rob Herring wrote: > > On Wed, May 3, 2023 at 9:37 AM Ilias Apalodimas > > wrote: > > > > > > Hi Krzysztof, > > > > > > On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-04 Thread Ilias Apalodimas
Hi Rob, On Wed, May 03, 2023 at 12:24:39PM -0500, Rob Herring wrote: > On Wed, May 3, 2023 at 9:37 AM Ilias Apalodimas > wrote: > > > > Hi Krzysztof, > > > > On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof Kozlowski wrote: > > > On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: > > > >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-03 Thread Rob Herring
On Wed, May 3, 2023 at 9:37 AM Ilias Apalodimas wrote: > > Hi Krzysztof, > > On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof Kozlowski wrote: > > On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: > > > From: Jassi Brar > > > > > > Any requirement of FWU should not require changes to

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-03 Thread Krzysztof Kozlowski
On 03/05/2023 18:26, Tom Rini wrote: I think we do not review U-Boot bindings usually, except these put in the Linux kernel. There were few targeting U-Boot specifically (e.g. Documentation/devicetree/bindings/mtd/partitions/u-boot.yaml and

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-03 Thread Tom Rini
On Wed, May 03, 2023 at 04:54:45PM +0200, Krzysztof Kozlowski wrote: > On 03/05/2023 16:37, Ilias Apalodimas wrote: > > Hi Krzysztof, > > > > On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof Kozlowski wrote: > >> On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: > >>> From: Jassi Brar >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-03 Thread Krzysztof Kozlowski
On 03/05/2023 16:37, Ilias Apalodimas wrote: > Hi Krzysztof, > > On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof Kozlowski wrote: >> On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: >>> From: Jassi Brar >>> >>> Any requirement of FWU should not require changes to bindings >>> of other

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-05-03 Thread Ilias Apalodimas
Hi Krzysztof, On Tue, Apr 11, 2023 at 07:38:11AM +0200, Krzysztof Kozlowski wrote: > On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: > > From: Jassi Brar > > > > Any requirement of FWU should not require changes to bindings > > of other subsystems. For example, for mtd-backed storage we >

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-04-14 Thread Michal Simek
On 4/11/23 07:38, Krzysztof Kozlowski wrote: On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: From: Jassi Brar Any requirement of FWU should not require changes to bindings of other subsystems. For example, for mtd-backed storage we can do without requiring 'fixed-partitions'

Re: [PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-04-11 Thread Krzysztof Kozlowski
On 11/04/2023 01:21, jaswinder.si...@linaro.org wrote: > From: Jassi Brar > > Any requirement of FWU should not require changes to bindings > of other subsystems. For example, for mtd-backed storage we > can do without requiring 'fixed-partitions' children to also > carry 'uuid', a property

[PATCH] dt/bindings: fwu-mdata-mtd: drop changes outside FWU

2023-04-10 Thread jaswinder . singh
From: Jassi Brar Any requirement of FWU should not require changes to bindings of other subsystems. For example, for mtd-backed storage we can do without requiring 'fixed-partitions' children to also carry 'uuid', a property which is non-standard and not in the bindings. There exists no code