Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-09 Thread Simon Glass
Hi Rob, On Fri, 6 Oct 2023 at 10:11, Rob Herring wrote: > > On Fri, Oct 06, 2023 at 10:37:41AM +0200, Michael Walle wrote: > > Hi, > > > > > > I'm still not sure why that compatible is needed. Also I'd need to > > > > change > > > > the label which might break user space apps looking for that spe

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-06 Thread Rob Herring
On Fri, Oct 06, 2023 at 10:37:41AM +0200, Michael Walle wrote: > Hi, > > > > I'm still not sure why that compatible is needed. Also I'd need to > > > change > > > the label which might break user space apps looking for that specific > > > name. > > > > > > Also, our board might have u-boot/spl or

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-06 Thread Simon Glass
Hi Michael, On Fri, 6 Oct 2023 at 02:37, Michael Walle wrote: > > Hi, > > >> I'm still not sure why that compatible is needed. Also I'd need to > >> change > >> the label which might break user space apps looking for that specific > >> name. > >> > >> Also, our board might have u-boot/spl or u-bo

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-06 Thread Michael Walle
Hi, I'm still not sure why that compatible is needed. Also I'd need to change the label which might break user space apps looking for that specific name. Also, our board might have u-boot/spl or u-boot/spl/bl31/bl32, right now that's something which depends on an u-boot configuration variable

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-05 Thread Simon Glass
Hi Michael, On Thu, 5 Oct 2023 at 07:28, Simon Glass wrote: > > Hi Michael, > > On Thu, 5 Oct 2023 at 02:54, Michael Walle wrote: > > > > Hi, > > > > >> >> >> Add a compatible string for binman, so we can extend > > >> >> >> fixed-partitions > > >> >> >> in various ways. > > >> >> > > > >> >> >

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-05 Thread Simon Glass
Hi Michael, On Thu, 5 Oct 2023 at 02:54, Michael Walle wrote: > > Hi, > > >> >> >> Add a compatible string for binman, so we can extend fixed-partitions > >> >> >> in various ways. > >> >> > > >> >> > I've been thinking at the proper way to describe the binman > >> >> > partitions. > >> >> > I a

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-05 Thread Michael Walle
Hi, >> >> Add a compatible string for binman, so we can extend fixed-partitions >> >> in various ways. >> > >> > I've been thinking at the proper way to describe the binman partitions. >> > I am wondering if we should really extend the fixed-partitions >> > schema. This description is really bas

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Simon Glass
Hi Michael, On Wed, 4 Oct 2023 at 11:17, Michael Walle wrote: > > Hi, > > >> >> Add a compatible string for binman, so we can extend fixed-partitions > >> >> in various ways. > >> > > >> > I've been thinking at the proper way to describe the binman partitions. > >> > I am wondering if we should r

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Michael Walle
Hi, >> Add a compatible string for binman, so we can extend fixed-partitions >> in various ways. > > I've been thinking at the proper way to describe the binman partitions. > I am wondering if we should really extend the fixed-partitions > schema. This description is really basic and kind of sup

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Simon Glass
Hi Michael, On Wed, 4 Oct 2023 at 05:35, Michael Walle wrote: > > Hi, > > >> Add a compatible string for binman, so we can extend fixed-partitions > >> in various ways. > > > > I've been thinking at the proper way to describe the binman partitions. > > I am wondering if we should really extend th

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Simon Glass
Hi Miquel, On Wed, 4 Oct 2023 at 01:36, Miquel Raynal wrote: > > Hi Simon, > > s...@chromium.org wrote on Mon, 2 Oct 2023 11:49:40 -0600: > > > Add a compatible string for binman, so we can extend fixed-partitions > > in various ways. > > I've been thinking at the proper way to describe the binm

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Michael Walle
Hi, >> Add a compatible string for binman, so we can extend fixed-partitions >> in various ways. > > I've been thinking at the proper way to describe the binman partitions. > I am wondering if we should really extend the fixed-partitions > schema. This description is really basic and kind of supp

Re: [PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-04 Thread Miquel Raynal
Hi Simon, s...@chromium.org wrote on Mon, 2 Oct 2023 11:49:40 -0600: > Add a compatible string for binman, so we can extend fixed-partitions > in various ways. I've been thinking at the proper way to describe the binman partitions. I am wondering if we should really extend the fixed-partitions

[PATCH v2 1/3] dt-bindings: mtd: fixed-partitions: Add binman compatible

2023-10-02 Thread Simon Glass
Add a compatible string for binman, so we can extend fixed-partitions in various ways. Signed-off-by: Simon Glass --- Changes in v2: - Drop mention of 'enhanced features' in fixed-partitions.yaml - Mention Binman input and output properties - Use plain partition@xxx for the node name .../bindi