On Sat, Sep 09, 2017 at 08:30:44AM +0800, Bin Meng wrote:
> Hi Tom,
>
> On Wed, Sep 6, 2017 at 3:30 PM, Bin Meng <bmeng...@gmail.com> wrote:
> > This converts the IDE driver to driver model so that block read and
> > write are fully functional.
> >
> > Fixes: b7c6baef ("x86: Convert MMC to driver model")
> > Reported-by: Heinrich Schuchardt <xypron.g...@gmx.de>
> > Signed-off-by: Bin Meng <bmeng...@gmail.com>
> >
> > Reviewed-by: Simon Glass <s...@chromium.org>
> > Tested-by: Heinrich Schuchardt <xypron.g...@gmx.de>
> > ---
> >
> > Changes in v3:
> > - Updated BLK_XXX_SIZE usage
> >
> > Changes in v2:
> > - Fixed 'fatls ide 0' issue
> >
> > drivers/block/blk-uclass.c | 2 +-
> > drivers/block/ide.c | 70
> > ++++++++++++++++++++++++++++++++++++++++++++++
> > include/dm/uclass-id.h | 1 +
> > 3 files changed, 72 insertions(+), 1 deletion(-)
> >
>
> Will you take this series in 2017.09?I would swear that we declare vendor/etc as size + 1 so that we can ensure that size+1==NULL and only write up to size into the field. But the rest of this patch strncpy's to size+1. That feels wrong, but I'm not totally sure. Are you? Thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
