Just replying because I want to understand certain choices you make,
absolutely not questioning your code!
On 05/26/13 12:20, Maxime Ripard wrote:
+ * warranty of any kind, whether express or implied.
+ */
+
+#include <linux/clk.h>
+#include <linux/completion.h>
You forgot to add #include <linux/bitops.h> for BIT()
+
+static void sunxi_i2c_write(struct sunxi_i2c_dev *i2c_dev, u16 reg, u8 value)
+{
+ writel(value, i2c_dev->membase + reg);
Why writel? and why without (u32)value? I thought iowrite* where the
preferred calls and in this case, wouldn't we want writeb since value is u8?
+}
+
+static u32 sunxi_i2c_read(struct sunxi_i2c_dev *i2c_dev, u16 reg)
+{
+ return readl(i2c_dev->membase + reg);
And here, readl does match the return of u32, but aren't we always
reading 8 bits since the TWI Data Register only uses the first 8 bits?
So wouldn't we want to return u8 and readb?
+static int sunxi_i2c_probe(struct platform_device *pdev)
+{
+ struct sunxi_i2c_dev *i2c_dev;
+ struct device_node *np;
+ u32 freq, div_m, div_n;
+ struct resource res;
I feel stupid for questioning this, since it only shows my lack of
knowledge, but
If you declare all the memory here, isn't all the data lost after
exiting the _probe function? we pass a pointer to this memory in the
of_address_to_resource() function so that fills it, right?
Or does after devm_ioremap_resource it no longer matter, since that
function got what it needed and useless after?
Just asking because of the wdt driver and possibly mine that you told us
to change.
Sorry for asking (again) maybe very obvious things. Just trying to learn.
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
