From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rajeev kumar Subject: Re: [PATCH V2] i2c-designware: Change readl to readw and writel to writew Date: Thu, 27 Oct 2011 11:01:34 +0530 Message-ID: <4EA8ECB6.50600@st.com> References: <1319450282-914-1-git-send-email-rajeev-dlh.kumar@st.com> <20111024103316.GC26649@sapphire.tkos.co.il> <4EA5472A.1000400@st.com> <20111024122349.GD26649@sapphire.tkos.co.il> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20111024122349.GD26649-MwjkAAnuF3khR1HGirfZ1z4kX+cae0hd@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Baruch Siach Cc: "linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Shiraz HASHIM , Viresh KUMAR , Bhupesh SHARMA , Pratyush ANAND , Vipin KUMAR , Deepak SIKRI , Amit VIRDI , Vipul Kumar SAMAR , Armando VISCONTI List-Id: linux-i2c@vger.kernel.org Hello Baruch On 10/24/2011 5:53 PM, Baruch Siach wrote: > Hi Rajeev, > > On Mon, Oct 24, 2011 at 04:38:26PM +0530, Rajeev kumar wrote: >> On 10/24/2011 4:03 PM, Baruch Siach wrote: >>> On Mon, Oct 24, 2011 at 03:28:02PM +0530, Rajeev Kumar wrote: >>>> Since I2C designware registers are 16 bit wide and so we should use >>>> readw/writew. >>>> >>>> Signed-off-by: Rajeev Kumar >>>> --- >>>> drivers/i2c/busses/i2c-designware.c | 104 +++++++++++++++++----------------- >>>> 1 files changed, 52 insertions(+), 52 deletions(-) >>>> >>>> diff --git a/drivers/i2c/busses/i2c-designware.c b/drivers/i2c/busses/i2c-designware.c >>>> index 6eaa681..5149a10 100644 >>>> --- a/drivers/i2c/busses/i2c-designware.c >>>> +++ b/drivers/i2c/busses/i2c-designware.c >>>> @@ -216,11 +216,11 @@ struct dw_i2c_dev { >>>> u32 abort_source; >>>> int irq; >>>> struct i2c_adapter adapter; >>>> - unsigned int tx_fifo_depth; >>>> - unsigned int rx_fifo_depth; >>>> + u16 tx_fifo_depth; >>>> + u16 rx_fifo_depth; >>>> }; >>> >>> This looks wrong. The {tx,rx}_fifo_depth fields do not represent bit fields, >>> but numbers. So unsigned int should be better here. >> >> Yes, I agree with you, but I do not see any possibility of value of >> {tx,rx}_fifo_depth fields greater than 2^^16 - 1. So, would not it >> be better to keep them as u16 and save just 4 bytes. > > Well, if you are after these 4 bytes just make them 'unsigned short'. > Sorry, I could not understand preference of unsigned short over u16. Although, its not a big deal to keep either unsigned short or u16 or unsigned int. More or less all are fine. But, should we not keep what is most appropriate? Is it not correct that u16 will always be 16 bit , but unsigned short may not be guaranteed to be 16 bit on every platform? Regards ~Rajeev > baruch >