Thanks.
Regards Haijun. > -----Original Message----- > From: David Laight [mailto:david.lai...@aculab.com] > Sent: Thursday, July 11, 2013 4:44 PM > To: Wood Scott-B07421; Zhang Haijun-B42677 > Cc: Zhao Chenhui-B35336; linux-...@vger.kernel.org; Wrobel Heinz-R39252; > Fleming Andy-AFLEMING; Zhang Haijun-B42677; cbouatmai...@gmail.com; > c...@laptop.org; linuxppc-dev@lists.ozlabs.org; Xie Xiaobo-R63061 > Subject: RE: [PATCH 1/4 V2] powerpc/85xx: Add support for 85xx cpu type > detection > > > > +#define IS_SVR_REV(svr, maj, min) \ > > > + ((SVR_MAJ(svr) == (maj)) && (SVR_MIN(svr) == (min))) > > > > I don't think IS_SVR_REV is needed. Callers can just do "if > > (SVR_REV(svr) == 0x30)" or whatever, especially since we're relying on > > them to do this for greater/less than comparisons. > > Not only that, I'd guess that 'maj' and 'min' are likely to be constants > - so you'd want to combine them and compare against 'svr' > rather than have two conditionals. [Haijun Wrote:] yes, e.g: 1.0, 1.1 or 2.0. 'Major revision field' + '.' + 'Minor revision field ' I had resent the patch(V2). I'm not so clear about what you want to express.^_^ > > David > > > _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev