[PATCH] patman: Resolve python string vs. regex escaping syntax

2024-07-26 Thread Brian Norris
ect_tag = re.compile('([^:\s]*):\s*(.*)') We should use a raw string (r'...') so that all escaping is passed into the regex module, not interpreted within the string itself. Signed-off-by: Brian Norris --- tools/patman/commit.py | 2 +- tools/patman/patchstream.py | 2

Re: [PATCH 4/6] patman: Make most bool arguments BooleanOptionalAction

2022-06-30 Thread Brian Norris
action=BooleanOptionalAction, >default=False, >help='Ignore bad tags / aliases (default=warn)') I know you mentioned --help ugliness, but this one ends up looking like: (default=warn) (default: False) Perhaps we should drop the baked-in "(default=warn)" text? Otherwise: Reviewed-by: Brian Norris Tested-by: Brian Norris

Re: [PATCH 6/6] patman: Take project defaults into account for --help

2022-06-30 Thread Brian Norris
On Thu, Jun 30, 2022 at 02:08:09PM -0700, Doug Anderson wrote: > I'd like it so that when you do "patman send --help" and you're using > Linux that it show it the proper defaults for Linux. > > Signed-off-by: Douglas Anderson > --- > > tools/patman/main.py | 13 + > 1 file changed,

Re: [PATCH 0/6] patman: Small fixes plus remove --no-tree from checkpatch for linux

2022-06-30 Thread Brian Norris
go used to using --no-tags. Glad to see you're fixing it. For the whole series: Tested-by: Brian Norris As noted, my brain isn't working so well on patch 6. But for patches 1-5 (modulo a small nit on 4): Reviewed-by: Brian Norris

Re: [U-Boot] [PATCH v1 0/2] mtd: nand: omap: booting from NAND using u-boot

2014-01-25 Thread Brian Norris
Hi Pekon, Sorry, I'm revisiting your patch series a bit late. There are a few factors that contributed to this, though. 1. This patch series talks extensively about U-Boot. U-Boot is not my interest, nor should it be the focus of kernel (driver) development. Any work done here should be fra

Re: [U-Boot] [PATCH v1 2/2] mtd: nand: omap: fix ecclayout to be in sync with u-boot NAND driver

2014-01-25 Thread Brian Norris
On Fri, Dec 13, 2013 at 02:42:58PM +0530, Pekon Gupta wrote: > @@ -1851,7 +1854,13 @@ static int omap_nand_probe(struct platform_device > *pdev) > ecclayout->eccbytes = nand_chip->ecc.bytes * > (mtd->writesize / >

Re: [U-Boot] [PATCH v1 1/2] mtd: nand: omap: fix ecclayout->oobfree->offset

2014-01-25 Thread Brian Norris
On Fri, Dec 13, 2013 at 02:42:57PM +0530, Pekon Gupta wrote: > This patch updates starting offset for free bytes in OOB which can be used by > file-systems to store their metadata (like clean-marker in case of JFFS2). This should be describing a regression fix, right? We don't just arbitrarily cha

Re: [U-Boot] [PATCH v1 0/2] mtd: nand: omap: booting from NAND using u-boot

2014-01-27 Thread Brian Norris
On Mon, Jan 27, 2014 at 9:46 AM, Gupta, Pekon wrote: >>From: Brian Norris >> >>1. This patch series talks extensively about U-Boot. U-Boot is not my >> interest, nor should it be the focus of kernel (driver) development. >> Any work done here should be framed in

Re: [U-Boot] NAND bad block table

2014-02-11 Thread Brian Norris
[BTW, I'm not a U-Boot subscriber, so this cross-post probably won't go through] Hi Scott, On Tue, Feb 11, 2014 at 4:06 PM, Scott Wood wrote: > On Mon, 2014-02-10 at 15:05 +, Naveen Mamindlapalli wrote: >> Yes, the searching of BBT happens from the end of flash when >> NAND_BBT_USE_FLASH opt

Re: [U-Boot] [PATCH v1 0/2] mtd: nand: omap: booting from NAND using u-boot

2014-02-11 Thread Brian Norris
Hi Pekon, On Tue, Jan 28, 2014 at 07:42:09AM +, Pekon Gupta wrote: > >From: Brian Norris > > > >On Fri, Dec 13, 2013 at 02:42:56PM +0530, Pekon Gupta wrote: > >> As there were parallel set of patches running between u-boot and kernel. > > > >I don&#

Re: [U-Boot] [PATCH v2 1/3] mtd: nand: omap: fix ecclayout->oobfree->offset

2014-02-14 Thread Brian Norris
Hi Pekon, On Fri, Feb 14, 2014 at 10:31:46PM +0530, Pekon Gupta wrote: > 1) In current implementation, ecclayout->oobfree->offset is calculated with > respect to ecclayout->eccpos[0] which is incorrect because ECC bytes may not > be stored contiguously in OOB. > So, this patch calculates ecclay

Re: [U-Boot] [PATCH v3 1/3] mtd: nand: omap: fix ecclayout to be in sync with u-boot NAND driver

2014-02-23 Thread Brian Norris
On Mon, Feb 17, 2014 at 10:11:52AM +0100, Albert ARIBAUD wrote: > On Mon, 17 Feb 2014 13:11:23 +0530, Pekon Gupta wrote: > > Fixes: commit a919e51161b58ed7e6e663daba99ab7d558808f3 > >mtd: nand: omap2: clean-up BCHx_HW and BCHx_SW ECC configurations in > > device_probe > > > > Fixes eccla

Re: [U-Boot] [PATCH v3 0/3] mtd: nand: omap: fix ecc-layout

2014-02-23 Thread Brian Norris
calculation + adjust for reserved-marker of > last sector > [PATCH 2/3] > [PATCH 3/3] refactor code as suggested by Brian Norris > > > > *original v1* > [PATCH 1/2] > http://lists.infradead.org/pipermail/linux-mtd/2013-December/050946.html > [PATCH 2/2] >

Re: [U-Boot] A way to change the status of the whole series on Patchwork ?

2014-07-21 Thread Brian Norris
On Tue, Jul 22, 2014 at 11:37:26AM +0900, Masahiro Yamada wrote: > On Wed, 16 Jul 2014 19:49:09 -0700 > Brian Norris wrote: > > On Thu, Jul 17, 2014 at 11:19:42AM +0900, Masahiro Yamada wrote: > > > Does Patchwork have any good way to change the status > > > of the

Re: [U-Boot] A way to change the status of the whole series on Patchwork ?

2014-07-22 Thread Brian Norris
On Tue, Jul 22, 2014 at 09:50:51AM -0400, Tom Rini wrote: > On Wed, Jul 16, 2014 at 07:49:09PM -0700, Brian Norris wrote: > > On Thu, Jul 17, 2014 at 11:19:42AM +0900, Masahiro Yamada wrote: > > > Does Patchwork have any good way to change the status > > > of the whole s

Re: [U-Boot] A way to change the status of the whole series on Patchwork ?

2014-07-23 Thread Brian Norris
On Wed, Jul 23, 2014 at 09:10:08AM +0100, Ian Campbell wrote: > On Tue, 2014-07-22 at 12:45 -0700, Brian Norris wrote: > > > Then some sort of grepping/regex/etc. can get you the patchwork ID from > > the Message-ID. e.g.: > > > > pwclient list -m "$MESSAGEI

Re: [U-Boot] A way to change the status of the whole series on Patchwork ?

2014-07-16 Thread Brian Norris
On Thu, Jul 17, 2014 at 11:19:42AM +0900, Masahiro Yamada wrote: > When I change the status of patches, > I always do it one by one. > > But it is painful if the series consists > of a bunch of patches. > > Does Patchwork have any good way to change the status > of the whole series? You can use

Re: [U-Boot] [PATCH v2 05/10] video: add MIPI DSI host controller bridge

2018-03-03 Thread Brian Norris
Hi, On Fri, Mar 02, 2018 at 04:44:06PM +0100, yannick fertre wrote: > Add a Synopsys Designware MIPI DSI host bridge driver, based on the > Rockchip version from rockchip/dw-mipi-dsi.c with phy & bridge APIs. How many times are we going to allow copy-and-pasting the same driver? Last time we want

Re: [U-Boot] [PATCH v2 05/10] video: add MIPI DSI host controller bridge

2018-03-03 Thread Brian Norris
On Fri, Mar 02, 2018 at 10:57:59AM -0800, Brian Norris wrote: > Hi, > > On Fri, Mar 02, 2018 at 04:44:06PM +0100, yannick fertre wrote: > > Add a Synopsys Designware MIPI DSI host bridge driver, based on the > > Rockchip version from rockchip/dw-mipi-dsi.c with phy & b

Re: [U-Boot] [PATCH v3 00/10] splash screen on the stm32f769 disco board

2018-03-13 Thread Brian Norris
On Tue, Mar 13, 2018 at 1:50 PM, Anatolij Gustschin wrote: > On Tue, 13 Mar 2018 16:23:10 +0100 > Daniel Vetter dan...@ffwll.ch wrote: > ... >> Shouldn't we patch the drivers/gpu/drm/stm driver instead of the >> drivers/video one? fbdev is kinda a dead end and not for adding new hw >> support ...

[PATCH] patman: Support absolute and ~user-relative alias files

2022-01-08 Thread Brian Norris
following: Warning: Cannot find alias file '/path/to/source/tree/~/.git-email' Seen when git-config is like: $ git config sendemail.aliasesfile ~/.git-email Signed-off-by: Brian Norris --- tools/patman/gitutil.py | 11 --- 1 file changed, 8 insertions(+), 3 deleti