Re: [PATCH V3 1/1] X86: Probe for PIC and set legacy_pic appropriately

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

 



On 04/14/2014 10:46 AM, K. Y. Srinivasan wrote:
> This patch implements the proposal put forth by H. Peter Anvin <hpa@xxxxxxxxxxxxxxx>.
> 
> In  version V1 of the patch, I had cleaned up the code based on comments from Peter.
> In  version V2 of the patch, I have addressed additional comments from Peter.
> In this version of the patch, I have addressed Jan's comments (JBeulich@xxxxxxxx).
> 
> Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>

Technically the patch looks good, but I'm going to have to reject it
based on the patch description.

Keep in mind that the above description is the only thing that would go
into the git repository, and needs to be understandable to someone, say,
10 years from now.

The above says nothing about what the proposal was, and the motivation,
and the problems it addresses.  So the patch is good, but please write a
proper, standalone description for it.

Thanks,

	-hpa


_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux