linux-kernel - Re: [PATCHv4 0/4] staging: adding OMAP bandgap driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 18 Jul 2012 19:24:37 +0300
From:	"Valentin, Eduardo" <eduardo.valentin@...com>
To:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:	devel@...verdev.osuosl.org, b-cousson@...com, j-keerthy@...com,
	linux-kernel@...r.kernel.org, kbaidarov@....rtsoft.ru,
	grant.likely@...retlab.ca
Subject: Re: [PATCHv4 0/4] staging: adding OMAP bandgap driver

Hello Greg,

On Tue, Jul 17, 2012 at 5:05 AM, Greg Kroah-Hartman
<gregkh@...uxfoundation.org> wrote:
> On Thu, Jul 12, 2012 at 07:02:28PM +0300, Eduardo Valentin wrote:
>> Greg,
>>
>> Here is again the OMAP BG driver, under staging area.
>> I fixed the compilation issue I didn't see, wrt implicit function
>> declarations. As I mentioned on other thread, my compilation test
>> didn't see it, that's why I thought it was fine to send it.
>>
>> I also limited the driver compilation to OMAP4/5.
>
> I edited the Makefile part of the patch by hand, and applied these
> patches, can you pull my tree and verify that I didn't break anything?

I checked the Makefile change, it is fine for me. Thanks for fixing it.

I also pulled your staging-next branch and verified that the code is
working fine. And it is OK.

>
> thanks,

Thanks to you for accepting it. We will be sending patches to you on
this driver from now on, in order to improve it's level of quality.

All best,

>
> greg k-h



-- 

Eduardo Valentin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/