ain.c | 64 +++-
1 file changed, 39 insertions(+), 25 deletions(-)
Nice code simplification.
Reviewed-by: Philippe Mathieu-Daudé
| 29 +++
23 files changed, 463 insertions(+), 445 deletions(-)
create mode 100644 arch/sparc/mm/execmem.c
Reviewed-by: Philippe Mathieu-Daudé
On 5/4/24 13:58, Paolo Bonzini wrote:
The only user was kvm_mmu_notifier_change_pte(), which is now gone.
Signed-off-by: Paolo Bonzini
---
virt/kvm/kvm_main.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
--
kernel/events/uprobes.c | 5 ++---
mm/ksm.c | 4 ++--
mm/memory.c | 7 +--
mm/migrate_device.c | 8 ++--
5 files changed, 7 insertions(+), 19 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
Hi Bhaskar,
On Fri, Mar 5, 2021 at 3:06 AM Bhaskar Chowdhury wrote:
>
>
>
> s/revelant/relevant/
Let me recommend you this great post explaining why and how you could
improve your patch subject and description:
On Fri, Mar 5, 2021 at 11:16 AM Rafał Miłecki wrote:
>
> Hi,
>
> On 05.03.2021 10:58, Philippe Mathieu-Daudé wrote:
> > On Fri, Mar 5, 2021 at 6:55 AM Rafał Miłecki wrote:
> >>
> >> From: Rafał Miłecki
> >>
> >> 1. Use meaningful
Hi,
On Thu, Mar 4, 2021 at 5:35 PM Qing Zhang wrote:
>
> The purpose of separating loongson_system_configuration from boot_param.h
> is to keep the other structure consistent with the firmware.
This is supposed to be a trivial patch, but the description actually
confuses me.
Why is the move
Hi Rafał,
On Fri, Mar 5, 2021 at 6:55 AM Rafał Miłecki wrote:
>
> From: Rafał Miłecki
>
> 1. Use meaningful variable names (e.g. "flash_start", "res_size" instead
>of e.g. "iobase", "end")
> 2. Always operate on "offset" instead of mix of start, end, size, etc.
"instead of a mix"
> 3. Add
/github.com/ClangBuiltLinux/linux/issues/1299
> Signed-off-by: Nathan Chancellor
> ---
> drivers/firmware/qemu_fw_cfg.c | 8 +++-
> 1 file changed, 3 insertions(+), 5 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
On Tue, Feb 23, 2021 at 1:33 PM Maciej W. Rozycki wrote:
> On Tue, 23 Feb 2021, Philippe Mathieu-Daudé wrote:
>
> > > diff --git a/arch/mips/sgi-ip27/ip27-timer.c
> > > b/arch/mips/sgi-ip27/ip27-timer.c
> > > index 79c434fece52..444b5e0e935f 100644
> >
On Mon, Feb 22, 2021 at 5:22 PM Lukas Bulwahn wrote:
>
> The domain lookup for linux-mips.org fails for quite some time now.
> Further, the two links:
>
> http://decstation.unix-ag.org/
https://web.archive.org/web/20070609185805/http://decstation.unix-ag.org/
>
On Mon, Feb 22, 2021 at 5:22 PM Lukas Bulwahn wrote:
>
> This is a Copyright line, and just a typo slipped through.
>
> Signed-off-by: Lukas Bulwahn
> ---
> arch/mips/sgi-ip27/ip27-timer.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git
+-
> arch/mips/sgi-ip32/ip32-irq.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé
On Mon, Feb 22, 2021 at 5:22 PM Lukas Bulwahn wrote:
>
> The domain lookup for linux-mips.org fails for quite some time now. Hence,
> webpages, the patchwork instance and Ralf Baechle's email there is not
> reachable anymore.
>
> Remove all references of webpages from linux-mips.org in
t; ---
> arch/mips/boot/compressed/decompress.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé
On Wed, Oct 28, 2020 at 9:01 PM Colin King wrote:
>
> From: Colin Ian King
>
> There are a few spelling mistakes in the Kconfig, fix these.
>
> Signed-off-by: Colin Ian King
> ---
> arch/mips/Kconfig | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
Re
On Sun, Sep 20, 2020 at 11:09 PM Thomas Bogendoerfer
wrote:
>
> Remove unused heasder file asm/mach-malta/malta-pm.h.
Typo "header".
>
> Signed-off-by: Thomas Bogendoerfer
> ---
> arch/mips/include/asm/mach-malta/malta-pm.h | 33
> -
> 1 file changed, 33
On Sun, Sep 6, 2020 at 9:31 PM Paul Cercueil wrote:
>
> Check for the (already existing) "system_type" variable in the
> get_system_type() function. If non-NULL, return it as the system type.
Nitpick: It is not already "existing", simply declared in asm/bootinfo.h.
>
> Signed-off-by: Paul
On Thu, Sep 3, 2020 at 11:28 AM Greg Kroah-Hartman
wrote:
>
> On Wed, Aug 26, 2020 at 02:06:28PM -0700, Guenter Roeck wrote:
> > Hi,
> >
> > On Wed, Oct 16, 2019 at 02:51:17PM -0700, Greg Kroah-Hartman wrote:
> > > From: Paul Burton
> > >
> > > commit 2f2b4fd674cadd8c6b40eb629e140a14db4068fd
On Wed, Aug 12, 2020 at 9:53 AM Álvaro Fernández Rojas
wrote:
>
> Current board declarations are a mess. Let's put some order and make them
> follow the same structure.
What a mess indeed. Patch easier to review using 'git-diff --ignore-all-space'.
Reviewed-by: Philippe Math
dy reviewed, again:
Reviewed-by: Philippe Mathieu-Daudé
> ---
> v4: no changes.
> v3: Reword commit description to avoid possible confusions.
> v2: no changes.
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/m
On Wed, Aug 12, 2020 at 9:53 AM Álvaro Fernández Rojas
wrote:
>
> There's no EHCI controller on BCM6348.
>
> Signed-off-by: Álvaro Fernández Rojas
> Acked-by: Florian Fainelli
v3 already reviewed, again:
Reviewed-by: Philippe Mathieu-Daudé
> ---
> v4: no changes.
>
On Tue, Aug 11, 2020 at 6:48 PM Philippe Mathieu-Daudé wrote:
> On Tue, Aug 11, 2020 at 5:01 PM Álvaro Fernández Rojas
> wrote:
> >
> > Current board declarations are a mess. Let's put some order and make them
> > follow the same structure. Also board declarations
On Tue, Aug 11, 2020 at 5:01 PM Álvaro Fernández Rojas
wrote:
>
> Current board declarations are a mess. Let's put some order and make them
> follow the same structure. Also board declarations tabs.
> Switch to SPDX license identifier.
>
> Signed-off-by: Álvaro Fernández Rojas
> Acked-by:
On Tue, Aug 11, 2020 at 5:05 PM Álvaro Fernández Rojas
wrote:
>
> There's no EHCI controller on BCM6348.
>
> Signed-off-by: Álvaro Fernández Rojas
> Acked-by: Florian Fainelli
Reviewed-by: Philippe Mathieu-Daudé
> ---
> v3: no changes.
> v2: no changes.
>
iewed-by: Philippe Mathieu-Daudé
> ---
> v3: Reword commit description to avoid possible confusions.
Thanks!
> v2: no changes.
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/mips/bcm63xx/boards/board_bcm963xx.c
>
Hi Florian,
On Tue, Aug 11, 2020 at 4:33 AM Florian Fainelli wrote:
> On 8/9/2020 11:49 AM, Philippe Mathieu-Daudé wrote:
> > Hi Álvaro,
> >
> > On Fri, Aug 7, 2020 at 12:05 PM Álvaro Fernández Rojas
> > wrote:
> >>
> >> EHCI and OHCI share the sam
Hi Álvaro,
On Fri, Aug 7, 2020 at 12:05 PM Álvaro Fernández Rojas
wrote:
>
> EHCI and OHCI share the same USB ports. Therefore, if the board has OHCI
> it should also have EHCI.
This statement isn't correct. OHCI doesn't imply EHCI...
(although the opposite is almost always true).
Now per
On Fri, Aug 7, 2020 at 12:06 PM Álvaro Fernández Rojas
wrote:
>
> There are 3 duplicated new lines, let's remove them.
>
> Signed-off-by: Álvaro Fernández Rojas
Reviewed-by: Philippe Mathieu-Daudé
=)
> ---
> v2: no changes.
>
> arch/mips/bcm63xx/boards/board_bcm9
Hi Alexander,
On Tue, Jul 14, 2020 at 10:18 PM Alexander A. Klimov
wrote:
>
> Rationale:
> A completely white page with just an in the .
>
> Signed-off-by: Alexander A. Klimov
> ---
> Yes, I noted that some of the links removed by these "CREDITS: remove link:"
> patches have email addresses
Hi Alexander,
On Tue, Jul 14, 2020 at 10:18 PM Alexander A. Klimov
wrote:
>
> Rationale:
> The way it redirects looks like a fallback from a dead URL to a generic one.
>
> Signed-off-by: Alexander A. Klimov
> ---
> Yes, I noted that some of the links removed by these "CREDITS: remove link:"
>
On Wed, Jun 10, 2020 at 5:32 PM Florian Fainelli wrote:
> On 6/10/2020 1:29 AM, Philippe Mathieu-Daudé wrote:
> > Hi,
> >
> > On Wed, Jun 10, 2020 at 8:13 AM Álvaro Fernández Rojas
> > wrote:
> >>
> >> Hi Florian,
> >>
> >>
Hi,
On Wed, Jun 10, 2020 at 8:13 AM Álvaro Fernández Rojas
wrote:
>
> Hi Florian,
>
> > El 10 jun 2020, a las 4:27, Florian Fainelli
> > escribió:
> >
> >
> >
> > On 6/9/2020 4:30 AM, Álvaro Fernández Rojas wrote:
> >> +static const struct clk_bcm63xx_table_entry bcm6318_clocks[] = {
> >> +
Hi Linus,
On 5/29/20 9:19 PM, Linus Torvalds wrote:
> On Fri, May 29, 2020 at 6:08 AM David Laight wrote:
>>
>> A wide monitor is for looking at lots of files.
>
> Not necessarily.
>
> Excessive line breaks are BAD. They cause real and every-day problems.
>
> They cause problems for things
On Wed, May 20, 2020 at 11:28 AM Thomas Bogendoerfer
wrote:
>
> On Wed, May 20, 2020 at 11:19:49AM +0200, Philippe Mathieu-Daudé wrote:
> > Hi Thomas,
> >
> > On Wed, May 20, 2020 at 8:50 AM Thomas Bogendoerfer
> > wrote:
> > >
> > > On Tue, May 1
Hi Thomas,
On Wed, May 20, 2020 at 8:50 AM Thomas Bogendoerfer
wrote:
>
> On Tue, May 19, 2020 at 11:22:30PM +0200, Paul Cercueil wrote:
> > Add missing include which adds the prototype to plat_time_init().
> >
> > Fixes: f932449c11da ("MIPS: ingenic: Drop obsolete code, merge the rest in
> >
long __xchg(volatile void *ptr, unsigned long x, int size)
{
switch (size) {
case 1:
Reviewed-by: Philippe Mathieu-Daudé
+}
+
#endif /* CONFIG_NVMEM */
#if IS_ENABLED(CONFIG_NVMEM) && IS_ENABLED(CONFIG_OF)
Reviewed-by: Philippe Mathieu-Daudé
Hi Thomas,
On 8/11/19 9:29 AM, Greg Kroah-Hartman wrote:
> On Sat, Aug 10, 2019 at 04:22:23PM +0300, Andy Shevchenko wrote:
>> On Fri, Aug 9, 2019 at 1:34 PM Thomas Bogendoerfer
>> wrote:
>>>
>>> Our chosen byte swapping, which is what firmware already uses, is to
>>> do readl/writel by normal
rough.
>
> Cc: Gustavo A. R. Silva
> Signed-off-by: Guenter Roeck
Reviewed-by: Philippe Mathieu-Daudé
> ---
> arch/mips/cavium-octeon/octeon-usb.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/mips/cavium-octeon/octeon-usb.c
> b/arch/mips/cavium-octeon/oct
On 5/27/19 5:56 PM, Guenter Roeck wrote:
> On 5/26/19 11:32 PM, Tony Lindgren wrote:
>> Hi,
>>
>> * Philippe Mathieu-Daudé [190523 12:01]:
>>> What I use as reference for testing ARM boards [*] is the work of
>>> Guenter Roeck:
>>> https://github.co
On 5/23/19 1:27 PM, Thomas Huth wrote:
> On 22/05/2019 20.19, Aaro Koskinen wrote:
>> Hi,
>>
>> On Wed, May 22, 2019 at 11:33:41AM +0200, Corentin Labbe wrote:
>>> qemu-system-arm -M help |grep OMAP
>>> cheetah Palm Tungsten|E aka. Cheetah PDA (OMAP310)
>>> n800 Nokia
if (nvram_len >= NVRAM_SPACE) {
> - pr_err("nvram on flash (%i bytes) is bigger than the
> reserved space in memory, will just copy the first %i bytes\n",
> + pr_err("nvram on flash (%zu bytes) is bigger than the
> reserved space in memory, will just copy the first %i bytes\n",
> header.len, NVRAM_SPACE);
Ditto.
Regardless:
Reviewed-by: Philippe Mathieu-Daudé
> nvram_len = NVRAM_SPACE - 1;
> }
>
l at offset 0 */
> - lw a1, PT_R4(sp) /* Arg1 for __NR_syscall case */
> + ld a1, PT_R4(sp) /* Arg1 for __NR_syscall case */
> .setpop
>
> 1: jal syscall_trace_enter
>
>
Reviewed-by: Philippe Mathieu-Daudé
On 2/26/19 5:10 PM, Michael S. Tsirkin wrote:
> On Tue, Feb 26, 2019 at 03:31:59PM +0800, Wei Yang wrote:
>> Leverage __ATTR_RO_MODE to define rev sysfs instead of using open code
>> to define the attribute.
>>
>> Signed-off-by: Wei Yang
>> ---
>> drivers/firmware/qemu_fw_cfg.c | 13
Hi Wei,
On 2/26/19 8:31 AM, Wei Yang wrote:
> Leverage __ATTR_RO_MODE to define rev sysfs instead of using open code
> to define the attribute.
>
> Signed-off-by: Wei Yang
> ---
> drivers/firmware/qemu_fw_cfg.c | 13 -
> 1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff
..@anholt.net>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> drivers/char/hw_random/bcm2835-rng.c | 27 +++
> 1 file changed, 19 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/char/hw_random/bcm2835-rng.c
> b/driver
On 11/07/2017 09:44 PM, Florian Fainelli wrote:
> In preparation for allowing BCM63xx to use this driver, we abstract I/O
> accessors such that we can easily change those later on.
>
> Signed-off-by: Florian Fainelli
> Reviewed-by: Eric Anholt
Reviewed-by: Philipp
48 matches
Mail list logo
