Hi Alan,
On Thu, 1 Nov 2018 14:37:02 -0500
Alan Tull atull@xxxxxxxxxx wrote:
...
>> static const struct spi_device_id altera_ps_spi_ids[] = {
>> - {"cyclone-ps-spi", 0},
>> + {"cyclone-ps-spi", (kernel_ulong_t)&c5_data},
>> + {"fpga-passive-serial", (kernel_ulong_t)&c5_data},
>> + {"fpga-arria10-passive-serial", (kernel_ulong_t)&a10_data},
>
>I don't think this should be implemented as a pointer. This would be
>easy if driver_data were void* but it's a value that's not a pointer.
> I suggest using driver_data as a index to an array of pointers to the
>structs instead.
Thanks for review. I've sent v3 using array of pointers. It uses
the FPGA type in driver_data. It could be used as an index to the
array of pointers, but I'd prefer checking for this type explicitly,
so it will work even if the array is wrongly extended or reordered
by future changes.
Thanks,
Anatolij
