All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC for-v2019.01 4/4] dm: MIGRATION: Update migration plan for BLK
Date: Thu, 29 Nov 2018 20:17:11 +0100	[thread overview]
Message-ID: <3b1cac94-3eeb-c0e1-5d7d-09070480e4d5@gmail.com> (raw)
In-Reply-To: <20181129190601.GM5048@bill-the-cat>

On 29.11.2018 20:06, Tom Rini wrote:
> On Thu, Nov 29, 2018 at 11:43:14AM -0700, Simon Glass wrote:
>> Hi Tom,
>>
>> On Thu, 29 Nov 2018 at 11:34, Tom Rini <trini@konsulko.com> wrote:
>>> On Thu, Nov 29, 2018 at 10:10:01AM -0700, Simon Glass wrote:
>>>> Hi Tom,
>>>>
>>>> On Sun, 25 Nov 2018 at 11:19, Tom Rini <trini@konsulko.com> wrote:
>>>>> The biggest part of migration to using CONFIG_BLK is that we need to
>>>>> have the various subsystems migrated first, so reword the plan here to
>>>>> reference the new deadlines.
>>>>>
>>>>> Cc: Simon Glass <sjg@chromium.org>
>>>>> Signed-off-by: Tom Rini <trini@konsulko.com>
>>>>> ---
>>>>>   doc/driver-model/MIGRATION.txt | 12 +++++-------
>>>>>   1 file changed, 5 insertions(+), 7 deletions(-)
>>>>>
>>>>> diff --git a/doc/driver-model/MIGRATION.txt b/doc/driver-model/MIGRATION.txt
>>>>> index 6df7e02a63de..6b691338b4e7 100644
>>>>> --- a/doc/driver-model/MIGRATION.txt
>>>>> +++ b/doc/driver-model/MIGRATION.txt
>>>>> @@ -39,14 +39,12 @@ CONFIG_BLK
>>>>>   ----------
>>>>>
>>>>>   Status: In progress
>>>>> -Deadline: 2018.05
>>>>> -
>>>>> -Maintainers should submit patches for enabling CONFIG_BLK on all boards in
>>>>> -time for inclusion in the 2018.05 release. Boards not converted by this
>>>>> -time may be removed in a subsequent release.
>>>>> +Deadline: 2019.07
>>>>>
>>>>> -Note that this implies use of driver model for all block devices (e.g.
>>>>> -MMC, USB, SCSI, SATA).
>>>>> +In concert with maintainers migrating their block device usage to the
>>>>> +appropriate DM driver, CONFIG_BLK needs to be set as well.  The final deadline
>>>>> +here coincides with the final deadline for migration of the various block
>>>>> +subsystems.
>>>>>
>>>>>   CONFIG_DM_SPI
>>>>>   CONFIG_DM_SPI_FLASH
>>>> My only concern here is the long deadline, more than a year after the
>>>> original one. Granted I should have been more proactive in sending
>>>> patches in May/June, but this has been fairly widely telegraphed IMO.
>>>> I know that opinions different on this matter.
>>>>
>>>> I'm also concerned about dealing with the SPL size issues that are
>>>> likely to come up, but with two implementations these problems are
>>>> masked.
>>> Yes, but I think I've been overly optimistic in my mental map of what
>>> has/hasn't been converted.  While I hope we have what we need to make
>>> things work in SPL, or maybe only need to do some tweaking, I want a big
>>> enough window ahead that people aren't too stressed out when it turns
>>> out that we need, perhaps as came up in the SPI-nor series just now, a
>>> "tiny $something" subsystem and to work a bit more on the "tiny mmc"
>>> subsystem or what have you.  I do plan to make noise and deadlines about
>>> SPL but since "everyone else" got to worry about U-Boot first and SPL
>>> second, I don't want to add more stress to the folks just now seeing
>>> urgency here.
>>>
>>>> Can I suggest an earlier deadline of 2019.01 instead? That effectively
>>>> gives people until about March/April to send patches.
>>> The problem is that we cannot make BLK be declared done until these
>>> other subsystems are also marked done.  That was to me one of the big
>>> take-aways from the big series you did.  We have enough boards that
>>> aren't so much broken at the board level (we do have those) but are
>>> broken at the driver-needs-converting level.  And since it's really only
>>> "now" that everyone is seeing some of these, I don't think we can do
>>> things earlier than that.
>>>
>>> But that said, my meaning of deadline is that we drop things.  So a
>>> v2019.01 deadline means that second week of January we would be dropping
>>> things, if we pushed BLK up.  Since you said March/April there, I assume
>>> you're thinking that means a bit later on we drop things.  If I re-word
>>> things to be clearer that stuff will be dropped post v2019.07 will that
>>> be better?
>>
>> OK I see. Yes that seems reasonable.
>>
>> Reviewed-by: Simon Glass <sjg@chromium.org>
>>
>> It's been very quiet on this thread.
> That it's been so quiet is a bit worrying to me, yes.  I am going to fix
> the warning message that Chris pointed out and push this out for -rc1,
> which will I suppose be the next starting point for "We need more time!"
> requests.

I can't speak for everyone affected, but maybe I can give an example why 
it's so quiet:

At first I was shocked to see that all the socfpga boards should be 
removed. Then I read that Simon got something wrong and that it's 
unclear which boards were false positives. And now I'm confused and I'm 
waiting for you to decide on action to tell what's actually to be done.

I think we need a clear message of *what* needs to be upgraded. I do 
think it's hard to see that for developers that aren't involved everyday 
with U-Boot development.

I think a build warning would be best. But maybe this is already pushed 
and I'm not seeing it because socfpga was a false positive?

Regards,
Simon

  reply	other threads:[~2018-11-29 19:17 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-25 18:18 [U-Boot] [RFC for-v2019.01 1/4] dm: MIGRATION: Add migration plan for DM_MMC Tom Rini
2018-11-25 18:18 ` [U-Boot] [RFC for-v2019.01 2/4] dm: MIGRATION: Add migration plan for DM_USB Tom Rini
2018-11-25 18:18 ` [U-Boot] [RFC for-v2019.01 3/4] dm: MIGRATION: Add migration plan for CONFIG_SATA Tom Rini
2018-11-26  6:32   ` Chris Packham
2018-11-26 12:25     ` Tom Rini
2018-11-27  3:33       ` Chris Packham
2018-11-25 18:18 ` [U-Boot] [RFC for-v2019.01 4/4] dm: MIGRATION: Update migration plan for BLK Tom Rini
2018-11-29 17:10   ` Simon Glass
2018-11-29 18:34     ` Tom Rini
2018-11-29 18:43       ` Simon Glass
2018-11-29 19:06         ` Tom Rini
2018-11-29 19:17           ` Simon Goldschmidt [this message]
2018-11-29 19:19             ` Tom Rini
2018-11-26 15:09 ` [U-Boot] [RFC for-v2019.01 1/4] dm: MIGRATION: Add migration plan for DM_MMC Adam Ford
2018-11-26 15:29   ` 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=3b1cac94-3eeb-c0e1-5d7d-09070480e4d5@gmail.com \
    --to=simon.k.r.goldschmidt@gmail.com \
    --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.