RE: [PATCH v2 4/4] platform:x86: Makefile/Kconfig/MAINTAINERS changes for Intel Telemetry

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

 



Hi Darren,

I am a bit confused as checkpatch that I pulled from Linus' tree gave me all clear.
Can you please send me the log, because I have a clean checkpatch report at my end.

BR,
Souvik

skchakra@intel-desktop:~/disk2/linux-platform-drivers-x86/linux$ ./scripts/checkpatch.pl 0004-platform-x86-Makefile-Kconfig-MAINTAINERS-changes-fo.patch
total: 0 errors, 0 warnings, 36 lines checked

0004-platform-x86-Makefile-Kconfig-MAINTAINERS-changes-fo.patch has no obvious style problems and is ready for submission.
skchakra@intel-desktop:~/disk2/linux-platform-drivers-x86/linux$ ./scripts/checkpatch.pl 0003-platform-x86-Add-Intel-Telemetry-Debugfs-interfaces.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#29:
new file mode 100644

total: 0 errors, 1 warnings, 1094 lines checked

0003-platform-x86-Add-Intel-Telemetry-Debugfs-interfaces.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
skchakra@intel-desktop:~/disk2/linux-platform-drivers-x86/linux$ ./scripts/checkpatch.pl 0002-platform-x86-Add-Intel-telemetry-platform-device-dri.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#195:
new file mode 100644

total: 0 errors, 1 warnings, 1371 lines checked

0002-platform-x86-Add-Intel-telemetry-platform-device-dri.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
skchakra@intel-desktop:~/disk2/linux-platform-drivers-x86/linux$ ./scripts/checkpatch.pl 0001-platform-x86-Add-Intel-Telemetry-Core-Driver.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#34:
new file mode 100644

total: 0 errors, 1 warnings, 611 lines checked

0001-platform-x86-Add-Intel-Telemetry-Core-Driver.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

> -----Original Message-----
> From: platform-driver-x86-owner@xxxxxxxxxxxxxxx [mailto:platform-driver-x86-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Darren Hart
> Sent: Thursday, October 8, 2015 10:22 PM
> To: Chakravarty, Souvik K <souvik.k.chakravarty@xxxxxxxxx>
> Cc: platform-driver-x86@xxxxxxxxxxxxxxx; Kasagar, Srinidhi
> <srinidhi.kasagar@xxxxxxxxx>; Zha, Qipeng <qipeng.zha@xxxxxxxxx>
> Subject: Re: [PATCH v2 4/4] platform:x86: Makefile/Kconfig/MAINTAINERS
> changes for Intel Telemetry
> 
> On Tue, Sep 22, 2015 at 02:31:40PM +0530, Souvik Kumar Chakravarty wrote:
> > Makefile, Kconfig & MAINTAINERS changes for compiling Telemetry.
> > It depends on PUNIT and PMC IPC drivers.
> >
> > Signed-off-by: Souvik Kumar Chakravarty
> > <souvik.k.chakravarty@xxxxxxxxx>
> 
> Souvik, v2 seems to be the latest I have from you and checkpatch still reports
> several basic coding style errors about comment blocks and comparison
> constant location.
> 
> See my response to v1 regarding that on Sep 18.
> 
> --
> Darren Hart
> Intel Open Source Technology Center
> --
> To unsubscribe from this list: send the line "unsubscribe platform-driver-x86"
> in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo
> info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux