* [PATCH 0/3] Styleguide conformity
@ 2012-06-07 20:20 Sam Hansen
2012-06-07 20:20 ` [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
` (6 more replies)
0 siblings, 7 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 20:20 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
From: Sam Hansen <sam@meebo-inc.com>
This patch set addresses basic checkpatch.pl issues.
Sam Hansen (3):
staging: zram: conventions pr_warning -> pr_warn()
staging: zram: conventions, __aligned() attribute
staging: zram: conventions, line splitting
drivers/staging/zram/zram_drv.c | 14 ++++++--------
drivers/staging/zram/zram_drv.h | 2 +-
2 files changed, 7 insertions(+), 9 deletions(-)
--
1.7.5.4
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn()
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
@ 2012-06-07 20:20 ` Sam Hansen
2012-06-08 7:34 ` Nitin Gupta
2012-06-07 20:20 ` [PATCH 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
` (5 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 20:20 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Porting zram to use the pr_warn() function instead of the deprecated
pr_warning().
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
index 685d612..55775ce 100644
--- a/drivers/staging/zram/zram_drv.c
+++ b/drivers/staging/zram/zram_drv.c
@@ -724,7 +724,7 @@ static int create_device(struct zram *zram, int device_id)
zram->disk = alloc_disk(1);
if (!zram->disk) {
blk_cleanup_queue(zram->queue);
- pr_warning("Error allocating disk structure for device %d\n",
+ pr_warn("Error allocating disk structure for device %d\n",
device_id);
ret = -ENOMEM;
goto out;
@@ -755,7 +755,7 @@ static int create_device(struct zram *zram, int device_id)
ret = sysfs_create_group(&disk_to_dev(zram->disk)->kobj,
&zram_disk_attr_group);
if (ret < 0) {
- pr_warning("Error creating sysfs group");
+ pr_warn("Error creating sysfs group");
goto out;
}
@@ -789,7 +789,7 @@ static int __init zram_init(void)
int ret, dev_id;
if (num_devices > max_num_devices) {
- pr_warning("Invalid value for num_devices: %u\n",
+ pr_warn("Invalid value for num_devices: %u\n",
num_devices);
ret = -EINVAL;
goto out;
@@ -797,7 +797,7 @@ static int __init zram_init(void)
zram_major = register_blkdev(0, "zram");
if (zram_major <= 0) {
- pr_warning("Unable to get major number\n");
+ pr_warn("Unable to get major number\n");
ret = -EBUSY;
goto out;
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 2/3] staging: zram: conventions, __aligned() attribute
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
2012-06-07 20:20 ` [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
@ 2012-06-07 20:20 ` Sam Hansen
2012-06-08 7:37 ` Nitin Gupta
2012-06-07 20:20 ` [PATCH 3/3] staging: zram: conventions, line splitting Sam Hansen
` (4 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 20:20 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Using the __aligned() attribute in favor of __attribute__((aligned(size)))
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.h b/drivers/staging/zram/zram_drv.h
index fbe8ac9..cda89ec 100644
--- a/drivers/staging/zram/zram_drv.h
+++ b/drivers/staging/zram/zram_drv.h
@@ -85,7 +85,7 @@ struct table {
u16 size; /* object size (excluding header) */
u8 count; /* object ref count (not yet used) */
u8 flags;
-} __attribute__((aligned(4)));
+} __aligned(4);
struct zram_stats {
u64 compr_size; /* compressed size of pages stored */
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 3/3] staging: zram: conventions, line splitting
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
2012-06-07 20:20 ` [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
2012-06-07 20:20 ` [PATCH 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
@ 2012-06-07 20:20 ` Sam Hansen
2012-06-07 20:28 ` Joe Perches
2012-06-07 23:03 ` [PATCH V2 0/3] Styleguide conformity Sam Hansen
` (3 subsequent siblings)
6 siblings, 1 reply; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 20:20 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Opting to violate the 80-char limit in favor of not splitting strings onto
multiple lines (via pr_info()). This mostly to help readability.
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.c | 6 ++----
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
index 55775ce..8791a09 100644
--- a/drivers/staging/zram/zram_drv.c
+++ b/drivers/staging/zram/zram_drv.c
@@ -390,8 +390,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
clen = PAGE_SIZE;
page_store = alloc_page(GFP_NOIO | __GFP_HIGHMEM);
if (unlikely(!page_store)) {
- pr_info("Error allocating memory for "
- "incompressible page: %u\n", index);
+ pr_info("Error allocating memory for incompressible page: %u\n", index);
ret = -ENOMEM;
goto out;
}
@@ -407,8 +406,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
handle = zs_malloc(zram->mem_pool, clen + sizeof(*zheader));
if (!handle) {
- pr_info("Error allocating memory for compressed "
- "page: %u, size=%zu\n", index, clen);
+ pr_info("Error allocating memory for compressed page: %u, size=%zu\n", index, clen);
ret = -ENOMEM;
goto out;
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] staging: zram: conventions, line splitting
2012-06-07 20:20 ` [PATCH 3/3] staging: zram: conventions, line splitting Sam Hansen
@ 2012-06-07 20:28 ` Joe Perches
2012-06-07 20:30 ` sam hansen
0 siblings, 1 reply; 17+ messages in thread
From: Joe Perches @ 2012-06-07 20:28 UTC (permalink / raw)
To: Sam Hansen
Cc: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Seth Jennings, devel, linux-kernel
On Thu, 2012-06-07 at 13:20 -0700, Sam Hansen wrote:
> Opting to violate the 80-char limit in favor of not splitting strings onto
> multiple lines (via pr_info()). This mostly to help readability.
[]
> diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
[]
> @@ -390,8 +390,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
> clen = PAGE_SIZE;
> page_store = alloc_page(GFP_NOIO | __GFP_HIGHMEM);
> if (unlikely(!page_store)) {
> - pr_info("Error allocating memory for "
> - "incompressible page: %u\n", index);
> + pr_info("Error allocating memory for incompressible page: %u\n", index);
Hi Sam.
The general use has been to allow the format string to exceed
80 columns, but not any additional arguments on the same line.
i.e. it's more conformant to use:
pr_info("Error allocating memory for incompressible page: %u\n",
index);
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] staging: zram: conventions, line splitting
2012-06-07 20:28 ` Joe Perches
@ 2012-06-07 20:30 ` sam hansen
0 siblings, 0 replies; 17+ messages in thread
From: sam hansen @ 2012-06-07 20:30 UTC (permalink / raw)
To: Joe Perches
Cc: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Seth Jennings, devel, linux-kernel
> The general use has been to allow the format string to exceed
> 80 columns, but not any additional arguments on the same line.
Oh cool. Expect a follow up patch set later on.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH V2 0/3] Styleguide conformity
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
` (2 preceding siblings ...)
2012-06-07 20:20 ` [PATCH 3/3] staging: zram: conventions, line splitting Sam Hansen
@ 2012-06-07 23:03 ` Sam Hansen
2012-06-07 23:03 ` [PATCH V2 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
` (2 subsequent siblings)
6 siblings, 0 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 23:03 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
From: Sam Hansen <sam@meebo-inc.com>
This patch set addresses basic checkpatch.pl issues.
Sam Hansen (3):
staging: zram: conventions pr_warning -> pr_warn()
staging: zram: conventions, __aligned() attribute
staging: zram: conventions, line splitting
drivers/staging/zram/zram_drv.c | 16 ++++++++--------
drivers/staging/zram/zram_drv.h | 2 +-
2 files changed, 9 insertions(+), 9 deletions(-)
V2 - Moving arguments for pr_info() to the next line.
--
1.7.5.4
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH V2 1/3] staging: zram: conventions pr_warning -> pr_warn()
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
` (3 preceding siblings ...)
2012-06-07 23:03 ` [PATCH V2 0/3] Styleguide conformity Sam Hansen
@ 2012-06-07 23:03 ` Sam Hansen
2012-06-07 23:03 ` [PATCH V2 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
2012-06-07 23:03 ` [PATCH V2 3/3] staging: zram: conventions, line splitting Sam Hansen
6 siblings, 0 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 23:03 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Porting zram to use the pr_warn() function instead of the deprecated
pr_warning().
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
index 685d612..55775ce 100644
--- a/drivers/staging/zram/zram_drv.c
+++ b/drivers/staging/zram/zram_drv.c
@@ -724,7 +724,7 @@ static int create_device(struct zram *zram, int device_id)
zram->disk = alloc_disk(1);
if (!zram->disk) {
blk_cleanup_queue(zram->queue);
- pr_warning("Error allocating disk structure for device %d\n",
+ pr_warn("Error allocating disk structure for device %d\n",
device_id);
ret = -ENOMEM;
goto out;
@@ -755,7 +755,7 @@ static int create_device(struct zram *zram, int device_id)
ret = sysfs_create_group(&disk_to_dev(zram->disk)->kobj,
&zram_disk_attr_group);
if (ret < 0) {
- pr_warning("Error creating sysfs group");
+ pr_warn("Error creating sysfs group");
goto out;
}
@@ -789,7 +789,7 @@ static int __init zram_init(void)
int ret, dev_id;
if (num_devices > max_num_devices) {
- pr_warning("Invalid value for num_devices: %u\n",
+ pr_warn("Invalid value for num_devices: %u\n",
num_devices);
ret = -EINVAL;
goto out;
@@ -797,7 +797,7 @@ static int __init zram_init(void)
zram_major = register_blkdev(0, "zram");
if (zram_major <= 0) {
- pr_warning("Unable to get major number\n");
+ pr_warn("Unable to get major number\n");
ret = -EBUSY;
goto out;
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH V2 2/3] staging: zram: conventions, __aligned() attribute
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
` (4 preceding siblings ...)
2012-06-07 23:03 ` [PATCH V2 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
@ 2012-06-07 23:03 ` Sam Hansen
2012-06-07 23:03 ` [PATCH V2 3/3] staging: zram: conventions, line splitting Sam Hansen
6 siblings, 0 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 23:03 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Using the __aligned() attribute in favor of __attribute__((aligned(size)))
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.h b/drivers/staging/zram/zram_drv.h
index fbe8ac9..cda89ec 100644
--- a/drivers/staging/zram/zram_drv.h
+++ b/drivers/staging/zram/zram_drv.h
@@ -85,7 +85,7 @@ struct table {
u16 size; /* object size (excluding header) */
u8 count; /* object ref count (not yet used) */
u8 flags;
-} __attribute__((aligned(4)));
+} __aligned(4);
struct zram_stats {
u64 compr_size; /* compressed size of pages stored */
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
` (5 preceding siblings ...)
2012-06-07 23:03 ` [PATCH V2 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
@ 2012-06-07 23:03 ` Sam Hansen
2012-06-08 7:42 ` Nitin Gupta
2012-06-11 16:04 ` Greg Kroah-Hartman
6 siblings, 2 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-07 23:03 UTC (permalink / raw)
To: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Nitin Gupta,
Sam Hansen, Seth Jennings, devel, linux-kernel
Opting to violate the 80-char limit in favor of not splitting strings onto
multiple lines (via pr_info()). This mostly to help readability.
Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
---
drivers/staging/zram/zram_drv.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c
index 55775ce..6292045 100644
--- a/drivers/staging/zram/zram_drv.c
+++ b/drivers/staging/zram/zram_drv.c
@@ -390,8 +390,8 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
clen = PAGE_SIZE;
page_store = alloc_page(GFP_NOIO | __GFP_HIGHMEM);
if (unlikely(!page_store)) {
- pr_info("Error allocating memory for "
- "incompressible page: %u\n", index);
+ pr_info("Error allocating memory for incompressible page: %u\n",
+ index);
ret = -ENOMEM;
goto out;
}
@@ -407,8 +407,8 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index,
handle = zs_malloc(zram->mem_pool, clen + sizeof(*zheader));
if (!handle) {
- pr_info("Error allocating memory for compressed "
- "page: %u, size=%zu\n", index, clen);
+ pr_info("Error allocating memory for compressed page: %u, size=%zu\n",
+ index, clen);
ret = -ENOMEM;
goto out;
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn()
2012-06-07 20:20 ` [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
@ 2012-06-08 7:34 ` Nitin Gupta
0 siblings, 0 replies; 17+ messages in thread
From: Nitin Gupta @ 2012-06-08 7:34 UTC (permalink / raw)
To: Sam Hansen
Cc: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Seth Jennings,
devel, linux-kernel
On 06/07/2012 01:20 PM, Sam Hansen wrote:
> Porting zram to use the pr_warn() function instead of the deprecated
> pr_warning().
>
> Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
> ---
> drivers/staging/zram/zram_drv.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
Acked-by: Nitin Gupta <ngupta@vflare.org>
Thanks,
Nitin
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 2/3] staging: zram: conventions, __aligned() attribute
2012-06-07 20:20 ` [PATCH 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
@ 2012-06-08 7:37 ` Nitin Gupta
0 siblings, 0 replies; 17+ messages in thread
From: Nitin Gupta @ 2012-06-08 7:37 UTC (permalink / raw)
To: Sam Hansen
Cc: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Seth Jennings,
devel, linux-kernel
On 06/07/2012 01:20 PM, Sam Hansen wrote:
> Using the __aligned() attribute in favor of __attribute__((aligned(size)))
>
> Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
> ---
> drivers/staging/zram/zram_drv.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/staging/zram/zram_drv.h b/drivers/staging/zram/zram_drv.h
> index fbe8ac9..cda89ec 100644
> --- a/drivers/staging/zram/zram_drv.h
> +++ b/drivers/staging/zram/zram_drv.h
> @@ -85,7 +85,7 @@ struct table {
> u16 size; /* object size (excluding header) */
> u8 count; /* object ref count (not yet used) */
> u8 flags;
> -} __attribute__((aligned(4)));
> +} __aligned(4);
>
> struct zram_stats {
> u64 compr_size; /* compressed size of pages stored */
I plan to get rid of this table soon (as part of zsmalloc compaction
work). So, we can skip any changes to this part of the code.
Thanks,
Nitin
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-07 23:03 ` [PATCH V2 3/3] staging: zram: conventions, line splitting Sam Hansen
@ 2012-06-08 7:42 ` Nitin Gupta
2012-06-11 16:04 ` Greg Kroah-Hartman
1 sibling, 0 replies; 17+ messages in thread
From: Nitin Gupta @ 2012-06-08 7:42 UTC (permalink / raw)
To: Sam Hansen
Cc: Sam Hansen, Greg Kroah-Hartman, Jerome Marchand, Seth Jennings,
devel, linux-kernel
On 06/07/2012 04:03 PM, Sam Hansen wrote:
> Opting to violate the 80-char limit in favor of not splitting strings onto
> multiple lines (via pr_info()). This mostly to help readability.
>
> Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
> ---
> drivers/staging/zram/zram_drv.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
Acked-by: Nitin Gupta <ngupta@vflare.org>
Thanks,
Nitin
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-07 23:03 ` [PATCH V2 3/3] staging: zram: conventions, line splitting Sam Hansen
2012-06-08 7:42 ` Nitin Gupta
@ 2012-06-11 16:04 ` Greg Kroah-Hartman
2012-06-11 22:08 ` Sam Hansen
1 sibling, 1 reply; 17+ messages in thread
From: Greg Kroah-Hartman @ 2012-06-11 16:04 UTC (permalink / raw)
To: Sam Hansen
Cc: Sam Hansen, Jerome Marchand, Nitin Gupta, Seth Jennings, devel,
linux-kernel
On Thu, Jun 07, 2012 at 04:03:49PM -0700, Sam Hansen wrote:
> Opting to violate the 80-char limit in favor of not splitting strings onto
> multiple lines (via pr_info()). This mostly to help readability.
>
> Signed-off-by: Sam Hansen <solid.se7en@gmail.com>
> Acked-by: Nitin Gupta <ngupta@vflare.org>
> ---
> drivers/staging/zram/zram_drv.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
This patch no longer applies to the tree :(
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-11 16:04 ` Greg Kroah-Hartman
@ 2012-06-11 22:08 ` Sam Hansen
2012-06-11 22:26 ` Greg Kroah-Hartman
0 siblings, 1 reply; 17+ messages in thread
From: Sam Hansen @ 2012-06-11 22:08 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Sam Hansen, Jerome Marchand, Nitin Gupta, Seth Jennings, devel,
linux-kernel
> drivers/staging/zram/zram_drv.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> This patch no longer applies to the tree :(
Hi Greg,
I wrote the patch against Linus' tree
<http://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git>.
Is it more appropriate to write code against your staging tree
<http://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git>?
Have I been writing code against the wrong tree? If so, I'm a huge
tool and sorry for the confusion :p
I'm also doing a huge cleanup of the wan-pmc drivers (cxt1e1) and will
port over to your repo if necessary.
Thanks!
-sam
<solid.se7en@gmail.com>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-11 22:08 ` Sam Hansen
@ 2012-06-11 22:26 ` Greg Kroah-Hartman
2012-06-11 22:29 ` Sam Hansen
0 siblings, 1 reply; 17+ messages in thread
From: Greg Kroah-Hartman @ 2012-06-11 22:26 UTC (permalink / raw)
To: Sam Hansen
Cc: devel, Seth Jennings, Jerome Marchand, linux-kernel, Nitin Gupta
On Mon, Jun 11, 2012 at 03:08:13PM -0700, Sam Hansen wrote:
> > drivers/staging/zram/zram_drv.c | 8 ++++----
> > 1 files changed, 4 insertions(+), 4 deletions(-)
> >
> > This patch no longer applies to the tree :(
>
> Hi Greg,
> I wrote the patch against Linus' tree
> <http://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git>.
> Is it more appropriate to write code against your staging tree
> <http://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git>?
>
> Have I been writing code against the wrong tree? If so, I'm a huge
> tool and sorry for the confusion :p
Problem is, other zram changes just went into my tree, so they probably
conflicted with your changes as well.
So yes, it's best to work against my staging-next branch for coding
style cleanups, or the linux-next tree, either should be fine.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2 3/3] staging: zram: conventions, line splitting
2012-06-11 22:26 ` Greg Kroah-Hartman
@ 2012-06-11 22:29 ` Sam Hansen
0 siblings, 0 replies; 17+ messages in thread
From: Sam Hansen @ 2012-06-11 22:29 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: devel, Seth Jennings, Jerome Marchand, linux-kernel, Nitin Gupta
> Problem is, other zram changes just went into my tree, so they probably
> conflicted with your changes as well.
Ok cool. I was assuming me and someone else simply clobbered each
others work. I try and keep an eye on the mailing lists to avoid
duplicating work and causing a merge conflict. Thanks for the
feedback, it's appreciated!
-sam
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2012-06-11 22:29 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-07 20:20 [PATCH 0/3] Styleguide conformity Sam Hansen
2012-06-07 20:20 ` [PATCH 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
2012-06-08 7:34 ` Nitin Gupta
2012-06-07 20:20 ` [PATCH 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
2012-06-08 7:37 ` Nitin Gupta
2012-06-07 20:20 ` [PATCH 3/3] staging: zram: conventions, line splitting Sam Hansen
2012-06-07 20:28 ` Joe Perches
2012-06-07 20:30 ` sam hansen
2012-06-07 23:03 ` [PATCH V2 0/3] Styleguide conformity Sam Hansen
2012-06-07 23:03 ` [PATCH V2 1/3] staging: zram: conventions pr_warning -> pr_warn() Sam Hansen
2012-06-07 23:03 ` [PATCH V2 2/3] staging: zram: conventions, __aligned() attribute Sam Hansen
2012-06-07 23:03 ` [PATCH V2 3/3] staging: zram: conventions, line splitting Sam Hansen
2012-06-08 7:42 ` Nitin Gupta
2012-06-11 16:04 ` Greg Kroah-Hartman
2012-06-11 22:08 ` Sam Hansen
2012-06-11 22:26 ` Greg Kroah-Hartman
2012-06-11 22:29 ` Sam Hansen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).