[U-Boot] [PATCH] ARM: imx: cm_fx6: Drop ad-hoc SATA binding
Marek Vasut
marex at denx.de
Mon Jun 10 14:27:23 UTC 2019
On 6/10/19 4:24 PM, Stefano Babic wrote:
> Hi Marek,
>
> On 12/05/19 22:43, Marek Vasut wrote:
>> Drop the ad-hoc AHCI binding code, this is superseded by
>> CONFIG_DWC_AHSATA_AHCI=y resp. drivers/ata/dwc_ahsata.c
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> Cc: Christopher Spinrath <christopher.spinrath at rwth-aachen.de>
>> Cc: Fabio Estevam <fabio.estevam at nxp.com>
>> Cc: Igor Grinberg <grinberg at compulab.co.il>
>> Cc: Nikita Kiryanov <nikita at compulab.co.il>
>> Cc: Stefano Babic <sbabic at denx.de>
>> ---
>> board/compulab/cm_fx6/cm_fx6.c | 63 ----------------------------------
>> configs/cm_fx6_defconfig | 1 -
>> 2 files changed, 64 deletions(-)
>>
>> diff --git a/board/compulab/cm_fx6/cm_fx6.c b/board/compulab/cm_fx6/cm_fx6.c
>> index d42f57d4b7..b8f15cf3ab 100644
>> --- a/board/compulab/cm_fx6/cm_fx6.c
>> +++ b/board/compulab/cm_fx6/cm_fx6.c
>> @@ -724,66 +724,3 @@ U_BOOT_DEVICE(cm_fx6_serial) = {
>> .name = "serial_mxc",
>> .platdata = &cm_fx6_mxc_serial_plat,
>> };
>> -
>> -#if CONFIG_IS_ENABLED(AHCI)
>> -static int sata_imx_probe(struct udevice *dev)
>> -{
>> - int i, err;
>> -
>> - /* Make sure this gpio has logical 0 value */
>> - gpio_direction_output(CM_FX6_SATA_PWLOSS_INT, 0);
>> - udelay(100);
>> - cm_fx6_sata_power(1);
>
> cm_fx6_sata_power() is still defined and not used in the board, this
> raises at least a warning (apart of Christopher's comments).
>
AFAIR Christopher will be fixing this patch, so drop it.
--
Best regards,
Marek Vasut
More information about the U-Boot
mailing list