public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Marek Behún" <kabel@kernel.org>
To: Tom Rini <trini@konsulko.com>
Cc: "Pali Rohár" <pali@kernel.org>, "Stefan Roese" <sr@denx.de>,
	"Tony Dinh" <mibodhi@gmail.com>,
	"Martin Rowe" <martin.p.rowe@gmail.com>,
	"U-Boot Mailing List" <u-boot@lists.denx.de>,
	"Peng Fan" <peng.fan@oss.nxp.com>,
	"Jaehoon Chung" <jh80.chung@samsung.com>
Subject: Re: mmc: Read eMMC partition access bits before card reset
Date: Mon, 12 Jun 2023 21:11:38 +0200	[thread overview]
Message-ID: <20230612211138.77cb340d@thinkpad> (raw)
In-Reply-To: <20230612181141.GP835503@bill-the-cat>

On Mon, 12 Jun 2023 14:11:41 -0400
Tom Rini <trini@konsulko.com> wrote:

> On Mon, Jun 12, 2023 at 07:39:20PM +0200, Pali Rohár wrote:
> > On Monday 12 June 2023 11:22:12 Stefan Roese wrote:  
> > > Hi Pali,
> > > 
> > > (added Peng for PowerPC and Jaehoon for MMC to Cc)
> > > 
> > > On 6/11/23 15:03, Pali Rohár wrote:
> > > 
> > > <snip>
> > >   
> > > > > Pali, I will try to pull these not applied patches - if there are not
> > > > > issues of course. Could you please send me a list of these patches so
> > > > > that I can take look? Please note that I will probably not be able to
> > > > > work on this this week. But next week should be possible.
> > > > > 
> > > > > Thanks,
> > > > > Stefan  
> > > > 
> > > > Hello!
> > > > 
> > > > These patches are on the list for year:
> > > > https://patchwork.ozlabs.org/project/uboot/patch/20220516090119.20217-1-pali@kernel.org/  
> > > 
> > > This patch fails with multiple problems. Reason is, that it's very "old"
> > > and many of the Kconfig stuff was changed, which is of course not your
> > > fault. Still, I'm not in the right position to make these changes I'm
> > > afraid.
> > > 
> > > Peng, this patch is now assigned to you in patchwork. Could you please
> > > also take a look? And if Pali (or someone else) sends an updated
> > > version, could you please make sure that it's reviewed (and applied if
> > > no issues arise) in due time?  
> > 
> > Updated version was already sent but again no response...
> > https://patchwork.ozlabs.org/project/uboot/patch/20220817205622.26644-1-pali@kernel.org/  
> 
> I was expecting Marek Behún to follow up and take this as he's the one
> that volunteered to take over one of the powerpc custodian trees and
> push this through.  I've replied now with more feedback.

Please also look at my reply at
  https://lists.denx.de/pipermail/u-boot/2023-June/520031.html

To recapitulate
- Pali worked on this over a year ago, he has done quite a lot of work
- it wasn't accepted at the time due to how some things changed in
  U-Boot - things regarding the conversion of CONFIG_ options, which
  Pali disagreed with since some of it was done in a wrong way. He
  wanted those changes to be reverted whole
- I promised Pali to work it out, once I had time to look into it. I
  expected to be able to do it in october / november. Unfortunately I
  wasn't able to, things have been quite hectic since then (I was
  working on my diploma thesis and afterwards when I returned to work,
  more pressing things took precedence)

Thus I quite understand why Pali is frustrated, since he has done the
work over a year ago. I am awfully sorry about this :-(

By no means I have forgotten about this. We at Turris still want to
work on this, since we are still interested in maintaining Turris 1.x.
We are currently in the process of hiring new kernel developers, and so
hopefully I will be able to work on this issue soon.

Marek

  reply	other threads:[~2023-06-12 19:11 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-03  9:18 Please pull u-boot-marvell/master Stefan Roese
2023-05-03 17:14 ` Tom Rini
2023-05-05 19:37   ` mmc: Read eMMC partition access bits before card reset Pali Rohár
2023-05-07 13:54     ` Tom Rini
2023-05-07 14:01       ` Pali Rohár
2023-05-07 14:40         ` Tom Rini
2023-05-07 14:56           ` Pali Rohár
2023-05-07 16:45             ` Tom Rini
2023-05-07 20:36               ` Pali Rohár
2023-05-16  7:04                 ` Pali Rohár
2023-05-16 15:36                   ` Tom Rini
2023-05-16 18:52                     ` Pali Rohár
2023-05-16 18:56                       ` Tom Rini
2023-05-16 22:30                         ` Pali Rohár
2023-05-17  8:26                           ` Stefan Roese
2023-05-18 20:55                             ` Tony Dinh
2023-05-25 12:57                               ` Stefan Roese
2023-06-11 13:03                                 ` Pali Rohár
2023-06-12  9:22                                   ` Stefan Roese
2023-06-12 17:39                                     ` Pali Rohár
2023-06-12 18:11                                       ` Tom Rini
2023-06-12 19:11                                         ` Marek Behún [this message]
2023-06-24  9:26                                         ` Pali Rohár
2023-06-24 16:57                                           ` 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=20230612211138.77cb340d@thinkpad \
    --to=kabel@kernel.org \
    --cc=jh80.chung@samsung.com \
    --cc=martin.p.rowe@gmail.com \
    --cc=mibodhi@gmail.com \
    --cc=pali@kernel.org \
    --cc=peng.fan@oss.nxp.com \
    --cc=sr@denx.de \
    --cc=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox