lkml.org 
[lkml]   [2014]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 00/27] Fix common bug in most nand drivers not showing a device in sysfs
    Am 16.10.2014 08:37, schrieb Alexander Holler:
    > Hello,
    >
    > I wonder if anyone else will fix this in a maintainer-approved style
    > which doesn't use these evil leftovers from C666 called functions.
    >
    > Or will the sysfs for most NAND drivers be knowingly broken forever?


    To explain that a bit more:

    1. The reason I had to write 27 patches instead of just one is that such
    a function didn't exist. Therefor I've implemented one.

    2. I was kind to write 27 patches to fix that silly bug.

    3. I'm no fool and I will NOT write n * 21 patches in order to brutforce
    some maintainers prefered style.



    >
    > Regards,
    >
    > Alexander Holler
    >
    > Am 27.05.2014 00:12, schrieb Alexander Holler:
    >> A comment in mtdcore.c function add_mtd_device() which is called by
    >> mtd_device_parse_register() made me wonder:
    >>
    >> "Caller should have set dev.parent to match the physical device."
    >>
    >> In fact this is not done by most nand drivers.
    >>
    >> What follows is a series which fixes this.
    >>
    >> Tested: orion and omap2
    >> Compile-Tested: atmel, gpio, fsmc, gpmi, plat, pxa3xx, s3c2410, sh_flctl,
    >> sharpsl, tmio, docg4, davinci, lpc32xx_mlc, lpc32xx_slc, mxc
    >> Not tested at all (only be view, patches 19-27): bcm47, fsl_elbc,
    >> fsl_upm,
    >> fsl_ifc, jz4740, mpc5121m, ndfc, txx9ndfmx, socrates
    >>
    >> The overall stat is
    >>
    >> 27 files changed, 36 insertions(+), 80 deletions(-)
    >>
    >> and it fixes 21 of these bugs.
    >>
    >> Regards,
    >>
    >> Alexander Holler
    >>
    >> --
    >> To unsubscribe from this list: send the line "unsubscribe
    >> linux-kernel" in
    >> the body of a message to majordomo@vger.kernel.org
    >> More majordomo info at http://vger.kernel.org/majordomo-info.html
    >> Please read the FAQ at http://www.tux.org/lkml/
    >>
    >



    \
     
     \ /
      Last update: 2014-10-17 11:41    [W:2.798 / U:1.304 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site