On Fri, Nov 09, 2018 at 05:45:16PM +0100, Wolfram Sang wrote:
> On Mon, Oct 29, 2018 at 02:42:37PM +0200, Andy Shevchenko wrote:
> > On Fri, Oct 26, 2018 at 09:36:57AM -0700, Ajay Gupta wrote:
> > > Hi Heikki and Wolfram,
> > >
> > > These two changes add support for USB Type-C interface on latest NVIDIA
> > > GPU card.
> > > The Type-C controller used is Cypress CCGx and is over I2C interface.
> > >
> > > I2C host controller has known limitation of sending STOP after every
> > > read. Since
> > > each read can be of 4 byte maximum length so there is a limit of 4 byte
> > > read.
> > > This is mentioned in adapter quirks as "max_read_len = 4"
> > >
> > > I2C host controller is mainly used for "write-then-read" or "write"
> > > messages so added
> > > the flag I2C_AQ_COMB_WRITE_THEN_READ in adapter quirks.
> > >
> > > PATCH[2/2] on ucsi driver now have added logic to check i2c adapter
> > > quirks and
> > > issues i2c read transfer based on max_read_len quirk settings. This will
> > > make sure
> > > the read limitation is not affecting I2C host which do not have such
> > > limitation.
> > >
> > > I think the patches should through usb tree because the main
> > > functionality is
> > > usb Type-C.
> >
> > FWIW,
> > Reviewed-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>
>
> Thanks, Andy. Your review is worth something :)
>
> May I ask you to add your tag to the individual patches (as long as it
> is not a patch bomb), so patchwork will pick them up and I see them
> immediately?
Hmm... It would be a bit difficult (requires to download patches again as mbox).
I would rather give you a hint how I resolve such problem when apply a tag to a
pile of patches.
Pre-requisites:
- git branch with all patches in question located at the top (let's assume 2
patches)
You may just run the following:
git filter-branch --msg-filter \
'sed -e "/Signed-off-by:/ a Reviewed-by: Andy Shevchenko
<andriy.shevche...@linux.intel.com>"' \
-f HEAD~2..HEAD
It will append a given line after first match with SoB one (`man sed`
for details). Check without -f (which enforces to rewrite) first.
--
With Best Regards,
Andy Shevchenko
