Re: [pasemi] Internal CompactFlash (CF) card device not recognised after the powerpc-4.8-1 merge

From: Nicholas Piggin <npiggin@gmail.com>
To: Christian Zigotzky <chzigotzky@xenosoft.de>
Cc: "Michael Ellerman" <mpe@ellerman.id.au>,
	"Benjamin Herrenschmidt" <benh@kernel.crashing.org>,
	"luigi burdo" <intermediadc@hotmail.com>,
	"Darren Stevens" <darren@stevens-zone.net>,
	"Julian Margetson" <runaway@candw.ms>,
	"Steven Jeffery" <sajeffer@gmail.com>,
	"R.T.Dickinson" <rtd@a-eon.com>,
	"R.T.Dickinson" <rtd2@xtra.co.nz>, "Pat Wall" <pjwall@mac.com>,
	"Pat Wall" <pjwall@me.com>,
	"contact@a-eon.com" <contact@a-eon.com>,
	"matthew@a-eon.biz" <matthew@a-eon.biz>,
	"Kevin Saunders" <kfs1971@gmail.com>,
	"Lyle Hazelwood" <lylehaze@gmail.com>,
	EntwicklerX <mail@entwickler-x.de>,
	"xray@relec.ch" <xray@relec.ch>,
	"Steve-David Marguet" <steve@esdeem.ch>,
	"George Sokianos" <walkero@gmail.com>,
	"Sinan Gürkan" <sinan.amigaone@gmail.com>,
	Mike <michael.heltne@gmail.com>,
	"Thomas Ferreira" <macsociety@me.com>,
	"Christian Zigotzky" <info@xenosoft.de>,
	"pjs@pjsa.net" <pjs@pjsa.net>,
	"Ben Cato Malkenes" <ben.cato@gmail.com>,
	"Olof Johansson" <olof@lixom.net>,
	linuxppc-dev@lists.ozlabs.org
Subject: Re: [pasemi] Internal CompactFlash (CF) card device not recognised after the powerpc-4.8-1 merge
Date: Sat, 6 Aug 2016 01:24:54 +1000	[thread overview]
Message-ID: <20160806012454.0c9a26f9@roar.ozlabs.ibm.com> (raw)
In-Reply-To: <4168bd4c-f895-fd08-0d1a-ca4c999ab6b2@xenosoft.de>

On Fri, 5 Aug 2016 16:38:17 +0200
Christian Zigotzky <chzigotzky@xenosoft.de> wrote:

> Hi All,

Hi Christian,

Firstly, thanks for the report. As a suggestion, it can be
better to reduce the CC list unless you have found a specific
commit that causes the problem. powerpc developers read this
list so you can send initial bug reports there.

Have you been able to bisect the exact commit which caused the
regression? If it's possible, that is the fastest way to get a
response to your report.

As for your driver support, it would indeed be a good idea to
get it supported in the upstream kernel. You should post a
new mail about that. Take a look at these 3 commits:

61f7162117d4767875825abf2f6ed1eeebbcceed
9cd55be4d22376893d2818ce3c0e5706a3d74121
ca99140a63b7326ee9a38f64c326317f2c63b594

Your patch comes from code based on the second one. The last
commit removed it, and says that it is not the best way to
implement it. You could cc this list and some of the people
involved with those commits and ask ask for advice about
getting your driver supported.

Thanks,
Nick

> I have found some information about the electra IDE CF card device: 
> http://linuxppc-dev.ozlabs.narkive.com/kxQRFqGe/patch-pasemi-electra-ide-pata-platform-glue
> 
> I checked the kernel config and the merge but without any success.
> 
> Cheers,
> 
> Christian
> 
> On 05 August 2016 at 1:41 PM, Christian Zigotzky wrote:
> > Hi All,
> >
> > The internal PASEMI CompactFlash (CF) card device doesn't work anymore 
> > after the powerpc-4.8-1 merge. That means the code for the internal CF 
> > card device in the Nemo patch doesn't work after the first PowerPC 
> > merge. The CompactFlash (CF) card slot is wired to the CPU local bus. 
> > It is typically used to hold the Linux kernel. I know it isn't well to 
> > use an own patch for that but I think it is a good time to integrate 
> > the PASEMI internal CompactFlash (CF) card device to the official 
> > kernel. What do you think? I am not a programmer so I can't integrate 
> > the source code for the internal CF card device. But maybe you can 
> > take the patch and integrate it.
> >
> > We use the following patch for the kernel 4.7: