All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot-Custodians] [U-Boot-Board-Maintainers] [RFC] Eliminate boards not using CONFIG_DM=y
Date: Thu, 28 Nov 2019 10:40:38 +0100	[thread overview]
Message-ID: <0e9a704a-8c37-0924-cdc5-f5fe6acd2c10@denx.de> (raw)
In-Reply-To: <a1af91f2-ee71-ff25-5605-1dfeb838ef55@gmx.de>

On 11/28/19 7:22 AM, Heinrich Schuchardt wrote:
> On 11/26/19 6:07 PM, Marek Vasut wrote:
>> On 11/26/19 5:52 PM, Tom Rini wrote:
>>> On Tue, Nov 26, 2019 at 05:47:48PM +0100, Marek Vasut wrote:
>>>> On 11/26/19 5:26 PM, Tom Rini wrote:
>>>>> On Tue, Nov 26, 2019 at 09:11:51AM +0100, Marek Vasut wrote:
>>>>>> On 11/26/19 12:16 AM, Heinrich Schuchardt wrote:
>>>>>>> Dear maintainers,
>>>>>>
>>>>>> Hi,
>>>>>>
>>>>>>> we have been trying to move to the driver model for several years
>>>>>>> now.
>>>>>>> Starting in 2018 we have added warnings to the Makefile that
>>>>>>> boards not
>>>>>>> supporting the driver model will be eliminated. Still 24 % of the
>>>>>>> configuration files have not been converted and do not even use
>>>>>>> CONFIG_DM=y.
>>>>>>>
>>>>>>> If we want to get rid of legacy drivers, at some point we have to
>>>>>>> remove
>>>>>>> the 347 configuration files in the list below not using the
>>>>>>> driver model.
>>>>>>>
>>>>>>> I suggest to do this directly after the release of v2020.01
>>>>>>> scheduled
>>>>>>> January 6th.
>>>>>>>
>>>>>>> This should not stop the maintainers from reinserting the boards
>>>>>>> after
>>>>>>> conversion to the driver model.
>>>>>>
>>>>>> Some boards just cannot accommodate this DM stuff. For those boards,
>>>>>> it's just bloat without any useful added value. Hence, these boards
>>>>>> would be removed because they cannot accommodate arbitrary bloat.
>>>>>> This
>>>>>> makes U-Boot not-so-universal bootloader anymore, but rather a
>>>>>> bloated one.
>>>>>>
>>>>>> I don't think we can force boards out or impose DM on everyone
>>>>>> unless we
>>>>>> can solve this bloat issue first.
>>>>>
>>>>> As someone who was involved in creating this DM stuff, do you have
>>>>> some
>>>>> ideas on addressing things?  Given that you're responsible for a
>>>>> number
>>>>> of these platforms and can test out some ideas on them, what are you
>>>>> suggesting?
>>>>
>>>> How about directly calling driver functions for drivers which have
>>>> single instance only ? Then we could optimize out all the DM overhead
>>>> for that.
>>>
>>> And when are you hoping to post an RFC / example?
>>
>> Currently I have zero time available. Maybe someone else can look into
>> this option?
> 
> Dear Marex,
> 
> DM drivers make use of the DM infrastructure for instance for the
> allocation of the private data area. The uclass files often include
> common logic needed for accessing all drivers (see for example tpm_xfer()).
> 
> So which drivers do you think of that could be simplified?

UART for example ? You only usually have one.

> I would also be interested to learn which of the 347 boards not using
> CONFIG_DM=y are still actively maintained.

Probably quite a few of those iMX, omap and PPC ones.
The qemu ones are probably used for CI ?

  reply	other threads:[~2019-11-28  9:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-25 23:16 [U-Boot] [RFC] Eliminate boards not using CONFIG_DM=y Heinrich Schuchardt
2019-11-26  8:11 ` [U-Boot] [U-Boot-Board-Maintainers] " Marek Vasut
2019-11-26 16:26   ` [U-Boot] [U-Boot-Custodians] " Tom Rini
2019-11-26 16:31     ` [U-Boot] [U-Boot-Board-Maintainers] [U-Boot-Custodians] " Lukasz Majewski
2019-11-26 16:56       ` Tom Rini
2019-11-26 16:47     ` [U-Boot] [U-Boot-Custodians] [U-Boot-Board-Maintainers] " Marek Vasut
2019-11-26 16:52       ` Tom Rini
2019-11-26 17:07         ` Marek Vasut
2019-11-28  6:22           ` Heinrich Schuchardt
2019-11-28  9:40             ` Marek Vasut [this message]
2019-11-28 16:51               ` Tom Rini
2019-11-29  6:46 ` [U-Boot] [U-Boot-Custodians] " Priyanka Jain
2019-11-29 15:18   ` Tom Rini

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=0e9a704a-8c37-0924-cdc5-f5fe6acd2c10@denx.de \
    --to=marex@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 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.