public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v1 1/3] powerpc, 8xx: remove support for 8xx
Date: Wed, 14 Jun 2017 15:23:39 +0200	[thread overview]
Message-ID: <594138DB.1050300@denx.de> (raw)
In-Reply-To: <e14b80d8-e2b9-4f27-0191-bb64cd987edb@c-s.fr>

Hello Christophe,

Am 14.06.2017 um 09:40 schrieb Christophe LEROY:
>
>
> Le 13/06/2017 à 09:37, Heiko Schocher a écrit :
>> Hello Christophe,
>>
>> Am 13.06.2017 um 07:40 schrieb Christophe LEROY:
>>>
>>>
>>> Le 13/06/2017 à 07:26, Christophe LEROY a écrit :
>>>> There was for long time no activity in the 8xx area.
>>>> We need to go further and convert to Kconfig, but it
>>>> turned out, nobody is interested anymore in 8xx,
>>>> so remove it (with a heavy heart, knowing that I remove
>>>> here the root of U-Boot).
>>>>
>>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>>>
>>>
>>> Please don't do that.
>>
>> Tom already applied the patch to mainline ...
>
> Can be reverted ?
>
>>
>>> As you can see in Linux kernel activity, there have been a lot activity related to the 8xx,
>>> including but not limited to:
>>> 1/ HW Crypto for the 885 (Talitos SEC1)
>>> 2/ TX NAPI in the 8xx Ethernet driver
>>> 3/ Scatter/Gather support in the 8xx Ethernet driver
>>> 4/ Hugepages
>>> 5/ Perf events
>>> 6/ hw breakpoints
>>> 7/ Linear memory mapping via Large TLBs
>>
>> Thats linux ... not u-boot!
>
> Sure, but it shows there is still interest to that processor. The 885 is a good recent 8xx.
> u-boot is not really something we focus on. We update it once a year, as far as it can start our
> Linux box we are happy.

Hmm...

>>> The followings links give an overview of the activity:
>>>
>>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/log/drivers/crypto/talitos.c
>>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/log/drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c
>>>
>>>
>>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/log/arch/powerpc/kernel/head_8xx.S
>>>
>>>
>>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/log/arch/powerpc/mm/8xx_mmu.c
>>>
>>>
>>> We have thousands of boards with mpc885 running and requiring support for at least the next 10
>>> years.
>>
>> Ok, nice to hear!
>>
>>> How can the 8xx survive without U-boot support ?
>>
>> Tom asked (I think a lot of times) regarding converting mpc8xx to
>> Kconfig / DM and nobody did the necessary steps. We also asked
>> our customers if they can do the necessary changes, none was interested.
>
> I did't know. Indeed I was not following uboot activity until someone who knows the level of
> interest we have in 8xx alerted me yesterday.
> It would have been nice if you had notified linuxppc-dev list.
>
>>
>> So, if you need mpc8xx support in U-Boot, simply add it again with
>> Kconfig and DM support included!
>
> Ok, I'll try and come with a patch to convert 8xx in the coming weeks. In the mean time please
> revert the deletion in order to avoid nightmare conflicts when the converting patch comes.

I vote for making a patch which adds new mpc8xx support, as we had a lot
of crap in the code, but thats Tom decision!

bye,
Heiko
-- 
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

  reply	other threads:[~2017-06-14 13:23 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <8472b742-049b-070b-2627-27704f5bfc0a@c-s.fr>
2017-06-13  5:40 ` [U-Boot] [PATCH v1 1/3] powerpc, 8xx: remove support for 8xx Christophe LEROY
2017-06-13  7:37   ` Heiko Schocher
2017-06-13 12:15     ` Tom Rini
2017-06-14  7:40     ` Christophe LEROY
2017-06-14 13:23       ` Heiko Schocher [this message]
2017-06-14 13:27       ` Tom Rini
2017-06-07 15:33 [U-Boot] [PATCH v1 0/3] remove 8xx/82xx and mpc5xx Heiko Schocher
2017-06-07 15:33 ` [U-Boot] [PATCH v1 1/3] powerpc, 8xx: remove support for 8xx Heiko Schocher

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=594138DB.1050300@denx.de \
    --to=hs@denx.de \
    --cc=u-boot@lists.denx.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox