From: Wan Zongshun <vw@iommu.org>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
Brian Norris <computersforpeace@gmail.com>,
linux-mtd@lists.infradead.org,
David Woodhouse <dwmw2@infradead.org>,
Wan ZongShun <mcuos.com@gmail.com>,
linux-kernel@vger.kernel.org,
Mike Thompson <mpthompson@gmail.com>
Subject: Re: [PATCH] mtd: nuc900_nand: read correct SMISR register
Date: Fri, 15 Jan 2016 23:01:33 +0800 [thread overview]
Message-ID: <569909CD.1000901@iommu.org> (raw)
In-Reply-To: <11388611.tVUcx6RgHM@wuerfel>
> This sounds great, good to hear!
>
> I see that your last feature contribution to mach-w90x900 was a little over
> 5 years ago, before we started the current arm-soc process, after that
> we only had bug fixes and smaller cleanups.
>
> I don't know to what degree you have been following what the other platforms
> have done in the meantime, so let me try to get you up to speed:
>
> * All patches from platform maintainers by default get merged through the
> arm-soc tree at http://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git
> Please send patches "To: arm@kernel.org", Cc the relevant mailing lists,
> and split them up to you submit bug fixes, cleanups, and new features
> separately (they go into different branches)
>
> * All actively maintained platforms should use CONFIG_ARCH_MULTIPLATFORM.
> This will take significant work for mach-w90x900, but you don't have to
> do it all at once. As a rule of thumb, I'd expect that for any work
> you do on new features in the platform, an at least equal amount of work
> is spent on getting closer to multiplatform support until you are done.
> This has worked well for most other platforms, and with the work that
> Mike has done it is all complete.
>
> * We have basically stopped taking new board files and moved on to
> using devicetree descriptions of the hardware. It's probably ok to
> add one more board file for the nuc970 reference design here if you
> also work on the cleanup and you don't expect to add multiple other
> machines. Again, Mike's port seems already uses DT.
>
> * The multiplatform work does not require the dts conversion, but it
> does require converting the clock.c file into a driver for drivers/clk/,
> and it requires converting the interrupt handling to use an irqchip
> driver with CONFIG_IRQ_DOMAIN and CONFIG_MULTI_IRQ_HANDLER. These
> are usually not hard to do if you have the hardware for testing, but
> it's not easy otherwise.
>
> * I've merged a patch that moves all mach/*.h headers out of the globally
> visible directory to arch/arm/mach-w90x900/*.h directly unless they are
> used by some driver outside of mach-w90x900. To complete the multiplatform
> work, all remaining headers have to be moved as well, either into the
> driver or into the platform directory.
>
> * I'd suggest you start by looking at n329 code from Mike before you
> submit the nuc970 support. As I think it gets all of the above right
> already, it may even be possible to support the nuc970 through the
> new mach-n329 support, or to share a large portion of the code.
>
Arnd,
I really appreciate your great nice help.
I will fully consider your proposal and try my best to clean nuc900 plat
codes to keep up to date.
Wan Zongshun.
> Arnd
>
WARNING: multiple messages have this Message-ID (diff)
From: vw@iommu.org (Wan Zongshun)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] mtd: nuc900_nand: read correct SMISR register
Date: Fri, 15 Jan 2016 23:01:33 +0800 [thread overview]
Message-ID: <569909CD.1000901@iommu.org> (raw)
In-Reply-To: <11388611.tVUcx6RgHM@wuerfel>
> This sounds great, good to hear!
>
> I see that your last feature contribution to mach-w90x900 was a little over
> 5 years ago, before we started the current arm-soc process, after that
> we only had bug fixes and smaller cleanups.
>
> I don't know to what degree you have been following what the other platforms
> have done in the meantime, so let me try to get you up to speed:
>
> * All patches from platform maintainers by default get merged through the
> arm-soc tree at http://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git
> Please send patches "To: arm at kernel.org", Cc the relevant mailing lists,
> and split them up to you submit bug fixes, cleanups, and new features
> separately (they go into different branches)
>
> * All actively maintained platforms should use CONFIG_ARCH_MULTIPLATFORM.
> This will take significant work for mach-w90x900, but you don't have to
> do it all at once. As a rule of thumb, I'd expect that for any work
> you do on new features in the platform, an at least equal amount of work
> is spent on getting closer to multiplatform support until you are done.
> This has worked well for most other platforms, and with the work that
> Mike has done it is all complete.
>
> * We have basically stopped taking new board files and moved on to
> using devicetree descriptions of the hardware. It's probably ok to
> add one more board file for the nuc970 reference design here if you
> also work on the cleanup and you don't expect to add multiple other
> machines. Again, Mike's port seems already uses DT.
>
> * The multiplatform work does not require the dts conversion, but it
> does require converting the clock.c file into a driver for drivers/clk/,
> and it requires converting the interrupt handling to use an irqchip
> driver with CONFIG_IRQ_DOMAIN and CONFIG_MULTI_IRQ_HANDLER. These
> are usually not hard to do if you have the hardware for testing, but
> it's not easy otherwise.
>
> * I've merged a patch that moves all mach/*.h headers out of the globally
> visible directory to arch/arm/mach-w90x900/*.h directly unless they are
> used by some driver outside of mach-w90x900. To complete the multiplatform
> work, all remaining headers have to be moved as well, either into the
> driver or into the platform directory.
>
> * I'd suggest you start by looking at n329 code from Mike before you
> submit the nuc970 support. As I think it gets all of the above right
> already, it may even be possible to support the nuc970 through the
> new mach-n329 support, or to share a large portion of the code.
>
Arnd,
I really appreciate your great nice help.
I will fully consider your proposal and try my best to clean nuc900 plat
codes to keep up to date.
Wan Zongshun.
> Arnd
>
next prev parent reply other threads:[~2016-01-15 15:01 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-13 21:38 [PATCH] mtd: nuc900_nand: read correct SMISR register Arnd Bergmann
2016-01-13 21:38 ` Arnd Bergmann
2016-01-13 21:50 ` Brian Norris
2016-01-13 21:50 ` Brian Norris
2016-01-13 22:02 ` Arnd Bergmann
2016-01-13 22:02 ` Arnd Bergmann
2016-01-14 0:29 ` Brian Norris
2016-01-14 0:29 ` Brian Norris
2016-01-14 9:34 ` Wan Zongshun
2016-01-14 9:34 ` Wan Zongshun
2016-01-14 12:08 ` Arnd Bergmann
2016-01-14 12:08 ` Arnd Bergmann
2016-01-15 7:53 ` Wan Zongshun
2016-01-15 7:53 ` Wan Zongshun
2016-01-15 11:03 ` Arnd Bergmann
2016-01-15 11:03 ` Arnd Bergmann
2016-01-15 15:01 ` Wan Zongshun [this message]
2016-01-15 15:01 ` Wan Zongshun
2016-01-15 18:03 ` Brian Norris
2016-01-15 18:03 ` Brian Norris
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=569909CD.1000901@iommu.org \
--to=vw@iommu.org \
--cc=arnd@arndb.de \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=mcuos.com@gmail.com \
--cc=mpthompson@gmail.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.