2020-03-28 15:07:44

by Christian Lamparter

[permalink] [raw]
Subject: Re: [PATCH 0/2] powerpc: Remove support for ppc405/440 Xilinx platforms

(Sorry for the bounces, yes my smarthost mail setup breaks from time to time
and I had to cut the CC since it complained about the length.)

On Saturday, 28 March 2020 12:17:58 CET Christophe Leroy wrote:
>
> Le 27/03/2020 ? 15:14, Andy Shevchenko a ?crit :
>> On Fri, Mar 27, 2020 at 02:22:55PM +0100, Arnd Bergmann wrote:
>>> On Fri, Mar 27, 2020 at 2:15 PM Andy Shevchenko
>>> <[email protected]> wrote:
>>>> On Fri, Mar 27, 2020 at 03:10:26PM +0200, Andy Shevchenko wrote:
>>>>> On Fri, Mar 27, 2020 at 01:54:33PM +0100, Arnd Bergmann wrote:
>>>>>> On Fri, Mar 27, 2020 at 1:12 PM Michal Simek
>>>>>> <[email protected]> wrote:
>>>>>> It does raise a follow-up question about ppc40x though: is it time to
>>>>>> retire all of it?
>>>>>
>>>>> Who knows?
>>>>>
>>>>> I have in possession nice WD My Book Live, based on this
>>>>> architecture, and I won't it gone from modern kernel support.
>>>>> OTOH I understand that amount of real users not too big.
Hm, can't add much to Xilinx ppc405/440 removal patch debate.

But as for the APM82181 with it's PPC464:

The last time I checked was with 5.6-rc4, it worked fine on the APM82181
(a MyBook Live) device. I've made a "build your own powerpc debian sid"
image thing that takes the latest kernel git and up-to-date packages
from debian ports (they still make powerpc packages!):
<https://github.com/chunkeey/mbl-debian> .

Though, this is small potatoes. There exists a much more popular project
by Ewald Comhaire (CCed): <https://github.com/ewaldc/My-Book-Live>
that serves the largest userbase:
<https://community.wd.com/c/wd-legacy-products>

I guess we should think about upstreaming the MyBook Live DTS. Problem here
is that we deviated a bit from the canyonlands.dts/bluestone.dts structure
by having a skeleton apm82181.dtsi with most of the SoC defintition and a
wd-mybooklive.dts for the device.

<https://github.com/chunkeey/mbl-debian/blob/master/dts/apm82181.dtsi>
<https://github.com/chunkeey/mbl-debian/blob/master/dts/wd-mybooklive.dts>

Cheers,
Christian




2020-03-30 08:53:08

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 0/2] powerpc: Remove support for ppc405/440 Xilinx platforms

On Sat, Mar 28, 2020 at 4:07 PM Christian Lamparter <[email protected]> wrote:
>
> (Sorry for the bounces, yes my smarthost mail setup breaks from time to time
> and I had to cut the CC since it complained about the length.)
>
> I guess we should think about upstreaming the MyBook Live DTS. Problem here
> is that we deviated a bit from the canyonlands.dts/bluestone.dts structure
> by having a skeleton apm82181.dtsi with most of the SoC defintition and a
> wd-mybooklive.dts for the device.

I don't see much of a problem with that. If the chips are quite
different, then this
would be the right way to do it, and it matches how most dts files are
structured
on arm and others as well.

It would be nice to move over the the bluestone .dts to the apm82181.dtsi file
when that gets added, if only to ensure they use the same description for each
node, but that shouldn't stop the addition of the new file if that is needed for
distros to make use of a popular device.
I see a couple of additional files in openwrt [1],

[obviously, I'm not actually the one in charge here, the decision is with the
powerpc maintainers].

Arnd

[1] https://git.openwrt.org/?p=openwrt/openwrt.git;a=tree;f=target/linux/apm821xx/dts;h=d359ba4ed83547438581dd40105e5100d8240beb;hb=HEAD