Re: [PATCH v2 2/4] soc: ti: add firmware file name as part of the driver

From: Arnd Bergmann
Date: Tue Oct 13 2015 - 10:33:51 EST


On Monday 12 October 2015 15:46:49 Murali Karicheri wrote:
> Currently firmware file name is included in the DTS. This is not scalable
> as user has to change the DTS if they need upgrade to a new firmware.
> Instead, add the firmware file name in the driver itself. As long as there
> is no API change, new firmware upgrade is easy and require no driver
> change. User is expected to copy the firmware image to the file system
> and add a sym link to the new firmware for doing an upgrade. Driver add
> a array of firmware file names to search for the available firmware blobs.
> This scheme also prepare the driver for future changes to API if ever
> happens. In such case it is assumed that driver needs to change to
> accommodate the new firmware and new firmware file name will get added to
> the array.
>
> Also update the DT document to remove the firmware attribute and add
> description about firmware in the driver documentation.
>
> Signed-off-by: Murali Karicheri <m-karicheri2@xxxxxx>
>

Acked-by: Arnd Bergmann <arnd@xxxxxxxx>

Looks good.
--
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/