Re: [PATCH 1/1] mmc: Fixed bug in IDMAC_SET_BUFFER1_SIZE Macro in dw_mmc.c

2011-07-29 Thread Jaehoon Chung
Just add Chris to CC. Jaehoon Chung wrote: Hi.. this patch is right. Acked-by : Jaehoon Chung jh80.ch...@samsung.com Regards, Jaehoon Chung James Hogan wrote: On 07/28/2011 12:03 PM, Shashidhar Hiremath wrote: The mask used inside this macro was assuming Buffer_Size1's[BS1's]

Re: [PATCH 1/1] mmc: Fixed bug in IDMAC_SET_BUFFER1_SIZE Macro in dw_mmc.c

2011-07-29 Thread Chris Ball
Hi Shashidhar, On Fri, Jul 29 2011, Will Newton wrote: The mask used inside this macro was assuming Buffer_Size1's[BS1's] width to be 14 bits, it is actually of 13 bits, Modified masks used in IDMAC_SET_BUFFER1_SIZE such that they use only 13 bits instead of current 14. Signed-off-by:

Re: [PATCH 1/1] mmc: Fixed bug in IDMAC_SET_BUFFER1_SIZE Macro in dw_mmc.c

2011-07-28 Thread James Hogan
On 07/28/2011 12:03 PM, Shashidhar Hiremath wrote: The mask used inside this macro was assuming Buffer_Size1's[BS1's] width to be 14 bits, it is actually of 13 bits, Modified masks used in IDMAC_SET_BUFFER1_SIZE such that they use only 13 bits instead of current 14. Signed-off-by:

Re: [PATCH 1/1] mmc: Fixed bug in IDMAC_SET_BUFFER1_SIZE Macro in dw_mmc.c

2011-07-28 Thread Jaehoon Chung
Hi.. this patch is right. Acked-by : Jaehoon Chung jh80.ch...@samsung.com Regards, Jaehoon Chung James Hogan wrote: On 07/28/2011 12:03 PM, Shashidhar Hiremath wrote: The mask used inside this macro was assuming Buffer_Size1's[BS1's] width to be 14 bits, it is actually of 13 bits,