Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <0c252770-f06f-0bd8-79b0-af637ef27cc4@nokia.com>
Date:   Wed, 29 Jun 2022 14:07:59 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: [PATCH v2 02/10] i2c: xiic: Add standard mode support for > 255
 byte read transfers
Content-Language: en-US
To:     Michal Simek <michal.simek@amd.com>,
        Raviteja Narayanam <raviteja.narayanam@xilinx.com>
Cc:     linux-i2c@vger.kernel.org, michal.simek@xilinx.com,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        git@xilinx.com, marex@denx.de, joe@perches.com
References: <20210626102806.15402-1-raviteja.narayanam@xilinx.com>
 <20210626102806.15402-3-raviteja.narayanam@xilinx.com>
 <c8d4faf1-5f0c-4adb-e52a-92d46179ffa4@nokia.com>
 <27c88580-51bb-7813-c29f-b8a3faaaa294@amd.com>
From:   Krzysztof Adamski <krzysztof.adamski@nokia.com>
In-Reply-To: <27c88580-51bb-7813-c29f-b8a3faaaa294@amd.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UTFPeUlDdWVJTGQybXBWUnZ3dE44NVl5blBGS3RVWnFJbm85YjdxTWY2cm9S?=
 =?utf-8?B?MS8vK0VhejQvNi9Id3plQWQybmFicFR3QU1LVkNzYjFnN1JzRGRUTGZHTVFG?=
 =?utf-8?B?b09RdWQwQWdzbEMyN3o1Slc1djBNZmRPOVEyVkRzSlJ0d3VVQ3B1aTVOUENq?=
 =?utf-8?B?akR2TlM2RVZkSkVMalVoRGVLR0U5dmFyWWdtVTU2Ry9WR0hTbU4yK3NsOG1U?=
 =?utf-8?B?UE5laWJjQk9nMVJBYkgwaTVpeUtvM0RHSm1kUXlYUW51bkNlN3REaFRjaE96?=
 =?utf-8?B?U1F1K2w3ZnVzSWFhOHJrMFRjU1ZQbkhNTkxKZnZWbitJZlNQc2pPa2h3YUNH?=
 =?utf-8?B?alNobHYrYS9seWpNR2VoSTd2TmY0UHF2QkpsWmo1MzREeW85aDdwWmhETDlj?=
 =?utf-8?B?YWNwM2RCNVlscWFYY3JLVHV3dW1jZENDV2YxSHFTNmk1R1FwVHZaS2toelYv?=
 =?utf-8?B?MTRqRm5DVm52aUwzd2lNK29DNnExR1BVbVlNUkxlT2FjQll3UWQ3ZmsvWTNz?=
 =?utf-8?B?eXMzU1lOWlZKTEl5NGtoeHVQNC84cUdVbEF5c0pqVTZteXh3TmFvVnZjd0RI?=
 =?utf-8?B?TnVsdU9TdzdZbGRzeGFESlhwbUpEb1hYV01TbWRBa3NkVzhneUNPeEpUekM0?=
 =?utf-8?B?QzJkUXlXTHRVNW12UkJBQVJ5Y3BXN29aR3JkQUkwYzc3ZlMraVpiY3N5YUg2?=
 =?utf-8?B?QUVudzMyenVvTmtaVVkzV09ISDZXQkhkMkc3SHVkcW9kc2RFZ2toU1pJSDZW?=
 =?utf-8?B?dGwwc2dGVENkMGp6eXRmUTBIRE5LaDFHWHdabHJ6SWNlUGVRN2QvR0RBcXl1?=
 =?utf-8?B?U3ZUeDdyR3BILzZwcUdGanZ6cFFSZVVIZ2drbmVYUnorN1EweThrVEdEblBF?=
 =?utf-8?B?bDhtUWZDais1RVpBbTZBLzMzYXVwTDRPR1NRbTBLcVZJWUNseUZDSm5xYzlv?=
 =?utf-8?B?QUdDVDBPVFZybEF2SDNoTzRCZkhFckNpUG9FM0wxZ3R1dHVyQlZFeTBIdmtV?=
 =?utf-8?B?cUY3aWkrMndPKzdlbFduTHRYZldUTUtndExET2xDYTdJTnVYN2FoeWt2ckhk?=
 =?utf-8?B?cS90YnlHYWl1R3gzclIyT2cyREFrb3M1VkYzcUVkUmQ3OXlKTS9VVHFwOHkx?=
 =?utf-8?B?K0trTTZaU1k2Qm9kWEt6REJwamZKNXFVQmJOMUxSM3dTeCtWY2JpenRtV3Ji?=
 =?utf-8?B?b1I5YmYwU3cxQnNuQVVsUWhYbFZNckRNZGpCdDNsNi8wRFNScUo2Z1dJNXdT?=
 =?utf-8?B?SGwzbFRNeUkvSXpNOHkrQmpoYWlyL09ack1waHFjMHAzTHBsMVUwbzhCRkdR?=
 =?utf-8?B?NW1LU05TMW4vNkdUeFFyQjFUczJMRks0VDgzVjBXa2dTTkFpa1NtLzJ1OEta?=
 =?utf-8?B?aE5OdGluNVlJSHp3aG51QmtLamNnazJxQXpTQ1BZZ1o5MTlQZnRpOG84cEVJ?=
 =?utf-8?B?MVdTRUFQbzFHUU1xWFBucmZkeVdMcEFZcVZBa3gwT2FKYUh3UlJMWkc0bnpw?=
 =?utf-8?B?emxPTER4aGwyaCtSMU10ZnUxLzJXeHJKZWtOZ3JPa3ZNRGJsQUFIQ2FVSlFK?=
 =?utf-8?B?dXp5cnNEaWxhMXB3NVA0UFYxK1orNGhUVm1iNHBZQ0ZONUlwVVFpZCtWTy9p?=
 =?utf-8?B?eXVKcHc0c2lHdjlxSnJEVVpMNzVqdmhBYVJUNUNCQUxqQkFLZjFHbFFJL3FS?=
 =?utf-8?B?bGREdEplVG1QeDVmbE8rd3NzVVpxZDU0V05pWXpIWk0xZmtDSkhsR3FzaXRV?=
 =?utf-8?B?Rm4yNlJzQmdxdGlueTFFaFRjTVhWMGQwNUU4dUVhNHcvWG5teXdZRkg4Tkxo?=
 =?utf-8?B?aFRBRmEzazR3OUVlbktKM2VBQ0p4NnlhQk9mREN6clBadmRwbFVUc01aVFlE?=
 =?utf-8?B?Nkd5NS96MldqQlVQdjZGUmlpTkdobG8yckVQSDl5ZFh5UExhK0p3MDhoOXpn?=
 =?utf-8?B?YWYxdW94N3FoMUxLYStNRVhabHllSkZTY2pTeVpQRzZZMjVNd284eEI3OEd0?=
 =?utf-8?B?czdWcmw3QlNtbExOTnRxeUFvRmI1b1pWME9iVVIrRUhncjIvVVlEMVdPMXBM?=
 =?utf-8?B?YmtSUTN1NGZvQXpRcTRBbXplQ0hEd3ZnWHUzSlRDc0VYeU14TURNZC84R2xm?=
 =?utf-8?B?WGVCTTIvak5WcnlHUFRkRmN3NVR0VEV0S1grdktaMFVBSWVtbDFSTjlWWnhR?=
 =?utf-8?B?dnc9PQ==?=
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cdf05492-7d4b-450c-2e43-08da59c804ce
X-MS-Exchange-CrossTenant-AuthSource: DU2PR07MB8110.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jun 2022 12:08:04.1547
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: djAqJ857To9JbJMCzmJunkzFy2UqC6GlN81W4kBKX+FCM1TMoXHaZuiQHHH8RL14bMwDTTQpzLaFxWA43VnsMQPobHlb3wigA7ptW2ztOyk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB5742
X-Spam-Status: No, score=-1.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,FORGED_SPF_HELO,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Michal,

I did not realize there is a v3 to that, sorry. I have tested those 
patches untill the "Add smbus_block_read functionality" patch and it 
seems to be working fine for me. I will move my comments to this thread 
instead.

Krzysztof

W dniu 29.06.2022 o 13:39, Michal Simek pisze:
> Hi,
>
> On 6/29/22 13:02, Krzysztof Adamski wrote:
>>
>> Hi,
>>
>> I know this patch is quite old but we need the smbus_block_read
>> functionality, which depends on this one, so I would really like this to
>> be merged. See my comments below.
>
> Please try this series and let us know if it is working for you or not.
>
> https://lore.kernel.org/all/1656072327-13628-1-git-send-email-manikanta.guntupalli@xilinx.com/ 
>
>
> Thanks,
> Michal