Messages in this thread | | | From | marc.ferland@gmail ... | Subject | [PATCH v4 0/5] Add support for the ds28ec20 one-wire eeprom | Date | Mon, 18 Dec 2023 10:02:25 -0500 |
| |
From: Marc Ferland <marc.ferland@sonatest.com>
Hi,
Here is v4 of my ds2433 driver patch series, see [1] for v3.
Changes: v4: Drop last paragraph from the ds2490 patch commit message (suggested by Krzysztof). Rename the __ds_read_block function to read_block_chunk. Statically allocate the validcrc bitmap, suggested by David Laight. Remove both W1_PAGE_COUNT and W1_F23_TIME defines from the same patch (was previously in separate patches, suggested by Krzysztof). Nullify pointer earlier in w1_f23_remove_slave to better match w1_f23_add_slave (suggested by Krzysztof). v3: Do not use in-reply-to when sending a new patch series. v2: Incorporate suggestions from Krzysztof Kozlowski: drop the 'w1: ds2433: rename W1_EEPROM_DS2433' and 'w1: ds2433: rename w1_f23_data to w1_data' patches. Create a separate patch for the validcrc bitmap change (also suggested by Krzysztof). Fix build error: initializer element is not a compile-time constant. Rework the ds2490 patch and remove the ds_write_block changes: I have no way of reliably test this change with my current setup, and I did not experience any write failures. Let's not try to fix what already works. Rearrange commit order for a more logical order. Tested with the ds2433 eeprom. Rebased on v6.7-rc2.
[1] https://lore.kernel.org/lkml/20231130135232.191320-1-marc.ferland@sonatest.com/
Marc Ferland (5): w1: ds2490: support block sizes larger than 128 bytes in ds_read_block w1: ds2433: remove unused definitions w1: ds2433: introduce a configuration structure w1: ds2433: use the kernel bitmap implementation w1: ds2433: add support for ds28ec20 eeprom
drivers/w1/masters/ds2490.c | 25 +++++- drivers/w1/slaves/w1_ds2433.c | 162 ++++++++++++++++++++++++++++------ 2 files changed, 157 insertions(+), 30 deletions(-)
base-commit: 3f7168591ebf7bbdb91797d02b1afaf00a4289b1 -- 2.34.1
| |