All of lore.kernel.org
 help / color / mirror / Atom feed
From: Boris Brezillon <boris.brezillon@bootlin.com>
To: Bernhard Frauendienst <kernel@nospam.obeliks.de>
Cc: linux-mtd@lists.infradead.org
Subject: Re: [PATCH 0/3] Virtual MTD concat device driver
Date: Thu, 6 Sep 2018 23:23:41 +0200	[thread overview]
Message-ID: <20180906232341.0dd510e3@bbrezillon> (raw)
In-Reply-To: <d8c4e901-ae96-965c-0d01-4fa012da41b8@nospam.obeliks.de>

On Thu, 6 Sep 2018 23:16:01 +0200
Bernhard Frauendienst <kernel@nospam.obeliks.de> wrote:

> Hi Boris,
> 
> On 06/09/18 23:03, Boris Brezillon wrote:
> > On Thu, 6 Sep 2018 22:59:49 +0200
> > Boris Brezillon <boris.brezillon@bootlin.com> wrote:
> >  
> >> Hi Bernhard,
> >>
> >> On Thu,  6 Sep 2018 18:14:10 +0200
> >> Bernhard Frauendienst <kernel@nospam.obeliks.de> wrote:
> >>  
> >>> Hi everybody,
> >>>
> >>> when porting my router board from a mach-file based OpenWRT target to a
> >>> device-tree based target, I found that there is no generic way to create
> >>> a mtd_concat device from within the dts. The following patches attempt
> >>> to provide that possibility.  
> >> There's a fundamental issue with this patch series: DT is supposed to
> >> represent the hardware, not how you want to use it, and clearly, your
> >> concatenated MTD dev is a virtual device, not a real one, so I doubt it
> >> will pass DT maintainers' review.  
> > BTW, did you consider doing that through the command line?  
> 
> Would you mind elaborating on that?

Something like:

mtdconcat=<virtdev1>=<realdev1>,<realdev2>;<virtdev2>=<realdev3>,<realdev4>

> 
> The biggest problem that I see is that I have to specify the fixed 
> partitions somewhere, and I would really like to keep that in the dts 
> file, where they would also be if it was only a single chip.

Send a v2, add the DT maintainers+ML in Cc, and you'll see what
happens.

  parent reply	other threads:[~2018-09-06 21:24 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-06 16:14 [PATCH 0/3] Virtual MTD concat device driver Bernhard Frauendienst
2018-09-06 16:14 ` [PATCH 1/3] mtd: core: add get_mtd_device_by_node Bernhard Frauendienst
2018-09-06 18:23   ` Miquel Raynal
2018-09-06 16:14 ` [PATCH 2/3] dt-bindings: add bindings for mtd-concat devices Bernhard Frauendienst
2018-09-06 16:14 ` [PATCH 3/3] mtd: mtdconcat: add dt driver for concat devices Bernhard Frauendienst
2018-09-06 18:47   ` Miquel Raynal
2018-09-06 22:54     ` Bernhard Frauendienst
2018-09-07 13:13       ` Miquel Raynal
2018-09-06 20:59 ` [PATCH 0/3] Virtual MTD concat device driver Boris Brezillon
2018-09-06 21:03   ` Boris Brezillon
2018-09-06 21:19     ` Bernhard Frauendienst
     [not found]     ` <d8c4e901-ae96-965c-0d01-4fa012da41b8@nospam.obeliks.de>
2018-09-06 21:23       ` Boris Brezillon [this message]
2018-09-06 21:35         ` Bernhard Frauendienst
2018-09-06 21:11 ` Boris Brezillon
2018-09-06 21:32   ` Bernhard Frauendienst

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=20180906232341.0dd510e3@bbrezillon \
    --to=boris.brezillon@bootlin.com \
    --cc=kernel@nospam.obeliks.de \
    --cc=linux-mtd@lists.infradead.org \
    /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.