Re: [PATCH v1 01/13] iio: Fix typo in vendor name
From: Andy Shevchenko
Date: Sat Sep 05 2026 - 03:41:40 EST
On Sat, Sep 05, 2026 at 10:40:13AM +0300, Andy Shevchenko wrote:
> On Fri, Sep 04, 2026 at 04:41:22PM -0300, Marcelo Schmitt wrote:
> > On 09/03, Andy Shevchenko wrote:
> > > On Wed, Sep 02, 2026 at 02:21:34PM -0300, Marcelo Schmitt wrote:
...
> > > > Now
> > > > find drivers/iio/ -type f | xargs grep -i "Analog Device "
> > > > finds no leftover occurrence of such typo.
> > > >
> > > > Also,
> > > > grep -nrIi drivers/iio/ -e "Analog Device" | wc -l
> > > > and
> > > > grep -nrIi drivers/iio/ -e "Analog Devices" | wc -l
> > > > both count the same number of lines.
> > >
> > > Why not `git grep ...`?
> > >
> > Could be. I'm just more used to non-git grep.
>
> The rationale here is that `grep` goes over filesystem (and files) while
> `git grep` goes over index which requires much less resources to waste and
> hence 10x faster or so). That said, I highly recommend using `git grep`
> when it's applicable.
One more important thing, it skips ignored or temporary files, while `grep`
goes over all of them.
...
> > Thanks for suggestions to other patches too. Will apply them for v2.
>
> You're welcome!
--
With Best Regards,
Andy Shevchenko