[PATCH 2/6] spi/imx: use soc name in spi device type naming scheme
Shawn Guo
shawn.guo at freescale.com
Sat Jul 9 03:00:02 EDT 2011
On Fri, Jul 08, 2011 at 12:58:51PM +0200, Sascha Hauer wrote:
> On Fri, Jul 08, 2011 at 04:27:30PM +0800, Shawn Guo wrote:
> > Software defined version number is not stable enough to be used
> > in device type naming scheme. The patch changes it to use implicit
> > soc name for spi device type definition. In this way, we can easily
> > align the naming scheme with device tree binding, which comes later.
> >
> > Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> > Cc: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> > Cc: Sascha Hauer <s.hauer at pengutronix.de>
> > Cc: Grant Likely <grant.likely at secretlab.ca>
> > ---
> > arch/arm/mach-imx/clock-imx25.c | 7 +-
> > arch/arm/mach-mx5/clock-mx51-mx53.c | 11 +-
> > arch/arm/plat-mxc/devices/platform-spi_imx.c | 12 +-
> > drivers/spi/Kconfig | 15 -
> > drivers/spi/spi-imx.c | 423 ++++++++++++--------------
> > 5 files changed, 219 insertions(+), 249 deletions(-)
>
> Generally this patch is too big for me to get the warm feeling that
> everything will just work. Renaming from spi_ver_xy to imx_xy is fine,
> but should be done in smaller steps and where possible not across
> subsystems.
>
Okay, I will split the patch into small pieces to win your warm
feeling :)
> I increasingly see patches touching arch/ and drivers/ and I think
> there's a high risk of merge conflicts upstream. This will not help
> making Linus happy.
>
Yes, I increasingly run into the situation touching arch/ and drivers/
when working on dt migration because of the device renaming. May I
have to your suggestion on how to separate drivers/ patch from arch/
one in orthogonal way, while keeping both trees working/bisectable?
--
Regards,
Shawn
More information about the linux-arm-kernel
mailing list