* [PATCH] badblocks: Prefer unsigned int over unsigned
@ 2019-02-24 14:19 Bharath Vedartham
2019-02-24 15:15 ` Jens Axboe
0 siblings, 1 reply; 5+ messages in thread
From: Bharath Vedartham @ 2019-02-24 14:19 UTC (permalink / raw)
To: axboe; +Cc: linux-block, linux-kernel
Replace unsigned with unsigned int to improve readability.
Signed-off-by: Bharath Vedartham <linux.bhar@gmail.com>
---
block/badblocks.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/badblocks.c b/block/badblocks.c
index 91f7bcf..e3f90c9 100644
--- a/block/badblocks.c
+++ b/block/badblocks.c
@@ -66,7 +66,7 @@ int badblocks_check(struct badblocks *bb, sector_t s, int sectors,
u64 *p = bb->page;
int rv;
sector_t target = s + sectors;
- unsigned seq;
+ unsigned int seq;
if (bb->shift > 0) {
/* round the start down, and the end up */
--
2.7.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] badblocks: Prefer unsigned int over unsigned
2019-02-24 14:19 [PATCH] badblocks: Prefer unsigned int over unsigned Bharath Vedartham
@ 2019-02-24 15:15 ` Jens Axboe
2019-02-24 15:37 ` Bharath Vedartham
0 siblings, 1 reply; 5+ messages in thread
From: Jens Axboe @ 2019-02-24 15:15 UTC (permalink / raw)
To: Bharath Vedartham; +Cc: linux-block, linux-kernel
On 2/24/19 7:19 AM, Bharath Vedartham wrote:
> Replace unsigned with unsigned int to improve readability.
Frivolous change, would make a lot more sense to get checkpatch
to stop complaining about this.
--
Jens Axboe
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] badblocks: Prefer unsigned int over unsigned
2019-02-24 15:15 ` Jens Axboe
@ 2019-02-24 15:37 ` Bharath Vedartham
2019-02-24 15:50 ` Richard Weinberger
0 siblings, 1 reply; 5+ messages in thread
From: Bharath Vedartham @ 2019-02-24 15:37 UTC (permalink / raw)
To: Jens Axboe; +Cc: linux-block, linux-kernel
On Sun, Feb 24, 2019 at 08:15:30AM -0700, Jens Axboe wrote:
> On 2/24/19 7:19 AM, Bharath Vedartham wrote:
> > Replace unsigned with unsigned int to improve readability.
>
> Frivolous change, would make a lot more sense to get checkpatch
> to stop complaining about this.
>
> --
> Jens Axboe
>
I understand. I am starting out kernel hacking. I have been working on
fixing checkpatch.pl warnings. If you have any tasks for me in the
kernel, it would be great.
I have also noticed a lot of warnings to replace printk with subsystem
specific warning functions. Is that a valuable change?
Bharath
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] badblocks: Prefer unsigned int over unsigned
2019-02-24 15:37 ` Bharath Vedartham
@ 2019-02-24 15:50 ` Richard Weinberger
2019-02-25 17:03 ` Bharath Vedartham
0 siblings, 1 reply; 5+ messages in thread
From: Richard Weinberger @ 2019-02-24 15:50 UTC (permalink / raw)
To: Bharath Vedartham; +Cc: Jens Axboe, linux-block, LKML
On Sun, Feb 24, 2019 at 4:38 PM Bharath Vedartham <linux.bhar@gmail.com> wrote:
> I understand. I am starting out kernel hacking. I have been working on
> fixing checkpatch.pl warnings. If you have any tasks for me in the
> kernel, it would be great.
>
> I have also noticed a lot of warnings to replace printk with subsystem
> specific warning functions. Is that a valuable change?
This is a very good start:
https://lists.kernelnewbies.org/pipermail/kernelnewbies/2017-April/017765.html
Happy hacking and welcome! :-)
--
Thanks,
//richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] badblocks: Prefer unsigned int over unsigned
2019-02-24 15:50 ` Richard Weinberger
@ 2019-02-25 17:03 ` Bharath Vedartham
0 siblings, 0 replies; 5+ messages in thread
From: Bharath Vedartham @ 2019-02-25 17:03 UTC (permalink / raw)
To: Richard Weinberger; +Cc: Jens Axboe, linux-block, LKML
On Sun, Feb 24, 2019 at 04:50:51PM +0100, Richard Weinberger wrote:
> On Sun, Feb 24, 2019 at 4:38 PM Bharath Vedartham <linux.bhar@gmail.com> wrote:
> > I understand. I am starting out kernel hacking. I have been working on
> > fixing checkpatch.pl warnings. If you have any tasks for me in the
> > kernel, it would be great.
> >
> > I have also noticed a lot of warnings to replace printk with subsystem
> > specific warning functions. Is that a valuable change?
>
> This is a very good start:
> https://lists.kernelnewbies.org/pipermail/kernelnewbies/2017-April/017765.html
>
> Happy hacking and welcome! :-)
>
> --
> Thanks,
> //richard
Hey! Thanks for the link Richard! I have read it and clearly understood
the message. I will work on understanding the kernel and writing my own
code for my own personal learning before jumping into tasks! Thanks for
your time!
Bharath
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-02-25 17:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-24 14:19 [PATCH] badblocks: Prefer unsigned int over unsigned Bharath Vedartham
2019-02-24 15:15 ` Jens Axboe
2019-02-24 15:37 ` Bharath Vedartham
2019-02-24 15:50 ` Richard Weinberger
2019-02-25 17:03 ` Bharath Vedartham
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.