From: Thomas Zimmermann <tzimmermann@suse.de>
To: Sui Jingfeng <15330273260@189.cn>,
deller@gmx.de, geert@linux-m68k.org, javierm@redhat.com,
daniel@ffwll.ch, vgupta@kernel.org, chenhuacai@kernel.org,
kernel@xen0n.name, davem@davemloft.net,
James.Bottomley@HansenPartnership.com, arnd@arndb.de,
sam@ravnborg.org
Cc: linux-arch@vger.kernel.org, linux-fbdev@vger.kernel.org,
linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-m68k@lists.linux-m68k.org, loongarch@lists.linux.dev,
sparclinux@vger.kernel.org, linux-snps-arc@lists.infradead.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 1/6] fbdev/matrox: Remove trailing whitespaces
Date: Thu, 11 May 2023 09:55:05 +0200 [thread overview]
Message-ID: <a2315b9a-0747-1f0f-1f0a-1c6773931db4@suse.de> (raw)
In-Reply-To: <0e13efbf-9a48-6e70-fdf3-8290f28c6dc7@189.cn>
[-- Attachment #1.1: Type: text/plain, Size: 4433 bytes --]
Hi
Am 10.05.23 um 20:20 schrieb Sui Jingfeng:
> Hi, Thomas
>
>
> I love your patch, yet something to improve:
>
>
> On 2023/5/10 19:05, Thomas Zimmermann wrote:
>> Fix coding style. No functional changes.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>> Reviewed-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> Tested-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
>> drivers/video/fbdev/matrox/matroxfb_accel.c | 6 +++---
>> drivers/video/fbdev/matrox/matroxfb_base.h | 4 ++--
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> index 9cb0685feddd..ce51227798a1 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> +++ b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> @@ -88,7 +88,7 @@
>> static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x11111111U;
>> }
>> @@ -96,7 +96,7 @@ static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> static inline void matrox_cfb8_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x01010101U;
>> }
>> @@ -482,7 +482,7 @@ static void matroxfb_1bpp_imageblit(struct
>> matrox_fb_info *minfo, u_int32_t fgx,
>> /* Tell... well, why bother... */
>> while (height--) {
>> size_t i;
>> -
>> +
>> for (i = 0; i < step; i += 4) {
>> /* Hope that there are at least three readable
>> bytes beyond the end of bitmap */
>> fb_writel(get_unaligned((u_int32_t*)(chardata +
>> i)),mmio.vaddr);
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_base.h
>> b/drivers/video/fbdev/matrox/matroxfb_base.h
>> index 958be6805f87..c93c69bbcd57 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_base.h
>> +++ b/drivers/video/fbdev/matrox/matroxfb_base.h
>> @@ -301,9 +301,9 @@ struct matrox_altout {
>> int (*verifymode)(void* altout_dev, u_int32_t mode);
>> int (*getqueryctrl)(void* altout_dev,
>> struct v4l2_queryctrl* ctrl);
>
> Noticed that there are plenty of coding style problems in matroxfb_base.h,
>
> why you only fix a few of them? Take this two line as an example,
> shouldn't
>
> they be fixed also as following?
I configured my text editor to remove trailing whitespaces
automatically. That keeps my own patches free of them. But the editor
removes all trailing whitespaces, including those that have been there
before. If I encounter such a case, I split out the whitespace fix and
submit it separately.
But the work I do within fbdev is mostly for improving DRM. For the
other issues in this file, I don't think that matroxfb should even be
around any longer. Fbdev has been deprecated for a long time. But a
small number of drivers are still in use and we still need its
framebuffer console. So someone should either put significant effort
into maintaining fbdev, or it should be phased out. But neither is
happening.
Best regards
Thomas
>
>
> int (*verifymode)(void *altout_dev, u_int32_t mode);
> int (*getqueryctrl)(void *altout_dev,
> struct v4l2_queryctrl *ctrl);
>
>
>> - int (*getctrl)(void* altout_dev,
>> + int (*getctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> - int (*setctrl)(void* altout_dev,
>> + int (*setctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> };
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Zimmermann <tzimmermann@suse.de>
To: Sui Jingfeng <15330273260@189.cn>,
deller@gmx.de, geert@linux-m68k.org, javierm@redhat.com,
daniel@ffwll.ch, vgupta@kernel.org, chenhuacai@kernel.org,
kernel@xen0n.name, davem@davemloft.net,
James.Bottomley@HansenPartnership.com, arnd@arndb.de,
sam@ravnborg.org
Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-arch@vger.kernel.org, linux-snps-arc@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-ia64@vger.kernel.org,
loongarch@lists.linux.dev, linux-m68k@lists.linux-m68k.org,
sparclinux@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-parisc@vger.kernel.org
Subject: Re: [PATCH v6 1/6] fbdev/matrox: Remove trailing whitespaces
Date: Thu, 11 May 2023 09:55:05 +0200 [thread overview]
Message-ID: <a2315b9a-0747-1f0f-1f0a-1c6773931db4@suse.de> (raw)
In-Reply-To: <0e13efbf-9a48-6e70-fdf3-8290f28c6dc7@189.cn>
[-- Attachment #1.1: Type: text/plain, Size: 4433 bytes --]
Hi
Am 10.05.23 um 20:20 schrieb Sui Jingfeng:
> Hi, Thomas
>
>
> I love your patch, yet something to improve:
>
>
> On 2023/5/10 19:05, Thomas Zimmermann wrote:
>> Fix coding style. No functional changes.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>> Reviewed-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> Tested-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
>> drivers/video/fbdev/matrox/matroxfb_accel.c | 6 +++---
>> drivers/video/fbdev/matrox/matroxfb_base.h | 4 ++--
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> index 9cb0685feddd..ce51227798a1 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> +++ b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> @@ -88,7 +88,7 @@
>> static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x11111111U;
>> }
>> @@ -96,7 +96,7 @@ static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> static inline void matrox_cfb8_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x01010101U;
>> }
>> @@ -482,7 +482,7 @@ static void matroxfb_1bpp_imageblit(struct
>> matrox_fb_info *minfo, u_int32_t fgx,
>> /* Tell... well, why bother... */
>> while (height--) {
>> size_t i;
>> -
>> +
>> for (i = 0; i < step; i += 4) {
>> /* Hope that there are at least three readable
>> bytes beyond the end of bitmap */
>> fb_writel(get_unaligned((u_int32_t*)(chardata +
>> i)),mmio.vaddr);
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_base.h
>> b/drivers/video/fbdev/matrox/matroxfb_base.h
>> index 958be6805f87..c93c69bbcd57 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_base.h
>> +++ b/drivers/video/fbdev/matrox/matroxfb_base.h
>> @@ -301,9 +301,9 @@ struct matrox_altout {
>> int (*verifymode)(void* altout_dev, u_int32_t mode);
>> int (*getqueryctrl)(void* altout_dev,
>> struct v4l2_queryctrl* ctrl);
>
> Noticed that there are plenty of coding style problems in matroxfb_base.h,
>
> why you only fix a few of them? Take this two line as an example,
> shouldn't
>
> they be fixed also as following?
I configured my text editor to remove trailing whitespaces
automatically. That keeps my own patches free of them. But the editor
removes all trailing whitespaces, including those that have been there
before. If I encounter such a case, I split out the whitespace fix and
submit it separately.
But the work I do within fbdev is mostly for improving DRM. For the
other issues in this file, I don't think that matroxfb should even be
around any longer. Fbdev has been deprecated for a long time. But a
small number of drivers are still in use and we still need its
framebuffer console. So someone should either put significant effort
into maintaining fbdev, or it should be phased out. But neither is
happening.
Best regards
Thomas
>
>
> int (*verifymode)(void *altout_dev, u_int32_t mode);
> int (*getqueryctrl)(void *altout_dev,
> struct v4l2_queryctrl *ctrl);
>
>
>> - int (*getctrl)(void* altout_dev,
>> + int (*getctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> - int (*setctrl)(void* altout_dev,
>> + int (*setctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> };
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Zimmermann <tzimmermann@suse.de>
To: Sui Jingfeng <15330273260@189.cn>,
deller@gmx.de, geert@linux-m68k.org, javierm@redhat.com,
daniel@ffwll.ch, vgupta@kernel.org, chenhuacai@kernel.org,
kernel@xen0n.name, davem@davemloft.net,
James.Bottomley@HansenPartnership.com, arnd@arndb.de,
sam@ravnborg.org
Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-arch@vger.kernel.org, linux-snps-arc@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-ia64@vger.kernel.org,
loongarch@lists.linux.dev, linux-m68k@lists.linux-m68k.org,
sparclinux@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-parisc@vger.kernel.org
Subject: Re: [PATCH v6 1/6] fbdev/matrox: Remove trailing whitespaces
Date: Thu, 11 May 2023 09:55:05 +0200 [thread overview]
Message-ID: <a2315b9a-0747-1f0f-1f0a-1c6773931db4@suse.de> (raw)
In-Reply-To: <0e13efbf-9a48-6e70-fdf3-8290f28c6dc7@189.cn>
[-- Attachment #1.1.1: Type: text/plain, Size: 4433 bytes --]
Hi
Am 10.05.23 um 20:20 schrieb Sui Jingfeng:
> Hi, Thomas
>
>
> I love your patch, yet something to improve:
>
>
> On 2023/5/10 19:05, Thomas Zimmermann wrote:
>> Fix coding style. No functional changes.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>> Reviewed-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> Tested-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
>> drivers/video/fbdev/matrox/matroxfb_accel.c | 6 +++---
>> drivers/video/fbdev/matrox/matroxfb_base.h | 4 ++--
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> index 9cb0685feddd..ce51227798a1 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> +++ b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> @@ -88,7 +88,7 @@
>> static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x11111111U;
>> }
>> @@ -96,7 +96,7 @@ static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> static inline void matrox_cfb8_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x01010101U;
>> }
>> @@ -482,7 +482,7 @@ static void matroxfb_1bpp_imageblit(struct
>> matrox_fb_info *minfo, u_int32_t fgx,
>> /* Tell... well, why bother... */
>> while (height--) {
>> size_t i;
>> -
>> +
>> for (i = 0; i < step; i += 4) {
>> /* Hope that there are at least three readable
>> bytes beyond the end of bitmap */
>> fb_writel(get_unaligned((u_int32_t*)(chardata +
>> i)),mmio.vaddr);
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_base.h
>> b/drivers/video/fbdev/matrox/matroxfb_base.h
>> index 958be6805f87..c93c69bbcd57 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_base.h
>> +++ b/drivers/video/fbdev/matrox/matroxfb_base.h
>> @@ -301,9 +301,9 @@ struct matrox_altout {
>> int (*verifymode)(void* altout_dev, u_int32_t mode);
>> int (*getqueryctrl)(void* altout_dev,
>> struct v4l2_queryctrl* ctrl);
>
> Noticed that there are plenty of coding style problems in matroxfb_base.h,
>
> why you only fix a few of them? Take this two line as an example,
> shouldn't
>
> they be fixed also as following?
I configured my text editor to remove trailing whitespaces
automatically. That keeps my own patches free of them. But the editor
removes all trailing whitespaces, including those that have been there
before. If I encounter such a case, I split out the whitespace fix and
submit it separately.
But the work I do within fbdev is mostly for improving DRM. For the
other issues in this file, I don't think that matroxfb should even be
around any longer. Fbdev has been deprecated for a long time. But a
small number of drivers are still in use and we still need its
framebuffer console. So someone should either put significant effort
into maintaining fbdev, or it should be phased out. But neither is
happening.
Best regards
Thomas
>
>
> int (*verifymode)(void *altout_dev, u_int32_t mode);
> int (*getqueryctrl)(void *altout_dev,
> struct v4l2_queryctrl *ctrl);
>
>
>> - int (*getctrl)(void* altout_dev,
>> + int (*getctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> - int (*setctrl)(void* altout_dev,
>> + int (*setctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> };
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
[-- Attachment #2: Type: text/plain, Size: 170 bytes --]
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Zimmermann <tzimmermann@suse.de>
To: Sui Jingfeng <15330273260@189.cn>,
deller@gmx.de, geert@linux-m68k.org, javierm@redhat.com,
daniel@ffwll.ch, vgupta@kernel.org, chenhuacai@kernel.org,
kernel@xen0n.name, davem@davemloft.net,
James.Bottomley@HansenPartnership.com, arnd@arndb.de,
sam@ravnborg.org
Cc: linux-arch@vger.kernel.org, linux-fbdev@vger.kernel.org,
linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-m68k@lists.linux-m68k.org, loongarch@lists.linux.dev,
sparclinux@vger.kernel.org, linux-snps-arc@lists.infradead.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 1/6] fbdev/matrox: Remove trailing whitespaces
Date: Thu, 11 May 2023 07:55:05 +0000 [thread overview]
Message-ID: <a2315b9a-0747-1f0f-1f0a-1c6773931db4@suse.de> (raw)
In-Reply-To: <0e13efbf-9a48-6e70-fdf3-8290f28c6dc7@189.cn>
[-- Attachment #1.1: Type: text/plain, Size: 4433 bytes --]
Hi
Am 10.05.23 um 20:20 schrieb Sui Jingfeng:
> Hi, Thomas
>
>
> I love your patch, yet something to improve:
>
>
> On 2023/5/10 19:05, Thomas Zimmermann wrote:
>> Fix coding style. No functional changes.
>>
>> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
>> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>> Reviewed-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> Tested-by: Sui Jingfeng <suijingfeng@loongson.cn>
>> ---
>> drivers/video/fbdev/matrox/matroxfb_accel.c | 6 +++---
>> drivers/video/fbdev/matrox/matroxfb_base.h | 4 ++--
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> index 9cb0685feddd..ce51227798a1 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_accel.c
>> +++ b/drivers/video/fbdev/matrox/matroxfb_accel.c
>> @@ -88,7 +88,7 @@
>> static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x11111111U;
>> }
>> @@ -96,7 +96,7 @@ static inline void matrox_cfb4_pal(u_int32_t* pal) {
>> static inline void matrox_cfb8_pal(u_int32_t* pal) {
>> unsigned int i;
>> -
>> +
>> for (i = 0; i < 16; i++) {
>> pal[i] = i * 0x01010101U;
>> }
>> @@ -482,7 +482,7 @@ static void matroxfb_1bpp_imageblit(struct
>> matrox_fb_info *minfo, u_int32_t fgx,
>> /* Tell... well, why bother... */
>> while (height--) {
>> size_t i;
>> -
>> +
>> for (i = 0; i < step; i += 4) {
>> /* Hope that there are at least three readable
>> bytes beyond the end of bitmap */
>> fb_writel(get_unaligned((u_int32_t*)(chardata +
>> i)),mmio.vaddr);
>> diff --git a/drivers/video/fbdev/matrox/matroxfb_base.h
>> b/drivers/video/fbdev/matrox/matroxfb_base.h
>> index 958be6805f87..c93c69bbcd57 100644
>> --- a/drivers/video/fbdev/matrox/matroxfb_base.h
>> +++ b/drivers/video/fbdev/matrox/matroxfb_base.h
>> @@ -301,9 +301,9 @@ struct matrox_altout {
>> int (*verifymode)(void* altout_dev, u_int32_t mode);
>> int (*getqueryctrl)(void* altout_dev,
>> struct v4l2_queryctrl* ctrl);
>
> Noticed that there are plenty of coding style problems in matroxfb_base.h,
>
> why you only fix a few of them? Take this two line as an example,
> shouldn't
>
> they be fixed also as following?
I configured my text editor to remove trailing whitespaces
automatically. That keeps my own patches free of them. But the editor
removes all trailing whitespaces, including those that have been there
before. If I encounter such a case, I split out the whitespace fix and
submit it separately.
But the work I do within fbdev is mostly for improving DRM. For the
other issues in this file, I don't think that matroxfb should even be
around any longer. Fbdev has been deprecated for a long time. But a
small number of drivers are still in use and we still need its
framebuffer console. So someone should either put significant effort
into maintaining fbdev, or it should be phased out. But neither is
happening.
Best regards
Thomas
>
>
> int (*verifymode)(void *altout_dev, u_int32_t mode);
> int (*getqueryctrl)(void *altout_dev,
> struct v4l2_queryctrl *ctrl);
>
>
>> - int (*getctrl)(void* altout_dev,
>> + int (*getctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> - int (*setctrl)(void* altout_dev,
>> + int (*setctrl)(void *altout_dev,
>> struct v4l2_control* ctrl);
>> };
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
next prev parent reply other threads:[~2023-05-11 7:55 UTC|newest]
Thread overview: 120+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-10 11:05 [PATCH v6 0/6] fbdev: Move framebuffer I/O helpers to <asm/fb.h> Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` [PATCH v6 1/6] fbdev/matrox: Remove trailing whitespaces Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 18:20 ` Sui Jingfeng
2023-05-10 18:20 ` Sui Jingfeng
2023-05-10 18:20 ` Sui Jingfeng
2023-05-11 7:55 ` Thomas Zimmermann [this message]
2023-05-11 7:55 ` Thomas Zimmermann
2023-05-11 7:55 ` Thomas Zimmermann
2023-05-11 7:55 ` Thomas Zimmermann
2023-05-11 9:24 ` Sui Jingfeng
2023-05-11 9:24 ` Sui Jingfeng
2023-05-11 9:24 ` Sui Jingfeng
2023-05-11 13:05 ` Helge Deller
2023-05-11 13:05 ` Helge Deller
2023-05-11 13:05 ` Helge Deller
2023-05-11 13:05 ` Helge Deller
2023-05-11 13:10 ` Geert Uytterhoeven
2023-05-11 13:10 ` Geert Uytterhoeven
2023-05-11 13:10 ` Geert Uytterhoeven
2023-05-11 13:10 ` Geert Uytterhoeven
2023-05-11 16:23 ` Helge Deller
2023-05-11 16:23 ` Helge Deller
2023-05-11 16:23 ` Helge Deller
2023-05-11 16:23 ` Helge Deller
2023-05-11 14:27 ` Thomas Zimmermann
2023-05-11 14:27 ` Thomas Zimmermann
2023-05-11 14:27 ` Thomas Zimmermann
2023-05-11 14:27 ` Thomas Zimmermann
2023-05-11 17:02 ` Helge Deller
2023-05-11 17:02 ` Helge Deller
2023-05-11 17:02 ` Helge Deller
2023-05-11 17:02 ` Helge Deller
2023-05-12 7:14 ` Thomas Zimmermann
2023-05-12 7:14 ` Thomas Zimmermann
2023-05-12 7:14 ` Thomas Zimmermann
2023-05-12 7:14 ` Thomas Zimmermann
2023-05-12 10:04 ` Finn Thain
2023-05-12 10:04 ` Finn Thain
2023-05-12 10:04 ` Finn Thain
2023-05-12 10:04 ` Finn Thain
2023-05-10 11:05 ` [PATCH v6 2/6] ipu-v3: Include <linux/io.h> Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` [PATCH v6 3/6] fbdev: Include <linux/io.h> in various drivers Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` [PATCH v6 4/6] fbdev: Include <linux/fb.h> instead of <asm/fb.h> Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` [PATCH v6 5/6] fbdev: Move framebuffer I/O helpers into <asm/fb.h> Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 12:34 ` Geert Uytterhoeven
2023-05-10 12:34 ` Geert Uytterhoeven
2023-05-10 12:34 ` Geert Uytterhoeven
2023-05-10 12:34 ` Geert Uytterhoeven
2023-05-10 14:20 ` Thomas Zimmermann
2023-05-10 14:20 ` Thomas Zimmermann
2023-05-10 14:20 ` Thomas Zimmermann
2023-05-10 14:20 ` Thomas Zimmermann
2023-05-10 14:34 ` Geert Uytterhoeven
2023-05-10 14:34 ` Geert Uytterhoeven
2023-05-10 14:34 ` Geert Uytterhoeven
2023-05-10 14:34 ` Geert Uytterhoeven
2023-05-10 15:11 ` Thomas Zimmermann
2023-05-10 15:11 ` Thomas Zimmermann
2023-05-10 15:11 ` Thomas Zimmermann
2023-05-10 15:11 ` Thomas Zimmermann
2023-05-10 14:03 ` kernel test robot
2023-05-10 14:03 ` kernel test robot
2023-05-10 14:03 ` kernel test robot
2023-05-10 14:03 ` kernel test robot
2023-05-10 14:15 ` Arnd Bergmann
2023-05-10 14:15 ` Arnd Bergmann
2023-05-10 14:15 ` Arnd Bergmann
2023-05-10 14:15 ` Arnd Bergmann
2023-05-10 14:27 ` Thomas Zimmermann
2023-05-10 14:27 ` Thomas Zimmermann
2023-05-10 14:27 ` Thomas Zimmermann
2023-05-10 14:27 ` Thomas Zimmermann
2023-05-10 15:54 ` Arnd Bergmann
2023-05-10 15:54 ` Arnd Bergmann
2023-05-10 15:54 ` Arnd Bergmann
2023-05-10 15:54 ` Arnd Bergmann
2023-05-11 10:53 ` Thomas Zimmermann
2023-05-11 10:53 ` Thomas Zimmermann
2023-05-11 10:53 ` Thomas Zimmermann
2023-05-11 12:35 ` Geert Uytterhoeven
2023-05-11 12:35 ` Geert Uytterhoeven
2023-05-11 12:35 ` Geert Uytterhoeven
2023-05-11 12:35 ` Geert Uytterhoeven
2023-05-11 12:48 ` Arnd Bergmann
2023-05-11 12:48 ` Arnd Bergmann
2023-05-11 12:48 ` Arnd Bergmann
2023-05-11 12:48 ` Arnd Bergmann
2023-05-11 13:22 ` Artur Rojek
2023-05-11 13:22 ` Artur Rojek
2023-05-11 13:22 ` Artur Rojek
2023-05-11 13:22 ` Artur Rojek
2023-05-11 13:40 ` Arnd Bergmann
2023-05-11 13:40 ` Arnd Bergmann
2023-05-11 13:40 ` Arnd Bergmann
2023-05-11 13:40 ` Arnd Bergmann
2023-05-11 13:12 ` Thomas Zimmermann
2023-05-11 13:12 ` Thomas Zimmermann
2023-05-11 13:12 ` Thomas Zimmermann
2023-05-10 11:05 ` [PATCH v6 6/6] fbdev: Rename fb_mem*() helpers Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
2023-05-10 11:05 ` Thomas Zimmermann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a2315b9a-0747-1f0f-1f0a-1c6773931db4@suse.de \
--to=tzimmermann@suse.de \
--cc=15330273260@189.cn \
--cc=James.Bottomley@HansenPartnership.com \
--cc=arnd@arndb.de \
--cc=chenhuacai@kernel.org \
--cc=daniel@ffwll.ch \
--cc=davem@davemloft.net \
--cc=deller@gmx.de \
--cc=dri-devel@lists.freedesktop.org \
--cc=geert@linux-m68k.org \
--cc=javierm@redhat.com \
--cc=kernel@xen0n.name \
--cc=linux-arch@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-ia64@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-m68k@lists.linux-m68k.org \
--cc=linux-parisc@vger.kernel.org \
--cc=linux-snps-arc@lists.infradead.org \
--cc=loongarch@lists.linux.dev \
--cc=sam@ravnborg.org \
--cc=sparclinux@vger.kernel.org \
--cc=vgupta@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.