From: "Stephen Neuendorffer" <stephen.neuendorffer@xilinx.com>
To: Arnd Bergmann <arnd@arndb.de>, monstr@seznam.cz
Cc: John Williams <john.williams@petalogix.com>,
linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
John Linn <linnj@xilinx.com>,
matthew@wil.cx, will.newton@gmail.com, drepper@redhat.com,
microblaze-uclinux@itee.uq.edu.au, grant.likely@secretlab.ca
Subject: RE: [PATCH 24/56] microblaze_v2: time support
Date: Tue, 6 May 2008 09:36:34 -0700 [thread overview]
Message-ID: <20080506163629.B6B85EB804F@mail78-sin.bigfish.com> (raw)
In-Reply-To: <200805061338.24870.arnd@arndb.de>
> I think you're doing a great job on this. Any patch set of this size
that
> has been maintained for a long time outside of the kernel goes through
this
> phase where everyone tells you what to improve, based on their
experience,
> while everything you do right about the patches is just taken for
granted.
I completely agree on this. I think you're making a huge amount of
progress and spending alot of time incorporating alot of comments from
alot of people.
My main priority is to get as many of the improvements that we've had at
in our git tree, so as to be able to switch to the mainline code as
quickly as possible. If I keep bringing up the same things it is
because they are barriers to that happening. I very much want to put a
big ACK on everything, but I'm uncomfortable doing that until I can at
least boot it, and that hasn't happened yet.
> Don't let yourself get discouraged by the negative comments, we're
really
> all just trying to help you do the right thing!
One of the key things that many people take a long time to learn is that
negative comments about code are not a reflection of negativity towards
the person who wrote the code.
Steve
next prev parent reply other threads:[~2008-05-06 16:41 UTC|newest]
Thread overview: 192+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-04 11:40 Microblaze patches V2 monstr
2008-05-04 11:40 ` [PATCH 01/56] microblaze_v2: Kconfig patches monstr
2008-05-04 11:40 ` [PATCH 02/56] microblaze_v2: Makefiles for Microblaze cpu monstr
2008-05-05 1:47 ` John Williams
2008-05-06 8:51 ` Michal Simek
2008-05-04 11:40 ` [PATCH 03/56] microblaze_v2: Cpuinfo handling monstr
2008-05-05 1:52 ` John Williams
2008-05-05 14:19 ` Michal Simek
2008-05-04 11:40 ` [PATCH 04/56] microblaze_v2: Open firmware files monstr
2008-05-05 14:24 ` Grant Likely
2008-05-05 21:56 ` Stephen Neuendorffer
2008-05-06 7:27 ` Michal Simek
2008-05-07 16:04 ` [microblaze-uclinux] " Stephen Neuendorffer
2008-05-07 18:40 ` Michal Simek
2008-05-07 18:59 ` Grant Likely
2008-05-07 19:12 ` Michal Simek
2008-05-07 20:14 ` Stephen Neuendorffer
2008-05-04 11:40 ` [PATCH 05/56] microblaze_v2: Support for semaphores monstr
2008-05-04 11:40 ` [PATCH 06/56] microblaze_v2: exception handling monstr
2008-05-04 11:40 ` [PATCH 07/56] microblaze_v2: Signal support monstr
2008-05-04 19:52 ` Arnd Bergmann
2008-05-05 21:32 ` Stephen Neuendorffer
2008-05-05 23:33 ` John Williams
2008-05-06 0:13 ` Stephen Neuendorffer
2008-05-06 0:25 ` John Williams
2008-05-06 0:33 ` Stephen Neuendorffer
2008-05-06 9:41 ` Michal Simek
2008-05-04 11:40 ` [PATCH 08/56] microblaze_v2: Interrupt handling, timer support, supported function monstr
2008-05-05 1:58 ` John Williams
2008-05-05 14:20 ` Michal Simek
2008-05-05 14:20 ` Michal Simek
2008-05-07 7:04 ` Thomas Gleixner
2008-05-11 13:55 ` Michal Simek
2008-05-11 14:35 ` Thomas Gleixner
2008-05-11 21:55 ` Michal Simek
2008-05-11 21:55 ` Michal Simek
2008-05-04 11:40 ` [PATCH 09/56] microblaze_v2: cache support monstr
2008-05-05 2:09 ` John Williams
2008-05-05 17:37 ` [microblaze-uclinux] " Stephen Neuendorffer
2008-05-06 9:22 ` Michal Simek
2008-05-05 22:37 ` Stephen Neuendorffer
2008-05-04 11:40 ` [PATCH 10/56] microblaze_v2: Generic dts file for platforms monstr
2008-05-05 14:31 ` Grant Likely
2008-05-05 20:07 ` Michal Simek
2008-05-05 17:25 ` Stephen Neuendorffer
2008-05-05 23:16 ` John Williams
2008-05-05 23:32 ` Stephen Neuendorffer
2008-05-06 0:10 ` John Williams
2008-05-06 0:17 ` Stephen Neuendorffer
2008-05-06 7:50 ` Michal Simek
2008-05-06 7:38 ` Michal Simek
2008-05-04 11:41 ` [PATCH 11/56] microblaze_v2: kernel modules support monstr
2008-05-04 11:41 ` [PATCH 12/56] microblaze_v2: lmb support monstr
2008-05-05 2:11 ` John Williams
2008-05-05 21:32 ` Segher Boessenkool
2008-05-05 21:32 ` Segher Boessenkool
2008-05-04 11:41 ` [PATCH 13/56] microblaze_v2: PVR support, cpuinfo support monstr
2008-05-05 2:14 ` John Williams
2008-05-05 14:20 ` Michal Simek
2008-05-05 14:20 ` Michal Simek
2008-05-04 11:41 ` [PATCH 14/56] microblaze_v2: defconfig file monstr
2008-05-04 11:41 ` [PATCH 15/56] microblaze_v2: head.S + linker script monstr
2008-05-04 11:41 ` [PATCH 16/56] microblaze_v2: supported function for memory - kernel/lib monstr
2008-05-04 11:41 ` [PATCH 17/56] microblaze_v2: checksum support monstr
2008-05-04 19:59 ` Arnd Bergmann
2008-05-05 14:05 ` Michal Simek
2008-05-04 11:41 ` [PATCH 18/56] microblaze_v2: early_printk support monstr
2008-05-05 14:36 ` Grant Likely
2008-05-05 20:10 ` Michal Simek
2008-05-05 23:22 ` John Williams
2008-05-06 8:14 ` Michal Simek
2008-05-06 8:14 ` Michal Simek
2008-05-04 11:41 ` [PATCH 19/56] microblaze_v2: uaccess files monstr
2008-05-04 11:41 ` [PATCH 20/56] microblaze_v2: heartbeat file monstr
2008-05-04 11:41 ` [PATCH 21/56] microblaze_v2: setup.c - system setting monstr
2008-05-05 2:15 ` John Williams
2008-05-05 14:21 ` Michal Simek
2008-05-05 14:21 ` Michal Simek
2008-05-04 11:41 ` [PATCH 22/56] microblaze_v2: asm-offsets monstr
2008-05-04 11:41 ` [PATCH 23/56] microblaze_v2: process and init task function monstr
2008-05-04 11:41 ` [PATCH 24/56] microblaze_v2: time support monstr
2008-05-05 2:19 ` John Williams
2008-05-05 14:22 ` Michal Simek
2008-05-05 14:22 ` Michal Simek
2008-05-06 0:30 ` John Williams
2008-05-06 9:56 ` Michal Simek
2008-05-06 9:56 ` Michal Simek
2008-05-06 10:02 ` Michal Simek
2008-05-06 10:02 ` Michal Simek
2008-05-06 11:38 ` Arnd Bergmann
2008-05-06 13:26 ` Michal Simek
2008-05-06 22:50 ` John Williams
2008-05-06 14:28 ` Grant Likely
2008-05-06 14:28 ` Grant Likely
2008-05-06 16:36 ` Stephen Neuendorffer [this message]
2008-05-07 7:22 ` Thomas Gleixner
2008-05-04 11:41 ` [PATCH 25/56] microblaze_v2: ptrace support monstr
2008-05-04 11:41 ` [PATCH 26/56] microblaze_v2: traps support monstr
2008-05-04 11:41 ` [PATCH 27/56] microblaze_v2: support for a.out monstr
2008-05-04 11:41 ` [PATCH 28/56] microblaze_v2: memory inicialization, MMU, TLB monstr
2008-05-04 11:41 ` [PATCH 29/56] microblaze_v2: page.h, segment.h, unaligned.h monstr
2008-05-04 11:41 ` [PATCH 30/56] microblaze_v2: includes SHM*, msgbuf monstr
2008-05-04 21:10 ` Arnd Bergmann
2008-05-04 11:41 ` [PATCH 31/56] microblaze_v2: bug headers files monstr
2008-05-04 11:41 ` [PATCH 32/56] microblaze_v2: definitions of types monstr
2008-05-04 21:28 ` Arnd Bergmann
2008-05-04 11:41 ` [PATCH 33/56] microblaze_v2: ioctl support monstr
2008-05-04 21:34 ` Arnd Bergmann
2008-05-05 14:06 ` Michal Simek
2008-05-05 14:06 ` Michal Simek
2008-05-04 11:41 ` [PATCH 34/56] microblaze_v2: io.h IO operations monstr
2008-05-04 11:41 ` [PATCH 35/56] microblaze_v2: headers for executables format FLAT, ELF monstr
2008-05-05 2:24 ` John Williams
2008-05-04 11:41 ` [PATCH 36/56] microblaze_v2: dma support monstr
2008-05-05 2:25 ` John Williams
2008-05-05 6:45 ` Geert Uytterhoeven
2008-05-05 6:45 ` Geert Uytterhoeven
2008-05-06 9:16 ` Michal Simek
2008-05-06 9:16 ` Michal Simek
2008-05-06 9:48 ` Geert Uytterhoeven
2008-05-06 9:53 ` Michal Simek
2008-05-06 9:53 ` Michal Simek
2008-05-06 11:17 ` Geert Uytterhoeven
2008-05-06 11:17 ` Geert Uytterhoeven
2008-05-06 11:24 ` Arnd Bergmann
2008-05-06 13:20 ` Michal Simek
2008-05-06 15:36 ` Arnd Bergmann
2008-05-07 6:24 ` Michal Simek
2008-05-07 7:17 ` Geert Uytterhoeven
2008-05-07 9:21 ` Arnd Bergmann
2008-05-07 18:43 ` Michal Simek
2008-05-07 18:43 ` Michal Simek
2008-05-04 11:41 ` [PATCH 37/56] microblaze_v2: headers for irq monstr
2008-05-07 7:26 ` Thomas Gleixner
2008-05-11 13:56 ` Michal Simek
2008-05-11 13:56 ` Michal Simek
2008-05-04 11:41 ` [PATCH 38/56] microblaze_v2: atomic.h bitops.h byteorder.h monstr
2008-05-04 21:58 ` Arnd Bergmann
2008-05-05 2:28 ` John Williams
2008-05-06 8:42 ` Michal Simek
2008-05-04 11:41 ` [PATCH 39/56] microblaze_v2: headers pgalloc.h pgtable.h monstr
2008-05-04 11:41 ` [PATCH 40/56] microblaze_v2: system.h pvr.h processor.h monstr
2008-05-04 11:41 ` [PATCH 41/56] microblaze_v2: clinkage.h linkage.h sections.h kmap_types.h monstr
2008-05-04 11:41 ` [PATCH 42/56] microblaze_v2: stats headers monstr
2008-05-04 22:31 ` Arnd Bergmann
2008-05-04 11:41 ` [PATCH 43/56] microblaze_v2: termbits.h termios.h monstr
2008-05-05 9:50 ` Arnd Bergmann
2008-05-05 9:50 ` Arnd Bergmann
2008-05-04 11:41 ` [PATCH 44/56] microblaze_v2: sigcontext.h siginfo.h monstr
2008-05-04 11:41 ` [PATCH 45/56] microblaze_v2: headers simple files - empty or redirect to asm-generic monstr
2008-05-05 2:34 ` John Williams
2008-05-05 15:10 ` Grant Likely
2008-05-06 9:02 ` Arnd Bergmann
2008-05-06 14:47 ` Stephen Rothwell
2008-05-06 15:23 ` Michal Simek
2008-05-06 15:23 ` Michal Simek
2008-05-06 15:38 ` Grant Likely
2008-05-06 16:21 ` Arnd Bergmann
2008-05-06 16:21 ` Arnd Bergmann
2008-05-07 18:46 ` Michal Simek
2008-05-04 11:41 ` [PATCH 46/56] microblaze_v2: headers files entry.h current.h mman.h registers.h sembuf.h monstr
2008-05-05 9:54 ` Arnd Bergmann
2008-05-06 9:07 ` Michal Simek
2008-05-06 20:57 ` Geert Uytterhoeven
2008-05-04 11:41 ` [PATCH 47/56] microblaze_v2: device.h param.h topology.h monstr
2008-05-04 22:37 ` Arnd Bergmann
2008-05-05 14:07 ` Michal Simek
2008-05-04 11:41 ` [PATCH 48/56] microblaze_v2: pool.h socket.h monstr
2008-05-04 22:39 ` Arnd Bergmann
2008-05-04 11:41 ` [PATCH 49/56] microblaze_v2: fcntl.h sockios.h ucontext.h monstr
2008-05-04 22:43 ` Arnd Bergmann
2008-05-05 14:08 ` Michal Simek
2008-05-04 11:41 ` [PATCH 50/56] microblaze_v2: setup.h string.h thread_info.h monstr
2008-05-05 2:39 ` John Williams
2008-05-05 14:24 ` Michal Simek
2008-05-05 14:24 ` Michal Simek
2008-05-04 11:41 ` [PATCH 51/56] microblaze_v2: Kbuild file monstr
2008-05-04 11:41 ` [PATCH 52/56] microblaze_v2: pci headers monstr
2008-05-04 22:45 ` Arnd Bergmann
2008-05-05 14:08 ` Michal Simek
2008-05-04 11:41 ` [PATCH 53/56] microblaze_v2: IPC headers monstr
2008-05-04 11:41 ` [PATCH 54/56] microblaze_v2: entry.S monstr
2008-05-04 11:41 ` [PATCH 55/56] microblaze_v2: sys_microblaze.c monstr
2008-05-04 11:41 ` [PATCH 56/56] microblaze_v2: syscall_table.S and unistd.h monstr
2008-05-04 11:41 ` monstr
2008-05-04 21:24 ` [PATCH 01/56] microblaze_v2: Kconfig patches Grant Likely
2008-05-05 6:36 ` Michal Simek
2008-05-05 1:42 ` John Williams
2008-05-05 6:46 ` Michal Simek
2008-05-05 14:16 ` Michal Simek
2008-05-05 2:30 ` Microblaze patches V2 John Williams
2008-05-05 7:02 ` Michal Simek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080506163629.B6B85EB804F@mail78-sin.bigfish.com \
--to=stephen.neuendorffer@xilinx.com \
--cc=arnd@arndb.de \
--cc=drepper@redhat.com \
--cc=grant.likely@secretlab.ca \
--cc=john.williams@petalogix.com \
--cc=linnj@xilinx.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=matthew@wil.cx \
--cc=microblaze-uclinux@itee.uq.edu.au \
--cc=monstr@seznam.cz \
--cc=will.newton@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox