RE: [PATCH] Documentation: block: cmdline-partition.txt fixes and additions

2018-05-06 Thread Caizhiyong
> -Original Message- > From: Randy Dunlap [mailto:rdun...@infradead.org] > Sent: Monday, May 07, 2018 2:50 AM > To: linux-bl...@vger.kernel.org; axboe > Cc: LKML ; Caizhiyong > ; Andrew Morton ; > linux-...@vger.kernel.org; Jonathan Corbet > Subject: [PATCH] Docum

RE: [PATCH v3] block: add verifier for cmdline partition

2018-05-06 Thread Caizhiyong
> /* > * Purpose: allocate cmdline partitions. > * Returns: > @@ -93,6 +158,7 @@ int cmdline_partition(struct parsed_partitions *state) > disk_size = get_capacity(state->bdev->bd_disk) << 9; > > cmdline_parts_set(parts, disk_size, 1, add_part, (void *)state); > + cmdline_parts

RE: [PATCH] block: add verifier for cmdline partition

2018-05-04 Thread Caizhiyong
> -Original Message- > From: Wang YanQing [mailto:udkni...@gmail.com] > Sent: Thursday, May 03, 2018 7:18 PM > To: ax...@kernel.dk > Cc: gre...@linuxfoundation.org; pombreda...@nexb.com; > t...@linutronix.de; Caizhiyong ; linux- > bl...@vger.kernel.org; linux-k

[PATCH] devfreq: fix double call put_device

2016-05-13 Thread Caizhiyong
From: Cai Zhiyong Date: Sat, 14 May 2016 14:13:30 +0800 Subject: [PATCH] devfreq: fix double call put_device 1295 */ 1296 void device_unregister(struct device *dev) 1297 { 1298 pr_debug("device: '%s': %s\n", dev_name(dev), __func__); 1299 device_del(dev); 1300 put_device(

[PATCH] devfreq: fix double call put_device

2016-05-13 Thread Caizhiyong
From: Cai Zhiyong Date: Sat, 14 May 2016 14:13:30 +0800 Subject: [PATCH] devfreq: fix double call put_device 1295 */ 1296 void device_unregister(struct device *dev) 1297 { 1298 pr_debug("device: '%s': %s\n", dev_name(dev), __func__); 1299 device_del(dev); 1300 put_device(

RE: [RESEND x2][PATCH v2] block: partition: Add partition specific uevent callbacks for partition info

2016-02-05 Thread Caizhiyong
> > Interestingly, this feature appears to already be documented in > > Documentation/block/cmdline-partition.txt. I wonder how that happened. > > Maybe we used to do this but it got taken away? This documentation talk about the partition name usages, my patch support get partition name from and

RE: [RESEND PATCH] block: cmdline-parser: add support for hidden disk space

2015-09-28 Thread Caizhiyong
y, September 28, 2015 5:51 PM > To: Jens Axboe > Cc: Caizhiyong; linux-kernel@vger.kernel.org; Shawn Lin > Subject: [RESEND PATCH] block: cmdline-parser: add support for hidden disk > space > > cmdline-partition now only support continuous disk space taken from > cmdline.

patch "[SCSI] Fix out of spec CD-ROM problem with media change" will cause usb disk resume fail.

2014-01-26 Thread Caizhiyong
>[SCSI] Fix out of spec CD-ROM problem with media change > >Some CD-ROMs fail to report a media change correctly. The specific >one for this patch simply fails to respond to commands, then gives a >UNIT ATTENTION after being reset which returns ASC/ASCQ 28/00. This >is out of

RE: [PATCH] mtd: nand: assign mtd->name in find_full_id_nand

2014-01-12 Thread Caizhiyong
> -Original Message- > From: Brian Norris [mailto:computersforpe...@gmail.com] > Sent: Sunday, January 12, 2014 4:10 AM > To: Huang Shijie; Caizhiyong > Cc: linux-...@lists.infradead.org; Wanglin (Albert); David Woodhouse; > linux-kernel@vger.kernel.org; Artem Bityut

[PATCH] mtd: nand: remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE

2013-12-27 Thread Caizhiyong
From: Cai Zhiyong Date: Fri, 27 Dec 2013 20:23:09 +0800 Subject: [PATCH] mtd: nand: remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE, dynamic allocate NAND buffer after detecting NAND writesize and oobsize. - save memory. when use MACRO allocate static

mtd: nand: remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE

2013-12-27 Thread Caizhiyong
From: Cai Zhiyong Date: Fri, 27 Dec 2013 20:23:09 +0800 Subject: [PATCH] mtd: nand: remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE remove NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE, dynamic allocate NAND buffer after detecting NAND writesize and oobsize. - save memory. when use MACRO allocate static

[PATCH] mtd: nand: assign mtd->name in find_full_id_nand

2013-12-25 Thread Caizhiyong
From: Cai Zhiyong Date: Wed, 25 Dec 2013 21:19:21 +0800 Subject: [PATCH] mtd: nand: assign mtd->name in find_full_id_nand This patch assigned the type->name to mtd->name when mtd->name is NULL in function "find_full_id_nand". mtd->name is NULL may cause some problem. Signed-off-by: Cai Zhiyong

[PATCH] mtd: nand: remove unused function input parameter

2013-12-25 Thread Caizhiyong
From: Cai Zhiyong Date: Wed, 25 Dec 2013 20:11:15 +0800 Subject: [PATCH] mtd: nand: remove unused function input parameter The nand_get_flash_type parameter "busw" input value is not used by any branch, and it is updated before use it in the function, so remove it, define the "busw" as an intern

RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
> -Original Message- > From: Huang Shijie [mailto:b32...@freescale.com] > Sent: Monday, December 16, 2013 11:07 AM > To: Caizhiyong > Cc: Huang Shijie; Brian Norris; David Woodhouse; Quyaxin; > linux-kernel@vger.kernel.org; linux-...@lists.infradead.org; Artem Bityutskiy;

RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-15 Thread Caizhiyong
> > > > There are a lot of 16K page size NAND, they are used in android system. > > > > Many NAND need read retry and data randomization, kernel does not support > > this > feature. > Brian is adding the read-retry feature. > > I think the data randomization should be done by the hardware. > If w

RE: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-13 Thread Caizhiyong
> On Fri, Dec 13, 2013 at 05:03:49AM +0000, Caizhiyong wrote: > > From: Cai Zhiyong > > Date: Fri, 13 Dec 2013 12:52:46 +0800 > > Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize > > NAND. > > > > The Toshiba's TC58TEG5DCJTA pag

[PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Caizhiyong
From: Cai Zhiyong Date: Fri, 13 Dec 2013 12:52:46 +0800 Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND. The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes. So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE. Signed-off-by: Cai Zhiyong --- inc

[PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND

2013-12-12 Thread Caizhiyong
From: Cai Zhiyong Date: Fri, 13 Dec 2013 12:52:46 +0800 Subject: [PATCH] mtd: increase max page/OOB size to support 16K pagesize NAND. The Toshiba's TC58TEG5DCJTA pagesize is 16K, oob size is 1280 bytes. So increase the NAND_MAX_OOBSIZE and NAND_MAX_PAGESIZE. Signed-off-by: Huang Shijie Signed-

[PATCH] regmap: Fix 'ret' would return an uninitialized value

2013-11-18 Thread Caizhiyong
From: Cai Zhiyong Date: Mon, 18 Nov 2013 20:21:49 +0800 Subject: [PATCH] regmap: Fix 'ret' would return an uninitialized value This patch give a warning when calling regmap_register_patch with parameter num_regs <= 0. When the num_regs parameter is zero and krealloc doesn't fail, then the code w

[PATCH] regmap: Fix 'ret' would return an uninitialized value

2013-11-15 Thread Caizhiyong
From: Cai Zhiyong Date: Mon, 11 Nov 2013 19:26:14 +0800 Subject: [PATCH] regmap: Fix 'ret' would return an uninitialized value This patch give a warning when calling regmap_register_patch with parameter num_regs <= 0. When the num_regs parameter is zero and krealloc doesn't fail, then the code w

[PATCH] block: cmdline-parser: perfect cmdline format checking

2013-11-12 Thread Caizhiyong
From: Cai Zhiyong Date: Tue, 12 Nov 2013 15:50:42 +0800 Subject: [PATCH] block: cmdline-parser: perfect cmdline format checking -Fix compile warning with value and function undeclared. this reported by and Randy Dunlap -perfect cmdline format checking, make the error information clear

[PATCH] regmap: Fix 'ret' would return an uninitialized value

2013-11-11 Thread Caizhiyong
From: Cai Zhiyong Date: Mon, 11 Nov 2013 19:26:14 +0800 Subject: [PATCH] regmap: Fix 'ret' would return an uninitialized value - Fix 'ret' would return an uninitialized value. - Add a warning avoid invalid 'num_regs' value passed in. When the num_regs parameter is zero and krealloc doesn't fai

RE: [PATCH 2/2] mtd: cmdlinepart: support master name is not set when parser partition

2013-11-10 Thread Caizhiyong
> On Sat, Nov 09, 2013 at 11:43:06AM +0000, Caizhiyong wrote: > > From: Cai Zhiyong > > Date: Sat, 9 Nov 2013 17:54:12 +0800 > > > > @@ -143,5 +149,5 @@ MODULE_PARM_DESC(mtdparts, "Partitioning > > specification"); > > module_param(

[PATCH] block: cmdline-parser: perfect cmdline format checking

2013-11-09 Thread Caizhiyong
From: Cai Zhiyong Date: Sat, 9 Nov 2013 19:27:38 +0800 Subject: [PATCH] block: cmdline-parser: perfect cmdline format checking -Fix compile warning with value and function undeclared. this reported by and Randy Dunlap -perfect cmdline format checking, make the error information clear f

[PATCH 2/2] mtd: cmdlinepart: support master name is not set when parser partition

2013-11-09 Thread Caizhiyong
From: Cai Zhiyong Date: Sat, 9 Nov 2013 17:54:12 +0800 Subject: [PATCH 2/2] mtd: cmdlinepart: support master name is not set when parser partition -Fix compile warning with value and function undeclared. this reported by and Randy Dunlap -support master name is not set when parser parti

[PATCH] regmap: Fix compile warning with value uninitialized

2013-11-09 Thread Caizhiyong
From: Cai Zhiyong Date: Sat, 9 Nov 2013 14:14:36 +0800 Subject: [PATCH] regmap: Fix compile warning with value uninitialized Fix compile warning with value uninitialized: drivers/base/regmap/regmap.c:2173: warning: 'ret' may be used uninitialized in this function Signed-off-by: Cai Zhiyong ---

RE: [PATCH 2/2] mtd: cmdlinepart: use cmdline partition parser lib

2013-11-08 Thread Caizhiyong
> Such results was never posted and unless we see those, I think > I'd rather NACK this patch instead. I like the cleanup, but only > if it's guaranteed to _not_ brake things, specially when dealing > with a kernel parameter. Do you have some test case or test standard for me perform. > -- > Ezeq

RE: [PATCH 2/2] mtd: cmdlinepart: use cmdline partition parser lib

2013-11-07 Thread Caizhiyong
> Nobody has had time to test this on MTD, it seems, and as such, I > strongly recommend you do not force it through -mm. We are perfectly > capable of merging it through the MTD tree if it ever gets proper > vetting by people in MTD (not just on block devices), and I am well > aware of this patch

RE: [PATCH 2/2] mtd: cmdlinepart: use cmdline partition parser lib

2013-11-07 Thread Caizhiyong
>> For further information, see "https://lkml.org/lkml/2013/8/6/550"; > > Thanks for doing this. Could we please get some acked-by's or, > preferably, tested-by's from the MTD people? Acked-by: Ezequiel Garcia Acked-by: Andrew Morton -- To unsubscribe from this list: send the line "unsubscrib

RE: [PATCH] mtd: cmdlinepart: use cmdline partition parser lib

2013-10-25 Thread Caizhiyong
> > I'd like to review the patch in detail and test it, but being a bit old, > the patch doesn't apply as it is on v3.12-rc5. Care to resend an update? > I have sent you a 2 patch base on v3.12-rc6. Are there any problems? I tested it for a long time and found no problems. Are there any plan

[PATCH 2/2] mtd: cmdlinepart: use cmdline partition parser lib

2013-10-22 Thread Caizhiyong
From: CaiZhiyong Subject: mtd: cmdlinepart: use cmdline partition parser lib In the previous version, adjust the cmdline parser code to library-style code, and move it to a separate file "block/cmdline-parser.c", we can use it in some client code. there is no any functionality cha

[PATCH 1/2] block: remove unrelated header files and export symbol

2013-10-22 Thread Caizhiyong
From: CaiZhiyong Subject: block: remove unrelated header files and export symbol This patch fix up the following items: - remove unrelated header files. - export interface function. - modify function cmdline_parts_parse return value, this will make it more friendly for the caller. Signed

RE: [PATCH] block: support embedded device command line partition

2013-09-17 Thread Caizhiyong
> I saw this patch appear in kernel v3.12-rc1, and it's nice since it's exactly > what we need. So needed that we proposed it in 2010: > http://marc.info/?l=linux-kernel&m=127425650923757&w=2 > http://marc.info/?l=linux-kernel&m=127599718024364&w=2 > > Is this patch inspired by Ulfs patch? (I ca

RE: [PATCH -next] block: fix error return code in parse_parts()

2013-08-22 Thread Caizhiyong
> -Original Message- > From: Wei Yongjun [mailto:weiyj...@gmail.com] > Sent: Friday, August 23, 2013 10:48 AM > To: ax...@kernel.dk; a...@linux-foundation.org; Caizhiyong; k...@redhat.com; > m...@sysgo.de; dw...@infradead.org; computersforpe...@gmail.com; > dedek...@

RE: [PATCH -next] block: fix error return code in parse_parts()

2013-08-22 Thread Caizhiyong
> From: Wei Yongjun [mailto:weiyj...@gmail.com] > Sent: Friday, August 23, 2013 10:48 AM > To: ax...@kernel.dk; a...@linux-foundation.org; Caizhiyong; k...@redhat.com; > m...@sysgo.de; dw...@infradead.org; computersforpe...@gmail.com; > dedek...@infradead.org > Cc: yongjun_...

RE: [PATCH] block: support embedded device command line partition

2013-08-19 Thread Caizhiyong
> On 08/15/2013 08:54 PM, Caizhiyong wrote: > >>> +blkdevparts=[;] > >>> + := :[,] > >>> + := [@](part-name) > >>> + > >>> + > >>> +block device disk name, embedded device used fixed block device, &g

RE: [PATCH] block: support embedded device command line partition

2013-08-15 Thread Caizhiyong
> > +blkdevparts=[;] > > + := :[,] > > + := [@](part-name) > > + > > + > > +block device disk name, embedded device used fixed block device, > > +it's disk name also fixed. such as: mmcblk0, mmcblk1, mmcblk0boot0. > > The device-name isn't always fixed. > > For example, what if ther

[PATCH] mtd: cmdlinepart: use cmdline partition parser lib

2013-08-15 Thread Caizhiyong
From: Cai Zhiyong MTD cmdline partition use cmdline partition parser lib reference: https://lkml.org/lkml/2013/8/6/550 Signed-off-by: Cai ZhiYong --- Documentation/block/cmdline-partition.txt | 93 +--- block/cmdline-parser.c| 16 +- drivers/mtd/Kconfig

[PATCH] mtd: cmdlinepart: use cmdline partition parser lib

2013-08-15 Thread Caizhiyong
From: Cai Zhiyong MTD cmdline partition use cmdline partition parser lib reference: https://lkml.org/lkml/2013/8/6/550 Signed-off-by: Cai ZhiYong --- Documentation/block/cmdline-partition.txt | 93 +--- block/cmdline-parser.c| 6 +- drivers/mtd/Kconfig

RE: [PATCH] block: add command line partition parser

2013-08-15 Thread Caizhiyong
> -Original Message- > From: Brian Norris [mailto:computersforpe...@gmail.com] > Sent: Thursday, August 15, 2013 3:10 PM > To: Caizhiyong > Cc: Andrew Morton; Karel Zak; linux-...@lists.infradead.org; > linux-kernel@vger.kernel.org; Wanglin (Albert); Artem Bityutskiy;

RE: [PATCH] block: add command line partition parser

2013-08-14 Thread Caizhiyong
> -Original Message- > From: Brian Norris [mailto:computersforpe...@gmail.com] > Sent: Thursday, August 15, 2013 1:00 PM > To: Caizhiyong > Cc: Andrew Morton; Karel Zak; linux-...@lists.infradead.org; > linux-kernel@vger.kernel.org; Wanglin (Albert); Artem Bityutskiy;

RE: [PATCH] block: add command line partition parser

2013-08-14 Thread Caizhiyong
> -Original Message- > From: Brian Norris [mailto:computersforpe...@gmail.com] > Sent: Thursday, August 15, 2013 8:12 AM > To: Andrew Morton > Cc: Caizhiyong; Karel Zak; linux-...@lists.infradead.org; > linux-kernel@vger.kernel.org; Wanglin (Albert); Artem Bityutskiy;

[PATCH] block: add command line partition parser

2013-08-12 Thread Caizhiyong
From: Cai Zhiyong move the command line parser to a separate module, and change it into library-style code. reference: https://lkml.org/lkml/2013/8/6/550 Signed-off-by: Cai Zhiyong --- block/Kconfig | 6 + block/Makefile | 1 + block/cmdline-parser.c

RE: [PATCH] block: support embedded device command line partition

2013-08-06 Thread Caizhiyong
> -Original Message- > From: Andrew Morton [mailto:a...@linux-foundation.org] > Sent: Tuesday, August 06, 2013 6:22 AM > To: Caizhiyong > Cc: Karel Zak; linux-kernel@vger.kernel.org; Wanglin (Albert); Quyaxin; Jens > Axboe; > David Woodhouse; Marius Groeger > Su

[PATCH] block: support embedded device command line partition

2013-08-03 Thread Caizhiyong
From: Cai Zhiyong Read block device partition table from command line. The partition used for fixed block device (eMMC) embedded device. It is no MBR, save storage space. Bootloader can be easily accessed by absolute address of data on the block device. Users can easily change the partition. Thi

RE: [PATCH] block: support embedded device command line partition

2013-07-31 Thread Caizhiyong
> From: Karel Zak [mailto:k...@redhat.com] > Sent: Wednesday, July 31, 2013 9:25 PM > To: Caizhiyong > Cc: Andrew Morton; linux-kernel@vger.kernel.org; Wanglin (Albert); Quyaxin > Subject: Re: [PATCH] block: support embedded device command line partition > > On Sat, Jul 2

[PATCH] block: support embedded device command line partition

2013-07-27 Thread Caizhiyong
From: Cai Zhiyong Read block device partition table from command line. This partition used for fixed block device (eMMC) embedded device. It no MBR, can save storage space. Bootloader can be easily accessed by absolute address of data on the block device. It support partition name, provides pa