Hi Leo and Scott, > -----Original Message----- > From: Ulf Hansson [mailto:ulf.hansson@xxxxxxxxxx] > Sent: Wednesday, April 06, 2016 4:15 PM > To: Yangbo Lu; Scott Wood > Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; linuxppc-dev@xxxxxxxxxxxxxxxx; linux- > clk@xxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx; iommu@lists.linux- > foundation.org; netdev@xxxxxxxxxxxxxxx; linux-mmc; Rob Herring; Russell > King; Jochen Friedrich; Joerg Roedel; Claudiu Manoil; Bhupesh Sharma; > Zhao Qiang; Kumar Gala; Santosh Shilimkar; Yang-Leo Li; Xiaobo Xie > Subject: Re: [v7, 0/5] Fix eSDHC host version register bug > > >> > >> I was about to queue this for next, when I noticed that checkpatch is > >> complaining/warning lots about these patches. Can you please a round > >> of checkpatch and fix what makes sense. > >> > >> Kind regards > >> Uffe > > > > [Lu Yangbo-B47093] Sorry about this, Uffe... > > No worries! > > > Should I ignore the warnings that update MAINTAINERS? > > drivers/soc/fsl/guts.c isn't part of the MAINTAINERS file, it should be. > > I also realize that the FREESCALE QUICC ENGINE LIBRARY section > drivers/soc/fsl/qe/* also need an active maintainer, as it's currently > orphan. > > Perhaps we should have create a new section for drivers/soc/fsl/* instead > that covers all of the above? Maybe you or Scott are interested to pick > it up? > > I also noted that, "include/linux/fsl/" isn't present in MAINTAINERS, > please add that as well. [Lu Yangbo-B47093] Could give some advice on the MAINTAINERS for these 'fsl' files since I really don’t know who should be the right person? I will appreciate that! Thanks a lot. > > > Regarding the 'undocumented' warning, I will added a patch updates doc > before all the patches, Ok? > > Yes, good! > > > > > Thanks a lot :) > > > > Kind regards > Uffe ��.n��������+%������w��{.n�����{��i��)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥