From: Sergei Shtylyov <sshtylyov@mvista.com> To: Marc Zyngier <Marc.Zyngier@arm.com> Cc: linux-arm-kernel@lists.infradead.org, David Woodhouse <dwmw2@infradead.org>, linux-mtd@lists.infradead.org, Sergei Shtylyov <sshtylyov@mvista.com>, Mike Frysinger <vapier.adi@gmail.com> Subject: Re: [PATCH v6 1/8] MTD: Add integrator-flash feature to physmap Date: Wed, 01 Jun 2011 20:28:59 +0400 [thread overview] Message-ID: <4DE668CB.9040408@ru.mvista.com> (raw) In-Reply-To: <1306914546.27474.349.camel@e102391-lin.cambridge.arm.com> Marc Zyngier wrote: >>>>> In the process of moving platforms away from integrator-flash >>>>> (aka armflash), add to physmap the few features that make >>>>> armflash unique: >>>>> - optionnal probing for the AFS partition type >>>>> - init() and exit() methods, used by Integrator to control >>>>> write access to the various onboard programmable components >>>>> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com> >>>>> Acked-by: Catalin Marinas <catalin.marinas@arm.com> >>>>> Cc: David Woodhouse <dwmw2@infradead.org> >>>> [...] >>>>> diff --git a/include/linux/mtd/physmap.h b/include/linux/mtd/physmap.h >>>>> index bcfd9f7..d37cca0 100644 >>>>> --- a/include/linux/mtd/physmap.h >>>>> +++ b/include/linux/mtd/physmap.h >>>>> @@ -22,6 +22,8 @@ struct map_info; >>>>> >>>>> struct physmap_flash_data { >>>>> unsigned int width; >>>>> + int (*init)(struct platform_device *); >>>>> + void (*exit)(struct platform_device *); >>>> How about a declaration for 'struct platform_device' beforehand? This breaks >>>> build on other platforms at least... >>> Sure. Please send a patch. >> uhh, what ? you broke things, so you fix them. i already pointed out >> this breakage to you over a week ago. > Mike, > Sorry I missed your earlier comment about this. My bad. > Now, about my reply to Sergei: > * Sergei identified the problem, and proposed a valid fix. It's > only fair he gets credited for his work (yes, you did it before > he did, but I didn't know it). The SOB on a patch is the proper > way of managing the attribution. > * Posting a patch is the quickest way for the maintainer (in this > case David) to be able to push a fix to mainline. > * It also helps other folks reading the ML and facing the same > issue. They can grab the patch, apply it locally and go on with > their own stuff until the patch is merged. > * The person reporting the problem probably has a configuration > showing the problem. If any of my platforms had shown that > error, I would have fixed it long before it hit the mailing > list. Actually, I'm not a primary reporter. The issue came up on the #mipslinux IRC channel, I then just remembered that I saw the patch set touching this file, and it was easy enough to complain to you after finding the corresponding mails. This happens to break MIPS platforms because they use warnings-as-errors option of gcc. WBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: sshtylyov@mvista.com (Sergei Shtylyov) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v6 1/8] MTD: Add integrator-flash feature to physmap Date: Wed, 01 Jun 2011 20:28:59 +0400 [thread overview] Message-ID: <4DE668CB.9040408@ru.mvista.com> (raw) In-Reply-To: <1306914546.27474.349.camel@e102391-lin.cambridge.arm.com> Marc Zyngier wrote: >>>>> In the process of moving platforms away from integrator-flash >>>>> (aka armflash), add to physmap the few features that make >>>>> armflash unique: >>>>> - optionnal probing for the AFS partition type >>>>> - init() and exit() methods, used by Integrator to control >>>>> write access to the various onboard programmable components >>>>> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com> >>>>> Acked-by: Catalin Marinas <catalin.marinas@arm.com> >>>>> Cc: David Woodhouse <dwmw2@infradead.org> >>>> [...] >>>>> diff --git a/include/linux/mtd/physmap.h b/include/linux/mtd/physmap.h >>>>> index bcfd9f7..d37cca0 100644 >>>>> --- a/include/linux/mtd/physmap.h >>>>> +++ b/include/linux/mtd/physmap.h >>>>> @@ -22,6 +22,8 @@ struct map_info; >>>>> >>>>> struct physmap_flash_data { >>>>> unsigned int width; >>>>> + int (*init)(struct platform_device *); >>>>> + void (*exit)(struct platform_device *); >>>> How about a declaration for 'struct platform_device' beforehand? This breaks >>>> build on other platforms at least... >>> Sure. Please send a patch. >> uhh, what ? you broke things, so you fix them. i already pointed out >> this breakage to you over a week ago. > Mike, > Sorry I missed your earlier comment about this. My bad. > Now, about my reply to Sergei: > * Sergei identified the problem, and proposed a valid fix. It's > only fair he gets credited for his work (yes, you did it before > he did, but I didn't know it). The SOB on a patch is the proper > way of managing the attribution. > * Posting a patch is the quickest way for the maintainer (in this > case David) to be able to push a fix to mainline. > * It also helps other folks reading the ML and facing the same > issue. They can grab the patch, apply it locally and go on with > their own stuff until the patch is merged. > * The person reporting the problem probably has a configuration > showing the problem. If any of my platforms had shown that > error, I would have fixed it long before it hit the mailing > list. Actually, I'm not a primary reporter. The issue came up on the #mipslinux IRC channel, I then just remembered that I saw the patch set touching this file, and it was easy enough to complain to you after finding the corresponding mails. This happens to break MIPS platforms because they use warnings-as-errors option of gcc. WBR, Sergei
next prev parent reply other threads:[~2011-06-01 16:28 UTC|newest] Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-05-17 13:11 [PATCH v6 0/8] Switch ARM platforms from integrator-flash to physmap Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 1/8] MTD: Add integrator-flash feature " Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-19 11:14 ` Artem Bityutskiy 2011-05-19 11:14 ` Artem Bityutskiy 2011-05-31 19:11 ` Sergei Shtylyov 2011-05-31 19:11 ` Sergei Shtylyov 2011-05-31 21:44 ` Marc Zyngier 2011-05-31 21:44 ` Marc Zyngier 2011-06-01 2:20 ` Mike Frysinger 2011-06-01 2:20 ` Mike Frysinger 2011-06-01 7:42 ` Artem Bityutskiy 2011-06-01 7:42 ` Artem Bityutskiy 2011-06-01 16:32 ` Sergei Shtylyov 2011-06-01 16:32 ` Sergei Shtylyov 2011-06-01 7:49 ` Marc Zyngier 2011-06-01 7:49 ` Marc Zyngier 2011-06-01 9:43 ` Artem Bityutskiy 2011-06-01 9:43 ` Artem Bityutskiy 2011-06-01 16:28 ` Sergei Shtylyov [this message] 2011-06-01 16:28 ` Sergei Shtylyov 2011-05-17 13:11 ` [PATCH v6 2/8] ARM: Realview: Use physmap driver instead of integrator-flash Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 3/8] ARM: Versatile: " Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 4/8] ARM: VExpress: " Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 5/8] ARM: Integrator/AP: " Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 6/8] ARM: Integrator/CP: " Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-17 13:11 ` [PATCH v6 7/8] MTD: Remove integrator-flash Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-19 11:14 ` Artem Bityutskiy 2011-05-19 11:14 ` Artem Bityutskiy 2011-05-17 13:11 ` [PATCH v6 8/8] MTD: physmap: let set_vpp() pass a platform_device instead of a map_info Marc Zyngier 2011-05-17 13:11 ` Marc Zyngier 2011-05-19 11:17 ` Artem Bityutskiy 2011-05-19 11:17 ` Artem Bityutskiy 2011-05-19 17:37 ` Jean-Christophe PLAGNIOL-VILLARD 2011-05-19 17:37 ` Jean-Christophe PLAGNIOL-VILLARD 2011-05-19 11:19 ` [PATCH v6 0/8] Switch ARM platforms from integrator-flash to physmap Artem Bityutskiy 2011-05-19 11:19 ` Artem Bityutskiy 2011-05-19 12:53 ` Marc Zyngier 2011-05-19 12:53 ` Marc Zyngier 2011-05-20 21:28 ` Russell King - ARM Linux 2011-05-20 21:28 ` Russell King - ARM Linux 2011-05-23 15:01 ` Artem Bityutskiy 2011-05-23 15:01 ` Artem Bityutskiy
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=4DE668CB.9040408@ru.mvista.com \ --to=sshtylyov@mvista.com \ --cc=Marc.Zyngier@arm.com \ --cc=dwmw2@infradead.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-mtd@lists.infradead.org \ --cc=vapier.adi@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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.