Re: [PATCH v15 0/2] Add support for USB Type-C interface on latest NVIDIA GPU

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.shevchenko@xxxxxxxxxxxxxxx>
> 
> 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.shevchenko@xxxxxxxxxxxxxxx>"' \
	-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





[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux