linux-kernel - Re: [PATCH v5 0/2] Introducing Exynos ChipId 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 for Android: free password hash cracker in your pocket

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

Message-ID: <CAGcde9HN7AWge=b10OJ5Xn71GJUOJoxs8DJpoxWG=Jqp7yCg3g@mail.gmail.com>
Date:	Thu, 19 May 2016 19:02:38 +0530
From:	Pankaj Dubey <pankaj.dubey@...sung.com>
To:	Krzysztof Kozlowski <k.kozlowski@...sung.com>
Cc:	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	linux-samsung-soc <linux-samsung-soc@...r.kernel.org>,
	linux-kernel@...r.kernel.org, Kukjin Kim <kgene.kim@...sung.com>,
	Russell King <linux@....linux.org.uk>, heiko@...ech.de,
	Arnd Bergmann <arnd@...db.de>,
	Tomasz Figa <tomasz.figa@...il.com>,
	"thomas.ab@...sung.com" <thomas.ab@...sung.com>,
	Marek Szyprowski <m.szyprowski@...sung.com>,
	Bartłomiej Żołnierkiewicz 
	<b.zolnierkie@...sung.com>
Subject: Re: [PATCH v5 0/2] Introducing Exynos ChipId driver

Hi Krzysztof,

On 16 May 2016 at 18:52, Krzysztof Kozlowski <k.kozlowski@...sung.com> wrote:
>
> On Thu, Dec 11, 2014 at 9:07 AM, Pankaj Dubey <pankaj.dubey@...sung.com> wrote:
> > This patch series introduces Exynos Chipid platform driver.
> > Each Exynos SoC has ChipID block which can give information about SoC's
> > product Id and revision number.
> > At the same time it reduces dependency of mach-exynos files from plat-samsung,
> > by removing samsung_rev API, similar API is introduced in chipid driver itself
> > to get revision number and product id.
> >
> > This patch series is based on Kukjin Kim's for-next having SHA_ID:
> > 9db7d78aca2c7fbc19a26ce2ef01c805dc010c72
> >
> > It can also be cleanly applied on latest (next-20141209) linux-next.
> >
> > This has been tested against both tree on Exynos3250 and Exynos5250 (SMDK) board.
> >
> > Revision 4 and it's discussion can be found here
> >  - https://lkml.org/lkml/2014/12/3/115
> >
> > Changes since v4:
> >  - Removed custom sysfs entries as they were not providing any new information
> >    as pointed out by Arnd.
> >  - Removed functions exporting product_id and revision, instead we will export
> >    exynos_chipid_info structure. It will be helpfull when we need to provide more
> >    fields of chipid outside of chipid, as commented by Yadwinder
> >  - Converted all funcions as __init.
> >
> > Change since v3:
> >  - This patch set contains 5/6 and 6/6 patch from v3 series.
> >  - Made EXYNOS_CHIPID config option non-user selectable,
> >    as suggested by Tomasz Figa.
> >  - Made uniform macro for EXYNOS4/5_SOC_MASK as EXYNOS_SOC_MASK as
> >    suggested by Tomasz Figa.
> >  - Made local variables static in chipid driver.
> >  - Added existing SoC's product id's.
> >  - Added platform driver support.
> >
> > Changes since v2:
> >  - Reorganized patches as suggested by Tomasz Figa.
> >  - Addressed review comments of Tomasz Figa in i2c-s3c2410.c file.
> >
> > Changes since v1:
> >  - Added patch to move i2c interrupt re-configuration code from exynos.c
> >    to i2c driver, as suggested by Arnd.
> >  - After above patch only user of SYS_I2C_CFG register is pm.c so moving
> >    save/restore of this register also into i2c driver.
> >  - Spiltted up exynos4 and exynos5 machine descriptors to get rid from
> >    soc_is_exynos4/exynos5 kind of macros, as suggested by Arnd.
> >  - Changed location of chipid driver to "drivers/soc".
> >  - Added drivers/base/soc.c provided infrastructure to make SoC specific
> >    information avaible to user space via sysfs entry, as suggested by Arnd.
> >
> >
> > Pankaj Dubey (2):
> >   soc: samsung: add exynos chipid driver support
> >   ARM: EXYNOS: refactoring of mach-exynos to enable chipid driver
>
> Hi,
>
> This is quite old patchset and apparently was not merged. I quickly
> looked at previous discussions (for previous versions) and it seems
> all review feedback was addressed.
>
> Any other obstacles?
>
> Pankaj, maybe you could respin this (after merge window)?

Thanks for looking into this.
Infact I also wanted to respin this patchset. Will do it soon.

Thanks,
Pankaj Dubey
>
> Best regards,
> Krzysztof
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html