linux-kernel - Re: [PATCH v2 0/6] Provides bug fixes for rtc-mcp795.c

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:   Thu, 8 Dec 2016 01:34:11 +0100
From:   Alexandre Belloni <alexandre.belloni@...e-electrons.com>
To:     Emil Bartczak <emilbart@...il.com>
Cc:     a.zummo@...ertech.it, rtc-linux@...glegroups.com,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/6] Provides bug fixes for rtc-mcp795.c

On 08/12/2016 at 00:27:36 +0100, Emil Bartczak wrote :
> This patchset provides 4 bug fixes (patch 1, 2, 3, 4), one 
> improvement (patch 5) and whitespace, indention errors fixes 
> (patch 6) in the file driver/rtc/rtc-mcp795.c.
> Please review the changes and consider to apply them to the 
> main kernel tree.
> 
> Changes from v1:
> - Introduced SoB to all patches.
> - Patch which switches to bcd2bin/bin2bcd became as first now, 
>   it also solved the shift problem presented in the previous patch 1.
> - Added restoring EXTOSC bit to its previous value.
> - All whitespace and indention fixes moved to the new patch 6.
> - Added new patch which replaces all bitmask values with the 
>   BIT(x) macro.
> 
> Emil Bartczak (6):
>   rtc: mcp795: use bcd2bin/bin2bcd.
>   rtc: mcp795: fix bitmask value for leap year (LP).
>   rtc: mcp795: fix time range difference between linux and RTC chip.
>   rtc: mcp795: fix month write resetting date to 1.
>   rtc: mcp795: Prefer using the BIT() macro.
>   rtc: mcp795: Fix whitespace and indentation.
> 
>  drivers/rtc/rtc-mcp795.c | 122 ++++++++++++++++++++++++++++++++++++++---------
>  1 file changed, 100 insertions(+), 22 deletions(-)
> 

All applied, thanks!

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com