Hi,
> Snapshot gcc-11-20210425 is now available on
> https://gcc.gnu.org/pub/gcc/snapshots/11-20210425/
> and on various mirrors, see http://gcc.gnu.org/mirrors.html for details.
>
> This snapshot has been generated from the GCC 11 git branch
> with the following options: git://gcc.gnu.org/git/g
> Le 13/06/2020 à 18:28, Arseny Solokha a écrit :
>> Building the current 5.8 kernel for a e500 machine with
>> CONFIG_RANDOMIZE_BASE set yields the following failure:
>>
>>arch/powerpc/mm/nohash/kaslr_booke.c: In function 'kaslr_early_init':
>>
> Le 13/06/2020 à 18:28, Arseny Solokha a écrit :
>> Building the current 5.8 kernel for a e500 machine with
>> CONFIG_RANDOMIZE_BASE set yields the following failure:
>>
>>arch/powerpc/mm/nohash/kaslr_booke.c: In function 'kaslr_early_init':
>>
h CONFIG_RANDOMIZE_BASE
set.
Fixes: 2b0e86cc5de6 ("powerpc/fsl_booke/32: implement KASLR infrastructure")
Cc: sta...@vger.kernel.org
Signed-off-by: Arseny Solokha
---
arch/powerpc/mm/nohash/kaslr_booke.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/powerpc/mm/nohash/kaslr_bo
h CONFIG_RANDOMIZE_BASE
set.
Fixes: 2b0e86cc5de6 ("powerpc/fsl_booke/32: implement KASLR infrastructure")
Cc: sta...@vger.kernel.org
Signed-off-by: Arseny Solokha
---
arch/powerpc/mm/nohash/kaslr_booke.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/powerpc/mm/nohash/kaslr_bo
> Hi,
>
>
> PRs from the second group were filed by me, so if there's consensus to
> close all
> of them, the ones from this second group I can close myself. I don't have
> the
> right permissions to modify PRs reported by someone else, so I'd like to
> ask a
> volunt
Hi,
>> >> PRs from the second group were filed by me, so if there's consensus to
>> >> close all
>> >> of them, the ones from this second group I can close myself. I don't have
>> >> the
>> >> right permissions to modify PRs reported by someone else, so I'd like to
>> >> ask a
>> >> volunteer
Hi,
> When fixing up the template specialization hasher I was confused by the
> control flow through template_args_equal. This reorders the category
> checking, so it is clearer as to what kind of node can reach which point.
>
> nathan
>
> 2020-05-13 Nathan Sidwell
>
> * pt.c (template_
> Otherwise, I'm proposing to finally close all open PRs filed against
> powerpcspe. I've been able to identify the following ones:
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19490
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=30259
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37759
> http
> Hi!
>> But it is clearly obvious that the powerpcspe backend won't be revived. After
>> two years of development, rs6000 has diverged too much from the split point,
>> including LRA adoption.
>
> LRA has been supported by the rs6000 port since 2013 (01b1efaa1439),
> and made the default (and on
Hi,
over the course of two years that had passed since the deprecation of the
powerpcspe backend, and a year and a half since its removal from gcc, I've still
been speaking out several times against immediate closing of Bugzilla PRs
against that target[1,2]. IIRC, Andrew has been contemplating a r
Hi,
>> https://gcc.gnu.org/pipermail/gcc/2020-February/232205.html
>> Looking around, the last two months of gcc now have very small
>> numbers, but e.g. on gcc-patches the mails have very high numbers like
>> 545238.html. Can pipermail provide stable URLs at all? We really
>> need those, we re
Hi,
>> https://gcc.gnu.org/pipermail/gcc/2020-February/232205.html
>> Looking around, the last two months of gcc now have very small
>> numbers, but e.g. on gcc-patches the mails have very high numbers like
>> 545238.html. Can pipermail provide stable URLs at all? We really
>> need those, we re
Hi,
> The first release candidate for GCC 10.1 is available from
>
> https://gcc.gnu.org/pub/gcc/snapshots/10.1.0-RC-20200430/
> ftp://gcc.gnu.org/pub/gcc/snapshots/10.1.0-RC-20200430
>
> and shortly its mirrors. It has been generated from git revision
> r10-8080-g591d857164c37cd0bb96da2a29314
> On Fri, Mar 27, 2020 at 07:00:59AM +0100, Bernd Edlinger wrote:
>>PS: I would CC you, Christopher Faylor, but your email address is
>>"cgf-use-the-mailinglist-ple...@gnu.org", so whatever I send there
>>would not reach you.
>
> Well duh? Not being cc'ed is the literal point of the email address.
> On 3/25/20 8:58 AM, Bernd Edlinger wrote:
>> On 3/25/20 8:32 AM, Jonathan Wakely wrote:
>>> On Wed, 25 Mar 2020 at 04:48, Bernd Edlinger wrote:
Hi,
I do not want to start a flame war.
I just am curious what was the reason why
the old system cannot be used any mo
.
Signed-off-by: Arseny Solokha
---
drivers/net/ethernet/freescale/gianfar.c | 19 +--
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/net/ethernet/freescale/gianfar.c
b/drivers/net/ethernet/freescale/gianfar.c
index 17fb412e4bb4..24bf7f68375f 100644
--- a
> Hi Arseny.
>
> On Tue, 30 Jul 2019 at 17:40, Arseny Solokha wrote:
>>
>> > Hi Arseny,
>> >
>> > Nice project!
>>
>> Vladimir, Russell, thanks for your review. I'm on vacation now, so won't
>> fully
>> address your com
change any
functionality.
Signed-off-by: Arseny Solokha
---
drivers/net/ethernet/freescale/gianfar.c | 2849 -
drivers/net/ethernet/freescale/gianfar.h |7 -
.../net/ethernet/freescale/gianfar_ethtool.c | 13 -
3 files changed, 1404 insertions(+), 1465 deletions
Remove now unused macro and structure definitions from gianfar.h that have
accumulated there over time.
Signed-off-by: Arseny Solokha
---
drivers/net/ethernet/freescale/gianfar.h | 38
1 file changed, 38 deletions(-)
diff --git a/drivers/net/ethernet/freescale
Hi,
>> @@ -1964,7 +1966,7 @@ void stop_gfar(struct net_device *dev)
>> /* disable ints and gracefully shut down Rx/Tx DMA */
>> gfar_halt(priv);
>>
>> - phy_stop(dev->phydev);
>> + phylink_stop(priv->phylink);
>>
>> free_skb_resources(priv);
>> }
>> @@ -2219,1
call instead of open-coding the lookup
half-way correctly.
[1]
https://lore.kernel.org/netdev/ca+h21hruqt6ngg5ksdswrgh_w5gtgf4fjamcwjne7qjrjus...@mail.gmail.com/
Arseny Solokha (4):
gianfar: remove forward declarations
gianfar: make five functions static
gianfar: cleanup gianfar.h
gi
Make functions that do not have callers outside the translation unit they
are defined in static.
Signed-off-by: Arseny Solokha
---
drivers/net/ethernet/freescale/gianfar.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/freescale/gianfar.c
b
> Hi Arseny,
>
> Nice project!
Vladimir, Russell, thanks for your review. I'm on vacation now, so won't fully
address your comments in a few weeks: while I can build the code, I won't have
access to hardware to test.
So it seems this patch will turn into a series where we'll have some cleanup
pat
are not affected.
Place explicit calls to phy_start() and phy_stop() before enabling or after
disabling an attached SFP module, where phydev is not yet set (or is
already unset), so they will be made only from the inside of sfp-bus, if
needed.
Signed-off-by: Arseny Solokha
---
Changes i
>>-Original Message-
>>From: Arseny Solokha
>>Sent: Tuesday, July 23, 2019 6:17 PM
>>To: Claudiu Manoil ; Ioana Ciornei
>>; Russell King ; Andrew Lunn
>>
>>Cc: netdev@vger.kernel.org; Arseny Solokha
>>Subject: [RFC PATCH 1/2] gianfar: co
Hi,
may I also ask to provide ppc_clone3 symbol also for 32-bit powerpc? Otherwise
Michael's patch breaks build for me:
powerpc-e500v2-linux-gnuspe-ld: arch/powerpc/kernel/systbl.o: in function
`sys_call_table':
(.rodata+0x6cc): undefined reference to `ppc_clone3'
make: *** [Makefile:1060:
Hi,
may I also ask to provide ppc_clone3 symbol also for 32-bit powerpc? Otherwise
Michael's patch breaks build for me:
powerpc-e500v2-linux-gnuspe-ld: arch/powerpc/kernel/systbl.o: in function
`sys_call_table':
(.rodata+0x6cc): undefined reference to `ppc_clone3'
make: *** [Makefile:1060:
s = <&i2c1>;
+ mod-def0-gpios = <&gpio 4 GPIO_ACTIVE_LOW>;
+ rate-select0-gpios = <&gpio 13 GPIO_ACTIVE_HIGH>;
+ tx-disable-gpios = <&gpio 5 GPIO_ACTIVE_LOW>;
+ tx-fault-gpios = <&gpio 12 GPIO_ACTIVE
4c095aeaf6ea999 ]---
SFP modules with the 1000Base-X interface are not affected.
So, skip explicit calls to phy_start() and phy_stop() when phylink has just
enabled or disabled an attached SFP module.
Signed-off-by: Arseny Solokha
---
drivers/net/phy/phylink.c | 6 +++---
1 file changed, 3 in
start() or phy_stop()
twice in a row for such modules. It doesn't look like a proper fix to me,
though, thus the second reason for RFC.
Arseny Solokha (2):
gianfar: convert to phylink
net: phylink: don't start and stop SGMII PHYs in SFP modules twice
drivers/net/ethernet/freescale/
load again:
at24 0-0050: 32768 byte 24c256 EEPROM, writable, 64 bytes/write
Cc: Adrian Bunk
Cc: Bartosz Golaszewski
Cc: Srinivas Kandagatla
Cc: sta...@vger.kernel.org # v5.2+
Signed-off-by: Arseny Solokha
---
drivers/misc/eeprom/Kconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/dr
> First problem:
>
> The g++.dg/pr85039-2.C tests (I've looked in detail at -std=c++98, but
> -std=c++11 and -std=c++14 appear to follow the same pattern) see gcc
> garbage-collecting a live vector. A subsequent access to the vector with
> vec_quick_push causes a segmentation fault, as m_vecpfx.m
> Hi Arseny,
>
> On Fri, Nov 23, 2018 at 06:15:47PM +0700, Arseny Solokha wrote:
>> I've found recently that rs6000 and powerpcspe backends can easily trip over
>> various gcc_unreachable()'s and gcc_assert()'s in their respective copies of
>> print
Hi,
I've found recently that rs6000 and powerpcspe backends can easily trip over
various gcc_unreachable()'s and gcc_assert()'s in their respective copies of
print_operand() when provided with some invalid assembly (i.e. assembly written
for other architectures). For example, when feeding
gcc/te
> I was doing some benchmarking with SPEC 2017 fprate on aarch64
> (Thunderx2) and I am getting some segfaults from GCC while compiling.
>
> I am working with delta to try and cut down one of the test cases
> but I was wondering if anyone else has seen this problem. The
> three tests that segfault
.
Changes from v2:
- left Device Tree compatibles in place
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 37 +++--
1 file changed, 15 insertions(+), 22 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index
>> /*
>> * Map and check POR Device Status Register 2
>> - * (PORDEVSR2) at 0xE0014
>> + * (PORDEVSR2) at 0xE0014. Note than while MPC8533
>> + * and MPC8544 indicate SEC frequency ratio
>> +
.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 52 +---
1 file changed, 15 insertions(+), 37 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index aac0ec6dc5fc..ad9af3ca35aa 100644
--- a/drivers/i2c
those SoCs.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index 648a5afded64..aac0ec6dc5fc 100644
--- a/drivers/i2c/busses/i2c-mpc.c
+++ b
caller, resulting in a division by zero
for the majority of processors supported by the module.
Avoid division by zero by obtaining the actual I2C clock prescaler
in mpc_i2c_setup_8xxx() unconditionally regardless of the passed clock
value.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c
44 in this regard, according to AN2919 "Determining the I2C
Frequency Divider Ratio for SCL".
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/bu
urrently covers more SoCs. Hardcoding it
is also wrong for some SoCs as it can be configured on board during POR.
This series is an exact copy of [1] which has received no feedback so far.
[1] https://marc.info/?l=linux-i2c&m=151030076114573&w=2
Arseny Solokha (4):
i2c: mpc: get MPC8
urrently covers more SoCs. Hardcoding it
is also wrong for some SoCs as it can be configured on board during POR.
Arseny Solokha (4):
i2c: mpc: get MPC8xxx I2C clock prescaler before using it in
calculations
i2c: mpc: unify obtaining the MPC8533/44 I2C clock prescaler w/
MPC8xxx
those SoCs.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index f47916466b82..8d60db0080f6 100644
--- a/drivers/i2c/busses/i2c-mpc.c
+++ b
44 in this regard, according to AN2919 "Determining the I2C
Frequency Divider Ratio for SCL".
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/bu
.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c-mpc.c | 52 +---
1 file changed, 15 insertions(+), 37 deletions(-)
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index 8d60db0080f6..e0f059687c2d 100644
--- a/drivers/i2c
caller, resulting in a division by zero
for the majority of processors supported by the module.
Avoid division by zero by obtaining the actual I2C clock prescaler
in mpc_i2c_setup_8xxx() unconditionally regardless of the passed clock
value.
Signed-off-by: Arseny Solokha
---
drivers/i2c/busses/i2c
> On 02/09/2017 11:06 AM, Jakub Jelinek wrote:
>> On Thu, Feb 09, 2017 at 01:47:33PM -0500, David Edelsohn wrote:
>>> Freescale did not implement the POWER architecture. Again, POWER is a
>>> comment about the original IBM POWER architecture (RIOS processors)
>>> and used in RISC System/6000 compu
From: Arseny Solokha
In spite of switching to paged allocation of Rx buffers, the driver still
called dma_unmap_single() in the Rx queues tear-down path.
The DMA region unmapping code in free_skb_rx_queue() basically predates
the introduction of paged allocation to the driver. While being
From: Arseny Solokha
In spite of switching to paged allocation of Rx buffers, the driver still
called dma_unmap_single() in the Rx queues tear-down path.
The DMA region unmapping code in free_skb_rx_queue() basically predates
the introduction of paged allocation to the driver. While being
Hi,
creduce 2.5.0 (18e04be) on x86_64 reports an error (contained in subject)
in pass_clex::rm-tok-pattern-4 w/ the attached input.
Thanks,
Arseny
#include
uint16_t us_0;
uint8_t uc_1;
int16_t s_2;
int8_t c_3;
uint16_t us_4;
func_0() {
int8_t c_5;
if (s_2) {
for (; c_5; us_0 -= uc_1 ? us
Hello,
creduce 2.5.0 (18e04be) on x86_64 fails in pass_clang::rename-param
transformation w/ the attached input:
% /usr/libexec/clang_delta --transformation=rename-param --counter=1 \
/tmp/asan_interceptors.i
clang_delta: Transformation.cpp:101: void
Transformation::outputTransformedSourc
th MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Signed-off-by: Arseny Solokha
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/
th MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Signed-off-by: Arseny Solokha
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/
> On Fri, 2015-03-20 at 11:55 +0700, Arseny Solokha wrote:
>>
>> And by the way, while revisiting the series I've noticed that though the
>> patch
>> 4/4 basically reverts [1], it leaves
>>
>> #define MPIC_GREG_GLOBAL_CONF_10x000
th MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Signed-off-by: Arseny Solokha
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/
> On Fri, 2015-03-20 at 11:55 +0700, Arseny Solokha wrote:
>>
>> And by the way, while revisiting the series I've noticed that though the
>> patch
>> 4/4 basically reverts [1], it leaves
>>
>> #define MPIC_GREG_GLOBAL_CONF_10x000
th MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Signed-off-by: Arseny Solokha
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/
> On Fri, 2015-03-20 at 10:56 +0700, Arseny Solokha wrote:
>> This series removes unused functions from powerpc tree that I've been able
>> to discover.
>>
>> Two machines at hands, e300 and e500 based, boot and run without regressions
>> on my workload with
> On Fri, 2015-03-20 at 10:56 +0700, Arseny Solokha wrote:
>> This series removes unused functions from powerpc tree that I've been able
>> to discover.
>>
>> Two machines at hands, e300 and e500 based, boot and run without regressions
>> on my workload with
-June/023867.html
Signed-off-by: Arseny Solokha
Cc: Jia Hongtao
---
arch/powerpc/include/asm/mpic.h | 11 ---
arch/powerpc/sysdev/mpic.c | 25 -
2 files changed, 36 deletions(-)
diff --git a/arch/powerpc/include/asm/mpic.h b/arch/powerpc/include/asm/mpic.h
Drop ucc_slow_poll_transmitter_now() which has no users since its
inception in 2007 in commit 986585385131 ("[POWERPC] Add QUICC
Engine (QE) infrastructure").
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_s
85xx: workaround for chips with MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic
-June/023867.html
Signed-off-by: Arseny Solokha
Cc: Jia Hongtao
---
arch/powerpc/include/asm/mpic.h | 11 ---
arch/powerpc/sysdev/mpic.c | 25 -
2 files changed, 36 deletions(-)
diff --git a/arch/powerpc/include/asm/mpic.h b/arch/powerpc/include/asm/mpic.h
Drop unused static procedure which doesn't have callers within its
translation unit. It had been already removed independently in QEMU[1]
from the OpenPIC implementation borrowed from the kernel.
[1] https://lists.gnu.org/archive/html/qemu-devel/2014-06/msg01812.html
Signed-off-by: A
Drop ucc_slow_poll_transmitter_now() which has no users since its
inception in 2007 in commit 986585385131 ("[POWERPC] Add QUICC
Engine (QE) infrastructure").
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_s
Drop planetcore_set_serial_speed() which had no users since its
inception in commit fec6047047fd ("[POWERPC] bootwrapper: Add PlanetCore
firmware support") in 2007.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/po
Drop planetcore_set_serial_speed() which had no users since its
inception in commit fec6047047fd ("[POWERPC] bootwrapper: Add PlanetCore
firmware support") in 2007.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/po
Drop unused static procedure which doesn't have callers within its
translation unit. It had been already removed independently in QEMU[1]
from the OpenPIC implementation borrowed from the kernel.
[1] https://lists.gnu.org/archive/html/qemu-devel/2014-06/msg01812.html
Signed-off-by: A
85xx: workaround for chips with MSI hardware errata") makes
use of it.
v2: Added a brief explanation to each patch description of why removed
functions are unused, as suggested by Michael Ellerman.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic
> This has probably been reported already.
>
> In 3.19 this started showing up in the dmesg log when the mmcblk driver
> gets loaded:
>
> "Driver 'mmcblk' needs updating - please use bus_type methods"
>
> Looks like it is caused by the re-factoring needed after the removal of
> struct mmc_driver
> @@ -1676,31 +1666,6 @@ void __init mpic_init(struct mpic *mpic)
> mpic_err_int_init(mpic, MPIC_FSL_ERR_INT);
> }
>
> -void __init mpic_set_clk_ratio(struct mpic *mpic, u32 clock_ratio)
> -{
> - u32 v;
> -
> - v = mpic_read(mpic->gregs, MPIC_GREG_GLOBAL_CONF_1);
> - v
> @@ -1676,31 +1666,6 @@ void __init mpic_init(struct mpic *mpic)
> mpic_err_int_init(mpic, MPIC_FSL_ERR_INT);
> }
>
> -void __init mpic_set_clk_ratio(struct mpic *mpic, u32 clock_ratio)
> -{
> - u32 v;
> -
> - v = mpic_read(mpic->gregs, MPIC_GREG_GLOBAL_CONF_1);
> - v
ren't
exported.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2006-June/023867.html
Signed-off-by: Arseny Solokha
Cc: hongtao@freescale.com
---
arch/powerpc/include/asm/mpic.h | 20
arch/powerpc/sysdev/mpic.c | 35 ---
2 fil
Drop ucc_slow_poll_transmitter_now() which has no users since its
inception in 2007 in commit 986585385131 ("[POWERPC] Add QUICC
Engine (QE) infrastructure").
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_s
ren't
exported.
[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2006-June/023867.html
Signed-off-by: Arseny Solokha
Cc: hongtao@freescale.com
---
arch/powerpc/include/asm/mpic.h | 20
arch/powerpc/sysdev/mpic.c | 35 ---
2 fil
Drop ucc_slow_poll_transmitter_now() which has no users since its
inception in 2007 in commit 986585385131 ("[POWERPC] Add QUICC
Engine (QE) infrastructure").
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_s
Drop unused static procedure which doesn't have callers within its
translation unit. It had been already removed independently in QEMU[1]
from the OpenPIC implementation borrowed from the kernel.
[1] https://lists.gnu.org/archive/html/qemu-devel/2014-06/msg01812.html
Signed-off-by: A
st
in general. But I can't obviously express any strong point in support of
the series, so it's completely OK to leave things as is.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic: drop unused IRQ_testbit
powrepc/qe: drop unused ucc_slow_poll_
Drop unused static procedure which doesn't have callers within its
translation unit. It had been already removed independently in QEMU[1]
from the OpenPIC implementation borrowed from the kernel.
[1] https://lists.gnu.org/archive/html/qemu-devel/2014-06/msg01812.html
Signed-off-by: A
Drop planetcore_set_serial_speed() which had no users since its
inception in commit fec6047047fd ("[POWERPC] bootwrapper: Add PlanetCore
firmware support") in 2007.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/po
Drop planetcore_set_serial_speed() which had no users since its
inception in commit fec6047047fd ("[POWERPC] bootwrapper: Add PlanetCore
firmware support") in 2007.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/po
st
in general. But I can't obviously express any strong point in support of
the series, so it's completely OK to leave things as is.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic: drop unused IRQ_testbit
powrepc/qe: drop unused ucc_slow_poll_
> If I just get a patch saying "removed unused foo()", I have to go and dig and
> find out:
> - was it recently added and will be used soon?
> - is it ancient and never used, if so can we work out why, ie. feature X
> never landed so this code is no longer needed.
> - is it old code that
> If I just get a patch saying "removed unused foo()", I have to go and dig and
> find out:
> - was it recently added and will be used soon?
> - is it ancient and never used, if so can we work out why, ie. feature X
> never landed so this code is no longer needed.
> - is it old code that
> On Mon, 2015-02-16 at 17:56 +0700, Arseny Solokha wrote:
>> Drop unused fsl_mpic_primary_get_version(), mpic_set_clk_ratio(),
>> mpic_set_serial_int().
>
> I'm always happy to remove unused code, but the interesting question is why
> are
> they unused? Please tel
> On Mon, 2015-02-16 at 17:56 +0700, Arseny Solokha wrote:
>> Drop unused fsl_mpic_primary_get_version(), mpic_set_clk_ratio(),
>> mpic_set_serial_int().
>
> I'm always happy to remove unused code, but the interesting question is why
> are
> they unused? Please tel
Drop unused fsl_mpic_primary_get_version(), mpic_set_clk_ratio(),
mpic_set_serial_int().
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/mpic.h | 16
arch/powerpc/sysdev/mpic.c | 35 ---
2 files changed, 51 deletions(-)
diff
Drop ucc_slow_poll_transmitter_now() which has no users.
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_slow.c | 5 -
2 files changed, 18 deletions(-)
diff --git a/arch/powerpc/include/asm/ucc_slow.h
b/arch
Drop planetcore_set_serial_speed() which had no users since its inception.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/powerpc/boot/planetcore.h | 3 ---
2 files changed, 36 deletions(-)
diff --git a/arch/powerpc/boot
Drop unused static procedure which doesn't have callers within its
translation unit.
Signed-off-by: Arseny Solokha
Cc: Alexander Graf
Cc: Gleb Natapov
Cc: Paolo Bonzini
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/powerpc/kvm/mpic.c b/arch/po
Drop planetcore_set_serial_speed() which had no users since its inception.
Signed-off-by: Arseny Solokha
---
arch/powerpc/boot/planetcore.c | 33 -
arch/powerpc/boot/planetcore.h | 3 ---
2 files changed, 36 deletions(-)
diff --git a/arch/powerpc/boot
Drop ucc_slow_poll_transmitter_now() which has no users.
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/ucc_slow.h | 13 -
arch/powerpc/sysdev/qe_lib/ucc_slow.c | 5 -
2 files changed, 18 deletions(-)
diff --git a/arch/powerpc/include/asm/ucc_slow.h
b/arch
Drop unused static procedure which doesn't have callers within its
translation unit.
Signed-off-by: Arseny Solokha
Cc: Alexander Graf
Cc: Gleb Natapov
Cc: Paolo Bonzini
---
arch/powerpc/kvm/mpic.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/powerpc/kvm/mpic.c b/arch/po
Drop unused fsl_mpic_primary_get_version(), mpic_set_clk_ratio(),
mpic_set_serial_int().
Signed-off-by: Arseny Solokha
---
arch/powerpc/include/asm/mpic.h | 16
arch/powerpc/sysdev/mpic.c | 35 ---
2 files changed, 51 deletions(-)
diff
This series removes unused functions from powerpc tree that I've been
able to discover.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic: drop unused IRQ_testbit
powrepc/qe: drop unused ucc_slow_poll_transmitter_now
powerpc/mpic: remove unused func
This series removes unused functions from powerpc tree that I've been
able to discover.
Arseny Solokha (4):
powerpc/boot: drop planetcore_set_serial_speed
kvm/ppc/mpic: drop unused IRQ_testbit
powrepc/qe: drop unused ucc_slow_poll_transmitter_now
powerpc/mpic: remove unused func
Function __flush_tlb_page() must only be called for user contexts, so
put in extra hardening to warn on calling it for kernel context.
Signed-off-by: Arseny Solokha
---
arch/powerpc/mm/tlb_nohash.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/arch/powerpc/mm
Function __flush_tlb_page() must only be called for user contexts, so
put in extra hardening to warn on calling it for kernel context.
Signed-off-by: Arseny Solokha
---
arch/powerpc/mm/tlb_nohash.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/arch/powerpc/mm
1 - 100 of 129 matches
Mail list logo
