All of lore.kernel.org
 help / color / mirror / Atom feed
From: maramaopercheseimorto@gmail.com (Alberto Panizzo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] mx31pdk: Add NAND support
Date: Sat, 27 Feb 2010 17:17:12 +0100	[thread overview]
Message-ID: <1267287432.2723.90.camel@realization> (raw)
In-Reply-To: <952338.61061.qm@web51008.mail.re2.yahoo.com>

On sab, 2010-02-27 at 07:38 -0800, Fabio Estevam wrote:
> Hi Alberto,
> 
> --- On Sat, 2/27/10, Alberto Panizzo <maramaopercheseimorto@gmail.com> wrote:
> 
> > From: Alberto Panizzo <maramaopercheseimorto@gmail.com>
> > Subject: Re: [PATCH] mx31pdk: Add NAND support
> > To: "Fabio Estevam" <fabioestevam@yahoo.com>
> > Cc: linux-arm-kernel at lists.infradead.org, s.hauer at pengutronix.de
> > Date: Saturday, February 27, 2010, 12:25 PM
> > Dear Fabio,
> > 
> > On ven, 2010-02-26 at 17:34 -0800, Fabio Estevam wrote:
> > > Hi Sascha,
> > > 
> > ...
> > > 
> > > Do you think this patch is OK?
> > > 
> > > Regards,
> > > 
> > > Fabio Estevam
> > 
> > I do not know if Sascha own an i.MX31 PDK board and I 
> > do not know how Sascha could say if your patch is correct
> > or not
> > until WE (i.MX31 PDK owners) are in disagree on this code.
> > So _please_, be _kind_ and continue the debugging of this
> > code.
> 
> Ok, I just did what all the others i.MX boards did.
> 
> > (Why your bootloader correctly read the BBT and the driver
> > not?)
> > 
> > Another thing: I am a new graduated developer, I do this
> > work
> > keeping in mind my future and until now I am not paid to do
> > this.
> > What I gain is in experience and resume with my name in the
> > 
> > kernel Git log.
> 
> This is great. Please continue with this work.
> 
> > If you are thinking to push my entire work with your name
> > only
> > because I sent you my own mx31_3ds.c out of this mailing
> > list
> > you are in _wrong_.
> 
> This is not my intention. The mx31_3ds.c you sent me helped me to solve a different issue related to the LCD driver. The NAND patch I submitted in this list is not the same you had in your code as I did not pass flash_bbt.
> So your comment that I am pushing your entire work is not fair.
> 
> > Review of my work are welcome, BUT please, do not overcome
> > me
> > thinking that Sascha can merely arbitrate between us. 
> > 
> 
> Again, Alberto, I would love to work together with you and make i.MX support better.
> 
> Regards,
> 
> Fabio Estevam
> 

Ok Fabio, nevermind.. I trust the pure intentions of the kernel community
and yours. For one moment I've lost the compass (we say this in Italy, I don't
know if it is valid too here) and I apologize for unfairness (I've spent months
on this board! urk!)

Returning on the subject, I was not the first owner of the i.MX pdk board in my
possess, and since the freescale NAND driver uses a private OOB schema, others
could have soiled my nand chip overwriting the Bad block bits in the OOB.

The things are two: 
1- I have to clean my NAND chip. -> the bad block scanning could 
   get rid of true Bad blocks.
2- Why the mxc_nand driver do not read your bbt as the 
   bootloader do?
   What version of redboot are you using?
   I am Very curious on this disagreement..!

-- 
Alberto!

        Be Persistent!
                - Greg Kroah-Hartman (FOSDEM 2010)

  reply	other threads:[~2010-02-27 16:17 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-25  0:58 [PATCH] mx31pdk: Add NAND support Fabio Estevam
2010-02-25 10:58 ` Alberto Panizzo
2010-02-25 22:09   ` Fabio Estevam
2010-02-26  9:36     ` Uwe Kleine-König
2010-02-26 10:19       ` Alberto Panizzo
2010-02-26 10:46         ` Uwe Kleine-König
2010-02-26 11:05           ` Alberto Panizzo
2010-02-26 13:14             ` Ivo Clarysse
2010-02-25 11:12 ` Alberto Panizzo
2010-02-25 21:59   ` Fabio Estevam
2010-02-25 22:19     ` Alberto Panizzo
2010-02-26  9:56     ` Juergen Beisert
2010-02-26 10:33       ` Alberto Panizzo
2010-02-26 11:48         ` Juergen Beisert
2010-02-26 13:29           ` Alberto Panizzo
2010-02-26 13:40             ` Juergen Beisert
2010-02-27  1:34 ` Fabio Estevam
2010-02-27 14:25   ` Alberto Panizzo
2010-02-27 15:38     ` Fabio Estevam
2010-02-27 16:17       ` Alberto Panizzo [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-02-25 22:22 Fabio Estevam
2010-02-25 22:33 ` Alberto Panizzo
2010-02-25 22:52   ` Fabio Estevam

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=1267287432.2723.90.camel@realization \
    --to=maramaopercheseimorto@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.