Re: [PATCH v7 0/6] Intel MAX10 BMC Secure Update Driver

[Tom Rix]

On 1/5/21 3:08 PM, Russ Weight wrote:

...

>  .../testing/sysfs-driver-intel-m10-bmc-secure |  61 ++
>  MAINTAINERS                                   |   2 +
>  drivers/fpga/Kconfig                          |  11 +
>  drivers/fpga/Makefile                         |   3 +
>  drivers/fpga/intel-m10-bmc-secure.c           | 543 ++++++++++++++++++
>  include/linux/mfd/intel-m10-bmc.h             |  85 +++

I am having trouble pulling this into my testing branch where i am tracking some other changes to intel-m10-bmc.h

https://lore.kernel.org/lkml/20210114231648.199685-1-russell.h.weight@xxxxxxxxx/

https://lore.kernel.org/lkml/1609999628-12748-3-git-send-email-yilun.xu@xxxxxxxxx/

so I am wondering if it makes sense to split the intel-m10-bmc.h change out of this patchset and sent as a single patch to mfd subsystem ?  The change is a bunch of #defines that don't do anything on their own, but will conflict with other similar additions to the h file.

Tom

>  6 files changed, 705 insertions(+)
>  create mode 100644 Documentation/ABI/testing/sysfs-driver-intel-m10-bmc-secure
>  create mode 100644 drivers/fpga/intel-m10-bmc-secure.c
>