RE: [PATCH 0/5 v2] qedr: batch of fixes for 4.10

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

 



> I took these, but I would like to point out a couple things.  First,
> the commit messages could use some work.  I fixed a number of them up.
>  Also, a fix to formatting isn't really an -rc fix.  I took it, but
> next time let's be more selective on our submission please.
> 

Thanks.
I have reviewed your changes for next time. I understood all but these:

I thought that introducing formatting fixes and making functions static are
simplifying and risk free operations that are acceptable. I now understand,
and correct me if I'm wrong, that only if it's important fix it should be introduced.

Sometimes I see IB/vendor and sometimes RDMA/vendor. When to use each?

Thanks,
Ram

��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux