* [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig @ 2022-07-20 7:22 Liu Song 2022-07-20 15:17 ` Randy Dunlap ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: Liu Song @ 2022-07-20 7:22 UTC (permalink / raw) To: bhelgaas; +Cc: linux-pci, linux-kernel [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain; charset=y, Size: 891 bytes --] From: Liu Song <liusong@linux.alibaba.com> Read the help information of PCI_P2PDMA through make menuconfig, "Enables" is partially displayed as garbled characters, so fix it. Signed-off-by: Liu Song <liusong@linux.alibaba.com> --- drivers/pci/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig index 133c732..8102b78 100644 --- a/drivers/pci/Kconfig +++ b/drivers/pci/Kconfig @@ -166,7 +166,7 @@ config PCI_P2PDMA depends on ZONE_DEVICE select GENERIC_ALLOCATOR help - Enableѕ drivers to do PCI peer-to-peer transactions to and from + Enables drivers to do PCI peer-to-peer transactions to and from BARs that are exposed in other devices that are the part of the hierarchy where peer-to-peer DMA is guaranteed by the PCI specification to work (ie. anything below a single PCI bridge). -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig 2022-07-20 7:22 [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig Liu Song @ 2022-07-20 15:17 ` Randy Dunlap 2022-07-20 17:23 ` Bjorn Helgaas 2023-10-06 19:36 ` Bjorn Helgaas 2 siblings, 0 replies; 6+ messages in thread From: Randy Dunlap @ 2022-07-20 15:17 UTC (permalink / raw) To: Liu Song, bhelgaas; +Cc: linux-pci, linux-kernel On 7/20/22 00:22, Liu Song wrote: > From: Liu Song <liusong@linux.alibaba.com> > > Read the help information of PCI_P2PDMA through make menuconfig, > "Enables" is partially displayed as garbled characters, so fix it. > > Signed-off-by: Liu Song <liusong@linux.alibaba.com> Acked-by: Randy Dunlap <rdunlap@infradead.org> Tested-by: Randy Dunlap <rdunlap@infradead.org> Thanks. > --- > drivers/pci/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig > index 133c732..8102b78 100644 > --- a/drivers/pci/Kconfig > +++ b/drivers/pci/Kconfig > @@ -166,7 +166,7 @@ config PCI_P2PDMA > depends on ZONE_DEVICE > select GENERIC_ALLOCATOR > help > - Enableѕ drivers to do PCI peer-to-peer transactions to and from > + Enables drivers to do PCI peer-to-peer transactions to and from > BARs that are exposed in other devices that are the part of > the hierarchy where peer-to-peer DMA is guaranteed by the PCI > specification to work (ie. anything below a single PCI bridge). -- ~Randy ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig 2022-07-20 7:22 [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig Liu Song 2022-07-20 15:17 ` Randy Dunlap @ 2022-07-20 17:23 ` Bjorn Helgaas 2022-08-01 21:56 ` Bjorn Helgaas 2023-10-06 19:36 ` Bjorn Helgaas 2 siblings, 1 reply; 6+ messages in thread From: Bjorn Helgaas @ 2022-07-20 17:23 UTC (permalink / raw) To: Liu Song; +Cc: bhelgaas, linux-pci, linux-kernel, Andy Whitcroft, Joe Perches [+cc Andy, Joe, possible checkpatch question] On Wed, Jul 20, 2022 at 03:22:03PM +0800, Liu Song wrote: > From: Liu Song <liusong@linux.alibaba.com> > > Read the help information of PCI_P2PDMA through make menuconfig, > "Enables" is partially displayed as garbled characters, so fix it. > > Signed-off-by: Liu Song <liusong@linux.alibaba.com> > --- > drivers/pci/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig > index 133c732..8102b78 100644 > --- a/drivers/pci/Kconfig > +++ b/drivers/pci/Kconfig > @@ -166,7 +166,7 @@ config PCI_P2PDMA > depends on ZONE_DEVICE > select GENERIC_ALLOCATOR > help > - Enableѕ drivers to do PCI peer-to-peer transactions to and from > + Enables drivers to do PCI peer-to-peer transactions to and from I see this problem ("Enables" renders as "Enable ~U" because the "s" is actually UTF-8 D195, CYRILLIC SMALL LETTER DZE). "file" found the following other Kconfig files that also contain UTF-8: drivers/pci/Kconfig D195 CYRILLIC SMALL LETTER DZE, which looks like "s" "Enables" renders as "Enable ~U" net/netfilter/ipvs/Kconfig C2A0 NO-BREAK SPACE renders fine drivers/mtd/nand/raw/Kconfig MTD_NAND_CAFE C389 LATIN CAPITAL LETTER E WITH ACUTE "CAFÉ" renders as "CAF ~I" drivers/mtd/spi-nor/Kconfig MTD_SPI_NOR_USE_4K_SECTORS "16 × 4 KiB" renders as "16 ~W 4 KiB" drivers/net/can/usb/Kconfig drivers/net/can/peak_canfd/Kconfig drivers/gpu/drm/panel/Kconfig drivers/platform/mellanox/Kconfig kernel/time/Kconfig crypto/Kconfig arch/Kconfig Some of these are clearly wrong (Cyrillic letter), some are unnecessary (non-breakable space), some are arguable ("CAFÉ" and "16 × 4 KiB" -- these take advantage of UTF-8 in useful ways). Not being a charset guru, I dunno if the rendering problem means my terminal is set incorrectly or if they all need to be changed. But I think we should address all of them at the same time. If we do need to avoid UTF-8 in Kconfig help, maybe checkpatch should look for it. > BARs that are exposed in other devices that are the part of > the hierarchy where peer-to-peer DMA is guaranteed by the PCI > specification to work (ie. anything below a single PCI bridge). > -- > 1.8.3.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig 2022-07-20 17:23 ` Bjorn Helgaas @ 2022-08-01 21:56 ` Bjorn Helgaas 2022-08-02 1:11 ` Liu Song 0 siblings, 1 reply; 6+ messages in thread From: Bjorn Helgaas @ 2022-08-01 21:56 UTC (permalink / raw) To: Liu Song; +Cc: bhelgaas, linux-pci, linux-kernel, Andy Whitcroft, Joe Perches On Wed, Jul 20, 2022 at 12:23:29PM -0500, Bjorn Helgaas wrote: > [+cc Andy, Joe, possible checkpatch question] > > On Wed, Jul 20, 2022 at 03:22:03PM +0800, Liu Song wrote: > > From: Liu Song <liusong@linux.alibaba.com> > > > > Read the help information of PCI_P2PDMA through make menuconfig, > > "Enables" is partially displayed as garbled characters, so fix it. > > > > Signed-off-by: Liu Song <liusong@linux.alibaba.com> > > --- > > drivers/pci/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig > > index 133c732..8102b78 100644 > > --- a/drivers/pci/Kconfig > > +++ b/drivers/pci/Kconfig > > @@ -166,7 +166,7 @@ config PCI_P2PDMA > > depends on ZONE_DEVICE > > select GENERIC_ALLOCATOR > > help > > - Enableѕ drivers to do PCI peer-to-peer transactions to and from > > + Enables drivers to do PCI peer-to-peer transactions to and from > > I see this problem ("Enables" renders as "Enable ~U" because the "s" > is actually UTF-8 D195, CYRILLIC SMALL LETTER DZE). > > "file" found the following other Kconfig files that also contain > UTF-8: > > drivers/pci/Kconfig > D195 CYRILLIC SMALL LETTER DZE, which looks like "s" > "Enables" renders as "Enable ~U" > net/netfilter/ipvs/Kconfig > C2A0 NO-BREAK SPACE > renders fine > drivers/mtd/nand/raw/Kconfig > MTD_NAND_CAFE C389 LATIN CAPITAL LETTER E WITH ACUTE > "CAFÉ" renders as "CAF ~I" > drivers/mtd/spi-nor/Kconfig > MTD_SPI_NOR_USE_4K_SECTORS > "16 × 4 KiB" renders as "16 ~W 4 KiB" > drivers/net/can/usb/Kconfig > drivers/net/can/peak_canfd/Kconfig > drivers/gpu/drm/panel/Kconfig > drivers/platform/mellanox/Kconfig > kernel/time/Kconfig > crypto/Kconfig > arch/Kconfig > > Some of these are clearly wrong (Cyrillic letter), some are > unnecessary (non-breakable space), some are arguable ("CAFÉ" and "16 × > 4 KiB" -- these take advantage of UTF-8 in useful ways). > > Not being a charset guru, I dunno if the rendering problem means > my terminal is set incorrectly or if they all need to be changed. > > But I think we should address all of them at the same time. If we do > need to avoid UTF-8 in Kconfig help, maybe checkpatch should look for > it. Any progress on this problem? I certainly agree that it should be fixed; I would just like to fix all the problems, not just the PCI one. Bjorn ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig 2022-08-01 21:56 ` Bjorn Helgaas @ 2022-08-02 1:11 ` Liu Song 0 siblings, 0 replies; 6+ messages in thread From: Liu Song @ 2022-08-02 1:11 UTC (permalink / raw) To: Bjorn Helgaas Cc: bhelgaas, linux-pci, linux-kernel, Andy Whitcroft, Joe Perches 在 2022/8/2 05:56, Bjorn Helgaas 写道: > On Wed, Jul 20, 2022 at 12:23:29PM -0500, Bjorn Helgaas wrote: >> [+cc Andy, Joe, possible checkpatch question] >> >> On Wed, Jul 20, 2022 at 03:22:03PM +0800, Liu Song wrote: >>> From: Liu Song <liusong@linux.alibaba.com> >>> >>> Read the help information of PCI_P2PDMA through make menuconfig, >>> "Enables" is partially displayed as garbled characters, so fix it. >>> >>> Signed-off-by: Liu Song <liusong@linux.alibaba.com> >>> --- >>> drivers/pci/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig >>> index 133c732..8102b78 100644 >>> --- a/drivers/pci/Kconfig >>> +++ b/drivers/pci/Kconfig >>> @@ -166,7 +166,7 @@ config PCI_P2PDMA >>> depends on ZONE_DEVICE >>> select GENERIC_ALLOCATOR >>> help >>> - Enableѕ drivers to do PCI peer-to-peer transactions to and from >>> + Enables drivers to do PCI peer-to-peer transactions to and from >> I see this problem ("Enables" renders as "Enable ~U" because the "s" >> is actually UTF-8 D195, CYRILLIC SMALL LETTER DZE). >> >> "file" found the following other Kconfig files that also contain >> UTF-8: >> >> drivers/pci/Kconfig >> D195 CYRILLIC SMALL LETTER DZE, which looks like "s" >> "Enables" renders as "Enable ~U" >> net/netfilter/ipvs/Kconfig >> C2A0 NO-BREAK SPACE >> renders fine >> drivers/mtd/nand/raw/Kconfig >> MTD_NAND_CAFE C389 LATIN CAPITAL LETTER E WITH ACUTE >> "CAFÉ" renders as "CAF ~I" >> drivers/mtd/spi-nor/Kconfig >> MTD_SPI_NOR_USE_4K_SECTORS >> "16 × 4 KiB" renders as "16 ~W 4 KiB" >> drivers/net/can/usb/Kconfig >> drivers/net/can/peak_canfd/Kconfig >> drivers/gpu/drm/panel/Kconfig >> drivers/platform/mellanox/Kconfig >> kernel/time/Kconfig >> crypto/Kconfig >> arch/Kconfig >> >> Some of these are clearly wrong (Cyrillic letter), some are >> unnecessary (non-breakable space), some are arguable ("CAFÉ" and "16 × >> 4 KiB" -- these take advantage of UTF-8 in useful ways). >> >> Not being a charset guru, I dunno if the rendering problem means >> my terminal is set incorrectly or if they all need to be changed. >> >> But I think we should address all of them at the same time. If we do >> need to avoid UTF-8 in Kconfig help, maybe checkpatch should look for >> it. > Any progress on this problem? I certainly agree that it should be > fixed; I would just like to fix all the problems, not just the PCI > one. > > Bjorn Totally agree, according to your suggestion, I will resubmit with all the revisions. Thanks ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig 2022-07-20 7:22 [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig Liu Song 2022-07-20 15:17 ` Randy Dunlap 2022-07-20 17:23 ` Bjorn Helgaas @ 2023-10-06 19:36 ` Bjorn Helgaas 2 siblings, 0 replies; 6+ messages in thread From: Bjorn Helgaas @ 2023-10-06 19:36 UTC (permalink / raw) To: Liu Song; +Cc: bhelgaas, linux-pci, linux-kernel, Uwe Kleine-König On Wed, Jul 20, 2022 at 03:22:03PM +0800, Liu Song wrote: > From: Liu Song <liusong@linux.alibaba.com> > > Read the help information of PCI_P2PDMA through make menuconfig, > "Enables" is partially displayed as garbled characters, so fix it. > > Signed-off-by: Liu Song <liusong@linux.alibaba.com> I think I derailed this by suggesting that we fix other Kconfig files at the same time, which ended up with a cross-subsystem patch that nobody picked up. My fault, sorry. So I applied this now to pci/misc for v6.7, thank you! Uwe just posted an identical patch, so I updated the commit log to mention his patch as well: PCI: Replace unnecessary UTF-8 in Kconfig The CONFIG_PCI_P2PDMA Kconfig help text contains a Cyrillic small "Dze" (ѕ). When menuconfig renders it, it looks like "Enable ~U drivers" instead of "Enables drivers". Replace it by a plain "s" so the help text is displayed correctly by menuconfig. Uwe Kleine-König <u.kleine-koenig@pengutronix.de> later posted the same patch at https://lore.kernel.org/r/20231006150209.87666-1-u.kleine-koenig@pengutronix.de > --- > drivers/pci/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig > index 133c732..8102b78 100644 > --- a/drivers/pci/Kconfig > +++ b/drivers/pci/Kconfig > @@ -166,7 +166,7 @@ config PCI_P2PDMA > depends on ZONE_DEVICE > select GENERIC_ALLOCATOR > help > - Enableѕ drivers to do PCI peer-to-peer transactions to and from > + Enables drivers to do PCI peer-to-peer transactions to and from > BARs that are exposed in other devices that are the part of > the hierarchy where peer-to-peer DMA is guaranteed by the PCI > specification to work (ie. anything below a single PCI bridge). > -- > 1.8.3.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-10-06 19:36 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-07-20 7:22 [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI_P2PDMA" under menuconfig Liu Song 2022-07-20 15:17 ` Randy Dunlap 2022-07-20 17:23 ` Bjorn Helgaas 2022-08-01 21:56 ` Bjorn Helgaas 2022-08-02 1:11 ` Liu Song 2023-10-06 19:36 ` Bjorn Helgaas
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.