From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Packham Date: Mon, 07 May 2018 23:20:52 +0000 Subject: [U-Boot] [PATCH 2/2] spi: kirkwood: Full dm conversion In-Reply-To: References: <20180315113323.32056-1-jagan@amarulasolutions.com> <20180315113323.32056-2-jagan@amarulasolutions.com> <20180427085111.GA11559@kw.sim.vm.gnt> <20180501105458.GA5581@kw.sim.vm.gnt> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, May 3, 2018 at 11:21 PM Stefan Roese wrote: > Hi Chris, > On 02.05.2018 23:56, Chris Packham wrote: > > Hi All, > > On Wed, May 2, 2018 at 10:53 PM Stefan Roese wrote: > > > >> Hi Simon, > > > >> On 01.05.2018 12:54, Simon Guinot wrote: > >>> On Mon, Apr 30, 2018 at 11:28:28AM +0530, Jagan Teki wrote: > >>>> On Fri, Apr 27, 2018 at 2:21 PM, Simon Guinot < > > simon.guinot at sequanux.org> wrote: > >>>>> On Thu, Apr 26, 2018 at 11:30:00AM +0530, Jagan Teki wrote: > >>>>>> On Thu, Mar 15, 2018 at 5:03 PM, Jagan Teki < > > jagan at amarulasolutions.com> wrote: > >>>>>>> kirkwood now support dt along with platform data, > >>>>>>> respective boards need to switch into dm for the same. > >>>>>> > >>>>>> Added all board mainatiner, using this driver on their relevant > >>>>>> boards. So try to switch to DM_SPI(SPI_FLASH) before migration > >>>>>> deadline expires. > >>>>> > >>>>> Hi Jagan, > >>>>> > >>>>> And what is the deadline exactly ? > >>>> > >>>> See DM_SPI/SPI_FLASH migration details from, > > doc/driver-model/MIGRATION.txt > >>> > >>> Thanks for letting me know Jagan... > > > >> Just to be clear here. The older Marvell platforms Orion and Kirkwood > >> completely lack DM (Driver-Model) and DT (Device-Tree) support in > >> U-Boot. This needs to be added (similar to what I've done to the > >> newer parts beginning with Armada XP / 38x) so that the SPI driver > >> (and others) can be used as DM-enabled driver. > > > >> Please see arch/arm/mach-mvebu/ for more details here. > > > >> Any work on this is greatly appreciated as I fear that the support > >> for these older SoC's might get dropped completely otherwise soon. > > > > I had a quick try on one of the kirkwood based boards I have. It was pretty > > easy to bring in the dts files from Linux and get some basic stuff working. > > I started with i2c since I can still boot without it, I haven't been brave > > enough to try spi yet. > > > > Hopefully I can spend a bit more time on it over the weekend. > This sound just great. Thanks. > > In terms of a long-term plan. I could upstream support for our board, we > > still include it in the source we distribute as part of our GPL compliance. > > Since it's a older board that has been fairly stable we're not doing a lot > > of development on it. My motivation for retaining support for kirkwood is > > just in case we have some other EOL part that requires us to release a new > > bootloader. I could do a blind conversion of other in-tree kirkwood boards > > but my ability to test them would be quite limited. > This "blind conversion" is definitely much better, than the removal. > Others might be able to test some of the boards. I don't have any of > those - so I can't help out here. An update on this. I've tested these 2 patches from Jagan on my out-of-tree Kirkwood board with the addition of http://patchwork.ozlabs.org/patch/909973/ the chip seems accessible. So for both Tested-by: Chris Packham In terms of getting the conversion done. I'm about halfway through bringing in the dts files from Linux. My plan is to send an initial series for boards that don't enable CONFIG_CMD_SF that just imports the dts and enables CONFIG_OF_CONTROL without actually enabling the DM for any subsystem (that isn't already enabled). Then for the boards with SF enabled I'll need to build test them with and without Jagan's patches. So that could take a bit longer. Unfortunately in order to maintain bisect-ability these will have have to go in before Jagan's changes can be merged.