Re: [PATCH 1/3] spi_mpc83xx: Handles other Freescale processors

From: Gary Thomas
Date: Mon Apr 06 2009 - 10:44:15 EST


Kumar Gala wrote:
>
> On Apr 4, 2009, at 2:28 PM, Anton Vorontsov wrote:
>
>> With this patch we'll able to select spi_mpc83xx driver on the MPC86xx
>> platforms. Let the driver depend on FSL_SOC, so we don't have to worry
>> about Kconfig anymore.
>>
>> Also remove the "experimental" dependency, the driver has been tested
>> to work on a various hardware, and surely not experimental anymore.
>>
>> Signed-off-by: Anton Vorontsov <avorontsov@xxxxxxxxxxxxx>
>> ---
>> drivers/spi/Kconfig | 14 ++++++--------
>> 1 files changed, 6 insertions(+), 8 deletions(-)
>
> this looks fine, but why don't we go ahead and rename things as well?
> SPI_MPC83xx -> SPI_MPC8xxx, spi_mpc83xx.c -> spi_mpc8xxx.c

mpc8xxx implies mpc82xx (e.g. mpc8272) which is not the same controller
as on the mpc83xx

--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/