From mboxrd@z Thu Jan 1 00:00:00 1970 From: vaibhav.hiremath@linaro.org (Vaibhav Hiremath) Date: Tue, 25 Aug 2015 14:20:21 +0530 Subject: [PATCH-v2 1/5] mfd: 88pm800: Fix tabbing issue In-Reply-To: <20150825075246.GF19409@x1> References: <1440442594-3102-1-git-send-email-vaibhav.hiremath@linaro.org> <1440442594-3102-3-git-send-email-vaibhav.hiremath@linaro.org> <20150825075246.GF19409@x1> Message-ID: <55DC2C4D.1040304@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 25 August 2015 01:22 PM, Lee Jones wrote: > On Tue, 25 Aug 2015, Vaibhav Hiremath wrote: > >> Driver had coding style issues where spaces were used instead >> of tabs. This patch fixes them all. >> >> Signed-off-by: Vaibhav Hiremath >> --- >> drivers/mfd/88pm800.c | 125 ++++++++++++++++++++++++-------------------------- >> 1 file changed, 61 insertions(+), 64 deletions(-) > > Same comment as before. > > How similar is this file to the other one? I believe you are referring to include/linux/mfd/88pm80x.h > > Can you reduce the size by having shared register defines? > Not sure whether I understand your comment here, They are common defines. PM860 only defines are named as PM860_XXXX Similarly, PM805 only defines are named as PM805_XXXX and PM800_xxx should be common. Thanks, Vaibhav