| To: | "Steven J. Hill" <sjhill@mips.com> |
|---|---|
| Subject: | Re: [PATCH] [RFC] Proposed changes to eliminate 'union mips_instruction' type. |
| From: | Ralf Baechle <ralf@linux-mips.org> |
| Date: | Tue, 15 Jan 2013 09:24:56 +0100 |
| Cc: | linux-mips@linux-mips.org, cernekee@gmail.com, kevink@paralogos.com |
| In-reply-to: | <1358230420-3575-1-git-send-email-sjhill@mips.com> |
| List-archive: | <http://www.linux-mips.org/archives/linux-mips/> |
| List-help: | <mailto:ecartis@linux-mips.org?Subject=help> |
| List-id: | linux-mips <linux-mips.eddie.linux-mips.org> |
| List-owner: | <mailto:ralf@linux-mips.org> |
| List-post: | <mailto:linux-mips@linux-mips.org> |
| List-software: | Ecartis version 1.0.0 |
| List-subscribe: | <mailto:ecartis@linux-mips.org?subject=subscribe%20linux-mips> |
| List-unsubscribe: | <mailto:ecartis@linux-mips.org?subject=unsubscribe%20linux-mips> |
| References: | <1358230420-3575-1-git-send-email-sjhill@mips.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Tue, Jan 15, 2013 at 12:13:40AM -0600, Steven J. Hill wrote: > From: "Steven J. Hill" <sjhill@mips.com> > > This patch shows the use of macros in place of 'union mips_instruction' > type. I converted all usages of 'j_format' and 'r_format' to show how > the code and macros could look and be defined. I have tested these > changes on big and little endian platforms. > > I want input from everyone, please!!! I want consensus on the macro > definitions, placement of parenthesis for them, spacing in the header > file, etc. This is your chance to be completely anal and have fun > arguments over how things should be. I would also like input on how > the maintainers would like the patchsets to look like. For example: > > [1/X] - Convert 'j_format' > [2/X] - Convert 'r_format' > [3/X] - Convert 'f_format' > [4/X] - Convert 'u_format' > ... > [X/X] - Remove usage of 'union mips_instruction' type completely. > > Also, I noticed 'p_format' is not used anywhere. Can we kill it? Be > picky and help with this conversion. Thanks. Mayne not kill it completely abut leave a comment mentioning its existence for the sake of completeness. Ralf |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 03/10] MIPS: PCI: Byteswap not needed in little-endian mode, Jayachandran C |
|---|---|
| Next by Date: | Re: [PATCH 03/10] MIPS: PCI: Byteswap not needed in little-endian mode, Sergei Shtylyov |
| Previous by Thread: | [PATCH] [RFC] Proposed changes to eliminate 'union mips_instruction' type., Steven J. Hill |
| Next by Thread: | Re: [PATCH] [RFC] Proposed changes to eliminate 'union mips_instruction' type., David Daney |
| Indexes: | [Date] [Thread] [Top] [All Lists] |