From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Hogan Subject: Re: [PATCH v2 2/2] dts: Probe efuse for CI20 Date: Wed, 17 Jan 2018 20:54:00 +0000 Message-ID: <20180117205359.GA27409@jhogan-linux.mipstec.com> References: <20171228212954.2922-1-malat@debian.org> <20171228212954.2922-3-malat@debian.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ONmcTkPhtkbQYSLg" Return-path: Content-Disposition: inline In-Reply-To: <20171228212954.2922-3-malat@debian.org> Sender: linux-kernel-owner@vger.kernel.org To: Mathieu Malaterre Cc: Marcin Nowakowski , Greg Kroah-Hartman , Zubair.Kakakhel@mips.com, Srinivas Kandagatla , Rob Herring , Mark Rutland , Ralf Baechle , PrasannaKumar Muralidharan , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-mips@linux-mips.org List-Id: devicetree@vger.kernel.org --ONmcTkPhtkbQYSLg Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 28, 2017 at 10:29:53PM +0100, Mathieu Malaterre wrote: > MIPS Creator CI20 comes with JZ4780 SoC. Provides access to the efuse blo= ck > using jz4780 efuse driver. >=20 > Signed-off-by: Mathieu Malaterre > --- > arch/mips/configs/ci20_defconfig | 2 ++ > 1 file changed, 2 insertions(+) >=20 > diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_de= fconfig > index b5f4ad8f2c45..62c63617e97a 100644 > --- a/arch/mips/configs/ci20_defconfig > +++ b/arch/mips/configs/ci20_defconfig > @@ -171,3 +171,5 @@ CONFIG_STACKTRACE=3Dy > # CONFIG_FTRACE is not set > CONFIG_CMDLINE_BOOL=3Dy > CONFIG_CMDLINE=3D"earlycon console=3DttyS4,115200 clk_ignore_unused" > +CONFIG_NVMEM=3Dy > +CONFIG_JZ4780_EFUSE=3Dy NVMEM is already implied by RTC_CLASS (which turns on RTC_NVMEM by default). I would suggest loading the defconfig: make ARCH=3Dmips ci20_defconfig Then enabling the extra configuration options you need, then create a new minimal defconfig: make ARCH=3Dmips savedefconfig Then look at the new file called "defconfig" or copy it over arch/mips/configs/ci20_defconfig, and see what changes it adds. That way you'll get the minimum you need and in the right order in the defconfig. Don't feel like you have to submit other random changes due to config changes since the defconfig was added, but if you do please do it as a separate patch to bring the defconfig up to date (i.e. just load defconfig and save it) before the patch which actually enables the EFUSE. Cheers James > --=20 > 2.11.0 >=20 >=20 --ONmcTkPhtkbQYSLg Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEd80NauSabkiESfLYbAtpk944dnoFAlpft+cACgkQbAtpk944 dnrk3hAApfLDG//NIiWtvtmyTvMW0EFfxCpWDTA+zdGgPhLlM+qC58BhuBt+opqq iuGc4fxu8X4HgHyNhNlxj5A9Y+6fEPVfBHzaSVj5Bgxs/pl9SKhdT1+IZOJ3GJZ/ F2zG7OvO0IePC+4ssYQmyOvWsJwQWT50ZtHf3xrPIgnzwj6NCNRtTLZ2cfuOmszY dwKhfNILgX2Z3pgX2KbAo1/C5nG6vAmRA8lDCw/BhOILsbcvjDGArXMAtC9aGC9T zRykc1ABefERM7hSNZrbsAK1CZclmb2KaD8wdLjvk5Fgbl7T9X1X1gsyO9PDtL3f FNMao9SWEpS05ur69tsL+ARjv0XU5s+ma1a2IOqGo8JrQZBjiqzpOnt6tJHoD/eu UqOns4infil4g7OfawZ+8yYw6rutmDo6xUgYe0gaRhcu8LyZMxtjVXisGQHj74w8 Jt35NVuFzJD4E8friIwKFaU0QB5OJEhXdxkGLy725ZZjq1WdxFMhymBupnO6hU3K 8f358WvR0KyjXkYs5XZFQJPcwglv8hcjpYQVv3glIkTFkDhuRy8Svrq9LcxHZd4w eV3f52R45yJ5iOfb/1BOCVcy12ZhL8qkh9ud6d932zhqkQ4LOL8+2cE+615MWQb8 R2uJfeb9hebqH69kAHbbvyKRoGxII5IHaZJGOdxrdm+qm/vl3As= =XoTT -----END PGP SIGNATURE----- --ONmcTkPhtkbQYSLg--