Hi Eugeniu, > The idea was to push it to LKML, once/after you are happy with it. I see. > Thanks for the precious feedback. We've requested Renesas to revert the > obsolete BSP commit, based on your recommendation. Thank you! > In general, the Renesas kernel always carries a set of patches with > non-mainlined changes, Fortunately, for i2c specifically (as opposed > to other subsystems), it is narrow enough to not raise major concerns: Well, yes, that shows that I am mostly successful with reporting back to the BSP team :D But some are still missing, as you can see. > $ git log --oneline v5.10.41..rcar-5.1.2 -- drivers/i2c/busses/i2c-rcar.c > 6745303b2bfa i2c: rcar: Add support for r8a77961 (R-Car M3-W+) Can be dropped: "Driver matches against family-specific compatible value, which has always been present in upstream DTS" > 3422d3131700 i2c: rcar: Support the suspend/resume Already upstream: "18569fa89a4db9ed6b5181624788a1574a9b6ed7 # i2c: rcar: add suspend/resume support" > 5680e77f2427 i2c: rcar: Tidy up the register order for hardware specification ver1.00. Relevant parts upstream: "e7f4264821a4ee07775f3775f8530cfa9a6d4b5d # i2c: rcar: enable interrupts before starting transfer" Other parts can be dropped. > 41394ab7420f i2c: rcar: Fix I2C DMA transmission by setting sequence Upstream: "94e290b0e9a6c360a5660c480c1ba996d892c650 # i2c: rcar: wait for data empty before starting DMA" > Thank you for the review comments. We are still working on a cleaner > solution. In case it comes from you first, we are very much keen to > give it a try on the target and report the results. I have a cleaner solution quite ready. Give me another hour for testing before I send it out. All the best, Wolfram
Attachment:
signature.asc
Description: PGP signature
