[RFC/PATCH 4/8] LPDDR QINFO records definitions

[RFC/PATCH 4/8] LPDDR QINFO records definitions

[Alexey Korolev]

There are declaraton of structures and macros definitions necessary for
operations with QINFO in this patch.

Signed-off-by: Alexey Korolev <akorolev@infradead.org>
---

diff --git a/include/linux/mtd/qinfo.h b/include/linux/mtd/qinfo.h
new file mode 100644
index 0000000..c8f91ca
--- /dev/null
+++ b/include/linux/mtd/qinfo.h
@@ -0,0 +1,91 @@
+#include <linux/mtd/map.h>
+#include <linux/wait.h>
+#include <linux/spinlock.h>
+#include <linux/delay.h>
+#include <linux/mtd/mtd.h>
+#include <linux/mtd/flashchip.h>
+#include <linux/mtd/partitions.h>
+
+#ifndef __LINUX_MTD_QINFO_H
+#define __LINUX_MTD_QINFO_H
+
+/* lpddr_private describes lpddr flash chip in memory map
+ * @ManufactId - Chip Manufacture ID
+ * @DevId - Chip Device ID
+ * @qinfo - pointer to qinfo records describing the chip
+ * @numchips - number of chips including virual RWW partitions
+ * @chipshift - Chip/partiton size 2^chipshift
+ * @chips - per-chip data structure
+ */
+struct lpddr_private {
+	uint16_t ManufactId;
+	uint16_t DevId;
+	struct qinfo_chip *qinfo;
+	int numchips;
+	unsigned long chipshift;
+	struct flchip chips[0];
+};
+
+/* qinfo_query_info structure contains request information for
+ * each qinfo record
+ * @major - major number of qinfo record
+ * @major - minor number of qinfo record
+ * @id_str - descriptive string to access the record
+ * @desc - detailed description for the qinfo record
+ */
+struct qinfo_query_info {
+	uint8_t	major;
+	uint8_t	minor;
+	char *id_str;
+	char *desc;
+};
+
+/*
+ * qinfo_chip structure contains necessary qinfo records data
+ * @DevSizeShift - Device size 2^n bytes
+ * @BufSizeShift - Program buffer size 2^n bytes
+ * @TotalBlocksNum - Total number of blocks
+ * @UniformBlockSizeShift - Uniform block size 2^UniformBlockSizeShift bytes
+ * @HWPartsNum - Number of hardware partitions
+ * @SuspEraseSupp - Suspend erase supported
+ * @SingleWordProgTime - Single word program 2^SingleWordProgTime u-sec
+ * @ProgBufferTime - Program buffer write 2^ProgBufferTime u-sec
+ * @BlockEraseTime - Block erase 2^BlockEraseTime m-sec
+ */
+struct qinfo_chip {
+	/* General device info */
+	uint16_t DevSizeShift;
+	uint16_t BufSizeShift;
+	/* Erase block information */
+	uint16_t TotalBlocksNum;
+	uint16_t UniformBlockSizeShift;
+	/* Partition information */
+	uint16_t HWPartsNum;
+	/* Optional features */
+	uint16_t SuspEraseSupp;
+	/* Operation typical time */
+	uint16_t SingleWordProgTime;
+	uint16_t ProgBufferTime;
+	uint16_t BlockEraseTime;
+};
+
+/* defines for fixup usage */
+#define LPDDR_MFR_ANY		0xffff
+#define LPDDR_ID_ANY		0xffff
+#define NUMONYX_MFGR_ID		0x0089
+#define R18_DEVICE_ID_1G	0x893c
+
+static inline map_word lpddr_build_cmd(u_long cmd, struct map_info *map)
+{
+	map_word val = { {0} };
+	val.x[0] = cmd;
+	return val;
+}
+
+#define CMD(x) lpddr_build_cmd(x, map)
+#define CMDVAL(cmd) cmd.x[0]
+
+struct mtd_info *lpddr_cmdset(struct map_info *);
+
+#endif
+

Re: [RFC/PATCH 4/8] LPDDR QINFO records definitions

[David Woodhouse]

On Wed, 2008-10-08 at 22:29 +0100, Alexey Korolev wrote:
> 
> --- /dev/null
> +++ b/include/linux/mtd/qinfo.h
> @@ -0,0 +1,91 @@
> +#include <linux/mtd/map.h>
> +#include <linux/wait.h>
> +#include <linux/spinlock.h>
> +#include <linux/delay.h>
> +#include <linux/mtd/mtd.h>
> +#include <linux/mtd/flashchip.h>
> +#include <linux/mtd/partitions.h>
> +
> +#ifndef __LINUX_MTD_QINFO_H
> +#define __LINUX_MTD_QINFO_H

GCC does magic optimisations to avoid looking at include files for a
second time -- but only if your include guard is around the _whole_
file, iirc. So it's best to move those #includes down below the 
#ifndef __LINUX_MTD_QINFO_H

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@intel.com                              Intel Corporation

Re: [RFC/PATCH 4/8] LPDDR QINFO records definitions

[Alexey Korolev]

Hi
> > 
> > --- /dev/null
> > +++ b/include/linux/mtd/qinfo.h
> > @@ -0,0 +1,91 @@
> > +#include <linux/mtd/map.h>
> > +#include <linux/wait.h>
> > +#include <linux/spinlock.h>
> > +#include <linux/delay.h>
> > +#include <linux/mtd/mtd.h>
> > +#include <linux/mtd/flashchip.h>
> > +#include <linux/mtd/partitions.h>
> > +
> > +#ifndef __LINUX_MTD_QINFO_H
> > +#define __LINUX_MTD_QINFO_H
> 
> GCC does magic optimisations to avoid looking at include files for a
> second time -- but only if your include guard is around the _whole_
> file, iirc. So it's best to move those #includes down below the 
> #ifndef __LINUX_MTD_QINFO_H
>
Correct, it seems I missed this. Thank you for notice. It will be fixed.

Thanks,
Alexey