Re: [PATCH] ata: Clean up hard coded array size calculation.

[Jeff Garzik]

On 11/20/2009 02:15 PM, Thiago Farina wrote:
> Hi Jeff,
>
> On Tue, Nov 17, 2009 at 1:17 AM, Jeff Garzik<jgarzik@xxxxxxxxx>  wrote:
> > On 11/08/2009 02:30 PM, Thiago Farina wrote:
> > > Use ARRAY_SIZE macro of kernel api instead.
> > >
> > > Signed-off-by: Thiago Farina<tfransosi@xxxxxxxxx>
> > > ---
> > >   drivers/ata/sata_mv.c |    2 +-
> > >   1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/drivers/ata/sata_mv.c b/drivers/ata/sata_mv.c
> > > index 6f5093b..a8a7be0 100644
> > > --- a/drivers/ata/sata_mv.c
> > > +++ b/drivers/ata/sata_mv.c
> > > @@ -2217,7 +2217,7 @@ static unsigned int mv_qc_issue_fis(struct
> > > ata_queued_cmd *qc)
> > >         int err = 0;
> > >
> > >         ata_tf_to_fis(&qc->tf, link->pmp, 1, (void *)fis);
> > > -       err = mv_send_fis(ap, fis, sizeof(fis) / sizeof(fis[0]));
> > > +       err = mv_send_fis(ap, fis, ARRAY_SIZE(fis));
> > >         if (err)
> >
> > applied
>
> Was it applied to this tree
> http://git.kernel.org/?p=linux/kernel/git/jgarzik/libata-dev.git?

Yes.

	Jeff




--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/