From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753398AbdKCUSd (ORCPT ); Fri, 3 Nov 2017 16:18:33 -0400 Received: from anholt.net ([50.246.234.109]:40618 "EHLO anholt.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752045AbdKCUSb (ORCPT ); Fri, 3 Nov 2017 16:18:31 -0400 From: Eric Anholt To: Florian Fainelli , linux-kernel@vger.kernel.org Cc: Florian Fainelli , Matt Mackall , Herbert Xu , Rob Herring , Mark Rutland , Ray Jui , Scott Branden , "maintainer\:BROADCOM BCM281XX\/BCM11XXX\/BCM216XX ARM ARCHITE..." , Stefan Wahren , PrasannaKumar Muralidharan , Russell King , Krzysztof Kozlowski , Harald Freudenberger , Sean Wang , Martin Kaiser , Steffen Trumtrar , "open list\:HARDWARE RANDOM NUMBER GENERATOR CORE" , "open list\:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list\:BROADCOM BCM2835 ARM ARCHITECTURE" , "moderated list\:BRO ADCOM BCM2835 ARM ARCHITECTURE" Subject: Re: [PATCH 12/12] hwrng: bcm63xx-rng: Remove since bcm2835-rng takes over In-Reply-To: <20171102010408.27736-13-f.fainelli@gmail.com> References: <20171102010408.27736-1-f.fainelli@gmail.com> <20171102010408.27736-13-f.fainelli@gmail.com> User-Agent: Notmuch/0.22.2+1~gb0bcfaa (http://notmuchmail.org) Emacs/25.2.2 (x86_64-pc-linux-gnu) Date: Fri, 03 Nov 2017 13:18:28 -0700 Message-ID: <877ev7ru97.fsf@anholt.net> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Florian Fainelli writes: > bcm2835-rng is now capable of supporting the BCM63xx hardware, so remove > the driver which duplicates the same functionality. > > Signed-off-by: Florian Fainelli > --- > drivers/char/hw_random/Kconfig | 13 --- > drivers/char/hw_random/Makefile | 1 - > drivers/char/hw_random/bcm63xx-rng.c | 154 ----------------------------------- > 3 files changed, 168 deletions(-) > delete mode 100644 drivers/char/hw_random/bcm63xx-rng.c > > diff --git a/drivers/char/hw_random/bcm63xx-rng.c b/drivers/char/hw_random/bcm63xx-rng.c > deleted file mode 100644 > index 5132c9cde50d..000000000000 > --- a/drivers/char/hw_random/bcm63xx-rng.c > +++ /dev/null > -static int bcm63xx_rng_data_present(struct hwrng *rng, int wait) > -{ > - struct bcm63xx_rng_priv *priv = to_rng_priv(rng); > - > - return __raw_readl(priv->regs + RNG_STAT) & RNG_AVAIL_MASK; > -} It looks like this method isn't in the 2835 implementation. Should it get ported over? --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAln8zxQACgkQtdYpNtH8 nugR4xAAucbKZ13rgHcRie1UB9tJNqmo2nmwD5qX3+pZJbvTJ0knzqPCg52fJ/kB ylJkghYlK4HiL6wQlcfxSVR01YYhj4Sddk+zPvIzgsFPaPHHDv1151iUwhB41Wwd DlQaQl1HhY+DLFCP7pYLQSMZ5ZJQ+yBf4+HX05jSEq9UnzAL1WRxRbNK45EOG+mP Pfsek8p+zpk8nPJ1NTbZbosiX/nXdm0cOID2eS2x7PxGWVf85wdnM8UA/WfzLv3q u0o5hUs458BG/Gkpb2kwhu67lx1P0OTwMJsiee//G7xcuwBMN/962zzerndF0lkO Wb6kRHG8d9ISI8OKDlfZg5BEtsHk6SBMxE3xfowIadLSKbSrXeWPBqnpch27u5z5 AlL4DcrpUxJfa9ZMr9btCUcOzSsgf2uu4lLFvMHm4T9awHqHRRpa2mjSIArnckRk qD9Mty6SgHtPswX7bVl81IwKM/YDJ29hMZH+qjDQLB16qqvYnlQrAM2Z92B6myNK J07D3mdE2fX/64F6DQbSaDGygmQ6n5jRH+O0ONQWXrW4dKJoKUpKybQ31dhkM/ye OqSki0lhrI+ShOs5vNz+i7fBXWajJ8tTz801iNOE8CGr8qoKD243UeNLESZwfRpG 9fpDWfaY9sO/amsSwdoAuZEGXN5T+DzPIBhpFNTbharaT96FZHA= =Z7XA -----END PGP SIGNATURE----- --=-=-=--